? Pending

User tests: Successful: Unsuccessful:

avatar astridx
astridx
9 Aug 2020

Pull Request for Issue # #30302.

Summary of Changes

Add a GET-Variable for determining if client has changed.
If the filters are used, the form is submitted and the GET-variable is set. If the client changes via the menu or the Select, it is only a redirected.

Testing Instructions

See #30302.

Actual result BEFORE applying this Pull Request

See #30302.

Expected result AFTER applying this Pull Request

Filters are completely in default state (not only in view) if client changed.

Documentation Changes Required

No

avatar astridx astridx - open - 9 Aug 2020
avatar astridx astridx - change - 9 Aug 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Aug 2020
Category Administration com_modules
avatar astridx astridx - change - 9 Aug 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-08-09 12:26:39
Closed_By astridx
Labels Added: ?
avatar astridx astridx - close - 9 Aug 2020
avatar astridx astridx - change - 9 Aug 2020
Status Closed New
Closed_Date 2020-08-09 12:26:39
Closed_By astridx
avatar astridx astridx - change - 9 Aug 2020
Status New Pending
avatar astridx astridx - reopen - 9 Aug 2020
avatar richard67
richard67 - comment - 9 Aug 2020

@astridx Why sometimes double quotes "and sometimes single quotes ' for strings? As far as I know we shall use single quotes.

avatar richard67
richard67 - comment - 9 Aug 2020

@astridx Could you check and apply my above suggested changes? I could give your PR a good test then because I have already tested it with these changes.

avatar Quy
Quy - comment - 9 Aug 2020

The filter is collapsed upon switching, however, cpanel is still selected when filter is expanded, but not filtered.

a5574f3 10 Aug 2020 avatar astridx cast
avatar ceford
ceford - comment - 10 Aug 2020

Confirming what Quy said. Also, if I select a Site Position filter and switch back forth between Site and Administrator then the Site filter is still selected but the list is not filtered.


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

avatar astridx
astridx - comment - 10 Aug 2020

I thought I had a fix.
But this was bad and now I see, that  joomla.resetfilter

Joomla.resetFilters = (element) => {

does not work.

Not only in com_modules, in no component, or do I oversee something? In other components it is not so obviously.

So this need another fix and I close here.

Can someone reopen
Issue  #30302 please.

avatar astridx astridx - close - 10 Aug 2020
avatar astridx astridx - change - 10 Aug 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-08-10 13:20:38
Closed_By astridx
avatar richard67
richard67 - comment - 10 Aug 2020

Can someone reopen
Issue #30302 please.

Done.

Add a Comment

Login with GitHub to post a comment