Mahara

Unable to remove first auth method via institution edit

Bug #1595364 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Robert Lyon
15.10
Fix Released
Medium
Unassigned
Mahara 15.10.6
16.04
Fix Released
Medium
Unassigned
Mahara 16.04.4
16.10
Fix Released
Medium
Robert Lyon
Mahara 16.10.0

Bug Description

There is a bug in the code that stops one from removing the first option from auth methods for an institution

This is due to the way we are using jQuery.remove()

Get the console error:

TypeError: elem is null
while ((child = elem.firstChild)) {

Due to: instances.remove(i)

when i = 1 or above it removes the row relating to the count before rebuilding the rows but when i = 0 it removes all the rows and so nothing to rebuild

Will fix this up so it actually removes the row relating to the id

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/6612

Kristina Hoeppner (kris-hoeppner)
Changed in mahara:
milestone: none → 16.10.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/6612
Committed: https://git.mahara.org/mahara/mahara/commit/bcfc376633246f4b0c29b5c6b1acca493ee51f04
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit bcfc376633246f4b0c29b5c6b1acca493ee51f04
Author: Robert Lyon <email address hidden>
Date: Thu Jun 23 13:41:09 2016 +1200

Bug 1595364: removing the correct row in js

Rather than relying on loop

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

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

Patch for "16.04_STABLE" branch: https://reviews.mahara.org/6822

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "15.10_STABLE" branch: https://reviews.mahara.org/6823

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/6822
Committed: https://git.mahara.org/mahara/mahara/commit/0b3bb84830e574caf7e8e0ea2e85e6acee8eabf7
Submitter: Aaron Wells (<email address hidden>)
Branch: 16.04_STABLE

commit 0b3bb84830e574caf7e8e0ea2e85e6acee8eabf7
Author: Robert Lyon <email address hidden>
Date: Thu Jun 23 13:41:09 2016 +1200

Bug 1595364: removing the correct row in js

Rather than relying on loop

Change-Id: I4ff09cdd056cedb927a26fa6c0ba33ccb83d5ebc
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit bcfc376633246f4b0c29b5c6b1acca493ee51f04)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/6823
Committed: https://git.mahara.org/mahara/mahara/commit/5f4bcab33ed02896d656e1ddc566a0cb200ff883
Submitter: Aaron Wells (<email address hidden>)
Branch: 15.10_STABLE

commit 5f4bcab33ed02896d656e1ddc566a0cb200ff883
Author: Robert Lyon <email address hidden>
Date: Thu Jun 23 13:41:09 2016 +1200

Bug 1595364: removing the correct row in js

Rather than relying on loop

Change-Id: I4ff09cdd056cedb927a26fa6c0ba33ccb83d5ebc
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit bcfc376633246f4b0c29b5c6b1acca493ee51f04)

Robert Lyon (robertl-9)
Changed in mahara:
milestone: 16.10.0 → none
Robert Lyon (robertl-9)
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.
Subscribing...

Other bug subscribers

Remote bug watches

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