Skip to content

[BetterPhpDocParser] Resolve class references in Generic PHPDoc tag node containing "::"#7895

Open
ruudk wants to merge 1 commit intorectorphp:mainfrom
ruudk:custom-phpdoc-tags-with-class-reference
Open

[BetterPhpDocParser] Resolve class references in Generic PHPDoc tag node containing "::"#7895
ruudk wants to merge 1 commit intorectorphp:mainfrom
ruudk:custom-phpdoc-tags-with-class-reference

Conversation

@ruudk
Copy link
Contributor

@ruudk ruudk commented Feb 14, 2026

No description provided.

@TomasVotruba
Copy link
Member

I'd prefer generic solution that would not require configuration per project.

@ruudk
Copy link
Contributor Author

ruudk commented Feb 14, 2026

How would you like to do that with a custom phpdoc like this? I believe this was done for performance reasons?

@samsonasik
Copy link
Member

I am on mobile, how about just remove the : if (! in_array() check? The next line already check on explode on :.

This of course need to verify other decorator that doing similar logic on different docblock node to avoid overwritten.

@ruudk
Copy link
Contributor Author

ruudk commented Feb 14, 2026

Yeah i think that would be fine too!

@ruudk ruudk force-pushed the custom-phpdoc-tags-with-class-reference branch from bbaf88a to a111b8f Compare February 16, 2026 13:04
@ruudk ruudk changed the title Allow configuring custom PHPDoc tags that reference classes Resolve class references in any PHPDoc tag containing "::" Feb 16, 2026
@ruudk ruudk force-pushed the custom-phpdoc-tags-with-class-reference branch from a111b8f to bb39d91 Compare February 16, 2026 13:05
@ruudk
Copy link
Contributor Author

ruudk commented Feb 16, 2026

Updated the PR. This is much simpler. Thanks!

@ruudk ruudk force-pushed the custom-phpdoc-tags-with-class-reference branch from bb39d91 to 017d054 Compare February 16, 2026 13:05
@samsonasik
Copy link
Member

@ruudk please update PR title to something like:

[BetterPhpDocParser] Resolve class references in Generic PHPDoc tag node containing "::"

since it apply only on Generic.

@ruudk ruudk changed the title Resolve class references in any PHPDoc tag containing "::" [BetterPhpDocParser] Resolve class references in Generic PHPDoc tag node containing "::" Feb 16, 2026
@ruudk
Copy link
Contributor Author

ruudk commented Feb 16, 2026

Done, thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants