Bug #72209 [NEW]: ReflectionProperty#getValue() doesn't fail if object doesn't match type

From: Date: Thu, 12 May 2016 15:37:51 +0000
Subject: Bug #72209 [NEW]: ReflectionProperty#getValue() doesn't fail if object doesn't match type
Groups: php.bugs 
Request: Send a blank email to php-bugs+get-201043@lists.php.net to get a copy of this message
From: ocramius at gmail dot com Operating system: PHP version: 7.0.7RC1 Package: Reflection related Bug Type: Bug Bug description:ReflectionProperty#getValue() doesn't fail if object doesn't match type Description: ------------ ReflectionProperty#getValue($object) does not check whether the given $object matches the expected type. If $r = ReflectionProperty(Foo::class, 'bar'), then only instances of Foo should be allowed by ReflectionProperty#getValue() and ReflectionProperty#setValue(). All other values should cause an exception to be raised. As a reference, ReflectionMethod behaves correctly ( https://3v4l.org/U2u0h ), since ReflectionMethod#invoke($object) and ReflectionMethod#invokeArgs($object) reject incompatible $object instances. Test script: --------------- <?php class Foo { public $bar = __LINE__; } class Baz { public $bar = __LINE__; } var_dump((new ReflectionProperty(Foo::class, 'bar'))->getValue(new Baz())); Expected result: ---------------- Fatal error: Uncaught ReflectionException: Expected an instance of Foo, got 'Baz' instead does not exist in /.../test-script.php Actual result: -------------- (int) 10 -- Edit bug report at https://bugs.php.net/bug.php?id=72209&edit=1 -- Try a snapshot (PHP 5.4): https://bugs.php.net/fix.php?id=72209&r=trysnapshot54 Try a snapshot (PHP 5.5): https://bugs.php.net/fix.php?id=72209&r=trysnapshot55 Try a snapshot (trunk): https://bugs.php.net/fix.php?id=72209&r=trysnapshottrunk Fixed in SVN: https://bugs.php.net/fix.php?id=72209&r=fixed Fixed in release: https://bugs.php.net/fix.php?id=72209&r=alreadyfixed Need backtrace: https://bugs.php.net/fix.php?id=72209&r=needtrace Need Reproduce Script: https://bugs.php.net/fix.php?id=72209&r=needscript Try newer version: https://bugs.php.net/fix.php?id=72209&r=oldversion Not developer issue: https://bugs.php.net/fix.php?id=72209&r=support Expected behavior: https://bugs.php.net/fix.php?id=72209&r=notwrong Not enough info: https://bugs.php.net/fix.php?id=72209&r=notenoughinfo Submitted twice: https://bugs.php.net/fix.php?id=72209&r=submittedtwice register_globals: https://bugs.php.net/fix.php?id=72209&r=globals PHP 4 support discontinued: https://bugs.php.net/fix.php?id=72209&r=php4 Daylight Savings: https://bugs.php.net/fix.php?id=72209&r=dst IIS Stability: https://bugs.php.net/fix.php?id=72209&r=isapi Install GNU Sed: https://bugs.php.net/fix.php?id=72209&r=gnused Floating point limitations: https://bugs.php.net/fix.php?id=72209&r=float No Zend Extensions: https://bugs.php.net/fix.php?id=72209&r=nozend MySQL Configuration Error: https://bugs.php.net/fix.php?id=72209&r=mysqlcfg

« previous php.bugs (#201043) next »