User tests: Successful: Unsuccessful:
This PR adds an additional section to the installation process, allowing users to opt out from automated updates. Using that option will prevent a site from registering with the service in the first place, ensuring that no data is submitted.
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | Installation Language & Strings JavaScript |
Labels |
Added:
Language Change
PR-5.4-dev
|
Labels |
Added:
Feature
|
Tested with new instructions.
With patch 45696 the new installation seemed to be as expected. Auto-update was not enabled on installation.
With patch 45697 and Disable Automated Updates not selected the installation was normal. However:
With patch 45697 and Disable Automated Updates selected there was a snag: the Automated Updates box disappeared but the Install Additional Languages button and the Remove Installation Folder button were both greyed. The Open Administrator button worked but I can see that the installation folder is still present.
@ceford do make sure I got that correclty: the installation and languages button were disabled when you hit the "disabled auto updates" button?
Yes, as shown in the screen shots. In the second one the Open Site and Open Administrator buttons look dim but I am fairly sure they were not disabled. I logged in as Admin OK. I just logged in to check and Automatic Updates are disabled.
I have updated the branch to get progress on this PR, to be included within beta1, but PHPStan fails with Instantiated class JConfig not found.
Please assist.
I have tested this item ✅ successfully on 27dfd62
Beforehand: I could not reproduce the issue with the disabled 'Remove installation folder' button. I removed the installation folder each time.
Tested with manual installation from full package on local site
Tested with manual installation from full package on public site
I have updated the branch to get progress on this PR, to be included within beta1, but PHPStan fails with Instantiated class JConfig not found. Please assist.
Fixed
@ceford could you please test again? To see Automated Updates enabled, it is important to test from a publicly available site. If you encounter the disabled button issue again, could you please provide more details, such as the browser used, as I had no problems deleting the installation folder using the given button. Perhaps the save way for installation is from the full package. Thank you.
I've updated the links to packages in the description of this PR so testers can be sure to have packages which are up to date with latest 5.4-dev.
I have tested this item 🔴 unsuccessfully on c79fb16
If I proceed with the buttons from top to bottom, i.e. first opt out from the auto updates and then delete the installation folder, it works fine.
But if I decide to first use the button to delete the installation folder and then the button to opt out from the automated updates, then the latter button does not have any effect, and in network analysis of browser developer tools I see a POST request to https://test3.richard-fath.de/installation/index.php?task=installation.disableAutomatedUpdates&format=json
with a 404 result.
The button for installing additional languages is removed (or hidden) when the button for deleting the installation folder has been used and the installation folder has been deleted.
This is correct as that button will not work after the installation folder has been removed.
We need the same logic for removing (or hiding) the new button to opt out from the auto updates.
P.S.: I think that it is here where the buttons are disabled after deleting the installation folder: https://github.com/joomla/joomla-cms/pull/45697/files#diff-d9f8a6c89739f0469e6ad48530ab28a41675604c9aad1d871c83a5f3f8032285R109-R121
@richard67 good catch, fix has been pushed
@SniperSister Still not perfect. Now the button for opt out is deactivated, but in opposite to the additional languages installation thing not hidden. I will later look if I can find the reason.
I have tested this item ✅ successfully on e23dc6f
I've also tested this additional test case:
At the final page, directly use the button to remove the installation folder.
Result: Anything related to the languages installation, the new option added by this PR here and the button to remove the installation folder are removed from the page because all that stuff cannot be used after the installation folder has been removed, and only the buttons for going to administrator or site remain.
This is the existing behaviour without this PR. The PR does not change anything on it, it only makes sure the new option is also handled like this.
Maybe not good UX, but nothing to be fixed with this PR here.
@ceford Could you test this PR again? Just use the installation package created for this PR by drone. I've updated the link in the description. No need to apply any other PR in addition. The actual result without this PR you don't need to test again. The issue you had should be solved. Thanks in advance.
There is still something not quite right! I installed the installation package with the PR linked in the instructions and stepped through the process. This time, after using Disable Automatic Updates, when I selected Remove Installation Folder the response screen still had a 'We detected development mode' message and the red 'Remove "Installation" folder' greyed out and I can see the installation folder has not been deleted. I can go on to Site or Admin screens.
@ceford Is is a Windows server? If so, this is due to issue #37909 . That is the expected behaviour for that case. If everything else was working for you, I would call it a successful test. But if it is Linux then there might be a permission issue when deleting the installation folder. That would have the same result.
@ceford Is is a Windows server? If so, this is due to issue #37909 . That is the expected behaviour for that case. If everything else was working for you, I would call it a successful test. But if it is Linux then there might be a permission issue when deleting the installation folder. That would have the same result.
It is a Linux server. My own VPS with WHM and several cPanel accounts. The installation folder has 0775 permissions - exactly the same as all other folders.
I have tested this item 🔴 unsuccessfully on e23dc6f
I can reprpduce @ceford 's finding. Procedure:
@richard67 took a closer look at the button problem and suggested a bunch of changes, thank you!
Please retry.
I have tested this item ✅ successfully on 04ca14d
Now it works as expected.
@ceford Sorry to bother you again. Fist of all thanks for your help and your patience. Now with the latest fix it id working for me as expected. I've updated the link to the package in the testing instructions so you will get the newest one with these changes. Could you test again? That would be great. Thanks in advance.
I have tested this item ✅ successfully on 04ca14d
My previous test used my installation in a subfolder (not inside another joomla installation) because I forgot my set up. This one uses the subdomain I previously set up. I t now works as expected.
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
RTC
|
✅ Final test before merge with multiple manual full package installation on public site (always disabling AU afterwards)
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-08-15 04:32:55 |
Closed_By | ⇒ | muhme |
Thank you @SniperSister and @brianteeman for your contribution. Thank you @ceford and @richard67 for repeated testing.
I think the test instructions need some clarification. What I did seemed to work first time but I needed to check and what I did the second time did not work (the Autoupdates was still enabled). My procedure:
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45697.