[wp-trac] Re: [WordPress Trac] #569: Percent sign (%) in
WordPress Trac
wp-trac at lists.automattic.com
Sat Jul 8 07:14:48 GMT 2006
#569: Percent sign (%) in title leads to unusable permalinks
-----------------------------------------+----------------------------------
Reporter: MC_incubus | Owner: Kitten
Type: defect | Status: reopened
Priority: normal | Milestone:
Component: General | Version: 2.0.2
Severity: major | Resolution:
Keywords: bg|has-patch bg|2nd-opinion |
-----------------------------------------+----------------------------------
Changes (by majelbstoat):
* keywords: bg|has-patch => bg|has-patch bg|2nd-opinion
Comment:
Not sure this is useful as it stands. For me, all that this patch seems
to do is to make Firefox's status bar less readable:
/path/to/permalink/これは何ですか。/
which is a readable Japanese permalink in the status bar becomes:
/path/to/permalink/2006/07/05/%e3%81%93%e3%82%8c%e3%81%af%e4%bd%95%e3%81%a7%e3%81%99%e3%81%8b%ef%bd%a1/
which isn't so useful. In IE there is no difference, as with or without
this patch the URL is seemingly automatically encoded by the browser.
Even without the patch applied, Firefox automatically encodes the link
correctly. I haven't tried on other browsers, but I imagine that most
(all?) modern ones will behave nicely. Besides which doesn't
sanitize_title_with_dashes() already sort out the post-slug as it is
created?
In any case, this patch doesn't fix the problem as the above permalink
still gives a 404 for me on latest trunk even after it is applied. It
looks like the permalink containing octets isn't matching the regular
expression rules correctly, though I can't see why.
Jamie.
--
Ticket URL: <http://trac.wordpress.org/ticket/569>
WordPress Trac <http://wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list