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

Broken link on WP_SimplePie_Sanitize_KSES #919

Closed
crstauf opened this issue Jul 3, 2023 · 4 comments
Closed

Broken link on WP_SimplePie_Sanitize_KSES #919

crstauf opened this issue Jul 3, 2023 · 4 comments
Assignees
Labels
[Status] Done Issue is completed

Comments

@crstauf
Copy link

crstauf commented Jul 3, 2023

Issue Description

The link to the “SimplePie_Sanitize” class in the “See Also” section of the “wp_simplepie_sanitize_kses” documentation is broken.

The correct URL for the “SimplePie_Sanitize” class is {80% SURE}:

https://developer.wordpress.org/reference/classes/wp_simplepie_sanitize_kses/sanitize/

Please fix this broken link.

Thank you.

Reported by luckybhumkar via User Contributed Notes.

URL of the Page with the Issue

https://developer.wordpress.org/reference/classes/wp_simplepie_sanitize_kses/#see-also

Section of Page with the issue

See Also

Why is this a problem?

Broken link.

Suggested Fix

Change URL to https://developer.wordpress.org/reference/classes/wp_simplepie_sanitize_kses/sanitize/ (80% confidence).

@crstauf crstauf added the [Status] To do Issue marked as Todo label Jul 3, 2023
@stevenlinx
Copy link
Member

Thanks for filing.

However, the See Also section belongs to Core Trac
https://github.com/WordPress/wordpress-develop/blob/6.2/src/wp-includes/class-wp-simplepie-sanitize-kses.php#L18

@stevenlinx stevenlinx self-assigned this Jul 31, 2023
@github-actions github-actions bot added [Status] In progress Issue is in progress and removed [Status] To do Issue marked as Todo labels Jul 31, 2023
@github-actions github-actions bot added [Status] Done Issue is completed and removed [Status] In progress Issue is in progress labels Aug 8, 2023
@crstauf
Copy link
Author

crstauf commented Aug 8, 2023

@stevenlinx In what commit was this fixed? Or is there ticket on Trac?

@stevenlinx
Copy link
Member

@crstauf
The "See Also section" comes from an inline doc line in the source code.
This is not the domain of DevHub.
So you'll need to file a ticket over at Core Trac to fix this.

@crstauf
Copy link
Author

crstauf commented Aug 9, 2023

Reported in Trac #59030.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Done Issue is completed
Projects
None yet
Development

No branches or pull requests

2 participants