[wp-trac] [WordPress Trac] #17343: update_post_meta removes all slashes

WordPress Trac wp-trac at lists.automattic.com
Thu Sep 29 06:48:36 UTC 2011


#17343: update_post_meta removes all slashes
------------------------------+-----------------------
 Reporter:  5ubliminal        |       Owner:
     Type:  defect (bug)      |      Status:  reopened
 Priority:  normal            |   Milestone:  3.3
Component:  Plugins           |     Version:  3.1.1
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+-----------------------
Changes (by nacin):

 * keywords:   => has-patch commit
 * milestone:  Awaiting Review => 3.3


Comment:

 Replying to [comment:4 5ubliminal]:

 You do this all of the time. It's getting rather old. It's difficult
 enough keeping this on Twitter. Can you please confine it there? Some
 people here are trying to work.

 There are something like 3000 open tickets right now. Dozens opened every
 week. When they get triaged, small details sometimes get missed. It
 happens. This does not mean there is a conspiracy. There is no cartel. If
 there is, I'm certainly not a member, let alone the leader. (That'd be
 cool, though, right? Running a bug-hunting cartel? I digress.)

 You could have commented five months ago, offering clarification, but you
 didn't. scribu, a different contributor than the original reviewer, still
 didn't see it at first glance. Instead of clarifying -- in your first
 comment to your own ticket -- you went on a rant.

 When I saw this ticket, I missed it just the same. But looking at the code
 with scribu, yes, we now see the bug. I've committed unit tests to confirm
 the bug ([UT445]), and [attachment:17343.diff] fixes it. I'll leave it for
 ryan's review in the morning.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/17343#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list