Feeling Lucky
?
[#22341] - [4.0] Fix multiple Editor instances
- Fixed in Code Base
- 23 Sep 2018
- Medium
- Build: 4.0-dev
- # 22341
- Diff
- Fedik:fix-multi-editor
User tests: Successful: Unsuccessful:
Summary of Changes
Since an editors is a special plugins rendering through $this->getDispatcher()->dispatch('onDisplay', $event); leads to rendering of ALL loaded editors, instead of ONE.
I guess onDisplay should be called directly, per specific editor.
Testing Instructions
in mod_custom XML, in params fieldset add fields with 2 different type of editor, eg:
<field name="editor1" type="editor" label="None" editor="none" />
<field name="editor2" type="editor" label="Default"/>Expected result
Should be rendered 2 editor:
Editor type None
TinyMCE editor
Actual result
Rendered 4 editors.
For a first field:
- editor none with name
name="editor1" - editor TinyMce with name
name="editor1"
For a second field:
- editor none with name
name="editor2" - editor TinyMce with name
name="editor2"
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
Hackwar
- comment
- 23 Sep 2018
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22341.
| Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-09-23 18:15:04 |
| Closed_By | ⇒ | wilsonge |
wilsonge
- comment
- 23 Sep 2018
Thanks!
I've tested this successfully.? Does exactly what it should do.