module_invoke() doesn't allow passing nodes by reference. Passing by reference allows altering of nodes before the diff is performed.

CommentFileSizeAuthor
#1 diff-1310420-passby-ref.patch877 bytesskwashd

Comments

skwashd’s picture

skwashd commented
StatusFileSize
new diff-1310420-passby-ref.patch877 bytes

The attached patch passes the nodes by reference.

skwashd’s picture

skwashd commented
Status: Active » Needs review

fixing status

deciphered’s picture

deciphered
Primary language English
Location Ballarat, Victoria, Australia
commented
Status: Needs review » Reviewed & tested by the community

Reviewed and tested.

As mentioned via IRC, only change I would make would be to use a drupal_alter() instead, but as that would break the existing API I understand not doing so.

Cheers,
Deciphered.

realityloop’s picture

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

Thanks skwashd/Deciphered, commited.

Status: Fixed » Closed (fixed)

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