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

[4.0] Fix missing filters (com_menus & com_associations) #19044

Merged
merged 2 commits into from Dec 13, 2017
Merged

[4.0] Fix missing filters (com_menus & com_associations) #19044

merged 2 commits into from Dec 13, 2017

Conversation

C-Lodder
Copy link
Member

@C-Lodder C-Lodder commented Dec 12, 2017

Pull Request for Issue #18815

Summary of Changes

Fix the filters

Testing Instructions

  • Go to administrator/index.php?option=com_menus&view=menus
  • You should see a filter to switch between Site and Administrator

  • Go to administrator/index.php?option=com_menus&view=items
  • You should see a filter to switch between Site and Administrator
  • You should see a filter to select the menu type

  • Go to administrator/index.php?option=com_associations
  • You should see a filter to for Item type and Language

@infograf768
Copy link
Member

infograf768 commented Dec 13, 2017

This works great. Can you please also change the lang constant which has nothing to do here:
<joomla-alert type="warning"><?php echo JText::_('COM_LANGUAGES_MULTILANGSTATUS_NONE'); ?></joomla-alert>
to
<joomla-alert type="warning"><?php echo JText::_('JGLOBAL_NO_MATCHING_RESULTS'); ?></joomla-alert>
in com_menus/tmpl/items/default.php and in com_menus/tmpl/items/modal

EDIT: no use. Will correct with new PR including some other stuff

@ghost
Copy link

ghost commented Dec 13, 2017

I have tested this item ✅ successfully on c1edfed


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

@infograf768
Copy link
Member

@C-Lodder
I will correct the code to use the new attribute, and at the same time take off that wrong constant.
Therefore, let's merge this as I will have to modify the same file.

@infograf768
Copy link
Member

I have tested this item ✅ successfully on c1edfed


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

@infograf768
Copy link
Member

rtc

@wilsonge Please merge asap to let me make new PR


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 13, 2017
@infograf768 infograf768 added this to the Joomla 4.0 milestone Dec 13, 2017
@wilsonge wilsonge merged commit 3fe4b89 into joomla:4.0-dev Dec 13, 2017
@joomla-cms-bot joomla-cms-bot added PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Dec 13, 2017
@C-Lodder C-Lodder deleted the assoc-23 branch December 13, 2017 20:15
@infograf768
Copy link
Member

Thanks @wilsonge
@C-Lodder
We are having some discussions about this new noresults attribute in the maintainers chat.
Shall we keep it or not is the question.

@C-Lodder
Copy link
Member Author

I don't mind whether it's kept or not. I'm just a UI dev

@infograf768
Copy link
Member

You can see what we can get if we keep the attribute in 4.0
For com_associations, com_content (articles and Featured), com_categories, com_associations.
noresults.diff.zip

For com_associations, one has also to delete in its custom searchtools layout the bar.php

laoneo pushed a commit to Digital-Peak/joomla-cms that referenced this pull request Dec 18, 2017
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.

None yet

4 participants