QA: Add TODO comment

xp-framework · Aug 25, 2024 · fab6891 · fab6891
1 parent 90d4dd5
commit fab6891
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/main/php/lang/reflection/Property.class.php b/src/main/php/lang/reflection/Property.class.php
@@ -48,6 +48,9 @@ public function constraint() {
    */
   public function get(?object $instance) {
     try {
+
+      // TODO: Remove superfluous call to setAccessible() if on PHP8.1+
+      // see https://wiki.php.net/rfc/make-reflection-setaccessible-no-op
       $this->reflect->setAccessible(true);
       return $this->reflect->getValue($instance);
     } catch (ReflectionException $e) {
@@ -68,6 +71,9 @@ public function get(?object $instance) {
    */
   public function set(?object $instance, $value) {
     try {
+
+      // TODO: Remove superfluous call to setAccessible() if on PHP8.1+
+      // see https://wiki.php.net/rfc/make-reflection-setaccessible-no-op
       $this->reflect->setAccessible(true);
       $this->reflect->setValue($instance, $value);
       return $value;