ID: 47752 Updated by: [email protected] Reported By: for-bugs at hnw dot jp -Status: Bogus +Status: To be documented Bug Type: Filter related Operating System: * PHP Version: 5.2.9 Assigned To: pajoye New Comment:
Make it a documentation bug instead, it should be documented and will be, will add some other things while being at it :) Previous Comments: ------------------------------------------------------------------------ [2009-03-24 15:15:17] [email protected] We have double checked again the design choices about -/+ 0, it was on purpose. -0 or +0 are not valid integers. Sorry, but we have to reject this request. thanks for your understanding, ------------------------------------------------------------------------ [2009-03-24 15:03:53] peter at lvp-media dot com Hereby a simple bug47752.phpt file. Tested it on a patched- an an unpatched installation, results as expected. Strictly speaking +0 and -0 are valid integers, even though neither of them is positive or negative. Peter Beverloo --TEST-- Bug #47752 (FILTER_VALIDATE_INT doesn't allow "+0" and "-0") --FILE-- <?php $positive = filter_var ('+0', FILTER_VALIDATE_INT); $negative = filter_var ('-0', FILTER_VALIDATE_INT); $zero = filter_var ('0', FILTER_VALIDATE_INT); $octal = filter_var ('-0123', FILTER_VALIDATE_INT); var_dump ($positive); var_dump ($negative); var_dump ($zero); var_dump ($octal); ?> --EXPECT-- int(0) int(0) int(0) bool(false) ------------------------------------------------------------------------ [2009-03-24 12:44:19] [email protected] As far as I remember it is on purpose. 0 cannot be negative or positive. But if we agree on allowing them again, no problem here. Can you please provide tests as well (phpt)? ------------------------------------------------------------------------ [2009-03-24 11:50:38] php at lvp-media dot com False, this only allows +0 and -0 by checking the lenght of the string, it verifies that the null is the only character in it. ------------------------------------------------------------------------ [2009-03-24 11:49:06] [email protected] Right, so this patch is not correct as it would allow octal numbers. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/47752 -- Edit this bug report at http://bugs.php.net/?id=47752&edit=1
