[wp-trac] [WordPress Trac] #15627: wp_insert_user should not assume a default role

WordPress Trac wp-trac at lists.automattic.com
Tue Mar 22 11:52:05 UTC 2011


#15627: wp_insert_user should not assume a default role
-------------------------------------+-----------------------------
 Reporter:  nickmomrik               |       Owner:  westi
     Type:  defect (bug)             |      Status:  accepted
 Priority:  low                      |   Milestone:  Future Release
Component:  Users                    |     Version:  3.1
 Severity:  minor                    |  Resolution:
 Keywords:  needs-patch westi-likes  |
-------------------------------------+-----------------------------
Changes (by westi):

 * keywords:  3.2-early => needs-patch westi-likes
 * priority:  normal => low
 * severity:  normal => minor


Comment:

 If this had a patch on it I would consider it for 3.2

 Without a patch I am not going to prioritise working on it for now.

 I think we need to review all calls to wp_insert_user in core and set the
 role arg to the value of {{{get_option('default_role'))}}} when
 appropriate.

 Then we can remove:

 {{{
         elseif ( !$update )
                 $user->set_role(get_option('default_role'));
 }}}

 From the function itself.

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


More information about the wp-trac mailing list