In _bakery_register_submit() there is no checking if "profile" module exists like it's in bakery_settings().
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 967682_profile_bakery_register_submit.patch | 1.38 KB | greggles |
In _bakery_register_submit() there is no checking if "profile" module exists like it's in bakery_settings().
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 967682_profile_bakery_register_submit.patch | 1.38 KB | greggles |
Comments
Comment #1
gregglesCan you specify what kinds of problems that causes?
Comment #2
mjanouch commentedIt can result to SQL error "Table profile_fields doesn't exist query: SELECT name FROM profile_fields WHERE register = 1..."
Comment #3
gregglesMarked #971252: profile table database error as a duplicate.
Comment #4
gregglesSomething like this?
Comment #5
gregglesI was able to test this by registering on a subsite. Without the patch we get the message:"User warning: Table 'sd6.profile_fields' doesn't exist query: SELECT name FROM profile_fields WHERE register = 1 in _db_query() (line 141 of /home/greggles/workspace/d6/includes/database.mysqli.inc)."
With the patch no error message about a missing table. I tested that the functionality still works as expected with editing a profile field on master and getting updated on slave and it does.
I plan to commit this shortly but leave it RTBC for others to provide input.
Comment #6
gábor hojtsyThe same error pops up all the time on localize.drupal.org. #971252: profile table database error was submitted based on that (and was marked duplicate). Please commit, so we can roll it out on l.d.o as well.
Comment #7
gregglesNow committed - http://drupal.org/cvs?commit=460026
Comment #8
gábor hojtsyThanks, rolled out on localize.drupal.org with other Drupal 6.x-2.x changes with the alpha2.
Comment #9
drummI don't think this needs to be backported to 6.x, since Drupal 6 will be unsupported in a few months.