[wp-trac] Re: [WordPress Trac] #7540: allow multiple values for key in usermeta

WordPress Trac wp-trac at lists.automattic.com
Tue Apr 21 11:30:44 GMT 2009


#7540: allow multiple values for key in usermeta
--------------------------------------------------+-------------------------
 Reporter:  wnorris                               |        Type:  enhancement
   Status:  new                                   |    Priority:  normal     
Milestone:  2.8                                   |   Component:  Users      
  Version:  2.6.1                                 |    Severity:  normal     
 Keywords:  needs-patch 2nd-opinion dev-feedback  |  
--------------------------------------------------+-------------------------
Changes (by Denis-de-Bernardy):

  * keywords:  needs-patch => needs-patch 2nd-opinion dev-feedback


Comment:

 adding to the previous comment, this line in update_user_meta():

         $wpdb->update( $wpdb->usermeta, $data, $where );

 is plain wrong, in the sense that it would need to do:

  - if single, make sure a single key is left in there
  - if multi, make sure all keys are marked as not single

 but I'd rather get a 2nd opinion/dev feedback before things move any
 further. the likelyhood is this ticket may end up as wontfix due to the
 huge potential for bugs.

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


More information about the wp-trac mailing list