Closed (fixed)
Project:
Subscriptions
Version:
5.x-2.0-beta2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Oct 2007 at 15:20 UTC
Updated:
19 Jul 2008 at 23:22 UTC
I'm starting a new thread here for an issue that was erroneously appended to an old unrelated issue. Two users have independently reported it:
The foreach() error is followed by two aftereffects:
It's this code inside node_load(), node.module, called from cron:
// Turn the conditions into a query.
foreach ($param as $key => $value) {
$cond[] = 'n.'. db_escape_string($key) ." = '%s'";
$arguments[] = $value;
}
$cond = implode(' AND ', $cond);
The SQL-errors even remains during cron even after deleting all comments and content on the site.
The errors during cron disappear if i disable the subsriptions module. But initially after the first enabling of the subscription module there were no errors. There start a few days ago.
This is nasty! I don't know how to reproduce it here — any ideas on how to tackle this, chx?
Comments
Comment #1
skizzo commentedI confirm the problem on D52, Subscriptions 15 Sep, PrivMsg 16 Sep.
Trying to upgrade to PrivMsg 2 Oct I get the following error for Update #6:
Other actions ALTER TABLE {users} and UPDATE {users} are ok.
Maybe it is just an issue with mySql permission settings, but I never saw this before.
# user warning: Access denied; you need the SUPER privilege for this operation query: CREATE TRIGGER user_name_length_insert BEFORE INSERT ON users FOR EACH ROW SET NEW.name_length = LENGTH(NEW.name) in /var/www/cms50/includes/database.mysql.inc on line 172.
Comment #2
salvisPlease, please, please do not mix issues!
That other issue is discussed in http://drupal.org/node/180549.
Comment #3
McPille commentedHi,
I tried the new October versions of privatemsg and subscriptions and the errormessages during the cron runs have disapeared.
Thank you very much for fixing the problem that fast!!!!
Andreas
P.S. Sorry for mixing up the problems in the first place. I thought the SQL problems were somehow related. My fault, I should have read more carefully.
Comment #4
(not verified) commentedComment #5
Saganesque commentedSubscribing
Comment #6
salvisThis issue is obsolete. There's no point in "subscribing" and bumping the version number.
Create a new issue, if you have a problem with the current version.