User tests: Successful: Unsuccessful:
Pull Request for Issue # .
This pull request (PR) removes the methods "contactItems" and "finderItems" and their usage in the "postflight" method from file "administrator/components/com_admin/script.php".
The removed methods and the removed code in the "postflight" method once had been added with version 4.0.0 with PR #26540 and then had been modified and extended with PR #27032 for adding missing admin menu items of com_contact and com_finder when updating from 3.10 to 4.x.
The PR is one in a sequence of PR's for removing obsolete actions from script.php which are not relevant anymore because updates to 5.x will have to be done from a 4.4 version, like it is now with 4.x from 3.10.
When this PR here and PR #40281 have been merged, the "postflight" method will not do anything anymore.
I will make a follow up PR when that has happened to change the version number in the version_compare
call from 4.0.0
to 5.0.0
here https://github.com/richard67/joomla-cms/blob/5.0-dev/administrator/components/com_admin/script.php#L885-L887 and below that add a comment like // Add here code which shall only be performed when updating from 4.4 to 5.x
or so.
The "postflight" method should be kept for such future use, I think.
Other opinions are welcome.
Code review: Make sure that the removed methods are not called anywhere else and that they do something which is definitely not needed when updating from 4.4 to 5.
Optional real test: Make sure that updating the latest 4.4-dev nightly build to the update package created by drone works when using the upload & update method.
Obsolete code for adding admin menu items of com_contact and com_finder when updating from 3.10 to 4.x is present in script.php.
Obsolete code for adding admin menu items of com_contact and com_finder when updating from 3.10 to 4.x is not present anymore in script.php.
Updating from a 4.4-dev nightly build works.
Please select:
No documentation changes for docs.joomla.org needed
No documentation changes for manual.joomla.org needed
Category | ⇒ | Administration com_admin |
Status | New | ⇒ | Pending |
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-04-02 07:34:28 |
Closed_By | ⇒ | HLeithner | |
Labels |
Added:
PR-5.0-dev
|
thanks