ID: 45956
Updated by: [EMAIL PROTECTED]
Reported By: bugs at timj dot co dot uk
-Status: Open
+Status: Assigned
Bug Type: Filesystem function related
-Operating System: irrelevant
+Operating System: *
-PHP Version: 5.2.6
+PHP Version: 5CVS, 6CVS (2008-08-31)
-Assigned To:
+Assigned To: jani
New Comment:
It's a bug. Oversight, but bug nevertheless.
Previous Comments:
------------------------------------------------------------------------
[2008-08-30 18:59:21] bugs at timj dot co dot uk
Description:
------------
This seems to be a design error rather than a bug: parse_ini_file()
does not provide any way to handle errors gracefully. In the event of a
syntax error, it throws an E_WARNING *but* the return value is simply an
empty array (the same as a - valid - empty file) instead of FALSE. Thus
there is no way to trap the error (via @ prefix) and handle it in a more
graceful way than an E_WARNING.
I would argue that changing the return value to FALSE in the case of an
invalid file is a reasonable change, since the only situations it will
break are ones where currently PHP throws a warning anyway.
Reproduce code:
---------------
This is what it would be nice to be able to do:
$ini = @parse_ini_file('some_invalid.ini');
if ($ini === false) {
// error: do something graceful
}
// normal execution
Expected result:
----------------
The code section "// do something graceful" executes
Actual result:
--------------
The code section "// normal execution" executes
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=45956&edit=1
