Closed (fixed)
Project:
Internationalization
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 May 2006 at 21:04 UTC
Updated:
13 May 2006 at 17:23 UTC
From what I see, adding .mysql to the db and running .install (update.php) does not add column 'language' to table 'node' anymore. But current cvs version of i18n seems to depend on that. I'm getting this error (4x) on every attempt to edit a previously created node.
user warning: Unknown column 'language' in 'field list' query: SELECT language FROM node WHERE nid=67 in
/www/htdocs/.../includes/database.mysql.inc on line 120.
Comments
Comment #1
profix898 commentedSorry, this was meant for cvs ...
Comment #2
klance commentedYeah, it seems like the i18n.install script isn't working. I downloaded a fresh copy of Drupal 4.7 and i18n, and when I activated i18n, it didn't seem to modify the database. When I tried to create a new Page node, the following error confirmed this:
warning: pg_query(): Query failed: ERROR: column v.language does not exist in database.pgsql.inc on line 84.
Whatever you do, don't use the old i18n.pgsql that's still bundled with this module, however, because it contains the old i18n schema.
Comment #3
klance commentedI should mention that I'm using PostgreSQL 8.x, and I realize that Drupal leans toward MySQL, so the install script will likely do something different in MySQL.
Comment #4
klance commentedMy mistake; I did another update and it looks like the taxonomy tables have been removed from i18n.pgsql, so it's up-to-date with the current cvs. I'm going to try it again, first installing i18n.pgsql and then the module, to see if i18n.install works in that order.
Comment #5
jose reyero commentedI've updated again the install file.
However, I'm not using PostgreSQL at all, so I appreciate any feedback and/or help with that update scripts for pgsql
Then I can set it to fixed?
Comment #6
profix898 commentedSorry, but there is still code depending on column 'language' in table 'node'. Look at i18n.inc function i18n_node_get_lang (line 297). I think this must be changed to i18n_node ...
Comment #7
jose reyero commentedFixed, thanks again.
Comment #8
jose reyero commentedComment #9
klance commentedSorry if this isn't the right place for this question, but I see that i18n_node still exists while i18n_taxonomy has been deprecated for 'language' and 'trid' columns in the term_data table. Will nodes and their translations be eventually correlated in this way as well?
Comment #10
jose reyero commented> i18n_node still exists while i18n_taxonomy has been deprecated for 'language' and 'trid' columns in the term_data table. Will nodes and their translations be eventually correlated in this way as well?
I cannot tell you, it will depend on the evolution of Drupal I guess.
It doesn't really make too much difference whether they're on their own table or in a different one. But for this version of Drupal -and without patching the core- it had some advantages having it like that, like saving database queries.
Comment #11
klance commentedOkay, thanks.
Comment #12
jose reyero commented