Opened 6 years ago
Closed 6 years ago
#495 closed enhancement (fixed)
make bb_new_user() pluggable
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | high | Milestone: | 0.8 |
| Component: | Administration | Version: | 0.7.3 |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
the new user creation process is currently very rigid and cannot be overridden without making changes to the core files.
Attachments (1)
Change History (12)
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.
if we put it in pluggable.. nope we wouldnt need to return the username and pass
comment:10
mdawaffe — 6 years ago
Looks great!
comment:11
mdawaffe — 6 years ago
- Resolution set to fixed
- Status changed from assigned to closed
Relation Discussion