Skip to content

ext/reflection: ignore leading namespace separator in ReflectionParameter - #22325

Closed
jorgsowa wants to merge 1 commit into
php:PHP-8.4from
jorgsowa:fix/reflectionparameter-leading-backslash
Closed

ext/reflection: ignore leading namespace separator in ReflectionParameter#22325
jorgsowa wants to merge 1 commit into
php:PHP-8.4from
jorgsowa:fix/reflectionparameter-leading-backslash

Conversation

@jorgsowa

Copy link
Copy Markdown
Contributor

Extracted from #22260

Strip a leading '\' before lowercasing, matching ReflectionFunction::__construct().

Closes #22324

@jorgsowa
jorgsowa force-pushed the fix/reflectionparameter-leading-backslash branch from c2ad475 to fa44a2e Compare June 15, 2026 22:33
…eter

Strip a leading '\\' before lowercasing, matching ReflectionFunction::__construct().
@jorgsowa
jorgsowa force-pushed the fix/reflectionparameter-leading-backslash branch from fa44a2e to 0b942fe Compare June 16, 2026 22:12
@jorgsowa
jorgsowa changed the base branch from master to PHP-8.4 June 16, 2026 22:12

@iluuu1994 iluuu1994 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks correct.

@iluuu1994 iluuu1994 closed this in 47f060c Jun 17, 2026
@iluuu1994

Copy link
Copy Markdown
Member

Thanks!

@DanielEScherzer

Copy link
Copy Markdown
Member

I was planning to review this today, looks like @iluuu1994 beat me to it

@jorgsowa
jorgsowa deleted the fix/reflectionparameter-leading-backslash branch July 10, 2026 14:29
adrian-enspired pushed a commit to adrian-enspired/php-src that referenced this pull request Aug 4, 2026
Strip a leading '\\' before lowercasing, matching ReflectionFunction::__construct().

Fixes phpGH-22324
Closes phpGH-22325
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.

ext/reflection: ReflectionParameter fails to load function with leading \

3 participants