PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
1 Apr 2023

Pull Request for Issue # .

Summary of Changes

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.0to 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.

Testing Instructions

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.

Actual result BEFORE applying this Pull Request

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.

Expected result AFTER applying this Pull Request

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.

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 1 Apr 2023
Category Administration com_admin
avatar richard67 richard67 - open - 1 Apr 2023
avatar richard67 richard67 - change - 1 Apr 2023
Status New Pending
avatar richard67 richard67 - change - 1 Apr 2023
The description was changed
avatar richard67 richard67 - edited - 1 Apr 2023
avatar richard67 richard67 - change - 1 Apr 2023
The description was changed
avatar richard67 richard67 - edited - 1 Apr 2023
avatar richard67 richard67 - change - 1 Apr 2023
The description was changed
avatar richard67 richard67 - edited - 1 Apr 2023
avatar HLeithner HLeithner - change - 2 Apr 2023
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
avatar HLeithner HLeithner - close - 2 Apr 2023
avatar HLeithner HLeithner - merge - 2 Apr 2023
avatar HLeithner
HLeithner - comment - 2 Apr 2023

thanks

Add a Comment

Login with GitHub to post a comment