Can you explain why the dumper behaves differently on PHP 8.1?

@derrabus looks like the behavior of ReflectionGenerator::getTrace() changed in PHP 8.1, returning some info when there was none before.

@derrabus also, some other functions seem to have changed their behaviour. For example, when null, ReflectionParameter::getDefaultValue() returns null on PHP 8.0, and ReflectionException: Internal error: Failed to retrieve the default value on PHP 8.1. Searched why in recent PRs of https://github.com/php/php-src/, but couldn't find which commit is responsible of it.

@derrabus For example, when null, ReflectionParameter::getDefaultValue() returns null on PHP 8.0, and ReflectionException: Internal error: Failed to retrieve the default value on PHP 8.1.

I'd like to clarify first, if that change was intentional: https://bugs.php.net/bug.php?id=81147

The other changes look reasonable to me.

Okay, that was fast and we've got the answer we were looking for.

Just to confirm, this is indeed an intentional change to ensure that optional-before-required is consistently treated as a required parameter. Previously it was treated as such in nearly all situations, but there were loopholes. One, as you have discovered, is reflection. The other is named arguments, where it was possible to not specify such a parameter if later required parameters were specified.

Can you have a look at the Travis failure? It looks related.

Thank you so much for your investigation, super good to know! I'll definitely take a look at Travis tomorrow to finish this PR. 👍

@derrabus done 👌 This failing build for 7.4 seems unrelated to this PR.

I'm not sure I understand the difference between "Tests / Integration (x.x)" and Travis CI tests ?

Thank you @alexandre-daubois.