It would be great to have more variables available in theme_views_bulk_operations_confirmation().
The attached patch adds "operation" and "form_state['values']".

CommentFileSizeAuthor
vbo-form_state_values-theme_confirmation.patch1.68 KBstborchert

Comments

McGo’s picture

McGo commented

patch works great.

bojanz’s picture

bojanz commented
Issue summary: View changes
Status: Needs review » Needs work

I am fine with adding 'operation' => $operation, 'select_all_pages' => $form_state['select_all_pages'] but I don't see a point in adding 'values' when we are already passing rows.
So how about just adding those two?

stborchert’s picture

stborchert
he/him
Primary language German
commented

Sounds good.
I must admit, I have now idea why $row wasn't enough in our case ...

bojanz’s picture

bojanz commented
Title: Make additional variables available in theme_views_bulk_operations_confirmation » Pass the operation to theme_views_bulk_operations_confirmation
Status: Needs work » Fixed

Committed:
http://drupalcode.org/project/views_bulk_operations.git/commitdiff/a67fa...

Thanks.

Status: Fixed » Closed (fixed)

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