ID: 41681 User updated by: rich at corephp dot co dot uk Reported By: rich at corephp dot co dot uk Status: Closed Bug Type: Filter related Operating System: Windows XP SP2 PHP Version: 5.2.3 Assigned To: pajoye New Comment:
By 'the decimal option was missing as well' do you mean the FILTER_FLAG_ALLOW_FRACTION flag? If so I have found that using that makes no difference at all: $val = 33.4; $result = filter_var($val, FILTER_VALIDATE_FLOAT, FILTER_FLAG_ALLOW_FRACTION); var_dump($val); var_dump($result); $val = 4.99; $result = filter_var($val, FILTER_VALIDATE_FLOAT); var_dump($val); var_dump($result); The first allows decimals and the float is created. The second doesn't specify decimals, but the float is still created. Using the German decimal notation (3,99) causes a boolean(false) regardless of the allow_fraction flag being present or not. Unless there is another flag you meant? Previous Comments: ------------------------------------------------------------------------ [2007-06-14 08:04:13] [EMAIL PROTECTED] "Then the 'bug' is simply in the documentation, because the thousands flag is *not* listed as a valid flag for that filter, and according to what you said, it should be." Good point, and the decimal option was missing as well. It is fixed in CVS, it will show up in the next manual update. Thanks for your report! ------------------------------------------------------------------------ [2007-06-14 07:53:31] rich at corephp dot co dot uk Then the 'bug' is simply in the documentation, because the thousands flag is *not* listed as a valid flag for that filter, and according to what you said, it should be. ------------------------------------------------------------------------ [2007-06-14 05:30:03] [EMAIL PROTECTED] "In which case why does the "allow thousands flag work" Because the thousand separator can be , ' or even . There is also an option to define it. The scientific format is support by default, always. Just like a (float)$var. Finally, "e34200" is not a valid scientific format. Please refer to the various tests cases in the filter extensions to see other examples (http://cvs.php.net/viewvc.cgi/php-src/ext/filter/tests/). ------------------------------------------------------------------------ [2007-06-13 23:22:02] rich at corephp dot co dot uk I understand that, in which case why does the "allow thousands" flag work? One way or the other it is inconsistent. Either the bug is that the allow thousands flag works, or the bug is that allow scientific doesn't. ------------------------------------------------------------------------ [2007-06-13 20:45:19] [EMAIL PROTECTED] It is a flag for the sanitizing function not the validate filter, see the main page of the filter manual function. (Not a bug > bogus) ------------------------------------------------------------------------ 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/41681 -- Edit this bug report at http://bugs.php.net/?id=41681&edit=1
