User tests: Successful: Unsuccessful:
Also fixes feature request
#6656
Possible conflicts with other PRs (but easy to fix):
1) #6673
https://github.com/bertmert/joomla-cms/blob/patch-9/administrator/components/com_modules/views/modules/tmpl/default.php#L249-L254
Test:
All filters in sidebar.
Check also issue #6661
Check also issue/request #6656
Labels |
Added:
?
|
Labels |
Added:
?
|
Category | ⇒ | Templates (admin) UI/UX |
@test @aasimali confirmed another successful test here: #6656 (comment)
@zero-24 Thank you for code review!
@test
Patch is working and better than current previous filtering.
Status | Pending | ⇒ | Ready to Commit |
Thanks to the tester and coder! RTC
Labels |
Added:
?
|
Labels |
Added:
?
|
Milestone |
Added: |
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-04-10 14:53:24 |
This has caused a regression in Hathor where we can't change the client anymore.
See #6673 (comment)
@zero-24 @roland-d @infograf768 Yes, buggy. Please remove RTC and close/remove this PR. I don't find a way to solve conflicts between Hathor and Isis in model modules.php described by infograf768. I don't know why all my tests then seemed to be successfull.
(Maybe because I tested with applied patch #6677 but I don't know).
Maybe someone will take over later. Then it's better to close this one because I will not have time for longer investigations. Sorry.
Milestone |
Removed: |
||
Labels |
Removed:
?
|
Labels |
Removed:
?
|
Labels |
Added:
?
|
Labels |
Added:
?
|
Milestone |
Removed: |
Reverted this PR due to the issues mentioned.
Feel free to propose a new PR.
Looks pretty good here.