ReflectionClass: show enums differently from classes, test output #15956
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While internally enums are mostly the same as classes, their output in
ReflectionClass::__toString()
should show the enum as the developer wrote it,rather than as the engine stored it. Accordingly
Say that the enum is an enum, not a final class
Include the backing type, if any, in the declaration line
Remove the
UnitEnum
andBackedEnum
interfaces from the list of interfacesimplemented
List enum cases separately from constants, and show the underlying values, if
any
Also add tests for the output - the tests are added in the first commit, so
that the changes in the second commit can be more easily confirmed.
GH-15766