[wp-trac] [WordPress Trac] #11083: wp_posts.post_name gets too long to save in DB, after repeated updates
WordPress Trac
wp-trac at lists.automattic.com
Fri Nov 6 10:33:43 UTC 2009
#11083: wp_posts.post_name gets too long to save in DB, after repeated updates
----------------------------+-----------------------------------------------
Reporter: knutsp | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 2.9
Component: Administration | Version: 2.8.5
Severity: normal | Keywords: reporter-feedback
----------------------------+-----------------------------------------------
Comment(by knutsp):
Installed WordPress 2.9 and I can no longer reproduce this behaviour
(growth, the main reported bug).
Details:
Version 2.8.5: The character ø (o-with-slash) gets encoded to %c3%b8 in
the first save. The next save encodes %c3%b8 to %25c3%25b8 (as you can see
the %-sign gets re-encoded.
Version 2.9: The character ø (o-with-slash) gets encoded to %c3%b8 in the
first save. The next save keeps %c3%b8 with no additional encoding.
It seems fixed nicely. Before submitting, I tried to do a search for
tickets addressing such an issue, but could not find any.
But there may still be situations where this field, or a field, is allowed
to grow, by applying encoding/escaping for "illegal" characters. Stings
longer than the database defined maximum length should not be allowed to
enter a SQL INSERT or SQL UPDATE. At least as long as such exceptions are
not handeled properly.
May be close this an open another ticket when the second issue is
investigated further. I have no idea on the what checks are made on string
lengths, inside or outside $wpdb or other classes/functions.
The third issue (no error message on failed save) seems to me to be quite
serious.
I leave it up to you how to close, split or re-summarize this ticket,
according to best practices here.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/11083#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list