[#15268] - Improve the override creator for JLayouts
- Fixed in Code Base
- 14 Apr 2017
- Medium
- Build: staging
- # 15268
- Diff
- Bakual:ImproveOverrideCreatorForJLayouts
User tests: Successful: Unsuccessful:
Summary of Changes
Currently, the override creator only allows to create overrides for JLayouts located withing /layouts/joomla.
This PR changes the code so it can also create overrides for the other global JLayouts plus for JLayouts located within a component folder (compontens/com_foo/layouts).
Testing Instructions
- Create various overrides for JLayouts as well as view ones
- Adjust the created file a bit (you'll see the path to it in the success message) so you can see the override actually works
Expected result
- You can create overrides for all global JLayouts as well as for component specific ones (eg com_fields)
- Your changes in the created override will have an effect
Actual result
Only possible to create overrides for Jlayouts in /layouts/joomla
Documentation Changes Required
Not that I'm aware
Pinging @angieradtke
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_templates |
(creating backups with dated filenames is not new)
Nice thing. Thanks!
Cool, thank you
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15268.
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-04-14 14:59:37 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
?
|
||
cool
Am 14.04.2017 um 16:59 schrieb Robert Deutz:
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#15268 (comment), or
mute the thread
https://github.com/notifications/unsubscribe-auth/ABHjs-RMLLVpt6pzIY1pqgCOw6VFLBscks5rv4ncgaJpZM4M769V.
I have tested this item✅ successfully on f631ee3
Nice one. Seems to be working well.
Mistakenly did same create twice and nice to see the override system add a date/timestamp to the naming.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15268.