? ? Pending

User tests: Successful: Unsuccessful:

avatar uglyeoin
uglyeoin
29 Apr 2019

… can have multiple login forms.

Pull Request for Issue # .

Summary of Changes

Added the module ID to the HTML ID making it unique and allowing for multiple login forms.

Testing Instructions

Add a login module to the front end
Inspect the source code and see that the login module has an id of id="login-form"
Install the patch
Inspect the login form and see that the module now has an ID with a module number added to the end of it. e.g. id="login-form-16"

Expected result

ID's should always be unique

Actual result

ID is static but module can be replicated.
You see the correct module ID attached to the ID making it unique

Documentation Changes Required

n/a

avatar uglyeoin uglyeoin - open - 29 Apr 2019
avatar uglyeoin uglyeoin - change - 29 Apr 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Apr 2019
Category Modules Front End
avatar franz-wohlkoenig franz-wohlkoenig - change - 29 Apr 2019
Title
Added the module ID to the forms HTML id so that it is unqiue and you…
[4.0] Added the module ID to the forms HTML id so that it is unqiue and you…
avatar franz-wohlkoenig franz-wohlkoenig - edited - 29 Apr 2019
avatar sanderpotjer
sanderpotjer - comment - 4 May 2019

I have tested this item successfully on 6a0a65f


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24757.

avatar sanderpotjer sanderpotjer - test_item - 4 May 2019 - Tested successfully
avatar alikon
alikon - comment - 5 May 2019

I have tested this item successfully on 6a0a65f


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24757.

avatar alikon alikon - test_item - 5 May 2019 - Tested successfully
avatar alikon alikon - change - 5 May 2019
Status Pending Ready to Commit
avatar alikon
alikon - comment - 5 May 2019

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24757.

avatar SniperSister SniperSister - change - 5 May 2019
Labels Added: ?
avatar wilsonge wilsonge - change - 5 May 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-05 11:55:58
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 5 May 2019
avatar wilsonge wilsonge - merge - 5 May 2019
avatar wilsonge
wilsonge - comment - 5 May 2019

Thanks!

Add a Comment

Login with GitHub to post a comment