#34764 [Csd->Bgs]: Acc. to the manual exif_read_data() should always return an array, it doesn't

ID: 34764 Updated by: simp@php.net Reported By: avarab at gmail dot com -Status: Closed +Status: Bogus Bug Type: Documentation problem Operating System: GNU/Linux PHP Version: Irrelevant New Comment: Thank you for taking the time to write to us, but this is not a bug. Please double-check the documentation available at http://www.php.net/manual/ and the instructions on how to report a bug at http://bugs.php.net/how-to-report.php A function returning false only in case of errors is consistently described with its main return type throughout the whole documentation. Previous Comments: ------------------------------------------------------------------------ [2005-10-06 19:06:30] simp@php.net This bug has been fixed in the documentation's XML sources. Since the online and downloadable versions of the documentation need some time to get updated, we would like to ask you to be a bit patient. Thank you for the report, and for helping us make our documentation better. ------------------------------------------------------------------------ [2005-10-06 18:34:09] avarab at gmail dot com Description: ------------ The prototype given for exif_read_data() at http://www.php.net/exif_read_data is: array exif_read_data ( string filename [, string sections [, bool arrays [, bool thumbnail]]] ) According to this prototype the function should always return an array, yet it returns false when fed with invalid input: Reproduce code: --------------- $ dd if=/dev/urandom of=foo count=1024 $ php -r "var_dump( exif_read_data( 'foo' ) );" Warning: exif_read_data(foo): file not supported in Command line code on line 1 bool(false) Expected result: ---------------- The documentation should be updated to report that exif_read_data() returns mixed (not array), array upon success, false upon failure. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=34764&edit=1