[wp-trac] [WordPress Trac] #57574: Add a Download row action to the Media List Table
WordPress Trac
noreply at wordpress.org
Sat Feb 4 07:05:35 UTC 2023
#57574: Add a Download row action to the Media List Table
---------------------------------------------+-----------------------------
Reporter: pbiron | Owner: audrasjb
Type: enhancement | Status: reopened
Priority: normal | Milestone: 6.2
Component: Media | Version:
Severity: normal | Resolution:
Keywords: has-screenshots needs-user-docs | Focuses: ui,
| administration
---------------------------------------------+-----------------------------
Changes (by audrasjb):
* keywords: has-patch has-screenshots needs-user-docs commit => has-
screenshots needs-user-docs
Comment:
Indeed, it is recommended to escape use `esc_url` here, as @costdev
pointed out [https://core.trac.wordpress.org/ticket/57541#comment:9 in
another ticket], see the function description:
> > eliminates invalid characters and removes dangerous characters
>
> So, even though esc_url() is mainly used for security, this also helps
when extenders return a value that has invalid characters. I guess this is
one reason why the pattern exists elsewhere in Core.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/57574#comment:31>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list