[6.0] Added possibility to batch remove a tag

[beni71]

Pull Request for Feature request #23185 .

Summary of Changes

In the articles view there is a batch action which can add/assign one tag from the tag list to the selected articles. This PR introduces the possibility to remove/unassign one tag from the selected articles. It also extends batch processing for categories, contacts and news feeds.

How it looks like:

Testing Instructions

If you use patch tester, after applying the patch, call npm install from commandline to rebuild the .js files in folder media.

1. Make sure your Joomla tag system is active and the plugin "Behaviour - Taggable" is enabled.
2. Create a new article and in the detail view add/assign two tags, lets say "Tag1" and "Tag2".
3. Save and create a second article, add/assign it to tag "Tag2" and "Tag3".
4. Save and close.
5. In the articles list select both articles, click batch action and remove "Tag2". Apply batch processing.
6. Open the first article details and check whether the "Tag2" is removed. The tag "Tag1" must be still assigned.
7. Open the second article details and check whether the "Tag2" is removed. The tag "Tag3" must be still assigned.
8. A similar test should be done for categories, contacts and news feeds.

Actual result BEFORE applying this Pull Request

It is not possible to remove/unassign a tag from an article via batch processing.

Expected result AFTER applying this Pull Request

It is possible to remove/unassign a tag from an article via batch processing.

Link to documentations

Please select:

• Documentation link for docs.joomla.org:

• No documentation changes for docs.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed

There are new text keys that need to be translated.

[crommie]

I have tested this item 🔴 unsuccessfully on d9bf8bf

Not yet, this throws

There is no "joomla.batch-tag-addremove" asset of a "script" type in the registry.

---

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

[beni71]

Hi @crommie, thanks for testing.
I tested it again by installing patch tester on a fresh joomla installation version 5. After applying the patch the article view indeed displayed that error. After calling npm install from commandline it worked. This is needed to rebuild the .js files in folder media. I will update the testing instructions.

[crommie]

OK in that case I can't re-test since I'm testing on a PBF server site.

[ceford]

I have tested this item ✅ successfully on ba2293d

Tested batch tag removal from articles, article categories, contacts and news feeds. All OK.

---

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

[alikon]

I have tested this item ✅ successfully on ba2293d

---

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

[alikon]

RTC

---

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

[HLeithner]

This pull request has been automatically rebased to 5.1-dev.

[beni71]

@softforge Thanks for helping me.
I have solved the last comment from @HLeithner by adding a new function postStore in the TagsHelper and deprecated the function postStoreProcess.

(if the message is not in before 6 then we cannot remove in 7)

Does it mean that I have to create a PR for the version 5.x as well? If so, shouldn't the deprecation messages be adapted to something like: deprecated 5.x will be removed in 7.0 ?

[Bodge-IT]

@beni71 yes you would need to get something in 5.* for deprecation in 7. As soon as we see a pr against 5.2, we can merge this one.

Thanks for the time and work on this. We can help test that and discuss with the RM for that release.

[richard67]

@softforge @Bodge-IT Meanwhile @beni71 has made a PR for 5.2-dev: #44383 . But as far as I understand your comments above, you just wanted to have the deprecation of the old method batchTag in 5.2-dev, not the complete change. Or did I get something wrong? The new PR #44383 seems to contain all changes from this PR here but not do anything with deprecation message. Could you check and advise?

[softforge]

@softforge @Bodge-IT Meanwhile @beni71 has made a PR for 5.2-dev: #44383 . But as far as I understand your comments above, you just wanted to have the deprecation of the old method batchTag in 5.2-dev, not the complete change. Or did I get something wrong? The new PR #44383 seems to contain all changes from this PR here but not do anything with deprecation message. Could you check and advise?

You are right and I have commented on #44383
Thanks for spotting it

[Bodge-IT]

Appreciate all your work and patience on this @beni71. Finally merged.

[HLeithner]

please create a migration entry in the manual for 6.0