Skip to content

Fix rendering of attribute constants#249

Open
kocsismate wants to merge 2 commits intophp:masterfrom
kocsismate:fix-attribute-const-rendering
Open

Fix rendering of attribute constants#249
kocsismate wants to merge 2 commits intophp:masterfrom
kocsismate:fix-attribute-const-rendering

Conversation

@kocsismate
Copy link
Member

$this->cchunk["fieldsynopsis"]["modifier"] was not set for atrributes due to the early return, but this value is needed in order to be able to decide if the fieldsynopsis refers to a constant or a property at

if ($this->cchunk["fieldsynopsis"]["modifier"] === "const") {

$this->cchunk["fieldsynopsis"]["modifier"] was not set for atrributes due to the early return, but this value is needed in order to be able to decide if the fieldsynopsis refers to a constant or a property at https://github.com/php/phd/blob/d7f700463918a466705e92b4bfd4df0afa39d944/phpdotnet/phd/Package/Generic/XHTML.php#L1608
@Girgias
Copy link
Member

Girgias commented Mar 16, 2026

Could you add a test file? :)

@kocsismate kocsismate force-pushed the fix-attribute-const-rendering branch 3 times, most recently from fa2280b to eb1b0d2 Compare March 16, 2026 20:53
@kocsismate kocsismate force-pushed the fix-attribute-const-rendering branch from eb1b0d2 to b79ab5f Compare March 16, 2026 21:00
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.

2 participants