[4.0] Add tag filter to category list

[stutteringp0et]

#24398 requested the tag filter option available in Category Blog be made available in Category List. This simple XML change is the only thing necessary to accomplish this request.

Pull Request for Issue # .

Summary of Changes

Simple XML change to the Category List (default.xml) enables tag filtering

Testing Instructions

Create several articles in the same category using various tags
Create several Category List menu items for the category using the various tags used previously

Expected result

Tag filtered Category List

Actual result

Works for me

Documentation Changes Required

[ghost]

@rytechsites please test as Opener of #24398. how to test: https://docs.joomla.org/Testing_Joomla!_patches

[fastslack]

@test Filters seems to works fine using this PR stutteringp0et#1

[sanderpotjer]

I have tested this item 🔴 unsuccessfully on caf6974

This needs a fix as suggested by merging PR of @fastslack "Fix SQL issue for ajax search" into this PR.

Also: When a tag is removed the "old" tag is still in the Joomla state, so only in a new session the user will see all articles for a category again. Would be nice to make sure that if no tag is selected this is directly forced in the state.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24550.}

[stutteringp0et]

I can find no PR titled "Fix SQL issue for ajax search" and none of @fastslack 's PR's match that description.

Is there a # I can look up?

[fastslack]

Is there a # I can look up?

35b1163

[stutteringp0et]

How do I resolve the conflicts?

[ghost]

https://help.github.com/en/articles/about-merge-conflicts

[stutteringp0et]

Thank you for pointing me in the right direction @franz-wohlkoenig

[stutteringp0et]

This needs, what - 2 more human tests?

[ghost]

This needs, what - 2 more human tests?

Yes. After 2 successfully Tests Release Lead decide if merge.

[stutteringp0et]

@fastslack do you mind testing again?
@rytechsites this is on your request, can you test?

[rytechsites]

I would be interested in testing this. Can someone tell me the exact 'joomla 4' version I should download that includes this fix? I'm sorry for my ignorance, I haven't tested in this way before.

Thanks again for getting this done!

-- Laura

[alikon]

i think you should download & install the nigthly build from here https://developer.joomla.org/nightly-builds.html which doesn't include this pr and then install the com_patchtester a guide for using patchtester https://docs.joomla.org/Component_Patchtester_for_Testers

[rytechsites]

Thank you, this is excellent! I should be able to test in the next day or so!

…

-- Laura

On Tue, Jul 23, 2019 at 1:07 PM Nicola Galgano ***@***.***> wrote: i think you should download & install the nigthly build from here https://developer.joomla.org/nightly-builds.html which doesn't include this pr and then install the com_patchtester a guide for using patchtester https://docs.joomla.org/Component_Patchtester_for_Testers — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#24550?email_source=notifications&email_token=AEIIWYFC4OY6PRT7Q6G7LPDQA43FXA5CNFSM4HFAPRHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2TZLMY#issuecomment-514299315>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEIIWYBZ6NIYYBBXAEMRME3QA43FXANCNFSM4HFAPRHA> .

-- www.RytechSites.com <http://www.rytechsites.com/> Dynamic Websites for your company!

______________________________ Trainer with www.lynda.com <http://www.Lynda.com> Member www.JoomlaUsersNJ.com Proud Board Member of: www.aecdc.org

[tushar33]

I have tested this item 🔴 unsuccessfully on b96957d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24550.}

[pravinTek]

I have tested this item 🔴 unsuccessfully on b96957d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24550.}

[shinderohanoffical]

I have tested this item 🔴 unsuccessfully on b96957d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24550.}

[SharayuYadav]

I have tested this item 🔴 unsuccessfully on b96957d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24550.}

[HLeithner]

Since this is a missing xml field I would accept this als backport for j3 if someone is interested to do this.

[roland-d]

@stutteringp0et This has been awhile but I think the failed tests come from the file conflict. Could you fix the conflict please?

[Quy]

Ported from J3 in PR #27596