[wp-trac] Re: [WordPress Trac] #9397: Quick edit creates new tag from existing tag name

WordPress Trac wp-trac at lists.automattic.com
Thu Apr 30 00:16:33 GMT 2009


#9397: Quick edit creates new tag from existing tag name
--------------------------+-------------------------------------------------
 Reporter:  forposts      |       Owner:  ryan       
     Type:  defect (bug)  |      Status:  new        
 Priority:  normal        |   Milestone:  2.8        
Component:  Quick Edit    |     Version:  2.7.1      
 Severity:  normal        |    Keywords:  needs-patch
--------------------------+-------------------------------------------------

Comment(by jhodgdon):

 I don't think it's the JS that is causing tag duplication in #6593. At
 least last time I looked at it (around when I first created #6593, so
 several versions ago) the problem was in the PHP code that is taking the
 form POST submission, and from that figuring out which tags the user meant
 to associate with the post (i.e. corresponding the POST submission to term
 IDs).

 That is why I think if the POST had the term IDs in it rather than trying
 to do text matching, we would avoid all these problems. That's what the
 categories section does, and it doesn't suffer from this problem.

 I haven't ever looked into what Quick Edit is doing, though, so I have no
 idea if the problem is in JS or PHP for this bug. Likely it is in the PHP
 -- when you submit the quick edit, the info is submitted via AJAX (I
 think) and it is definitely PHP code that saves the post.

 Then again, maybe you meant to say "same PHP being used in the background"
 rather than JS. :) Not sure where you are, but it's definitely the end of
 a long day here...

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


More information about the wp-trac mailing list