Skip to content
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.

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

Fix GH-3353 incorrect function tag linking #123

Merged
merged 2 commits into from
May 22, 2024

Conversation

haszi
Copy link
Contributor

@haszi haszi commented Apr 30, 2024

Closes php/doc-en#3353.

When the list of refentrys (functions and methods) is retrieved from the indexing database the characters _, :: and -> in their names were replaced by - before storing them in the array $ref in Format.php. As this array is used to generate the links to and the index list of functions/methods, functions and methods with the same name after replacing the before mentioned characters (e.g. finfo_buffer()/finfo::buffer(), finfo_file()/finfo::file(), etc.) were overwriting each other's reference entries in memory (in the $ref array).

Fix this by not replacing the above listed characters.

@Girgias
Copy link
Member

Girgias commented May 13, 2024

Could you rebase to solve conflicts?

@haszi
Copy link
Contributor Author

haszi commented May 15, 2024

If you don't mind, I'll wait for #126 to be merged before rebasing.

@haszi haszi force-pushed the Fix-GH-3353-incorrect-function-tag-linking branch from a2a4589 to c3759b9 Compare May 20, 2024 14:56
@haszi
Copy link
Contributor Author

haszi commented May 20, 2024

Rebased and ready for review.

phpdotnet/phd/IndexRepository.php Show resolved Hide resolved
tests/data/bug_doc-en_GH-3353.xml Outdated Show resolved Hide resolved
@haszi haszi requested a review from Girgias May 21, 2024 20:22
@Girgias Girgias merged commit 8625d10 into php:master May 22, 2024
9 checks passed
@haszi haszi deleted the Fix-GH-3353-incorrect-function-tag-linking branch May 22, 2024 16:34
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.

See also <function> tag generates incorrect titleabbrv content
2 participants