minor #19914 [PropertyAccess] implement __isset() together with __get() (xabbuh)

OskarStark · OskarStark · commit 24c3fc2a265f · 2024-05-24T10:49:02.000+02:00
This PR was merged into the 5.4 branch. Discussion ---------- [PropertyAccess] implement `__isset()` together with `__get()` see symfony/symfony#54739 (comment) Commits ------- b38c05e implement __isset() together with __get()
diff --git a/components/property_access.rst b/components/property_access.rst
@@ -204,12 +204,22 @@ The ``getValue()`` method can also use the magic ``__get()`` method::
         {
             return $this->children[$id];
         }
+
+        public function __isset($id): bool
+        {
+            return true;
+        }
     }
 
     $person = new Person();
 
     var_dump($propertyAccessor->getValue($person, 'Wouter')); // [...]
 
+.. caution::
+
+    When implementing the magic ``__get()`` method, you also need to implement
+    ``__isset()``.
+
 .. versionadded:: 5.2
 
     The magic ``__get()`` method can be disabled since in Symfony 5.2.

Original file line number	Diff line number	Diff line change
@@ -204,12 +204,22 @@ The ``getValue()`` method can also use the magic ``__get()`` method::
`204`	`204`	`{`
`205`	`205`	`return $this->children[$id];`
`206`	`206`	`}`
	`207`	`+`
	`208`	`+ public function __isset($id): bool`
	`209`	`+ {`
	`210`	`+ return true;`
	`211`	`+ }`
`207`	`212`	`}`
`208`	`213`
`209`	`214`	`$person = new Person();`
`210`	`215`
`211`	`216`	`var_dump($propertyAccessor->getValue($person, 'Wouter')); // [...]`
`212`	`217`
	`218`	`+.. caution::`
	`219`	`+`
	`220`	+ When implementing the magic ``__get()`` method, you also need to implement
	`221`	+ ``__isset()``.
	`222`	`+`
`213`	`223`	`.. versionadded:: 5.2`
`214`	`224`
`215`	`225`	The magic ``__get()`` method can be disabled since in Symfony 5.2.