After I installed the module, everything appeared to be working fine. But after a day of working on the site, I got this error on page load (showing in the drupal error messages div):
Deprecated function: Call-time pass-by-reference has been deprecated in drupal_load() (line 1128 of /.../includes/bootstrap.inc)
When I tried reloading the page I got a blank white page. Looking in apache's error log, I could see the following apc error:
[Mon Jun 25 13:47:53 2012] [apc-error] Cannot redeclare class insertquery_mysql in /data/web/ext/drupal7/includes/database/database.inc on line 1742.
All page requests from here on generated a blank, white page. On a hunch, I disabled the publish_button module with drush (it was the last module I installed) and the problem went away.
Hope these bug reports help. I really like the idea behind this module. I can't tell you how many times I've been asked, "Where's the publish/unpublish setting again?"
Tom
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | publish_button-calltime-passbyreference-1659386-2.patch | 805 bytes | floretan |
Comments
Comment #1
misc commentedHm, I will try to reproduce this, I have been devolping with apc cache on, and did not get this, but I will sure look into it.
Comment #2
floretan commentedI'm not sure this is related to apc, it just happens that apc is reporting it. The source is the call-time pass-by-reference, an instance of http://drupal.org/node/1097310.
Calling
node_form_submit()should use$form_stateinstead of&$form_state. The attached patch fixes that.Comment #3
misc commentedOff course! Thanks @floretan, committed to latest dev.
Comment #5
dooblem2 commentedto fix this problem, we upgraded the APC version from 3.1.3p1 to 3.1.9