User tests: Successful: Unsuccessful:
Pull Request for Issue # .
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.
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.
Code review.
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.
File "administrator/components/com_admin/script.php" doesn't contain anymore the obsolete function "uninstallEosPlugin" and any call to it.
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
Category | ⇒ | Administration com_admin |
Status | New | ⇒ | Pending |
Should / can the language strings of the eos310 plugin also be removed yet for J5? What is the procedure for this?
See:
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?
I have tested this item
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
|
Thanks
Should / can the language strings of the eos310 plugin also be removed yet for J5? What is the procedure for this?
See:
joomla-cms/administrator/language/en-GB/plg_quickicon_eos.ini
Line 22 in 934c75c