Skip to content
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

Don't show module chromes of disabled templates #21437

Merged
merged 1 commit into from
Aug 21, 2018
Merged

Don't show module chromes of disabled templates #21437

merged 1 commit into from
Aug 21, 2018

Conversation

SharkyKZ
Copy link
Contributor

@SharkyKZ SharkyKZ commented Aug 7, 2018

Pull Request for Issue #21429.

Summary of Changes

This filters out module chromes of disabled templates from chrome field.

Testing Instructions

Disable a template. Edit a module. Check available chromes in Module Style field.

Expected result

Modules chromes of disabled templates are not shown.

Actual result

Modules chromes of disabled templates are shown.

Documentation Changes Required

No.

@Quy
Copy link
Contributor

Quy commented Aug 12, 2018

I have tested this item ✅ successfully on 7d438bc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21437.

@brianteeman
Copy link
Contributor

I must be missing something but how do you disable a template

@SharkyKZ
Copy link
Contributor Author

Through Extension Manager.

@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 7d438bc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21437.

@ghost
Copy link

ghost commented Aug 20, 2018

Ready to Commit after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 20, 2018
@mbabker mbabker added this to the Joomla 3.8.12 milestone Aug 21, 2018
@mbabker mbabker merged commit 071476b into joomla:staging Aug 21, 2018
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Aug 21, 2018
@SharkyKZ SharkyKZ deleted the disabledModuleChromes branch August 21, 2018 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants