Opened 21 months ago
Closed 21 months ago
#58678 closed defect (bug) (fixed)
Site Health: Escaped HTML in test label
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | 6.3 |
Component: | Site Health | Keywords: | has-patch has-screenshots |
Focuses: | Cc: |
Description
The Site Health test for writeable backup directories added in [55720] includes a check for the wp-content
directory path. When this test fails, the label for the result displays (<code>wp-content</code>)
, as these labels are HTML-escaped.
The linked PR would remove the HTML from the label. wp-content
still appears in the result description, and keeping the directory name out of the label is still consistent with at least some of the other labels in the test, such as The temporary backup directory exists but is not writable
.
Attachments (2)
Change History (13)
This ticket was mentioned in PR #4763 on WordPress/wordpress-develop by dlh01.
21 months ago
#1
- Keywords has-patch added
#4
follow-up:
↓ 8
@
21 months ago
By default, $wp_content
returns a path, like for instance /home/htdocs/wordpress/wp-content/
. Not sure we should display the whole path in the description?
@SergeyBiryukov commented on PR #4763:
21 months ago
#7
Thanks for the PR! Merged in r56117.
#8
in reply to:
↑ 4
;
follow-up:
↓ 10
@
21 months ago
Replying to audrasjb:
By default,
$wp_content
returns a path, like for instance/home/htdocs/wordpress/wp-content/
. Not sure we should display the whole path in the description?
I might be missing something, but I don't see the full path displayed anywhere in the description, could you clarify? :)
#10
in reply to:
↑ 8
@
21 months ago
Replying to SergeyBiryukov:
I might be missing something, but I don't see the full path displayed anywhere in the description, could you clarify? :)
My bad, I misunderstood the PR 🙃
Thanks @dlh for the ticket and PR. Look good to me and approved.
Added
commit
for consideration.