[wp-trac] [WordPress Trac] #35032: Shiny plugin updates fails if slug has special characters
WordPress Trac
noreply at wordpress.org
Sun Dec 13 11:36:38 UTC 2015
#35032: Shiny plugin updates fails if slug has special characters
-------------------------------------+------------------------------
Reporter: khag7 | Owner: adamsilverstein
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: Awaiting Review
Component: Upgrade/Install | Version:
Severity: normal | Resolution:
Keywords: has-patch needs-testing | Focuses:
-------------------------------------+------------------------------
Comment (by SergeyBiryukov):
> It will look odd to have sanitize_key used for escaping HTML attributes,
so a comment is recommended to explain why it is being used.
I'd suggest keeping the `esc_attr()` for clarity: `esc_attr( sanitize_key(
$plugin['slug'] ) )`.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/35032#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list