? ? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
20 Aug 2021

Pull Request for Issue #35199

Summary of Changes

[3.10] Backport Add folder permissions check for com_joomlaupdate from @PhilETaylor

Add folder permissions check for administrator/components/com_joomlaupdate

This is important as Joomla will write a restoration.php file in this folder during Joomla Core Upgrade (this file contains the kickstart.security.password and other params which is used by the restore.php extraction process.

If this folder is not writable then you can see ajax errors like Access Denied

Testing Instructions

Apply this patch
Go to System -> System Information -> Permissions

Actual result BEFORE applying this Pull Request

No check for com_joomlaupdate write permissions

Expected result AFTER applying this Pull Request

There is a check for com_joomlaupdate write permissions

Documentation Changes Required

none

avatar zero-24 zero-24 - open - 20 Aug 2021
avatar zero-24 zero-24 - change - 20 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Aug 2021
Category Administration com_admin
avatar ChristineWk ChristineWk - test_item - 21 Aug 2021 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 21 Aug 2021

I have tested this item successfully on 5f09b84


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

avatar richard67 richard67 - test_item - 21 Aug 2021 - Tested successfully
avatar richard67
richard67 - comment - 21 Aug 2021

I have tested this item successfully on 5f09b84

New check is there and works (tested with chmod on Linux).


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

avatar richard67 richard67 - change - 21 Aug 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 21 Aug 2021

RTC


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

avatar zero-24 zero-24 - close - 22 Aug 2021
avatar zero-24 zero-24 - merge - 22 Aug 2021
avatar zero-24 zero-24 - change - 22 Aug 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-08-22 10:33:59
Closed_By zero-24
Labels Added: ? ?
avatar zero-24
zero-24 - comment - 22 Aug 2021

Merging thanks

Add a Comment

Login with GitHub to post a comment