PR-5.0-dev b/c break Removal Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
3 Jun 2023

Pull Request for Issue # .

Summary of Changes

This pull request (PR) removes the "uninstallEosPlugin" function from script.php and the one call to in the "update" function of the same file.

The function is obsolete in the 5.0 core because it handles uninstallation of the 3.10 EOS plugin, which is already uninstalled when updating from 3.10 to 4.x, and beginning with 4.4 and 5.0 there is a permanently installed EOS plugin which has been introduced with PR #40565 so it doesn't need to keep that function.

Possible B/C break to be approved by release managers

In opposite to previously removed, obsolete code in script.php for 5.0, this time the removed function is not private but protected.

That means that theoretically the method could be overridden in the installation scripts of 3rd party extension developers with code for doing own stuff when their scripts is based on the "JoomlaInstallerScript" class of our script.php.

But the name of that method is so specific to the core that I don't really think anybody has ever done that.

Testing Instructions

Code review.

Actual result BEFORE applying this Pull Request

File "administrator/components/com_admin/script.php" contains the obsolete function "uninstallEosPlugin" which is called only at one place in the public function "Update" of the same file.

Expected result AFTER applying this Pull Request

File "administrator/components/com_admin/script.php" doesn't contain anymore the obsolete function "uninstallEosPlugin" and any call to it.

Link to documentations

Not sure if it needs to be documented somewhere for the "JoomlaInstallerScript" class on manual.joomla.org that this method has been removed.

Please select:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 3 Jun 2023
Category Administration com_admin
avatar richard67 richard67 - open - 3 Jun 2023
avatar richard67 richard67 - change - 3 Jun 2023
Status New Pending
avatar richard67 richard67 - change - 3 Jun 2023
The description was changed
avatar richard67 richard67 - edited - 3 Jun 2023
avatar sandewt
sandewt - comment - 3 Jun 2023

Should / can the language strings of the eos310 plugin also be removed yet for J5? What is the procedure for this?

See:

; Below is a copy of the 3.10 text definitions for reference.

avatar richard67
richard67 - comment - 3 Jun 2023

Should / can the language strings of the eos310 plugin also be removed yet for J5? What is the procedure for this?

See:

; Below is a copy of the 3.10 text definitions for reference.

I think that either can be done in 4.4-dev and then be merged up into 5.0-dev because they are obsolete and only used by the core. But if we are strict regarding removal of language strings we might have to deprecate them now and remove with 6.0.

=> @HLeithner Any suggestions?

avatar sandewt sandewt - test_item - 3 Jun 2023 - Tested successfully
avatar sandewt
sandewt - comment - 3 Jun 2023

I have tested this item successfully on 1fd0b2d


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

avatar HLeithner HLeithner - change - 7 Jun 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-06-07 17:46:47
Closed_By HLeithner
Labels Added: PR-5.0-dev b/c break Removal
avatar HLeithner HLeithner - close - 7 Jun 2023
avatar HLeithner HLeithner - merge - 7 Jun 2023
avatar HLeithner
HLeithner - comment - 7 Jun 2023

Thanks

Add a Comment

Login with GitHub to post a comment