In some initial testing of the 6.x branch of the spam module, I enabled all modules and then on the admin pages I saw the following error:

user warning: Table 'spam68.profile_values' doesn't exist query: SELECT fid, value FROM profile_values WHERE uid = 3 in /var/www/spam68/sites/default/modules/spam/modules/spam_user.inc on line 228.

I've not yet looked into the code to debug.

CommentFileSizeAuthor
#3 351138-profile-table-doesnt-exist.patch983 bytescoltrane
#1 diff.patch633 bytescoreyp_1

Comments

coreyp_1’s picture

coreyp_1 commented
Status: Active » Needs review
StatusFileSize
new diff.patch633 bytes

It looks like the module assumes that the profile module is enabled.

Here's a quick and dirty solution.

TBarregren’s picture

TBarregren commented

I've reviewed and tested the patch. It looks correct and works as expected.

coltrane’s picture

coltrane
he/him
commented
StatusFileSize
new 351138-profile-table-doesnt-exist.patch983 bytes

Profile serializes arrays, such as in the case of profile date fields. spam_user_profile_update() doesn't handle serialized data just putting whatever is returned into the user object. spam_user_profile_update() should just use profile_load_profile instead.

gnassar’s picture

gnassar commented
Status: Needs review » Fixed

#3 committed. Thank you.

Status: Fixed » Closed (fixed)

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