Views bulk operations node export fails with undefined function _views_bulk_operations_adjust_selection

New correct patch

I've tested the patch in #2 with success. Thank you rimen !

This patch is the same as #2 but rolled from the node_export directory.

Fixes the problem for me.

Marking major as I cannot use the module otherwise.

Patch #4 worked for me was well.

I'm unable to use the module without the patch. Please commit this patch

The same problem exists in 6.x, and the same solution fixes it.

+1 for committing this

#4 works for me, thanks for the patch

#4 fixed it for me as well, thanks @rooby

I can also confirm that the patch in #4 solves the problem.

For me it seems to work somewhat.. In views bulk operations you have the ability to select all nodes instead of what is just displayed, but the export seemed to be still limited to 50 (which was on the view) even though there were more nodes than that.

Sometimes I will get the following error after clicking the export button (it will start out looking ok then error):

Notice: Undefined variable: nids in node_export_gui() (line 332 of /home/site/public_html/site/sites/all/modules/node_export/node_export.pages.inc).
Notice: Undefined variable: code_string in node_export_gui() (line 332 of /home/site/public_html/site/sites/all/modules/node_export/node_export.pages.inc).

Also after applying the patch I am randomly getting these errors as well:

Notice: Undefined variable: nids in node_export_gui() (line 332 of /home/site/public_html/site/sites/all/modules/node_export/node_export.pages.inc).
Notice: Undefined variable: code_string in node_export_gui() (line 332 of /home/site/public_html/site/sites/sites/all/modules/node_export/node_export.pages.inc).
Warning: implode() [function.implode]: Invalid arguments passed in node_export_get_file() (line 440 of home/site/public_html/site/sites/sites/all/modules/node_export/node_export.pages.inc).
Notice: Undefined index: in node_export_get_file() (line 447 of home/site/public_html/site/sites/sites/all/modules/node_export/node_export.pages.inc).

Path node_export-_views_bulk_operations_adjust_selection-1869918-4.patch works for me ..

Tested #4 and it works.
Please commit.

Like mentioned in #12, I'm also seeing that the patch in #4 does not work when choosing to extend the selection to all pages of the view. Only the current page of selections is exported.

I don't think the views_bulk_operations_execute() in the patch in #4 is having much, if any, effect. I see the batch operations being queued up, including views_bulk_operations_adjust_selection, showing the right new number of total rows (with selection expanded to all pages,) but I don't see the batch ops actually being called, (using dpm() and dd().)

At any rate, rows on all pages were not being exported -- only the current page.

The attached patch is successfully selecting all rows. It's using the views_bulk_operations_direct_adjust() function, which seems to be the new (though deprecated) incarnation of _views_bulk_operations_adjust_selection().

tested #16 and it did not resolve the problem.

Updated issue link

Tried #2, #4, and #16, and none of them worked. Still getting the same error message on line 300. Until this is fixed, the module is useless for bulk exports.

For me, #16 selects all rows every time, no matter if the respective checkbox is checked. I adapted the patch to fix that.

Many thanks @Ronino - I can confirm that your updated patch works and only exports what I select in VBO, also the orig error has been fixed:

views 7.x-3.5
VBO 7.x-3.1
node_export 7.x-3.0

Seems like i ran into this error yet again on another project. Every time i tried to export I would get a white screen of death. I applied the patch in #19 and it works. Thanks.

It worked for me with one content type, then on another content type it looks like I am getting errors related to this issue: https://drupal.org/node/1100024 i don't know if it is related though.

#19 worked for me too!

#19 resolve problem with _views_bulk_operations_adjust_selection();
But I try to export in DSV, and I get Drupal var format.

Maybe it's another issue.
Here is a screenshot:

I've found, if i use VBO to export nodes. It's ignore value of format in Views filter.
And just export data to format, which highest of enabled on node_export modules settings page (/admin/config/content/node_export).
So to export node in DSV, I enable only DSV in node_export module settings.

I don't mean to be pushy but why the heck isn't this committed yet? This makes the VBO functionality completely unusable and it's been open for 2 years!!

@Traverus this issue has been open for two years, but only been RTBC for a month. The node_export project is seeking co-maintainers, perhaps you can help??

Patch #19 doesn't fix the error undefined function _views_bulk_operations_adjust_selection with VBO.

Edit: A more detailed error spit by Drupal when batch mode is enabled in the Views settings:

Missing argument 4 for views_bulk_operations_adjust_selection(), called in /…/sites/all/modules/node_export/node_export.module on line 304 and defined dans views_bulk_operations_adjust_selection() (line 847 in /…/sites/all/modules/views_bulk_operations/views_bulk_operations.module

I had the same issue as #24; it was exporting in 'drupal' format regardless of what was selected.
Updated the patch from #19 to make other formats (e.g. dsv) working again.

#28 works for me!
Thank you Thijs

First time I have used this and I was just getting a blank screen where the exported code should have appeared. Patch in #28 fixes it for me.

The patch attached to #1869918-28: Views bulk operations node export fails with undefined function _views_bulk_operations_adjust_selection applies cleanly to 7.x-3.x and works. Thanks.

Changing the status back to "Needs work".
I realized not all the instances of variable_get('node_export_format', array('drupal')); is updated.

trying again...

Patch in #33 applies and works. Marking it RTBC!

also verified: patch #33 worked

when patch #33 will be included in dev version ??

EDIT : no, in stable version, 2 years after !?

Thanks

#19 worked for me. I was getting the white screen of death. Thank you!

#33 worked for me so this is still RTBC!

#33 isn't applying for me :(

EDIT: Never mind. I confused myself and was trying to apply the patch to views_bulk_operations instead of node_export. Long day...

Another confirmation, patch #33 solves the issue.

Yet another confirmation. Patch is working and required for this module to work at all with VBO.

Same issue. #33 fixed it.

I think this meets the definition of critical.

Fix #33 works also for me.
IMO, it's time this patch should be apply.

#33 Work. I have edit the .module

GIT here

https://github.com/mattyy21/node_export/tree/master

#33 worked for me... thanks!

We should try to get #33 in the dev version.

#33 works for me.

#33 also works for me.

Hope this helps someone else, but after stumbling across this yet again, 6 months later, I decided to go with Views Data Export instead. It offers more control over the data exported and is at least being actively maintained.

#33 worked great for me. Thanks everyone!

!!! Thanks for committing.

#33 works.