Closed (fixed)
Project:
Weather_es
Version:
6.x-1.6
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Nov 2009 at 12:51 UTC
Updated:
14 Feb 2010 at 16:54 UTC
The module is throwing this warning, right after installing and going to the settings page:
user warning: Unknown column 'n.language' in 'where clause' query: SELECT * FROM weather_es_config wec WHERE (n.language ='es' OR n.language ='' OR n.language IS NULL) AND ( wec.uid = 0) in /xxxxxxxx/sites/all/modules/weather_es/weather_es.module on line 125.
Besides, when i try to add a new location, it always stays in the province selection form, which never gets submitted.
I don't know if this is relevant, but i have the i18n module installed.
(On a side note, the INSTALL.txt file says the module works with drupal 5.x, though it's the 6.x version.)
Comments
Comment #1
egarias commentedI have exactly the same problem. I am not using i18n.
Comment #2
bullakio@drupal.org commentedHi,
I've installed weather_es 6.x-1.4 from scratch and I've had NO PROBLEMS IN THE INSTALLATION PROCESS (please try to download the module again and ensure that the md5sum is ok), but with Drupal-6.14 it always stays in the province selection form (like scb sais), but this problem does not exist in Drupal-6.13!!!
I'm asking for help at the forums...
P.S. Could you send information about your MySQL, and Apache versions. It would be help me.
Si podéis ayudar en algo estaría muy agradecido (la paginación no me convenció nunca...)
Comment #3
bullakio@drupal.org commentedAl last I've found a change in the Form API between Drupal 6.x-1.4 and 6.x-1.3... so the weather_es module code must be reviewed...
See: #302240: button broken due to fix various problems when using form storage
Comment #4
egarias commentedUnfortunately I am using Drupal 6.14
Here information about my MySQL, and Apache versions:
MySQL: 5.0.81
Apache: Apache/2.2.11 (Unix) mod_ssl/2.2.11 OpenSSL/0.9.7a Phusion_Passenger/2.2.4 mod_auth_passthrough/2.1 mod_bwlimited/1.4 FrontPage/5.0.2.2635
In the error report "SELECT * FROM weather_es_config wec INNER JOIN node_access na ON na.nid = n.nid" i think here is a problem too, query is using n.nid when should be wec.nid i think.
Gracias
Comment #5
bullakio@drupal.org commentedThanks for the information!!
For the info, I see that other module is trying to change the original weather_es sql (like the Drupal SQL programming style recommends) but it does in a wrong way... so that is not a weather_es problem. You could change the line 400 & 403 in that way and see:
400:
$usrcnfresult = db_query(db_rewrite_sql($sql), 0);to$usrcnfresult = db_query($sql, 0);403:
$usrcnfresult = db_query(db_rewrite_sql($sql), $user->uid);to$usrcnfresult = db_query($sql, $user->uid);The original SQL is:
396:
$sql = "SELECT * FROM {weather_es_config} wec WHERE wec.uid = %d";I hope I can submit a new weather_es to work with Drupal-6.14 soon (but I'm promise nothing).
Comment #6
bullakio@drupal.org commentedInstall weather_es 6.x-1.5 to fix it.
Comment #8
FVSmaster commentedFirst of all, thank you very much for this module, it's exactly what I'm looking for.
The weather module only works for airports, and the nearest I've got it's too far way to obtain relevant information.
Unfortunately, the problem already reported is back. I'm using drupal 6.15 and I've dowloaded your latest version 6.x-1.6
The site I'm developping is multilingual (French and Catalan) and this is the error I'm getting:
user warning: Unknown column 'n.nid' in 'on clause' query: SELECT * FROM weather_es_config wec INNER JOIN active_translation ON n.nid = active_translation.fr WHERE 1 = 1 AND ( wec.uid = 0) in C:\xampp\htdocs\modules\weather_es\weather_es.module on line 404.
I've discovered some more aspects:
-If I'm in http://localhost/admin the module owrks fine.
-If I'm in http://localhost/fr/admin or http://localhost/ca/admin the weather text information is displayed, but the icons are missing
-In any other "regular" page, I'm getting the error.
Any suggestions?
Comment #9
bullakio@drupal.org commentedHola,
Thank you for the feedback.
The weather_es module is not ready for multilingual sites (I never worry about this...).
May be you coud try to do what is said at #5 here or you could change the weather_es_config table to add the n.nid column and the data fr or ca...
To see the images see what the actual path for they is and change the weather_es.tpl.php properly...
All this things are patchings...
Multilingual support will be a good add.
Trebajaré en això, però com sempre... no promet res (es més, ara estic fins al cap de feina...). Bon nadal!!!
Comment #10
egarias commentedHello I was suscribing to the first problem and did not check the weather_es 6.x-1.5. My problem is with the customer he wanted a more customizable weather module.
He looked at the one at elmundo.es or elpais.es where for example in elmundo the widget is very nice and you can click to have more information and select different sites,etc from there.
I am still looking forward for this, i installed the module weather but it has no such functionality (either you define global weather or if you are authenticated you can define several modules but this is not so dinamic)
Enfin, esto esta pendiente.
Gràcies
Comment #11
bullakio@drupal.org commentedLike it is said in the module page, Weather_es doesn't support multilingual sites. All help is welcome. I'm looking for co-maintainers.
With Weather_es user can:
# Supports an "unlimited" number of locations per block. --> one user, one or more locations. The whole site --> one or more locations.
You can modify the wheather_es.tp.php to view in a different way the block (even some data instead of all data).
I'll close this bug because now it is a task, not a bug.
Comment #12
bullakio@drupal.org commented