Failure

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
31 Aug 2017

Makes web links installable and working on the current Joomla 4 branch. This fixes only the PHP errors. The layouts are not touched as we are still working on a proper BC solution joomla/joomla-cms#17190.

avatar laoneo laoneo - open - 31 Aug 2017
avatar laoneo laoneo - change - 31 Aug 2017
The description was changed
avatar laoneo laoneo - edited - 31 Aug 2017
avatar yvesh yvesh - assigned - 31 Aug 17
avatar yvesh yvesh - change - 31 Aug 2017
Milestone Added: Weblinks 3.7.1
avatar zero-24 zero-24 - assigned - 31 Aug 17
avatar zero-24 zero-24 - unassigned - 31 Aug 17
avatar wilsonge
wilsonge - comment - 19 Nov 2017

We need to make sure that parent::check is called in WeblinksTable so that we can have that plugin event properly triggered in the parent method

avatar laoneo
laoneo - comment - 7 Feb 2018

Added the parent check call. All should be fine now.

avatar astridx
astridx - comment - 7 Feb 2018

@laoneo Can we close this PR as this branch is merged into #384 ?

avatar laoneo
laoneo - comment - 7 Feb 2018

I would like to have a not namespaced version for Joomla 4 which is working and a namespaced version for Joomla 3.9 for testing. So we need this one here.

avatar mastacheata
mastacheata - comment - 8 Mar 2019

How is this PR related to the 4.0-dev branch?

This PR seems to only perform the minimally neccessary changes for it to work with Joomla 4.x
The Branch on the other hand is in a really strange state of something in between 3.x and 4.x style.
All of the Administrator stuff is namespaced and using 4.x style, while the component/frontend part seems to be untouched and still stuck in 3.x

Yet neither of the two seems to have adjusted to moving the dispatcher into it's own namespace as well.

avatar wilsonge wilsonge - close - 16 Aug 2019
avatar wilsonge wilsonge - merge - 16 Aug 2019
avatar wilsonge wilsonge - change - 16 Aug 2019
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-08-16 13:52:56
Closed_By wilsonge

Add a Comment

Login with GitHub to post a comment