? ? Pending

User tests: Successful: Unsuccessful:

avatar YatharthVyas
YatharthVyas
28 Apr 2021

Pull Request for Issue #33378

Summary of Changes

Wrapped the form(s) for Account Details and Two Factor Authentication in <fieldset class="options-form"> <legend>
Added indentations on line 66 after if block opens and after uitab.addTab on lIne 67 to make the indentations consistent

Testing Instructions

Joomla Admin Panel -> Users -> Manage -> Click on a user to edit their profile
Verify that all forms follow a uniform style (fieldset with legends)

Note: You will have to enable 2FA to check the Two Factor Authentication tab or you can comment the following lines

<?php if (!empty($this->tfaform) && $this->item->id) : ?>

Actual result BEFORE applying this Pull Request

image
image

Expected result AFTER applying this Pull Request

image
image

Documentation Changes Required

None

avatar YatharthVyas YatharthVyas - open - 28 Apr 2021
avatar YatharthVyas YatharthVyas - change - 28 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Apr 2021
Category Administration com_users
avatar YatharthVyas YatharthVyas - change - 28 Apr 2021
The description was changed
avatar YatharthVyas YatharthVyas - edited - 28 Apr 2021
avatar YatharthVyas YatharthVyas - change - 28 Apr 2021
The description was changed
avatar YatharthVyas YatharthVyas - edited - 28 Apr 2021
avatar drmenzelit drmenzelit - test_item - 28 Apr 2021 - Tested successfully
avatar drmenzelit
drmenzelit - comment - 28 Apr 2021

I have tested this item successfully on 738c8f9


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

avatar ChristineWk
ChristineWk - comment - 28 Apr 2021

Made this: Note: You will have to enable 2FA to check the Two Factor Authentication tab ...

and maybe something wrong, as my Backend is broken now. Can't neither log out nor use Close button.

screen shot 2021-04-28 at 11 19 10

Will (or should) try to delete Plugin via phpMyAdmin. Never used 2FA before, but wanted to test this PR.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33389.
avatar rjharishabh rjharishabh - test_item - 28 Apr 2021 - Tested successfully
avatar rjharishabh
rjharishabh - comment - 28 Apr 2021

I have tested this item successfully on 738c8f9


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

avatar YatharthVyas
YatharthVyas - comment - 28 Apr 2021

Made this: Note: You will have to enable 2FA to check the Two Factor Authentication tab ...

and maybe something wrong, as my Backend is broken now. Can't neither log out nor use Close button.

I was curious as to why this might've occurred and decided to give it a try myself:
For some reason, I wasn't able to reproduce this, added a gif below:

2fa

Will (or should) try to delete Plugin via phpMyAdmin. Never used 2FA before, but wanted to test this PR.

Yes please try that! I hope it fixes your backend. I'm sorry for the inconvenience caused and thanks for testing!

avatar Kostelano Kostelano - test_item - 28 Apr 2021 - Tested successfully
avatar Kostelano
Kostelano - comment - 28 Apr 2021

I have tested this item successfully on 738c8f9


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

avatar drmenzelit drmenzelit - change - 28 Apr 2021
Status Pending Ready to Commit
avatar drmenzelit
drmenzelit - comment - 28 Apr 2021

RTC


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

avatar ChristineWk
ChristineWk - comment - 28 Apr 2021

@YatharthVyas
Thanks for your gif :-) Here my steps:

Authentication Methode selected: G Authenticator.
In Frontend: Logged In with the Security Code (Key) successfully.
But in Backend: Still no Log Out button and Close button didn‘t work.

Therefore I was searching then in phpMyAdmin for the correct table …..

plg_twofactorauth_totp = from enabled (1) changed to (0)
Backend was available again.

Under Users: Selected: Enforce Two Factor Authentication from „both“ back to: „No“
Guess won't attack 2FA anytime soon :-)

avatar chmst chmst - close - 28 Apr 2021
avatar chmst chmst - merge - 28 Apr 2021
avatar chmst chmst - change - 28 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-28 19:49:32
Closed_By chmst
Labels Added: ? ?
avatar chmst
chmst - comment - 28 Apr 2021

Thanks!

Add a Comment

Login with GitHub to post a comment