-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Template Override Actions #38447
Template Override Actions #38447
Conversation
There was a button to remove a record but not to mark it as checked or unchecked. You had to do each one individually by clicking on the icon. This PR lets you select as many records as you want and then use the button to change all the selected records. This is based on user feedback who complained that there was no way to bulk mark records.
I have tested this item ✅ successfully on 0af4b65 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38447. |
If someone could help with the drone error I would aappreciate it as I dont understand it. |
@brianteeman The file contains a class definition and also execute code outside the class here:
|
I have tested this item ✅ successfully on 2b6b9f0 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38447. |
I have tested this item ✅ successfully on 2b6b9f0 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38447. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38447. |
administrator/components/com_templates/src/View/Template/HtmlView.php
Outdated
Show resolved
Hide resolved
…iew.php Co-authored-by: heelc29 <66922325+heelc29@users.noreply.github.com>
Thank you |
thanks |
joomla/joomla-cms#38638 + joomla/joomla-cms#38622 + joomla/joomla-cms#38619 + joomla/joomla-cms#38592 + joomla/joomla-cms#38541 - (только для en-GB) joomla/joomla-cms#38537 - (только для en-GB) joomla/joomla-cms#38570 + joomla/joomla-cms#38459 + joomla/joomla-cms#38451 + joomla/joomla-cms#38449 + joomla/joomla-cms#38313 + joomla/joomla-cms#38447 +
This fixes a bug exposed in my own pr joomla#38447 That PR added an action dropdown button When you have overrides. See the original PR for details However it also added the dropdown when the file was open for editing but at that point it can be selected because it is never active. This PR removes the button when the file is open for editing. A better PR would be to make it (or a similar button) work in the edit mode but I don't have the time (and probably the skill) to do that. So at this time I consider this PR a bug fix and the functionality can be added at a later date in another pr Testing requires overrides that need checking. You can simulate this by 1. Create template overrides for com_content/article and com_newsfeed/feed 2. Run this sql query replacing #_ with your db prefix INSERT INTO `#__template_overrides` (`id`, `template`, `hash_id`, `extension_id`, `state`, `action`, `client_id`, `created_date`, `modified_date`) VALUES (1, 'cassiopeia', 'L2h0bWwvY29tX25ld3NmZWVkcy9uZXdzZmVlZC9kZWZhdWx0LnBocA==', 223, 0, 'Joomla Update', 0, '2022-09-01 13:51:54', '2022-09-01 13:51:54'), (2, 'cassiopeia', 'L2h0bWwvY29tX2NvbnRlbnQvYXJ0aWNsZS9kZWZhdWx0X2xpbmtzLnBocA==', 223, 0, 'Joomla Update', 0, '2022-09-01 13:51:54', '2022-09-01 13:51:54'); You will now have two override files to update and can test this PR
This fixes a bug exposed in my own pr #38447 That PR added an action dropdown button When you have overrides. See the original PR for details However it also added the dropdown when the file was open for editing but at that point it can be selected because it is never active. This PR removes the button when the file is open for editing. A better PR would be to make it (or a similar button) work in the edit mode but I don't have the time (and probably the skill) to do that. So at this time I consider this PR a bug fix and the functionality can be added at a later date in another pr Testing requires overrides that need checking. You can simulate this by 1. Create template overrides for com_content/article and com_newsfeed/feed 2. Run this sql query replacing #_ with your db prefix INSERT INTO `#__template_overrides` (`id`, `template`, `hash_id`, `extension_id`, `state`, `action`, `client_id`, `created_date`, `modified_date`) VALUES (1, 'cassiopeia', 'L2h0bWwvY29tX25ld3NmZWVkcy9uZXdzZmVlZC9kZWZhdWx0LnBocA==', 223, 0, 'Joomla Update', 0, '2022-09-01 13:51:54', '2022-09-01 13:51:54'), (2, 'cassiopeia', 'L2h0bWwvY29tX2NvbnRlbnQvYXJ0aWNsZS9kZWZhdWx0X2xpbmtzLnBocA==', 223, 0, 'Joomla Update', 0, '2022-09-01 13:51:54', '2022-09-01 13:51:54'); You will now have two override files to update and can test this PR
There was a button to remove a record but not to mark it as checked or unchecked. You had to do each one individually by clicking on the icon.
This PR lets you select as many records as you want and then use the button to change all the selected records. This is based on user feedback who complained that there was no way to bulk mark records.
Before
chrome_cPbU2OavOI.mp4
After
chrome_MlHkWUo6dg.mp4