? Release Blocker NPM Resource Changed ? Pending

User tests: Successful: 0 Unsuccessful: 0

avatar fancyFranci
fancyFranci
18 Aug 2022

Pull Request for Issue #38513.

Summary of Changes

Going to the joomla updater can lead to a js error because an optional element can not be found. This PR checks the element before it is used.

Testing Instructions

Short: Follow the instructions in the linked issue.
Long:
Open the developer tools of your browser to see the console.
Go to System -> Global Configuration -> Joomla! Update.
Try out different channels.
Everytime you click "Save & Close" you are redirected to the update page.
There you can click buttons like "Reinstall Joomla! core files"

Actual result BEFORE applying this Pull Request

An js error saying "TypeError: updateCheck is null" appears in the browser console. Depending on your debug settings it could say "a is null" or something like this. Buttons are not working.

Expected result AFTER applying this Pull Request

There is no js error and buttons should work as always.

Documentation Changes Required

Nope.

avatar fancyFranci fancyFranci - open - 18 Aug 2022
avatar fancyFranci fancyFranci - change - 18 Aug 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Aug 2022
Category JavaScript Repository NPM Change
avatar zero-24
zero-24 - comment - 18 Aug 2022

I have tested this item successfully on 65394e0


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

avatar zero-24 zero-24 - test_item - 18 Aug 2022 - Tested successfully
avatar richard67
richard67 - comment - 18 Aug 2022

I have tested this item successfully on 65394e0

I did not get a JS error in my browser console without this PR. But I can confirm that without this PR, the button to reinstall the core files doesn't work, and with this PR it does work.


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

avatar richard67 richard67 - test_item - 18 Aug 2022 - Tested successfully
avatar richard67 richard67 - change - 18 Aug 2022
Status Pending Ready to Commit
Labels Added: NPM Resource Changed Release Blocker ?
avatar roland-d roland-d - change - 18 Aug 2022
Status Ready to Commit Pending
Labels
avatar richard67
richard67 - comment - 18 Aug 2022

RTC


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

avatar richard67 richard67 - change - 18 Aug 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 18 Aug 2022

RTC !


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

avatar roland-d roland-d - change - 18 Aug 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-08-18 18:46:34
Closed_By roland-d
Labels Added: ?
avatar roland-d roland-d - close - 18 Aug 2022
avatar roland-d roland-d - merge - 18 Aug 2022
avatar roland-d
roland-d - comment - 18 Aug 2022

Thank you

Add a Comment

Login with GitHub to post a comment