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']);

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2’s picture

erikhopp’s picture

A custom subtheme of Adaptive theme. Could it be something I'm doing?

erikhopp’s picture

And I do have jquery update installed, if that is relevant.

mikeytown2’s picture

Status: Active » Needs review
FileSize
1.01 KB

I 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.

erikhopp’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
931 bytes

Hi!

Yes that idea worked. I rerolled the patch due to a minor error and it is attached.

Thanks for the quick responses.

mikeytown2’s picture

Status: Reviewed & tested by the community » Fixed

Awesome, thanks for the patch! It has been committed :)

mikeytown2’s picture

Followup patch for this one. It has been committed.

Status: Fixed » Closed (fixed)

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