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.
After my web host moved me to a new server running PHP 5.3.6, I started getting *piles* of these errors in my log file: "Parameter 2 to me_load() expected to be a reference, value given in /var/www/html/includes/menu.inc on line 407."
I'm running Drupal 6.2, which is supposedly PHP 5.3-friendly.
I'm not sure if me_load needs to be more tolerant or if menu needs to be more careful, but somebody is irritating PHP 5.3.
Thanks for your hard work on this,
- Tim.
Comment | File | Size | Author |
---|---|---|---|
#4 | me_noreferencemapd7.patch | 1.97 KB | deggertsen |
#2 | me-noreferencemap-1221902-2.patch | 1.81 KB | neilnz |
Comments
Comment #1
neilnz CreditAttribution: neilnz commentedThis sounds like the same issue as I'm having with me_uid_optional_load() expecting $map to be a reference. See also #1252424: Incompatible with 'me' aliases for this causing a problem with the UC Addresses module indirectly.
Is there a good reason for $map to be taken by reference? If not, let's patch to remove the &
Comment #2
neilnz CreditAttribution: neilnz commentedWell here's a patch to do it in case anyone needs it.
Basically just replaced all instances of &$map with $map in the loader function definitions.
This may or may not apply to the D7 version, depending on how menu.inc works. I can't see any sign that _menu_check_access() in D7 behaves any differently than the D6 one. The same call_user_func_array() call is used to invoke the loader function, which will probably cause the same problem.
Comment #3
tomsm CreditAttribution: tomsm commentedI had the same problem after my webhost updated php from 5.2.x to 5.3.10.
My drupal version is 6.25.
Applying this patch solved it. Thank you!
Comment #4
deggertsen CreditAttribution: deggertsen commentedHere is a patch for the D7 version. Unfortunately, even though this patch fixes this error I'm getting a large number of other errors that will require me to uninstall =(
Comment #5
berenddeboer CreditAttribution: berenddeboer commentedGet "The message: Warning: Parameter 2 to me_load() expected to be a reference, value given in _menu_load_objects() (line 590 of /includes/menu.inc)."
Installing me actually breaks a site.
Comment #6
berenddeboer CreditAttribution: berenddeboer commentedComment #7
gaele CreditAttribution: gaele commentedComment #8
gaele CreditAttribution: gaele commentedPatch applies and works for me on 1.2.
Comment #9
Anybody