implements dynamic property access through __get()

jakubmisek · jakubmisek · commit d1fc1792fb73 · 2025-04-18T14:37:10.000+02:00
fixes #1165
diff --git a/src/Peachpie.Runtime/Operators.cs b/src/Peachpie.Runtime/Operators.cs
@@ -1200,9 +1200,9 @@ public static PhpValue PropertyGetValue(RuntimeTypeHandle caller, object instanc
 
             // 3. __get
             var getter = tinfo.RuntimeMethods[TypeMethods.MagicMethods.__get];
-            if (getter != null)
+            if (getter != null) // NOTE: magic methods must be public
             {
-                // return getter.Invoke(ctx, instance, propertyName);
+                return getter.Invoke(null/*expecting non-static PHP method which doesn't need Context, otherwise nullref exception*/, instance, propertyName);
             }
 
             // error

Original file line number	Diff line number	Diff line change
`@@ -1200,9 +1200,9 @@ public static PhpValue PropertyGetValue(RuntimeTypeHandle caller, object instanc`
`1200`	`1200`
`1201`	`1201`	`// 3. __get`
`1202`	`1202`	`var getter = tinfo.RuntimeMethods[TypeMethods.MagicMethods.__get];`
`1203`		`- if (getter != null)`
	`1203`	`+ if (getter != null) // NOTE: magic methods must be public`
`1204`	`1204`	`{`
`1205`		`- // return getter.Invoke(ctx, instance, propertyName);`
	`1205`	`+ return getter.Invoke(null/expecting non-static PHP method which doesn't need Context, otherwise nullref exception/, instance, propertyName);`
`1206`	`1206`	`}`
`1207`	`1207`
`1208`	`1208`	`// error`