Copy warnings and errors from xxx_writer to tag_writer

[fred727]

Not a bug but a suggestion.

If there is errors or warnings in an object getid3_write_id3v2 for exemple, these errors are not copied into the "top" object getid3_writetags, so we do not know they occured !!

Example :

Code: Select all

		$tagwriter = new getid3_writetags() ;
		$tagwriter->filename = 'path/to/file.mp3';
		$tagwriter->tagformats = [ 'id3v2.3' ] ;
		$tagwriter->tag_data = array(
			'title'                     => [ 'my title' ],
			'url_artist'                      => [ 'not a @url !' ],
		);
		$ok = $tagwriter->WriteTags() ;  /* uses getid3_write_id3v2 ! */
		/* We don't avec access to the [i]getid3_write_id3v2[/i] object here ! No property for that in $tagwriter... */
		var_dump($tagwriter->errors); /* nothing... */

[James Heinrich]

Actually they are copied, albeit in a slightly modified form. For example:
https://github.com/JamesHeinrich/getID3 ... #L281-L283

[fred727]

The problem is that $id3v2_writer->WriteID3v2() can return true even if errors/warnings occur.
(this is the case in my exemple above)

[James Heinrich]

As with everywhere in getID3, a warning is non-fatal and an error is fatal -- WriteID3v2() should return false if any errors occurred (write.id3v2.php lines 127-134):

Code: Select all

if (!empty($this->errors)) {
	return false;
}
return true;