Skip to content

Conversation

@haszi
Copy link
Contributor

@haszi haszi commented Nov 3, 2024

Add links to constants with replaceable tags by processing them when the constant tag is being closed. Also add a test.

This currently only works with one set of <replaceable> tags per constant (due to issues making the regex non-greedy). In practice, this shouldn't be an issue as we have no constants with more than one set of these tags.

The implementation feels a little hacky but more or less matches the processing of all other tags where post-processing is needed (all the class-based synposis* tags and their children).

Add links to constants with replaceable tags by processing them when the constant tag is being closed.
Add a test.
Copy link
Member

@Girgias Girgias left a comment

Choose a reason for hiding this comment

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

Thank you!

@Girgias Girgias merged commit 99c70b7 into php:master Nov 3, 2024
9 checks passed
Girgias pushed a commit to php/doc-en that referenced this pull request Nov 3, 2024
This will link to the constant pages since php/phd#175 has been marged.

Co-authored-by: haszi <haszika80@gmail.com>
@haszi haszi deleted the Add-links-to-constants-with-replaceable-tags branch November 3, 2024 14:29
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