Feeling Lucky
?
[#8272] - Sidebar changes broke the modules xtd button
- Closed
- 4 Nov 2015
- Medium
- Build: staging
- # 8272
- Diff
- dgt41:_fixModulesBtns
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
The Problem
Login in front end and try to edit an article
use the XTD button modules to insert a module
oops error 500
The fix
The changes for the sidebar broke it so lets be more specific on those lines and exempt them from the modal
Testing
Apply patch
Repeat the above procedure, the modal gets the correct content
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
roland-d
- comment
- 4 Nov 2015
I have tested this item 
successfully on eb142c8
The modal gives an error when trying to open it in a frontend article. After applying the fix, the modal lets me choose a module and inserts it into the text.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8272.
| Status | Pending | ⇒ | Ready to Commit |
zero-24
- comment
- 4 Nov 2015
@joomla-cms-bot the label please 
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8272.
| Labels |
Added:
?
|
||
zero-24
- comment
- 4 Nov 2015
After testing an CS. Good to merge. Thanks.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8272.
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-04 14:48:45 |
| Closed_By | ⇒ | Kubik-Rubik |
| Labels |
Removed:
?
|
||
| Milestone |
Added: |
||
I have tested this item
successfully on eb142c8
Works. To get the insert finaly work you need also apply this: #8269
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8272.