Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
This is really a more general issue involving unlogical way of core BulkForm
plugin operation, but I expect people to search for it by the current title.
The core BulkForm
doesn't support confirmation form step in a logical way, implementation requires saving of selected results into some temporary storage in the action execute method and actually executing the action in the form submit handler.
Proposed resolution
Replacement of the core DeleteNode
action plugin with a dedicated implementation and remove the core action in ViewsBulkOperationsActionManager
.
Comments
Comment #2
Graber CreditAttribution: Graber as a volunteer commentedComment #3
Graber CreditAttribution: Graber as a volunteer commentedComment #4
Graber CreditAttribution: Graber as a volunteer commentedComment #6
Graber CreditAttribution: Graber as a volunteer commentedComment #7
joelpittetFrom a nitpick
uncompatible
is not a word. Could you replace this withincompatible
?Comment #9
Graber CreditAttribution: Graber as a volunteer commentedDone. I'll use patches from now on so we don't have correction commits like this one, hopefully there will be reviews. Thanks!
Comment #10
joelpittetSorry, reviews from me are sporadic. If you find nobody else has reviewed in a while, feel free to ping me on the ones that are holding you back.
Comment #11
Graber CreditAttribution: Graber as a volunteer commented