feat: support for magic methods registered via __get and __isset

Ref: ramsey#125
tcarrio · Aug 22, 2024 · 4e368d4 · 4e368d4
1 parent 929049d
commit 4e368d4
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 0 deletions.
diff --git a/src/Tool/ValueExtractorTrait.php b/src/Tool/ValueExtractorTrait.php
@@ -72,6 +72,10 @@ protected function extractValue(mixed $element, ?string $propertyOrMethod): mixe
             return $element->{$propertyOrMethod}();
         }
 
+        if (isset($element->$propertyOrMethod)) {
+            return $element->$propertyOrMethod;
+        }
+
         throw new InvalidPropertyOrMethod(sprintf(
             'Method or property "%s" not defined in %s',
             $propertyOrMethod,

diff --git a/tests/Tool/ValueExtractorTraitTest.php b/tests/Tool/ValueExtractorTraitTest.php
@@ -88,4 +88,35 @@ public function getType(): string
 
         $this->assertSame('works!', $test('testProperty'), 'Could not extract value by property');
     }
+
+    public function testShouldExtractValueByMagicMethod(): void
+    {
+        $test = new class {
+            use ValueExtractorTrait;
+
+            /**
+             * @return mixed
+             */
+            public function __invoke(string $propertyOrMethod)
+            {
+                return $this->extractValue($this, $propertyOrMethod);
+            }
+
+            public function __get(string $name): mixed
+            {
+                if ($name === 'magic_property') {
+                    return 'value';
+                }
+
+                return null;
+            }
+
+            public function __isset(string $name): bool
+            {
+                return $name === 'magic_property';
+            }
+        };
+
+        $this->assertSame('value', $test('magic_property'), 'Could not extract value by magic method');
+    }
 }