? ? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
2 Feb 2019

Pull Request for Issue #23604 (comment)

Summary of Changes

After that discussion and merge of #23604 @HLeithner and I thought that we may have the wrong code since 1.5 (see discussion above) concerning the way to get an editor field in Custom modules or similar using < customContent /> in their manifest.

This patch also simplifies the custom type module edit in frontend as we now check the xml, thus normalizing the code vs backend.

Testing Instructions

Install mod_customadv.zip

Create or edit a custom module and the module above in back-end and frontend to confirm the editor field displays.

@HLeithner

Pinging also @johanjanssens (see discussion) as the original code comes from 1.5 and, before merging this, it would be important to know if there was a reason for it or if it was just a typo that made gently its way until now...

avatar infograf768 infograf768 - open - 2 Feb 2019
avatar infograf768 infograf768 - change - 2 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Feb 2019
Category Administration com_modules Front End com_config
avatar infograf768 infograf768 - change - 2 Feb 2019
Title
[staging] Follow up on #23604
[staging] Follow up on #23604 Custom modules editor field
avatar infograf768 infograf768 - edited - 2 Feb 2019
avatar infograf768 infograf768 - change - 2 Feb 2019
The description was changed
avatar infograf768 infograf768 - edited - 2 Feb 2019
d63bb6a 2 Feb 2019 avatar infograf768 cs
avatar infograf768 infograf768 - change - 2 Feb 2019
Labels Added: ?
avatar infograf768
infograf768 - comment - 3 Feb 2019

@HLeithner
done

avatar bembelimen bembelimen - test_item - 24 Feb 2019 - Tested successfully
avatar bembelimen
bembelimen - comment - 24 Feb 2019

I have tested this item successfully on 3e88d2a


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

avatar bembelimen
bembelimen - comment - 24 Feb 2019

I have tested this item successfully on 3e88d2a


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

avatar Quy Quy - test_item - 25 Feb 2019 - Tested successfully
avatar Quy
Quy - comment - 25 Feb 2019

I have tested this item successfully on 3e88d2a


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

avatar Quy
Quy - comment - 25 Feb 2019

I have tested this item successfully on 3e88d2a


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

avatar Quy Quy - change - 25 Feb 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 25 Feb 2019

RTC


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

avatar infograf768 infograf768 - change - 25 Feb 2019
Labels Added: ?
avatar HLeithner HLeithner - change - 25 Feb 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-02-25 17:10:49
Closed_By HLeithner
avatar HLeithner HLeithner - close - 25 Feb 2019
avatar HLeithner HLeithner - merge - 25 Feb 2019
avatar HLeithner
HLeithner - comment - 25 Feb 2019

thx

Add a Comment

Login with GitHub to post a comment