Opened 5 years ago
Closed 5 years ago
#48467 closed defect (bug) (fixed)
Update broken library links in PHP docblocks
Reported by: | mryoga | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | General | Keywords: | good-first-bug has-patch |
Focuses: | docs | Cc: |
Description
Update broken link
Attachments (1)
Change History (14)
#1
in reply to:
↑ description
@
5 years ago
#2
@
5 years ago
- Focuses docs added
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
Hi @mryoga,
Thanks for this ticket, and welcome to Trac!
Are you able to create a proper patch for this one so it can be tested/verified? There are some pages in the WordPress Core Contributor Handbook that will help.
When you refresh, please make sure to avoid the extra white space changes that were included in a few files.
Moving to Future Release, but it can be moved to 5.4 once patched.
#3
@
5 years ago
- Summary changed from Update broken link to Update broken library links in PHP docblocks
#5
@
5 years ago
when I scanned into the wordpress project I found a lot of broken links, like how I reported the problem
#7
follow-up:
↓ 11
@
5 years ago
@mryoga thank you for this ticket and the PR you provided as it was very helpful.
This patch addresses only the PHP files linked in the original Github PR here: https://github.com/WordPress/WordPress/pull/441
For the other files, these should be changed upstream in the original repos. I'll go ahead and submit PRs to those as well, and update this ticket with those links as well. As they are accepted, they will update WP Core on subsequent builds.
This ticket was mentioned in Slack in #core by sergey. View the logs.
5 years ago
#10
@
5 years ago
- Milestone changed from Future Release to 5.4
- Owner set to SergeyBiryukov
- Status changed from new to reviewing
#11
in reply to:
↑ 7
@
5 years ago
Replying to FinchPS:
This patch addresses only the PHP files linked in the original Github PR here: https://github.com/WordPress/WordPress/pull/441
For the other files, these should be changed upstream in the original repos. I'll go ahead and submit PRs to those as well, and update this ticket with those links as well. As they are accepted, they will update WP Core on subsequent builds.
Thanks for the patch! Just noting that PHPMailer is an external library too.
Looking at the original PR, only these files can be patched directly:
wp-admin/includes/class-wp-filesystem-ssh2.php js/_enqueues/vendor/imgareaselect/jquery.imgareaselect.js
jquery.imgareaselect.js
used to be an external library, but appears to be abandoned upstream and is treated as "adopted" in WordPress core.
Any changes to other libraries and components from the initial pull request should be submitted to the respective GitHub repositories:
Replying to mryoga:
my link pull request https://github.com/WordPress/WordPress/pull/441