[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