Add support for Views Bulk Operations (VBO)

I agree.

Patch?

patch attached - works for me, etc. One small caveat - I had to bump the max_allowed_packet size in MySQL to get this to work for views with many thousands of rows. (interaction with storing the information necessary in the session, I'm assuming)

#2: subscriptions-add_bulk_subscribe_to_vbo-1712734-2.patch queued for re-testing.

Any interest in this?

@MadEye4: Good — please check out the patch, install it, test it, and provide feedback on what you find.

Thank you, MadEye4!

The code does a great job of using what's already there — there's just one documentation issue:

+++ b/subscriptions.module
@@ -982,6 +982,97 @@ function _subscriptions_bulk_operation(array $uids, $bulk_op) {
+/**
+ * Helper function - abstracts the actions necessary to hijack the bulk
+ * subscribe/unsubscribe action form and redirect to the subscriptions bulk
+ * operations form.
+ */

... does not follow the one-line-plus-empty-line convention.

However, I'm encountering some issues in testing:

1. I've created a users view that includes the "Send e-mail" operation, and executing it gives me

Warning: array_unshift() expects parameter 1 to be array, null given in array_unshift()

in subscriptions_views_bulk_operations_form_alter().

2. Selecting two users and executing the "Subscribe the selected users to..." operation gives me

9 users selected for bulk subscribing.

which is all users on my test installation.

I'm using VBO 7.x-3.1, Views 7.x-3.7.

I would really like to be able to modify subscriptions in batches of more than 50 at a time.

You may want to take a look at the Pager Limits module...

I confirm #9 point 2.
It selects all of your first page users. That's 50 in my case.
I have the option to select all 85 users in my view. When I do that I can subscribe all 85.
It is not possible for me to select only a handfull of users. Its all or nothing.

Also I can only choose to subscribe users. Not unsubscribe.

Drupal 7.56 / Subscriptions 7.x-1.3+0-dev / VBO 7.x-1.3+0-dev

Updated patch to recognize the selected rows in the view. There still seems to be an issue with Select All across multiple pages. I did not look into this issue.

Thank you, pattersonc, for working on this!

Anyone willing to take on the Select All issue?