I think this error occurs because spam_filter_duplicate_update_6101 changes the names of columns in the table, but no-one bothered to change spam_filter_duplicate_schema().

Unknown column 'spam_filter_duplicate_hash' in 'field list' query: UPDATE spam_filter_duplicate SET spam_filter_duplicate_hash = spam_filter_duplicate_hash + 1 WHERE content_hash = '1d9eadc2a7be1958c1c21257ea2b89fa' in /f5/greatsunflower/public/profiles/custom/modules/spam/filters/spam_filter_duplicate/spam_filter_duplicate.module on line 315.

CommentFileSizeAuthor
#2 spam-6.x-spam_filter_duplicate_hash_column.patch825 bytesAlexisWilke

Comments

Dave Cohen’s picture

Dave Cohen commented

Correction, spam_filter_duplicate_update_6101() changes variables, not columns. I don't know where the bug lies here, but I dont see how the code could work for anybody.

AlexisWilke’s picture

AlexisWilke commented
Status: Active » Needs review
StatusFileSize
new spam-6.x-spam_filter_duplicate_hash_column.patch825 bytes

Actually the update_sql() function does not take %s arguments either... At least last time I checked. 8-)

I suggest the following patch for the 1st error reported (not the misuse of update_sql()).

Thank you.
Alexis Wilke

Dave Cohen’s picture

Dave Cohen commented
Status: Needs review » Reviewed & tested by the community

Looks resonable to me, I don't know if I have the expertise to "review" it. Have applied it and haven't seen errors.

jeremy’s picture

jeremy commented
Status: Reviewed & tested by the community » Fixed

Thanks, fix committed:
http://drupal.org/cvs?commit=468960

gnassar’s picture

gnassar commented
Status: Fixed » Needs work

This misses the point of update 6101, doesn't it?

The problem was, in the first place, that variables weren't properly namespaced. The fix needs to go in the other direction.

AlexisWilke’s picture

AlexisWilke commented

@gnassar,

Ah. That isn't a variable though... It's a field in a namespaced table. Are you sure you want to do that on all the fields?

Thank you.
Alexis

gnassar’s picture

gnassar commented
Status: Needs work » Fixed

No, I think you're correct. I was working under the presumption that the project description was correct and that this has something to do with update_6101. But on second glance, this problem actually has nothing to do with that. It is simply a misnamed field.

Remarking "fixed."

Status: Fixed » Closed (fixed)

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