Req #60957 [NEW]: if function returns false on error, don't emit a warning

From: Date: Thu, 02 Feb 2012 16:58:25 +0000
Subject: Req #60957 [NEW]: if function returns false on error, don't emit a warning
Groups: php.bugs 
Request: Send a blank email to php-bugs+get-167067@lists.php.net to get a copy of this message 
From:             
Operating system: 
PHP version:      Irrelevant
Package:          Filesystem function related
Bug Type:         Feature/Change Request
Bug description:if function returns false on error, don't emit a warning

Description:
------------
functions that return FALSE on error should not also emit a warning.

Example: filemtime(). it is sufficient to check if the file exists and
retrieve the mtime by doing:
if ($mtime = filemtime()) {
 echo date('ymd', $mtime);
} else {
 echo 'file does not exist';
}

supressing the warning with "@" is slow and generates an error in the log
(also slow). checking if the file exists before retrieving the mtime is
also wasteful.

Expected result:
----------------
filemtime and other functions that emit a warning on error when false is
also returned should not emit a warning.


-- 
Edit bug report at https://bugs.php.net/bug.php?id=60957&edit=1
-- 
Try a snapshot (PHP 5.4):            https://bugs.php.net/fix.php?id=60957&r=trysnapshot54
Try a snapshot (PHP 5.3):            https://bugs.php.net/fix.php?id=60957&r=trysnapshot53
Try a snapshot (trunk):              https://bugs.php.net/fix.php?id=60957&r=trysnapshottrunk
Fixed in SVN:                        https://bugs.php.net/fix.php?id=60957&r=fixed
Fixed in SVN and need be documented: https://bugs.php.net/fix.php?id=60957&r=needdocs
Fixed in release:                    https://bugs.php.net/fix.php?id=60957&r=alreadyfixed
Need backtrace:                      https://bugs.php.net/fix.php?id=60957&r=needtrace
Need Reproduce Script:               https://bugs.php.net/fix.php?id=60957&r=needscript
Try newer version:                   https://bugs.php.net/fix.php?id=60957&r=oldversion
Not developer issue:                 https://bugs.php.net/fix.php?id=60957&r=support
Expected behavior:                   https://bugs.php.net/fix.php?id=60957&r=notwrong
Not enough info:                     https://bugs.php.net/fix.php?id=60957&r=notenoughinfo
Submitted twice:                     https://bugs.php.net/fix.php?id=60957&r=submittedtwice
register_globals:                    https://bugs.php.net/fix.php?id=60957&r=globals
PHP 4 support discontinued:          https://bugs.php.net/fix.php?id=60957&r=php4
Daylight Savings:                    https://bugs.php.net/fix.php?id=60957&r=dst
IIS Stability:                       https://bugs.php.net/fix.php?id=60957&r=isapi
Install GNU Sed:                     https://bugs.php.net/fix.php?id=60957&r=gnused
Floating point limitations:          https://bugs.php.net/fix.php?id=60957&r=float
No Zend Extensions:                  https://bugs.php.net/fix.php?id=60957&r=nozend
MySQL Configuration Error:           https://bugs.php.net/fix.php?id=60957&r=mysqlcfg

Thread (4 messages)

« previous php.bugs (#167067) next »