Problem/Motivation

Add a bulk action form to mass-delete revisions on the entity level. But first we should check if other modules don't already implement this.

I think that would then fully support the different visions described in #2925532: Provide bulk action to delete all old revisions of an entity

Steps to reproduce

Proposed resolution

Remaining tasks

Discuss the options of the action
Implement
Test
Release

User interface changes

API changes

Data model changes

Issue fork revisions_bulk_operations-3581733

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

anybody created an issue. See original summary.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Version: » 1.x-dev
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Related issues: +#3585494: Revision deletions en-masse?
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Issue summary: View changes

First we need to discuss which options the action should have. For example how many and which versions to keep.
Also we should very first check which other modules already provide similar solutions.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

We could start with an action to "Delete all old revisions" which removes all revisions and just keeps the current version of the entity.
For more fine-grained actions we could add a different one in a more detailed follow-up issue. See #4

grevil made their first commit to this issue’s fork.

grevil opened merge request !5

grevil’s picture

grevil commented
Status: Active » Needs review

Done. Update hook is tested and works great.

>For more fine-grained actions we could add a different one in a more detailed follow-up issue sure thing! For that we would need the class to extend "ConfigurableActionBase".

Please review!

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Issue tags: +Needs tests

Can we have tests eventually? Hopefully should be very similar to node delete action tests?

grevil’s picture

grevil commented
Status: Needs review » Needs work
Issue tags: -Needs tests

Added tests, but we are currently not taking language into account.

grevil’s picture

grevil commented
Status: Needs work » Needs review

Alright, that should be it! The language support will get implemented in the follow-up issue!

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Needs review » Reviewed & tested by the community

LGTM

  • grevil committed 667f836b on 1.x 
    feat: #3581733 Add a bulk action form to mass-delete revisions on the...
grevil’s picture

grevil commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.