Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[VarDumper] fix detecting anonymous exception classes on Windows and PHP 7 #58765

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Nov 5, 2024

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Issues
License MIT

such a class name can look like this: class@anonymous\x00D:\a\symfony\symfony\src\Symfony\Component\VarDumper\Tests\Caster\ExceptionCasterTest.php145D2631

@carsonbot carsonbot added this to the 5.4 milestone Nov 5, 2024
@carsonbot carsonbot changed the title [VarDumper] fix detecting anonymous exception classes on Windows and PHP 7 [VarDumper] fix detecting anonymous exception classes on Windows and PHP 7 Nov 5, 2024
Copy link
Member

@alexandre-daubois alexandre-daubois left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be testable by any chance?

@xabbuh
Copy link
Member Author

xabbuh commented Nov 5, 2024

tests in #58721 fail without these changes

@nicolas-grekas
Copy link
Member

Looks like we use this regexp in quite a few classes: grep 'ff.*@anonymous' src/ -r

@xabbuh xabbuh force-pushed the var-dumper-anonymous-class-detection-windows branch from 0c19e32 to f36e21d Compare November 5, 2024 14:17
@xabbuh
Copy link
Member Author

xabbuh commented Nov 5, 2024

I updated the other places the same way

@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit 5d5e728 into symfony:5.4 Nov 5, 2024
11 of 12 checks passed
@xabbuh xabbuh deleted the var-dumper-anonymous-class-detection-windows branch November 5, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants