? ? ? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
1 Jun 2017

As with the change to namespaces in #15893 the core XML files do need to be adapted to that scheme.

avatar laoneo laoneo - open - 1 Jun 2017
avatar laoneo laoneo - change - 1 Jun 2017
Status New Pending
avatar laoneo laoneo - change - 1 Jun 2017
Title
Fix include path of categories fields
[4.0] Fix include path of categories fields
avatar laoneo laoneo - edited - 1 Jun 2017
avatar joomla-cms-bot joomla-cms-bot - change - 1 Jun 2017
Category Administration com_banners com_contact com_fields com_newsfeeds com_users Front End com_content Modules
avatar brianteeman
brianteeman - comment - 1 Jun 2017

I have tested this item successfully on 825e5ab

code review


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

avatar brianteeman brianteeman - test_item - 1 Jun 2017 - Tested successfully
avatar Bakual
Bakual - comment - 1 Jun 2017

Works fine

Is that something that needs to be documented? And how will it work for 3rd party extension that need to work both in J3.9 and J4? Can they add both paths (old and new) and it will find the right one?

avatar Bakual
Bakual - comment - 1 Jun 2017

I have tested this item successfully on 825e5ab


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

avatar Bakual Bakual - test_item - 1 Jun 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 1 Jun 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 1 Jun 2017

RTC after two successful tests.

avatar mbabker
mbabker - comment - 1 Jun 2017

@wilsonge Please don't merge until there's an answer to @Bakual question (just so it doesn't get lost in the shuffle).

avatar laoneo
laoneo - comment - 2 Jun 2017

@Bakual, yes it will find both ones, on J3 it will use the addfieldpath attribute and on J4 the new one.

avatar Bakual
Bakual - comment - 2 Jun 2017

Added "Documentation Required" label. Should be fine to be merged based on Allons answer.

avatar laoneo laoneo - change - 3 Jun 2017
Labels Added: ? ?
avatar laoneo laoneo - change - 8 Jun 2017
Labels Added: ?
avatar wilsonge wilsonge - change - 8 Jun 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-06-08 14:54:21
Closed_By wilsonge
avatar wilsonge wilsonge - close - 8 Jun 2017
avatar wilsonge wilsonge - merge - 8 Jun 2017

Add a Comment

Login with GitHub to post a comment