Deprecate old ReflectionParameter type declaration APIs

This deprecates:

    ReflectionParameter::isArray()
    ReflectionParameter::isCallable()
    ReflectionParameter::getClass()

These APIs have been superseded by ReflectionParameter::getType()
since PHP 7.0. Types introduced since that time are not available
through the old APIs, and their behavior is getting increasingly
confusing. This is how they interact with PHP 8 union types:

 * isArray() will return true if the type is array or ?array,
   but not any other union type
 * Same for isCallable().
 * getClass() will return a class for T|int etc, as long as the
   union only contains a single type. T1|T2 will return null.

This behavior is not particularly reasonable or useful, and will
get more confusing as new type system extensions are added.

Closes GH-5209.

1 files changed, 12 insertions, 3 deletions

diff --git a/ext/reflection/php_reflection.stub.php b/ext/reflection/php_reflection.stub.php
index 1bcd4221cf..e629b8d845 100644
--- a/ext/reflection/php_reflection.stub.php
+++ b/ext/reflection/php_reflection.stub.php
@@ -493,7 +493,10 @@ class ReflectionParameter implements Reflector
     /** @return ReflectionClass|null */
     public function getDeclaringClass() {}
 
-    /** @return ReflectionClass|null */
+    /**
+     * @return ReflectionClass|null
+     * @deprecated Use ReflectionParameter::getType() instead
+     */
     public function getClass() {}
 
     /** @return bool */
@@ -502,10 +505,16 @@ class ReflectionParameter implements Reflector
     /** @return ReflectionType|null */
     public function getType() {}
 
-    /** @return bool */
+    /**
+     * @return bool
+     * @deprecated Use ReflectionParameter::getType() instead
+     */
     public function isArray() {}
 
-    /** @return bool */
+    /**
+     * @return bool
+     * @deprecated Use ReflectionParameter::getType() instead
+     */
     public function isCallable() {}
 
     /** @return bool */