According the api, it looks like it should not be expecting parameters to be passed by reference.

#1 pass_by_value-1282564-1.patch494 bytesDeeLay


DeeLay’s picture

new494 bytes

Attached patch

Isostar’s picture

Version:7.x-2.x-dev» 7.x-2.7

Still present.

Shnapoo’s picture

The error message reads "Warning: Parameter 1 to page_title_user_cancel() expected to be a reference, value given in module_invoke_all()". The function's signature doesn't match the requirement defined by hook_user_cancel().

The error is still in the dev. The patch #1 looks fine. Go commit it.

camprandall’s picture

Title:Warning: Parameter 1 to page_title_user_cancel() expected to be a reference, value given in module_invoke_all()» page_title_user_cancel() has bad signature
Version:7.x-2.7» 7.x-2.x-dev
Priority:Minor» Normal
Status:Active» Reviewed & tested by the community

This is just to confirm that the patch fixes the problem. We're using it in production and no longer see the error.

feddovdm’s picture

Patch #1 works for me! No longer do i see the error.

Echofive’s picture

Patch #1 works for me too!

Bug related in my project.

blasthaus’s picture

Issue summary:View changes

why is this not committed yet? it's impossible for a user to cancel without correcting this!

wzain’s picture

Hi all. I am new to Drupal. How do I apply this patch? Thanks.

hargobind’s picture

Priority:Normal» Major

Also confirming the patch in #1 works.

Escalating the issue to Major to get the attention of the module maintainers since it was RTBC 2 years ago and still has not been committed even though this module is listed as "actively maintained".

Michelle’s picture

Still applies cleanly and works.