Skip to content

Conversation

beni71
Copy link
Contributor

@beni71 beni71 commented May 16, 2023

Pull Request for Issue #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:

articles-batch-tag-removing

Testing Instructions

  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.

@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.3-dev labels May 16, 2023
@brianteeman
Copy link
Contributor

This is a new feature isn't it?

@richard67
Copy link
Member

This is a new feature isn't it?

@brianteeman To me it seems so, and the referred issue also was labelled as feature and not as bug. Am just clarifying with other maintainers what they think.

beni71 and others added 2 commits May 16, 2023 19:22
Co-authored-by: Richard Fath <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
@richard67
Copy link
Member

@beni71 Thank you very much for this PR. Due to our development roadmap, new features will go into version 5.0. Before you get shocked now: 5.0 will be released at the same time as 4.4 in October this year.

4.4 will serve as a bridge version for updating to 5 in the same way as the 3.10 was for 4, so we don't include new features into 4.4 if they are not needed for a smooth update to 5.

4.3.0 had feature freeze when it was in beta phase, and with patch versions we do not include new features in general.

This all for your understanding so you are not too disappointed.

Could you rebase this PR to 5.0-dev? Let us know here if you need help or advise with that.

Or maybe it is better if you create a new branch based on the 5.0-dev branch and make the PR again with that new branch, so your 4.3-dev branch will not be rebased to 5.0-dev because that could be confusing in future.

Thanks in advance.

@beni71
Copy link
Contributor Author

beni71 commented May 17, 2023

Ok, so I can close this PR.

@beni71 beni71 closed this May 17, 2023
@richard67
Copy link
Member

New PR for 5.0-dev is #40613 . Thanks @beni71 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants