]> granicus.if.org Git - php/commitdiff
Revert "Fixed bug #71053 (Type Hint about DateTime::diff does not match with informed)"
authorXinchen Hui <laruence@gmail.com>
Wed, 9 Dec 2015 04:00:23 +0000 (12:00 +0800)
committerXinchen Hui <laruence@gmail.com>
Wed, 9 Dec 2015 04:00:23 +0000 (12:00 +0800)
THis is bc break, which will break $date1->diff($date2 , 1), which is
common used

please do not do bc break without a asking

This reverts commit 943b23dcf6666a708c2ad4470cad5db36d47683a.

NEWS
ext/date/php_date.c
ext/date/tests/bug71053.phpt [deleted file]

diff --git a/NEWS b/NEWS
index aae08f26450b98e98504469cd332be0e1e3b09ad..33bd38cc5525057525a2a9c2ecf2fd09aed205a3 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -2,10 +2,6 @@ PHP                                                                        NEWS
 |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
 ?? Jan 2016 PHP 7.0.2
 
-- Date:
-  . Fixed bug #71053 (Type Hint about DateTime::diff does not match with
-    informed). (Reeze Xia)
-
 - Mbstring:
   . Fixed bug #71066 (mb_send_mail: Program terminated with signal SIGSEGV,
     Segmentation fault). (Laruence)
index b3125a9a5a129b19b20292ed9a24b07ce7b1b96b..50c48eb43936cbd31f73c3b207dd44a0809a7a2b 100644 (file)
@@ -3579,9 +3579,9 @@ PHP_FUNCTION(date_diff)
        zval         *object1, *object2;
        php_date_obj *dateobj1, *dateobj2;
        php_interval_obj *interval;
-       zend_bool absolute = 0;
+       zend_long          absolute = 0;
 
-       if (zend_parse_method_parameters(ZEND_NUM_ARGS(), getThis(), "OO|b", &object1, date_ce_interface, &object2, date_ce_interface, &absolute) == FAILURE) {
+       if (zend_parse_method_parameters(ZEND_NUM_ARGS(), getThis(), "OO|l", &object1, date_ce_interface, &object2, date_ce_interface, &absolute) == FAILURE) {
                RETURN_FALSE;
        }
        dateobj1 = Z_PHPDATE_P(object1);
diff --git a/ext/date/tests/bug71053.phpt b/ext/date/tests/bug71053.phpt
deleted file mode 100644 (file)
index 2c86f9f..0000000
+++ /dev/null
@@ -1,13 +0,0 @@
---TEST--
-Bug #71053 (Type Hint about DateTime::diff does not match with informed)
---FILE--
-<?php
-declare(strict_types=1);
-
-$date1 = new \DateTime();
-$date2 = clone $date1;
-
-var_dump(count($date1->diff($date2 , true)) > 0);
-?>
---EXPECT--
-bool(true)