When I installed i18n it didn't create a "trid" column in my term_data table, so I had a look at the i18n.install script and noticed two issues with the "pgsql" case.
First, in this block:
db_query("ALTER TABLE {term_data} ADD language varchar(12)");
db_query("UPDATE {term_data} SET language=''");
db_query("ALTER TABLE {term_data} ALTER COLUMN language SET NOT NULL");
db_query("ALTER TABLE {term_data} ALTER COLUMN language SET DEFAULT ''");
It looks like the following lines need to be added:
db_query("ALTER TABLE {term_data} ADD trid integer");
db_query("UPDATE {term_data} SET trid='0'");
db_query("ALTER TABLE {term_data} ALTER COLUMN trid SET NOT NULL");
db_query("ALTER TABLE {term_data} ALTER COLUMN trid SET DEFAULT '0'");
Then in this section...
db_query("ALTER TABLE {vocabulary} ADD language varchar(12)");
db_query("UPDATE {vocabulary} SET language=''");
The following lines are duplicates of lines that exist in the {term_data} section, so I assume that they were cut and pasted from there, and that {term_data} should have been replaced with {vocabulary}:
db_query("ALTER TABLE {term_data} ALTER COLUMN language SET NOT NULL");
db_query("ALTER TABLE {term_data} ALTER COLUMN language SET DEFAULT ''");
I am attaching a cumulative patch for review.
Comment | File | Size | Author |
---|---|---|---|
i18n_pgsql.patch.txt | 1.17 KB | klance |
Comments
Comment #1
klance CreditAttribution: klance commentedJust installed 4.7 and it looks like this issue is fixed, so I'm going to go ahead and tag it.
Comment #2
(not verified) CreditAttribution: commented