Add a ReflectionClass::newInstanceFromData() method that behaves the same as PDOStatement::fetchObject()

Author: mattdinthehouse

ext/reflection/php_reflection.c

@@ -5127,6 +5127,62 @@ ZEND_METHOD(ReflectionClass, newInstanceArgs)
 }
 /* }}} */
 
+/* {{{ Returns an instance of this class whose properties are filled with the given data before the constructor is called */
+ZEND_METHOD(ReflectionClass, newInstanceFromData)
+{
+	reflection_object *intern;
+	zend_class_entry *ce;
+	int argc = 0;
+	HashTable *data, *args = NULL;
+	zend_function *constructor;
+	zend_string *key;
+	zval *val;
+
+	GET_REFLECTION_OBJECT_PTR(ce);
+
+	ZEND_PARSE_PARAMETERS_START(1, 2)
+		Z_PARAM_ARRAY_HT(data)
+		Z_PARAM_OPTIONAL
+		Z_PARAM_ARRAY_HT(args)
+	ZEND_PARSE_PARAMETERS_END();
+
+	if (args) {
+		argc = zend_hash_num_elements(args);
+	}
+
+	if (UNEXPECTED(object_init_ex(return_value, ce) != SUCCESS)) {
+		return;
+	}
+
+	ZEND_HASH_FOREACH_STR_KEY_VAL(data, key, val) {
+		zend_update_property_ex(ce, Z_OBJ_P(return_value), key, val);
+	} ZEND_HASH_FOREACH_END();
+
+	const zend_class_entry *old_scope = EG(fake_scope);
+	EG(fake_scope) = ce;
+	constructor = Z_OBJ_HT_P(return_value)->get_constructor(Z_OBJ_P(return_value));
+	EG(fake_scope) = old_scope;
+
+	/* Run the constructor if there is one */
+	if (constructor) {
+		if (!(constructor->common.fn_flags & ZEND_ACC_PUBLIC)) {
+			zend_throw_exception_ex(reflection_exception_ptr, 0, "Access to non-public constructor of class %s", ZSTR_VAL(ce->name));
+			zval_ptr_dtor(return_value);
+			RETURN_NULL();
+		}
+
+		zend_call_known_function(
+			constructor, Z_OBJ_P(return_value), Z_OBJCE_P(return_value), NULL, 0, NULL, args);
+
+		if (EG(exception)) {
+			zend_object_store_ctor_failed(Z_OBJ_P(return_value));
+		}
+	} else if (argc) {
+		zend_throw_exception_ex(reflection_exception_ptr, 0, "Class %s does not have a constructor, so you cannot pass any constructor arguments", ZSTR_VAL(ce->name));
+	}
+}
+/* }}} */
+
 void reflection_class_new_lazy(INTERNAL_FUNCTION_PARAMETERS,
 		int strategy, bool is_reset)
 {

ext/reflection/php_reflection.stub.php

@@ -370,6 +370,9 @@ public function newInstanceWithoutConstructor(): object {}
     /** @tentative-return-type */
     public function newInstanceArgs(array $args = []): ?object {}
 
+    /** @tentative-return-type */
+    public function newInstanceFromData(array $data, array $args = []): ?object {}
+
     public function newLazyGhost(callable $initializer, int $options = 0): object {}
 
     public function newLazyProxy(callable $factory, int $options = 0): object {}

ext/reflection/php_reflection_arginfo.h

@@ -0,0 +1,90 @@
+--TEST--
+ReflectionClass::newInstanceFromData
+--FILE--
+<?php
+
+class A
+{
+    public int $a;
+    public string $b;
+
+    public function __construct($c, $d)
+    {
+        echo "In constructor of class A\n";
+    }
+}
+
+class B
+{
+    public int $a;
+    public string $b;
+}
+
+class C
+{
+}
+
+#[\AllowDynamicProperties]
+class D
+{
+}
+
+
+$rcA = new ReflectionClass('A');
+$rcB = new ReflectionClass('B');
+$rcC = new ReflectionClass('C');
+$rcD = new ReflectionClass('D');
+
+try
+{
+    $rcA->newInstanceFromData(['a' => 'bad', 'b' => 123], ['foo', 1337]);
+}
+catch(Throwable $e)
+{
+    echo "Exception: " . $e->getMessage() . "\n";
+}
+
+var_dump($rcA->newInstanceFromData(['a' => 123, 'b' => 'good'], ['foo', 1337]));
+
+var_dump($rcB->newInstanceFromData(['a' => 123, 'b' => 'good']));
+
+var_dump($rcC->newInstanceFromData(['a' => 123, 'b' => 'good']));
+
+var_dump($rcC->newInstanceFromData([]));
+
+var_dump($rcD->newInstanceFromData(['a' => 123, 'b' => 'good']));
+
+?>
+--EXPECTF--
+Exception: Cannot assign string to property A::$a of type int
+In constructor of class A
+object(A)#5 (2) {
+  ["a"]=>
+  int(123)
+  ["b"]=>
+  string(4) "good"
+}
+object(B)#5 (2) {
+  ["a"]=>
+  int(123)
+  ["b"]=>
+  string(4) "good"
+}
+
+Deprecated: Creation of dynamic property C::$a is deprecated in %s on line %d
+
+Deprecated: Creation of dynamic property C::$b is deprecated in %s on line %d
+object(C)#5 (2) {
+  ["a"]=>
+  int(123)
+  ["b"]=>
+  string(4) "good"
+}
+object(C)#5 (0) {
+}
+object(D)#5 (2) {
+  ["a"]=>
+  int(123)
+  ["b"]=>
+  string(4) "good"
+}

ext/reflection/tests/ReflectionClass_newInstanceFromData_001.phpt

@@ -30,7 +30,7 @@ Class [ <internal:Reflection> class ReflectionClass implements Stringable, Refle
     Property [ public string $name ]
   }
 
-  - Methods [64] {
+  - Methods [65] {
     Method [ <internal:Reflection> private method __clone ] {
 
       - Parameters [0] {
@@ -332,6 +332,15 @@ Class [ <internal:Reflection> class ReflectionClass implements Stringable, Refle
       - Tentative return [ ?object ]
     }
 
+    Method [ <internal:Reflection> public method newInstanceFromData ] {
+
+      - Parameters [2] {
+        Parameter #0 [ <required> array $data ]
+        Parameter #1 [ <optional> array $args = [] ]
+      }
+      - Tentative return [ ?object ]
+    }
+
     Method [ <internal:Reflection> public method newLazyGhost ] {
 
       - Parameters [2] {