[wp-trac] [WordPress Trac] #8418: Sample permalink URL-encoded
WordPress Trac
wp-trac at lists.automattic.com
Fri Nov 28 22:35:33 GMT 2008
#8418: Sample permalink URL-encoded
----------------------------+-----------------------------------------------
Reporter: janbrasna | Owner: anonymous
Type: enhancement | Status: new
Priority: normal | Milestone: 2.7
Component: Administration | Version: 2.7
Severity: minor | Keywords: permalink admin preview utf urlencoded has-patch needs-feedback
----------------------------+-----------------------------------------------
When using a non-latin post title, the previewed permalink appears URL-
encoded (see $ATT). Since it's just an informative display of the slug, I
believe that a non-encoded readable format would be more helpful for users
to make them able to check the slug.
It would also fix a bug (probably jquery-related somewhere in slug.js)
when an URL-encoded slug like "%e6%88%90%e7%9…" can't be edited inline,
since the input doesn't get populated by such slug.
A simple fix is to add a default filter for the 'editable_slug' hook, see
the attached patch. (This fixes the same issue for taxonomy slug edits
etc. as well.)
(Since no characters that would require converting to special chars are
ever produced by the permalink routine, I assume it's safe to URL-decode
the string and leaving it then as-is for HTML output.)
--
Ticket URL: <http://trac.wordpress.org/ticket/8418>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list