duplicated id="login_login_submitted" in login form

Bug #1770261 reported by Cecilia Vela Gurovic
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Cecilia Vela Gurovic

Bug Description

as a logged out user in home screen the login form is fine,
but if we add /?login to the url we can see that the id="login_login_submitted"
is duplicated in the DOM

It's easy to see in chrome since it shows and error in the browser console
[DOM] Found 2 elements with non-unique id #login_login_submitted:

Tags: code-cleanup
Changed in mahara:
milestone: none → 18.10.0
Revision history for this message
Cecilia Vela Gurovic (ceciliavg) wrote :

I can see we create the login form in a different way if it's not in a sideblock.
We call function auth_get_login_form from auth/lib.php that includes an extra 'login_submitted' pieform element but I'm not sure why we do that.

Changed in mahara:
status: New → Confirmed
tags: added: code-cleanup
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/9120

Changed in mahara:
status: Confirmed → In Progress
assignee: nobody → Cecilia Vela Gurovic (ceciliavg)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

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

commit 52661cb37dacf73245e7efef95401aedad3835d7
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Sep 7 18:24:21 2018 +1200

Bug 1770261: remove extra submit field in login form

behatnotneeded

Change-Id: Ie83774a85379f54b6de83cf68fffdfff21ec618f

Changed in mahara:
status: In Progress → Fix Committed
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.

Other bug subscribers

Remote bug watches

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