NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
20 May 2021

This is simply applying the same fix as in #33803 but this time on logout.

Steps to reproduce the issue

To EASILY replicate, edit libraries/src/Session/SessionManager.php and put

throw new \Exception();
on line 65 in the destroySessions method, and then LOGOUT of Joomla admin to cause the crash

Pull Request for Issue #33741

This changes the scss so npm run build:css

avatar brianteeman brianteeman - open - 20 May 2021
avatar brianteeman brianteeman - change - 20 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 May 2021
Category Administration Templates (admin) NPM Change
avatar sandramay0905 sandramay0905 - test_item - 21 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 21 May 2021

I have tested this item successfully on 7d32a7a

With pull request error page looks like:

image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34061.
avatar PhilETaylor PhilETaylor - test_item - 21 May 2021 - Tested successfully
avatar PhilETaylor
PhilETaylor - comment - 21 May 2021

I have tested this item successfully on 7d32a7a

Works as expected

Screenshot 2021-05-21 at 11 21 29


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34061.
avatar richard67 richard67 - change - 21 May 2021
Status Pending Ready to Commit
Labels Added: ? NPM Resource Changed
avatar richard67
richard67 - comment - 21 May 2021

RTC


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

avatar Quy Quy - change - 21 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-21 21:43:38
Closed_By Quy
Labels Added: ?
avatar Quy Quy - close - 21 May 2021
avatar Quy Quy - merge - 21 May 2021
avatar Quy
Quy - comment - 21 May 2021

Thanks

avatar brianteeman
brianteeman - comment - 21 May 2021

Thank you

Add a Comment

Login with GitHub to post a comment