[#35024] - [4.0] fix the path for Cassiopeia js
- Fixed in Code Base
- 4 Aug 2021
- Medium
- Build: 4.0-dev
- # 35024
- Diff
- dgrammatiko:4.0-dev-fix-path
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
- Moved the file
menu-metismenu.es6.jsfrombuild/media_source/templates/cassiopeia/js/mod_menu/tobuild/media_source/templates/site/cassiopeia/js/mod_menu/(there is asitemissing there) - Adjust the layout to use the correct path
Testing Instructions
Enable the dropdown layout for the frontend menu. Check that the metismenu script is loaded
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Documentation Changes Required
@wilsonge this is kinda important for the Child templates
@richard67 I think you need to add the old paths to the files removed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript Repository NPM Change Front End Templates (site) |
| Labels |
Added:
NPM Resource Changed
?
|
||
Why?
The child templates are already configured (check the HTML helper file for the overrides) to look at the folders media/templates/site and media/templates/administrator depending on if the template is front end or back end. The current path was picked out of the blue, it doesn't follow any rules...
Sorry was reading on my phone. Ignore my comment
OK Let's get a quick test but this looks good to me
I have tested this item
Path is correct and script is loaded after the patch.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35024.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35024.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35024.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-08-04 21:43:30 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
?
|
||
Thanks!
Why?
This is not Cassiopeia specific