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

Graber created an issue. See original summary.

Graber’s picture

Graber’s picture

Issue summary: View changes
Graber’s picture

Status: Active » Needs work

  • Graber committed 39c6cae on 8.x-1.x
    Issue #2907929 by Graber: replaced core node delete action by a more...
Graber’s picture

Assigned: Graber » Unassigned
Status: Needs work » Needs review
joelpittet’s picture

From a nitpick uncompatible is not a word. Could you replace this with incompatible?

  • Graber committed 943301a on 8.x-1.x
    Issue #2907929 by Graber: language correction.
    
Graber’s picture

Done. I'll use patches from now on so we don't have correction commits like this one, hopefully there will be reviews. Thanks!

joelpittet’s picture

Sorry, 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.

Graber’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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