Warning: Parameter 1 to nodequeue_node_view() expected to be a reference, value given in module_invoke_all() (line 817 of /var/www/includes/module.inc).

I'm seeing this today after the latest update (I tried both the DEV and Alpha branches as well as with Drupal RC4 and Today's Drupal 7 release with the same result.)

I know that this was an issue with panels as well (which I am using) but it is fixed in their latest dev branch (not sure what they did) so I can't say if this is a nodequeue issue or a Drupal core issue but I thought that I might as well get a bug report submitted just in case.

CommentFileSizeAuthor
#4 nodequeue_1016130.patch428 bytesR.Muilwijk
#2 nodequeue_1016130.patch837 bytesScott Reynolds

Comments

bacchus101’s picture

bacchus101 commented

An update: I removed all modules and left just nodequeue and core - and this error was still persistent.

Scott Reynolds’s picture

Scott Reynolds commented
Status: Active » Needs review
Issue tags: -panels, -nodequeue
StatusFileSize
new nodequeue_1016130.patch837 bytes

Simple fix

- all issue tags because they are worthless.

naheemsays’s picture

naheemsays commented
Status: Needs review » Reviewed & tested by the community

yes, its a PHP 5.3 call by reference bug.

The fix works.

(subscribe)

R.Muilwijk’s picture

R.Muilwijk commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new nodequeue_1016130.patch428 bytes

According to http://api.drupal.org/api/drupal/modules--node--node.api.php/function/ho... there should also be a $langcode.

Scott Reynolds’s picture

Scott Reynolds commented
Status: Needs review » Reviewed & tested by the community

Ahh good call.

Works as well.

slashrsm’s picture

slashrsm commented

I can confirm, that error disappears with this patch.

berenddeboer’s picture

berenddeboer commented

works for me too.

derekahmedzai’s picture

derekahmedzai commented

Works for me, too.

anon’s picture

anon
Primary language Swedish
Location Gothenburg
commented

This patch works, but Im not sure about the param $langcode
Why is that needed?

ceasero’s picture

ceasero commented

Where do I add this patch? I'm new to drupal, I know where the nodeque file is, but where do i add this patch?
Thank you

slashrsm’s picture

slashrsm commented

http://drupal.org/patch/apply

upupax’s picture

upupax commented

Works fine.

yykucuk’s picture

yykucuk commented

#4: nodequeue_1016130.patch queued for re-testing.

yykucuk’s picture

yykucuk commented

#2: nodequeue_1016130.patch queued for re-testing.

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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