Skip to content

[com_modules] Client filter not respected in fields after editing module#22040

Merged
mbabker merged 1 commit intojoomla:stagingfrom
SharkyKZ:comModules_1
Sep 8, 2018
Merged

[com_modules] Client filter not respected in fields after editing module#22040
mbabker merged 1 commit intojoomla:stagingfrom
SharkyKZ:comModules_1

Conversation

@SharkyKZ
Copy link
Contributor

@SharkyKZ SharkyKZ commented Sep 7, 2018

Pull Request for Issue #22037.

Summary of Changes

This fixes the issue of module positions and types not being filtered correctly in filter form fields.

Testing Instructions

Go to com_modules.
In filter form select Administrator client.
Check that administrator positions and types are shown in - Select Position - and - Select Type - fields.
Mark any module and click Publish or Unpublish.
Check position and type fields again.

Expected result

Administrator positions and types shown.

Actual result

Site positions and types shown.

Documentation Changes Required

No.

@alikon
Copy link
Contributor

alikon commented Sep 7, 2018

I have tested this item ✅ successfully on 894ccdb


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

1 similar comment
@infograf768
Copy link
Member

I have tested this item ✅ successfully on 894ccdb


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

@infograf768
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 8, 2018
@infograf768 infograf768 added this to the Joomla 3.9.0 milestone Sep 8, 2018
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Sep 8, 2018
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