Doc->Bug #71053 [ReO]: Type Hint about DateTime::diff does not match with informed

Edit report at https://bugs.php.net/bug.php?id=71053&edit=1 ID: 71053 Updated by: salathe@php.net Reported by: gabriel dot heming at hotmail dot com Summary: Type Hint about DateTime::diff does not match with informed Status: Re-Opened -Type: Documentation Problem +Type: Bug Package: Date/time related Operating System: Debian 8.2 PHP Version: 7.0.0 Block user comment: N Private report: N New Comment: This is not a documentation bug. The source code should stipulate boolean for the $absolute parameter. That is the correct type given the a) documentation, b) prototype in the source, c) the use of the argument in the source. If fixing the type breaks existing old (incorrect with strict types) code using a value of 1 for the argument, so be it. Previous Comments: ------------------------------------------------------------------------ [2015-12-09 14:40:23] laruence@php.net then what the problem if we change bool to int in doc? thanks ------------------------------------------------------------------------ [2015-12-09 13:35:28] gabriel dot heming at hotmail dot com If the PHP 7 was builded to be faster and more strict, why this could be accepted as a doc bug? I mean how could be more than a value, since the manual say about a boolean, how someone could use an integer instead? ------------------------------------------------------------------------ [2015-12-09 04:04:22] laruence@php.net I reverted the fix, because it break common used $data->diff($data2, 1); I'd prefer doc bug instead. thanks ------------------------------------------------------------------------ [2015-12-09 04:01:29] laruence@php.net Automatic comment on behalf of laruence@gmail.com Revision: http://git.php.net/?p=php-src.git;a=commit;h=f248309b7485fea08e3c112adab97bfa1fc74df4 Log: Revert "Fixed bug #71053 (Type Hint about DateTime::diff does not match with informed)" ------------------------------------------------------------------------ [2015-12-09 03:50:14] reeze@php.net @requinix I agree with you. It is a bug, at least in strict types context, there might be more cases in repo like this. I will try to find them. ------------------------------------------------------------------------ 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 https://bugs.php.net/bug.php?id=71053 -- Edit this bug report at https://bugs.php.net/bug.php?id=71053&edit=1