Open
Bug 1401962
Opened 7 years ago
Updated 7 years ago
Backout Bug 218917 (including schema change)
Categories
(Bugzilla :: Bugzilla-General, enhancement)
Tracking
()
NEW
People
(Reporter: dylan, Unassigned, Mentored)
Details
Attachments
(1 file)
In an effort to rebase off of BMO, we need the code and schema to be a bit in sync. In some cases BMO will adopt upstream's solutions, but the username stuff is totally different than how we need to handle accounts (because of multiple auth providers, privacy concerns and so on)
Reporter | ||
Updated•7 years ago
|
Assignee: dylan → general
Mentor: dylan
Reporter | ||
Comment 1•7 years ago
|
||
Okay, the backout is in https://github.com/bugzilla/bugzilla/pull/51. The schema change is going to go in Bugzilla/Install/DB.pm Around line 735 you'll add a call to a new function _restore_login_emails(); Then you'll define function, with a skeleton like this: sub _restore_login_emails { my $dbh = Bugzilla->dbh; my $params = Bugzilla->params; # code here } From there you'll need to check $params->{use_login_as_email}. If it is false, and if the column 'email' exists on the profiles table, set the login_name column to email Regardless of the value of use_login_as_email}, the email column needs to be dropped if it exists. There should be examples of this in the same file. Something like $dbh->bz_drop_column('profiles', 'email');
Comment 2•7 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•