Multicomune: problema con tutte le consultazioni chiuse

[margottid]

Salve a tutti,
l'avevo già segnalato in un altro post, adesso lo riporto perché ho analizzato la situazione...

In una installazione multicomune, se tutte le consultazioni di un Comune sono chiuse, questo sparisce dai Comuni visualizzati nell'interfaccia client.

Nel file "menu.php" (della cartella client\temi\facebook) c'è la riga 61:

Codice: Seleziona tutto

$sqlcomu="select t1.id_comune,t1.descrizione,count(0) from ".$prefix."_ele_comuni as t1, ".$prefix."_ele_cons_comune as t2 where t1.id_comune=t2.id_comune and t2.chiusa<2 group by t1.id_comune,t1.descrizione order by t1.descrizione asc";

che dovrebbe scegliere i Comuni che hanno consultazioni con stato chiusa<2, ovvero con consultazioni attive (0) o chiuse (1), visto che 2 equivale a nulla.

Andrebbe tutto bene, se non fosse che il campo "chiusa" della tabella _ele_cons_comune non è un intero, ma è definito come set('0', '1', '2'), che è un oggetto stringa.
Lanciando la query su MySQL, il risultato non è quello che ci si aspetta, forse perché si confronta una stringa con un valore numerico.

Modificando la riga in questo modo:

Codice: Seleziona tutto

$sqlcomu="select t1.id_comune,t1.descrizione,count(0) from ".$prefix."_ele_comuni as t1, ".$prefix."_ele_cons_comune as t2 where t1.id_comune=t2.id_comune and t2.chiusa<'2' group by t1.id_comune,t1.descrizione order by t1.descrizione asc";

(cioè, racchiudendo il "2" tra apici) si dovrebbe risolvere il problema.

Riuscite a verificare, ed eventualmente a tenerlo presente per la prossima revisione?
In questo modo sarà possibile chiudere le consultazioni (come è giusto fare) senza veder sparire i Comuni che non hanno più consultazioni attive.

   Daniele

[roby]

Hai ragione Daniele, il problema sta nell'aver trattato il dato come numerico mentre è un set, per sistemare (prima di mandare la prossima rev faccio altre modifiche) è meglio dichiararlo diverso da 2, cioè $chiusa!='2'

[margottid]

Ciao,
stesso problema con il tema Futura2 (me ne sono accorto scaricando l'app stamattina): nel file "index.php" del tema, la query per cambiare Comune nel caso di installazione multicomune (riga 400 del file) ha ancora la clausola where "t2.chiusa<2", e anche qui bisognerebbe cambiarla con t2.chiusa!='2'.

   Daniele

[luc]

Ciao, corretto.
Luc