Merged pull request #684

mongodb · Dec 1, 2017 · ea4d10a · ea4d10a
2 parents f6f455a + 1caffe0
commit ea4d10a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/BSON/UTCDateTime.c b/src/BSON/UTCDateTime.c
@@ -396,7 +396,7 @@ static PHP_METHOD(UTCDateTime, unserialize)
 } /* }}} */
 
 /* {{{ MongoDB\BSON\UTCDateTime function entries */
-ZEND_BEGIN_ARG_INFO_EX(ai_UTCDateTime___construct, 0, 0, 1)
+ZEND_BEGIN_ARG_INFO_EX(ai_UTCDateTime___construct, 0, 0, 0)
 	ZEND_ARG_INFO(0, milliseconds)
 ZEND_END_ARG_INFO()
 

diff --git a/tests/bson/bug1053.phpt b/tests/bson/bug1053.phpt
@@ -0,0 +1,12 @@
+--TEST--
+PHPC-1053: MongoDB\BSON\UTCDateTime's constructor has argument defined as required
+--FILE--
+<?php
+$reflection = new ReflectionMethod(MongoDB\BSON\UTCDateTime::class, '__construct');
+var_dump($reflection->getParameters()[0]->isOptional());
+?>
+===DONE===
+<?php exit(0); ?>
+--EXPECT--
+bool(true)
+===DONE===