[wp-trac] [WordPress Trac] #10201: Switch roles to use single role, and no user-specific caps

WordPress Trac wp-trac at lists.automattic.com
Wed Dec 2 15:09:33 UTC 2009


#10201: Switch roles to use single role, and no user-specific caps
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |       Owner:                
     Type:  task (blessed)     |      Status:  assigned      
 Priority:  normal             |   Milestone:  Future Release
Component:  Role/Capability    |     Version:  2.8           
 Severity:  normal             |    Keywords:  2nd-opinion   
-------------------------------+--------------------------------------------

Comment(by apljdi):

 Replying to [comment:37 Denis-de-Bernardy]:
 > Replying to [comment:36 apljdi]:
 > > But done this way, there is no good reason for item 1. (...)
 >
 > actually, there is... usermeta and postmeta are managed inconsistently:
 a user cannot have multiple usermeta with the same key (yet).

 Right, I realize that. I've ran into the problem before in other contexts,
 but that is just a consequence of how the usermeta functions are written.
 There is nothing in the usermeta table itself that prevents it-- ie., no
 UNIQUE index. I avoided this problem in my testing by manually writing the
 INSERTs and SELECTs.


 > > Adding a new table might ramp up the performance but I haven't tested
 that, and I'm not sure the extra complexity is worth it.
 >
 > It totally is. And as pointed out by Sam, it's what they ended up doing
 in BBPress.

 I saw Sam's posts and actually I'm not averse to the idea-- I'd say I even
 favor it--, but there are also references on the thread to 'insane' people
 propagating the idea of extra tables and, like I said, I haven't had time
 to play with that aspect so I'm holding off on judgment until I do get the
 time.

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


More information about the wp-trac mailing list