I just tried both the alpha version of this module and the dev version and I'm getting the same errors:
On 7.x-2.0-alpha1: [Fri May 03 14:12:53 2013] [error] [client 24.5.79.61] FastCGI: server "/web/website/fcgi/php-fcgi" stderr: PHP Fatal error: Only variables can be passed by reference in /web/website/beta/staging/www/sites/all/modules/advagg/advagg.module on line 504
On 7.x-2.x-dev: [Fri May 03 14:19:59 2013] [error] [client 24.5.79.61] FastCGI: server "/web/website/fcgi/php-fcgi" stderr: PHP Fatal error: Only variables can be passed by reference in /web/website/beta/staging/www/sites/all/modules/advagg/advagg.module on line 535, referer: https://staging.beta.website.com/admin/config/development/performance
Both of those line numbers correspond to this line:
$variables['page_top'] = drupal_render($variables['page']['page_top']);
Comment | File | Size | Author |
---|---|---|---|
#7 | advagg-1986326-7-fix-notice.patch | 807 bytes | mikeytown2 |
#5 | advagg-page-vars-error.diff | 931 bytes | erikhopp |
#4 | advagg-1986326-4-only-render-isset.patch | 1.01 KB | mikeytown2 |
Comments
Comment #1
mikeytown2 CreditAttribution: mikeytown2 commentedWhat theme are you using? Related #1945800: Fatal error: Only variables can be passed by reference
Comment #2
erikhopp CreditAttribution: erikhopp commentedA custom subtheme of Adaptive theme. Could it be something I'm doing?
Comment #3
erikhopp CreditAttribution: erikhopp commentedAnd I do have jquery update installed, if that is relevant.
Comment #4
mikeytown2 CreditAttribution: mikeytown2 commentedI assumed that
$variables['page']['page_top']
and$variables['page']['page_bottom']
exists as it does in core. Seeing how you're the second one the report this I have a patch for you to try out.Comment #5
erikhopp CreditAttribution: erikhopp commentedHi!
Yes that idea worked. I rerolled the patch due to a minor error and it is attached.
Thanks for the quick responses.
Comment #6
mikeytown2 CreditAttribution: mikeytown2 commentedAwesome, thanks for the patch! It has been committed :)
Comment #7
mikeytown2 CreditAttribution: mikeytown2 commentedFollowup patch for this one. It has been committed.