Skip to:

Opened 11 years ago

Closed 11 years ago

#495 closed enhancement (fixed)

make bb_new_user() pluggable

Reported by: so1o Owned by: so1o
Milestone: 0.8 Priority: high
Severity: normal Version: 0.7.3
Component: General - Administration Keywords: has-patch


the new user creation process is currently very rigid and cannot be overridden without making changes to the core files.

Attachments (1)

ticket-495.patch (3.3 KB) - added by so1o 11 years ago.
pluggable bb_new_user and change to return user_id always

Download all attachments as: .zip

Change History (12)

#2 @mdawaffe
11 years ago

  • Milestone set to 0.80

#3 @mdawaffe
11 years ago

  • Priority changed from normal to high

#4 @so1o
11 years ago

  • Owner set to so1o
  • Status changed from new to assigned

#5 @so1o
11 years ago

any other ideas guys.. im running out of steam here..

#6 @so1o
11 years ago

  • Keywords has-patch added

#7 @mdawaffe
11 years ago

I would probably just put bb_create_user() in bb-includes/pluggable.php so that it can be replaced by a plugin rather than using a filter to accomplish that. We only need it to be replaced by one function, we don't need the ability to hook many functions onto the filter.

Also, does bb_new_user() ever need to return both the user_id and the password? I'd rather it only returned the user_id even when BB_INSTALLING. We can rewrite the install script to take advantage of the bb_new_user hook to grab the password.

#8 @so1o
11 years ago

if we put it in pluggable.. nope we wouldnt need to return the username and pass

11 years ago

pluggable bb_new_user and change to return user_id always

#9 @so1o
11 years ago

how does this look..

#10 @mdawaffe
11 years ago

Looks great!

#11 @mdawaffe
11 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [638]) pluggable bb_create_user() from so1o. Fixes #495.

Note: See TracTickets for help on using tickets.