Feeling Lucky
?
?
[#14902] - fixes
- Fixed in Code Base
- 26 Mar 2017
- Medium
- Build: staging
- # 14902
- Diff
- dgt41:+++++++++++++++++++++XTDMenus
User tests: Successful: Unsuccessful:
Pull Request for Issue #14890 .
Summary of Changes
- Call the menu function with the right amount of variables
- Fix all the logic for the modals (since modals they need to be checked against
window.parent.JoomlaNOTJoomla). THIS CODE WILL NEVER BE EXECUTED IN 3.x DUE TO THE EXISTENCE ofjInsertEditorText(tag, editor);, BUT SINCE IT'S ALREADY FIXED IN J4 THIS ALIGNS THE 2 CODEBASES
Testing Instructions
Inside an article use the menu button to insert a link to a menu item in an article
Expected result
Actual result
Documentation Changes Required
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript |
| Labels |
Added:
?
|
||
infograf768
- comment
- 25 Mar 2017
will test tomorrow.
First grr solved the language. ;)
dgt41
- comment
- 25 Mar 2017
@infograf768 I should go back to school and re learn counting from 1 to 6 :)
infograf768
- comment
- 26 Mar 2017
I have tested this item
Ok now. (Did not test multiple editors)
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14902.
brianteeman
- comment
- 26 Mar 2017
Added release blocker and milestone as requested
wilsonge
- | Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-03-26 11:07:39 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on e70da54
Patch ok for me
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14902.