Skip to content

Fix gen_stub.php --verify-manual subcommand#21484

Closed
kocsismate wants to merge 1 commit intophp:PHP-8.5from
kocsismate:dom-const-links
Closed

Fix gen_stub.php --verify-manual subcommand#21484
kocsismate wants to merge 1 commit intophp:PHP-8.5from
kocsismate:dom-const-links

Conversation

@kocsismate
Copy link
Member

Before these changes, the command used to fail with an exception due to ext/DOM constants being wrongly formatted: the old and new constant counterparts were documented in the same row (e.g. DOM_VALIDATION_ERR / Dom\VALIDATION_ERR) which format cannot be recognized by gen_stub.php. Instead, let's document each constant on its own row.

Before these changes, the command used to fail with an exception due to ext/DOM constants being wrongly formatted: the old and new constant counterparts were documented in the same row (e.g. <constant>DOM_VALIDATION_ERR</constant> / <constant>Dom\VALIDATION_ERR</constant>) which format cannot be recognized by gen_stub.php. Instead, let's document each constant on its own row.
@kocsismate
Copy link
Member Author

Sorry, I realized that this PR is not needed: the @link tag is only used for class constants (and actually unused for global ones)

@kocsismate kocsismate deleted the dom-const-links branch March 21, 2026 13:47
kocsismate added a commit to kocsismate/doc-en that referenced this pull request Mar 21, 2026
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.

1 participant