Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
According the api, it looks like it should not be expecting parameters to be passed by reference.
http://api.drupal.org/api/drupal/modules--user--user.api.php/function/ho...
Comment | File | Size | Author |
---|---|---|---|
#1 | pass_by_value-1282564-1.patch | 494 bytes | DeeLay |
Comments
Comment #1
DeeLay CreditAttribution: DeeLay commentedAttached patch
Comment #2
Isostar CreditAttribution: Isostar commentedStill present.
Comment #3
Anonymous (not verified) CreditAttribution: Anonymous commentedThe 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.
Comment #4
camprandall CreditAttribution: camprandall commentedThis is just to confirm that the patch fixes the problem. We're using it in production and no longer see the error.
Comment #5
feddovdm CreditAttribution: feddovdm commentedPatch #1 works for me! No longer do i see the error.
Comment #6
Echofive CreditAttribution: Echofive commentedPatch #1 works for me too!
Bug related in my project.
https://drupal.org/node/2027615
Comment #7
blasthaus CreditAttribution: blasthaus commentedwhy is this not committed yet? it's impossible for a user to cancel without correcting this!
Comment #8
wzain CreditAttribution: wzain commentedHi all. I am new to Drupal. How do I apply this patch? Thanks.
Comment #9
hargobindAlso 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".
Comment #10
MichelleStill applies cleanly and works.
Comment #11
el_reverend CreditAttribution: el_reverend commentedAs of 10/20/2015 (!) this is still applicable and should be added to the module. Dear @nicholasThompson or @JohnAlbin and the other maintainers. Would you please apply the patch for the rest of the community?
Comment #12
Michelle@el_reverend: Please read the issue tag guidelines. Tags are for grouping issues, not for emphasizing your need.