[4.2] Correctly revert pull request no. 38244 for updating from 4.2.0 RC 1#38567
Conversation
Add back the update SQL script "4.2.0-2022-07-07.sql" which were added with PR joomla#38244 and released with 4.2.0-rc1 and then later removed before 4.2.0 stable when that PR was reverted, and comment out their content.
Add new update SQL script to correctly revert PR joomla#38244 when updating from a 4.2.0-rc1.
|
As the files were shipped with an rc dont they have to be removed as well? |
@brianteeman No. See the description for an explanation (I've updated it meanwhile to be more precise):
|
|
I meant the files for the fields not the files for the sql |
Hmm, wait, as I have added new scripts with a later version, I could indeed remove them on update (but then have to add them to script.php, too).
Oh yes, these should be removed on update. I will change this PR to draft / work in progress and later adjust it. Gotta do something else first. |
|
PR is ready for testing. See the updated description for my motivation to reintroduce the old update SQL scripts. In this special case here this would not have been necessary, but we had discussions with users who were confused when comparing their files system with a backup, and so I decided to do it the proper way, which also better for documentation of the changes. |
|
Thank you |
Pull Request for Issue #38554 (comment) .
Summary of Changes
With pull request (PR) #38244 , a new plugin "plg_fields_menuitem" was added, which has been released with 4.2.0 Release Candidate 1 on July 19.
This PR has been reverted after that on August 14 with this commit: 31a2677
But as we guarantee that an RC 1 can be correctly updated, it was not right and not sufficient just to remove the update SQL scripts "4.2.0-2022-07-07.sql".
Because it was always subject of discussions in past if old update SQL scripts can be touched or removed, this PR here adds the removed scripts "4.2.0-2022-07-07.sql" back and comment out their content for documentation. In this special case here this is not necessary because we add a newer update SQL script so we won't have a problem with the schema version. But in other cases deleting an update SQL script may cause problems, so here I keep the best practice to serve as an example, too.
In addition, this PR here adds new update SQL scripts "4.2.1-2022-08-23.sql" to remove the extension record when updating a 4.2.0-rc1. If the record is not there, the delete statement will not cause a failure, it will just not do anything.
Finally, this PR adds the obsolete files and folders from the reverted plugin to the lists of files and folders in script.php so they are deleted on update.
Testing Instructions
Update any version except of a 4.2.0-rc1 to 4.2.1-dev.
Update a 4.2.0-rc1 to 4.2.1-dev.
In both cases use the latest 4.2.1-dev nightly to get the actual result or use the update package or custom update URL created by drone for this PR to get the expected result.
After the update:
Actual result BEFORE applying this Pull Request
After update from any other version than 4.2.0-rc1 to 4.2.1-dev, there is no record for the "plg_fields_menuitem" in the extensions table.
After update from 4.2.0-rc1 to 4.2.1-dev, there is a record for the "plg_fields_menuitem" in the extensions table, and there are obsolete files and folders of the reverted plugin, see https://github.com/joomla/joomla-cms/pull/38567/files#diff-db7eb77540ff419bd7e6557d2779f4cf1e31158c20cb4b63dbbe8d6c426385adR6439-R6443 for the files.
Expected result AFTER applying this Pull Request
In any case there is no record for the "plg_fields_menuitem" in the extensions table, and in case of updating from 4.2.0-rc1 the obsolete files and folders are been removed.
Documentation Changes Required
None.