Implement Stringable automatically for internal classes

Requiring all internal classes (including those from 3rd-party
extensions) to implement Stringable if they provide __toString()

Implement Stringable automatically for internal classes

Requiring all internal classes (including those from 3rd-party
extensions) to implement Stringable if they provide __toString()
is too error prone. Case in point, our _ZendTestClass test class
was not doing so, resulting in preloading test failures after
recent changes.

Instead we automatically implement Stringable, the same as we do
for userland classes. We still allow explicit implementations,
but ignore them (normally they would result in an error due to
duplicate interface implementation). Finally, we need to be
careful about not trying to implement Stringable on Stringable
itself.

In some cases this changes the interface order, in particular the
automatic Stringable implementation will now come first.

show more ...

Require stubs to declare return types for magic methods when possible

Closes GH-6376

don't display default value for variadic

Make the $filter parameter of ReflectionClass::get*Constants() nullable

Use ZPP instead of custom type checks

We can add these types as a native type declaration to stubs as a side-effect. Closes GH-6068

Add a few missing parameter types in stubs

Related to GH-5627

Cleanup argument handling in ext/reflection

Closes GH-5850

Add $filter parameter for ReflectionClass::(getConstants|getReflectionConstants)

This solves [#79628](https://bugs.php.net/79628).

Similar to `ReflectionClass::getMethods()` and `Re

Add $filter parameter for ReflectionClass::(getConstants|getReflectionConstants)

This solves [#79628](https://bugs.php.net/79628).

Similar to `ReflectionClass::getMethods()` and `ReflectionClass::getProperties()`,
this new `$filter` argument allows the filtering of constants defined in a class by
their visibility.

For that, we create three new constants for `ReflectionClassConstant`:

* `IS_PUBLIC`
* `IS_PROTECTED`
* `IS_PRIVATE`

Closes GH-5649.

show more ...

Add Attributes

Co-authored-by: Martin Schröder <m.schroeder2007@gmail.com>

Store default parameter values of internal functions in arg info

Closes GH-5353. From now on, PHP will have reflection information
about default values of parameters of internal function

Store default parameter values of internal functions in arg info

Closes GH-5353. From now on, PHP will have reflection information
about default values of parameters of internal functions.

Co-authored-by: Nikita Popov <nikita.ppv@gmail.com>

show more ...

Remove <default> prefix from reflection dump

This really doesn't add anything, and only makes for confusing
terminology. Only marking properties as dynamic is sufficient.

Display property default value in reflection dumps

Define Stringable with __toString():string method

Remove the deprecated reflection export methods

Closes GH-5188

Revert "Replace @param annotations with type declarations"

This reverts commit c31029f335ca1b453af799805c43c37e959ad555.

Replace @param annotations with type declarations

Add stubs for ext-reflection

Deprecate Reflection export() methods

And remove the Reflector::export() interface method.

Fix variadic arginfos for Reflection functions

Fix arginfos of required arguments for some Reflection methods

Revert removal of private __clone() methods

I thought these were redundant, because we already NULL out the
clone_obj object handler. However, it turns out that reflection is
using p

Revert removal of private __clone() methods

I thought these were redundant, because we already NULL out the
clone_obj object handler. However, it turns out that reflection is
using private __clone() to determine clonability (isCloneable) for
the case where we only have a class, rather than an object.

As such, removing these methods would be a BC break.

This reverts commit e7131a4e9fa0acf8fc1e486b49851e71859ef5b8.
This reverts commit 55bd88ce0d1bf461546d5d0b40920491d566ed48.

show more ...

Remove redundant __clone() methods from Reflection

Reflection classes already use NULLed clone_obj to signal that they
cannot be cloned, so it's not necessary to additionally declare a

Remove redundant __clone() methods from Reflection

Reflection classes already use NULLed clone_obj to signal that they
cannot be cloned, so it's not necessary to additionally declare a
throwing __clone() method.

show more ...

Renumber ZEND_ACC_... flags

Use EXPECT instead of EXPECTF when possible

EXPECTF logic in run-tests.php is considerable, so let's avoid it.

Use int instead of integer in type errors

PHP requires integer typehints to be written "int" and does not
allow "integer" as an alias. This changes type error messages to
match the a

Use int instead of integer in type errors

PHP requires integer typehints to be written "int" and does not
allow "integer" as an alias. This changes type error messages to
match the actual type name and avoids confusing messages like
"must be of the type integer, integer given".

show more ...

Revert BC break caused by fixing bug #74035

This reverts commit 9ffc6ca62f53431a4b32b30cdda8180142f47cdb.