Adding user to new institution changes auth method for all users

Bug #1785514 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon

Bug Description

When testing on master I did the following:

1) Upgrade an existing site from 16.10 -> Master

2) Checked the auth instance of all of the users (it was 1 = mahara / internal)

3) Created a new institution and added 'webservice' auth and deleted internal auth and saved

4) Checked the auth_instance table - there was only mahara/internal and newinstitution/webservice options

5) On admin user edit page I add a user to the new institution - they go from mahara/internal to newistitution/internal auth.

6) On checking the auth_instance table there is now a newinstitution/internal option - which is correct as the user will still need internal auth

But the problem comes when I check the usr table - all the users have this new newinstitution/internal option no matter what institution they are currently in

There is something wrong with adding user to institution

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/9046

Robert Lyon (robertl-9)
Changed in mahara:
status: Confirmed → In Progress
assignee: nobody → Robert Lyon (robertl-9)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/9046
Committed: https://git.mahara.org/mahara/mahara/commit/6b1e661837a1737b0ab9d67750c320dada49a82d
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit 6b1e661837a1737b0ab9d67750c320dada49a82d
Author: Robert Lyon <email address hidden>
Date: Mon Aug 6 10:04:38 2018 +1200

Bug 1785514: Only update auth instance for this user

When we add a user to an institution that doesn't have internal auth
on the admin/users/edit.php page

behatnotneeded

Change-Id: I7395d1481b15c0e1e82eec0a61b3cca891df72f9
Signed-off-by: Robert Lyon <email address hidden>

Changed in mahara:
status: In Progress → Fix Committed
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.