Closed (outdated)
Project:
Advertisement
Version:
6.x-2.x-dev
Component:
adserve.php
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Dec 2010 at 07:48 UTC
Updated:
19 Sep 2024 at 11:07 UTC
Jump to comment: Most recent
Comments
Comment #1
Shutnic commentedComment #2
bbcSame problem here on 6.x-2.2. I got the error on attempts to flush the cache, run database updates and when running cron. I tried the suggested change from Shutnic in comment #1 above and it solved the problem for me.
Also, this appears to be a duplicate of http://drupal.org/node/877482
Thanks Shutnic!
Comment #3
WeRockYourWeb.com commentedThanks Shutnic, I replaced lines 130-145 with the following and flushing cache works now:
Comment #4
Priyanka Singh commentedwow ..thanks Shutnic..it worked...
Comment #5
bbcIt seems this issue remains in 6.x-2.3 and 6.x-2.4. In this case, I was getting the error "The external command could not be executed due to an application error" when attempting to upgrade the module using drush. Downloading the latest version manually and reapplying the patch from above to adserve.inc took care of the issue.
Comment #6
john franklin commentedCould you do me a favor? On a test version of your site add a "var_dump(array_keys($conf));" between the isset() and foreach() lines. Somewhere in your $conf you you have an empty key, NULL or 0, none of which belong in your site's config array.
Comment #7
WeRockYourWeb.com commentedGetting the error again with the latest release (with or without the patch), and when I try to add
var_dump(array_keys($conf));or update the module, this error shows up as well:Parse error: syntax error, unexpected '!' in /modules/ad/adserve.inc on line 342Comment #8
andrewfn commentedThe fix in #3 worked perfectly for me,
Thanks WeRockYourWeb.com
Can we get this into a patch?
Comment #9
andrewfn commentedThis problem is still here.
Just had the annoyance of having to re-patch the latest dev
Please could it get committed?
Comment #10
lrwebks commentedDrupal 6 is EOL and no longer supported. Closing this as outdated for that reason. Thanks for your contribution!