[#11843] - CS after merge https://github.com/joomla/joomla-cms/pull/7540
- Fixed in Code Base
- 30 Aug 2016
- Medium
- Build: staging
- # 11843
- Diff
- zero-24:patch-18
- Success continuous-integration/drone the build was successful Details
- Error continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #7540 @rdeutz
Summary of Changes
CS after merge: #7540 (comment)
Testing Instructions
Edit the index.php of the protostar template and rename the module position in line 177 from
<jdoc:include type="modules" name="position-8" style="xhtml" />
to
<jdoc:include type="modules" name="test123" style="xhtml" />
Now go to the module and create a new custom module and place it in position test123 (it wont be on the list you have to manually enter it)
Go to global configuration and make sure that you have "Mouse-over Edit Icons for" set for modules
Go to the front end of the site and check the new module is displayed and then log in
Hover over the module and you get an icon to let you open the module for editing. Change the module title and press save. The module will have been saved in a different position.
Now apply the patch. Go to module manager and put the module back into test123 and repeat
You will now see that when you edit the module you will see at the bottom of the list of module positions a section "Active positions" and test123 is there. Dont change the position. Edit the title and sabe the module. The module will have been saved in the same position
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Components |
| Labels |
Added:
?
|
||
wilsonge
- | Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-08-30 08:29:35 |
| Closed_By | ⇒ | wilsonge |
it was/is the javascript part, doing my best to make it green
can't make it green and it was failing at the moment it was merged not after
Hmm travis fails on the last commit? But it don't let me go into the report. Can you double check that?