Skip to content

ext/reflection: Add ReflectionConstant::inNamespace() method #20902

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.

Sign up for GitHub

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

Jump to bottom
Open
khaledalam wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ext/reflection: Add ReflectionConstant::inNamespace() method #20902

khaledalam wants to merge 1 commit into from
+68 −1

Conversation

@khaledalam
Copy link

@khaledalam khaledalam commented Jan 11, 2026 •
edited
Loading

Description

Fix #19557

This PR adds the missing inNamespace() method to ReflectionConstant to achieve full parity with ReflectionClass and ReflectionFunctionAbstract for API consistency.

Background

ReflectionConstant already has:

  • getShortName() - returns the short name without namespace
  • getNamespaceName() - returns the namespace name

But was missing:

  • inNamespace() - returns boolean indicating if in a namespace

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kocsismate kocsismate Awaiting requested review from kocsismate kocsismate is a code owner

@DanielEScherzer DanielEScherzer Awaiting requested review from DanielEScherzer DanielEScherzer is a code owner

Assignees

No one assigned

Labels

Extension: reflection

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ext/reflection: Add missing ReflectionConstant::inNamespace() method

1 participant

@khaledalam