I have been writing a form field that extends ViewsBulkOperationsBulkForm. If I use both vbo and my field and an action is executed for my field, vbo throws an error because it can't find the action.

CommentFileSizeAuthor
#7 3066470-7.patch1.21 KBNWOM
#6 3064508-6.patch2.26 KBNWOM
#2 3066470-no-action-no-processing.patch1.16 KBkilles@www.drop.org
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

killes@www.drop.org created an issue. See original summary.

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented
FileSize
3066470-no-action-no-processing.patch1.16 KB
Graber’s picture

Graber CreditAttribution: Graber as a volunteer commented
Status: Active » Needs review
pawel_r’s picture

pawel_r CreditAttribution: pawel_r commented

Any update of the patch from #2 for Drupal 8.8, 8.9 ?

Graber’s picture

Graber CreditAttribution: Graber as a volunteer commented
Status: Needs review » Needs work

Needs re-roll.

Also for code review:
Don't we need to add checking logic to viewsFormValidate() as well?

NWOM’s picture

NWOM CreditAttribution: NWOM commented
Status: Needs work » Needs review
FileSize
3064508-6.patch2.26 KB

Here is a re-roll against the newest dev.

NWOM’s picture

NWOM CreditAttribution: NWOM commented
FileSize
3066470-7.patch1.21 KB

I accidentally uploaded the wrong patch. Here is the re-roll.

NWOM’s picture

NWOM CreditAttribution: NWOM commented
Status: Needs review » Needs work

The patch doesn't seem to work with Views Bulk Edit actions. Instead the form just refreshes when clicking on the Modify field values button