? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
1 Mar 2020

Summary of Changes

Namespaces the JAdapter classes for 3.10-dev (which will then be ported to 4.0-dev)

Testing Instructions

Ensure functionality using these classes (the ability to install/uninstall/discover install extensions) continues to work the same as before.

Expected result

Everything still works

Documentation Changes Required

None

avatar wilsonge wilsonge - open - 1 Mar 2020
avatar wilsonge wilsonge - change - 1 Mar 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Mar 2020
Category Libraries
avatar wilsonge wilsonge - change - 1 Mar 2020
Title
Namespace JAdapter
[3.10] Namespace JAdapter
avatar wilsonge wilsonge - edited - 1 Mar 2020
avatar richard67
richard67 - comment - 2 Mar 2020

Namespaces the JAdapter classes for 3.10-dev (which will then be ported to 3.10-dev)

@wilsonge Is there maybe some typo in the description? The PR is for the 3.10-dev, and you want to port it to 3.10-dev, too? That doesn't make sense.

avatar wilsonge wilsonge - change - 2 Mar 2020
The description was changed
avatar wilsonge wilsonge - edited - 2 Mar 2020
avatar wilsonge
wilsonge - comment - 2 Mar 2020

Yup. Fixed :)

avatar laoneo
laoneo - comment - 2 Mar 2020

Why did you namespace it? It is deprecated.

avatar wilsonge
wilsonge - comment - 2 Mar 2020

It's deprecated for 5.x - we're still using it through 4.x for the Installer classes. And I want the last of the non-namespaced classes in the libraries/joomla folder in the 4.x branch to get namespaced.

avatar laoneo
laoneo - comment - 2 Mar 2020

It's better to remove it from the installer classes and then leave it, same as we did with the "New MVC". I guess we even discussed that long time ago, to keep it not namespaced ?

avatar wilsonge
wilsonge - comment - 2 Mar 2020

That would involve another large refactor of the installer classes. It's not happening - we're keeping this one until J5. Choose the right fights :) New MVC removal is going to be decided by Production in their next meeting ;)

avatar rdeutz rdeutz - change - 3 Mar 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-03-03 11:42:51
Closed_By rdeutz
Labels Added: ? ?
avatar rdeutz rdeutz - close - 3 Mar 2020
avatar rdeutz rdeutz - merge - 3 Mar 2020
avatar richard67
richard67 - comment - 14 May 2021

Sorry for being a bit late to the party, but shouldn't the 2 files libraries/joomla/base/adapter.php and libraries/joomla/base/adapterinstance.php and also the folder libraries/joomla/base which will be empty without these 2 files be deleted on update, i.e. added to the files and folder deletion in script.php in 3.10?

avatar richard67
richard67 - comment - 14 May 2021

I just see the files have been later added to script.php with this commit: 7466dab .

And again later they were added to the list of folders to be removed with this commit: 3200667 . And here is the mistake: It should not be the files but only the folder which has been added there.

Will make PR in a few minutes.

avatar richard67
richard67 - comment - 14 May 2021

See #33868 .

Add a Comment

Login with GitHub to post a comment