The reset() function should have its variables passed by reference. I've uploaded a patch that removes the notice.

CommentFileSizeAuthor
#5 variables-passed-by-reference-3030478-5.patch882 bytescollinhaines
#3 variables-passed-by-reference-3030478-2.patch883 bytescollinhaines
variables-passed-by-reference.patch882 bytescollinhaines

Comments

collinhaines created an issue. See original summary.

junaidpv’s picture

junaidpv
Location Kannur, Kerala
commented

We also getting this. Some of our Ajax calls fails because of this error, of course in dev site.

collinhaines’s picture

collinhaines commented
StatusFileSize
new variables-passed-by-reference-3030478-2.patch883 bytes

Updated for 1.62.

cboyden’s picture

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

Tested the patch; it's working as expected.

collinhaines’s picture

collinhaines commented
StatusFileSize
new variables-passed-by-reference-3030478-5.patch882 bytes

Updated for 1.71.

  • dsnopek committed fabfb7a on 7.x-1.x authored by collinhaines 
    Issue #3030478 by collinhaines, junaidpv, cboyden: Notice: Only...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks! Committed with a minor change to variable names

Status: Fixed » Closed (fixed)

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