I've got message: warning: array_merge_recursive() [function.array-merge-recursive]: recursion detected in \includes\common.inc on line 2170, after i selected some nodes and submitted form with 'Delete selected nodes'. When I confirm operation (or cancel it), previuosly mentioned warning is occured.

Modules in my installation:
Drupal 6.8
Views Bulk Operations 6.x-1.x-dev 2008-12-20
Views 6.x-2.2
and many more...

I've noticed that source of problem lies inside function views_bulk_operations_form. I don't know the reason, but this function is called twice after submitting and that is why drupal_add_js raises warning. I suggest to move the call of drupal_add_js under conditional statement.

Comments

infojunkie’s picture

Assigned: Unassigned » infojunkie
Status: Needs review » Postponed (maintainer needs more info)

I don't get that problem. I wonder why the form would be called more than once. Does it happen only on the "Delete selected nodes" action or on any action? At which step in the form does this warning occur?

kndr’s picture

StatusFileSize
new120.34 KB
new178.89 KB
new201.61 KB
new6.36 KB

I have this warning in clean Drupal 6.8 installation in XAMPP environment.

Drupal 6.8 with modules:
1) Views
2) Views Bulk Operation
3) Locale

I've created two test pages and try to delete one of it by 'Delete node' button. I chose 'Cancel' on confirmation page ('Are you sure you want to perform 'Delete node' on selected rows?') and got mentioned warning.

deviantintegral’s picture

I also encountered this while testing Signup integration with VBO. It happens whenever I submit the view to make changes. This is on a stock D6.8 + Views 2.2 + VBO. What I found surprising was that the call actually is in the theme('page') call in index.php.

deviantintegral’s picture

Status: Postponed (maintainer needs more info) » Needs review

Applying this patch fixes the issue for me. Any ideas as to what could be causing it not to show up on some setups? It was appearing for me but not for dww during our Signup testing.

I'm running on PHP 5.2.3 + suPHP.

infojunkie’s picture

Status: Needs review » Fixed

I committed the patch, even though I don't understand why the problem occurs. Anyway, the JS setting is only needed on the first page so no harm done - hopefully :-)

infojunkie’s picture

I committed the patch, even though I don't understand why the problem occurs. Anyway, the JS setting is only needed on the first page so no harm done - hopefully :-)

Status: Fixed » Closed (fixed)

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

Tony Sharpe’s picture

Version: 6.x-1.x-dev » 6.x-1.7
Status: Closed (fixed) » Active

I'm getting the same problem although on much later version.
warning: array_merge_recursive() [function.array-merge-recursive]: recursion detected in .....\includes\common.inc on line 2174.
VBO 6.x - 1.7
Drupal 6.12
Views 6.x- 2.6
This occurring when deleting large number of comments (batch mode). 100 at once is OK 330 (and larger) gives error. Comments listed as 100 per page.

Tony Sharpe’s picture

StatusFileSize
new12.07 KB

It is not the number that causes it, it is using the button to select more than one page's worth, whether there's 10 to a page or 100 to a page. Using full pager. View attached.

infojunkie’s picture

Does it also happen on the stock VBO (admin/content/node2) ?

Please comment out the lines 82-95 in the file views_bulk_operations.module, function theme_views_node_selector. Does the error still occur? If not, then your report is a separate issue.

Tony Sharpe’s picture

Thanks for the fast reply.
It does not happen on stock VBO, nor on my created view for forum topic removal. It only happens removing comments not nodes.
I've upgraded to latest dev version. No difference.
If I remove lines 82-95 in latest dev the problem no longer occurs. (The line numbers only make sense on latest dev, not on 1.7).
Thanks
Tony

infojunkie’s picture

Version: 6.x-1.7 » 6.x-1.x-dev
Component: Code » Core
Status: Active » Fixed

This should be fixed by now, I've introduced a different way of preventing multiple Javascript inclusions.

Status: Fixed » Closed (fixed)

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