query con join tra 2 tabelle, devo aggiungere un where

[allin81]

questa e' la query che sto utilizzando:

select a.id, a.albergo, a.titolo, DATE_FORMAT(a.valida_da, '%d/%m') AS valida_da, 
DATE_FORMAT(a.valida_fino, '%d/%m/%Y') AS valida_fino, a.price, a.prezzo_come, 
a.arrangiamento, a.notti, a.scadenza, a.pacchetto, b.hotel, b.stelle, b.comune from offerte a, 
albergo b where from_unixtime(scadenza, '%Y-%m-%d') >= CURDATE() and stelle = '3' and 
valida_da <= "20120630" and valida_fino >= "20120601" group by id order by price/prezzo_come 
asc 

pero' non riesco a filtrarli per le 3 stelle che ho impostato, mi visualizza tutte le offerte in generale.
Dove sbaglio?

[satifal]

Di che tipo è la colonna 'stelle' ? Se è di tipo numerico devi togliere gli apici. Inoltre, per correttezza aggiungici anche il riferimento alla tabella.

... and b.stelle = 3 ...

[allin81]

La colonna stelle e' smallint ed ho provato anche come dici tu, ma non ne vuole sapere.
Questa e' la query completa:

$_dataInizio = "$data_giugno_dal";
$_dataFine = "$data_giugno_al";
$query=("select a.id, a.albergo, a.titolo, DATE_FORMAT(a.valida_da, '%d/%m') AS valida_da, DATE_FORMAT(a.valida_fino, '%d/%m/%Y') AS valida_fino, a.price, a.prezzo_come, a.arrangiamento, a.notti, a.scadenza, a.pacchetto, b.hotel, b.stelle, b.comune from offerte a, albergo b where from_unixtime(scadenza, '%Y-%m-%d') >= CURDATE() ");
$colore = 1;
$temp = " and ";

if(isset($_POST['stelle']) && $_POST['stelle']!="0"){
$query.= $temp. "b.stelle = \"$_POST[stelle]\"";
}

if($_dataInizio!=""&&$_dataInizio!=null){
    $query .= $temp."valida_da <= \"".$_dataFine."\"";
}
if($_dataFine!=""&&$_dataFine!=null){
    $query .= $temp."valida_fino >= \"".$_dataInizio."\"";
}

$query .= " group by id order by price/prezzo_come asc";
echo"$query"; 

[satifal]

Gli apici devi toglierli dalla query e NON dall'array $_POST:

...
$query .= $temp."b.stelle = ".$_POST['stelle']; 
... 

P.S.
A volte fate cascare proprio le braccia però

[allin81]

Originariamente inviato da satifal
Gli apici devi toglierli dalla query e NON dall'array $_POST:

...
$query .= $temp."b.stelle = ".$_POST['stelle']; 
... 

P.S.
A volte fate cascare proprio le braccia però

ma veramente non funziona nemmeno se nella query ci metto gia' il numero 3 o 4 (stelle)
Ho anche provato come dici tu, ma mi mostra tutti i record sempre

[allin81]

mi spiego meglio, usando nella query gia' questo valore and b.stelle=\"3\"

$query="select a.id, a.albergo, a.titolo, DATE_FORMAT(a.valida_da, '%d/%m') AS valida_da,
 DATE_FORMAT(a.valida_fino, '%d/%m/%Y') AS valida_fino, a.price, a.prezzo_come,
 a.arrangiamento, a.notti, a.scadenza, a.pacchetto, b.hotel, b.stelle, b.comune from offerte a, 
albergo b where from_unixtime(scadenza, '%Y-%m-%d') >= CURDATE() and b.stelle=\"3\" "; 

[satifal]

Originariamente inviato da allin81
mi spiego meglio, usando nella query gia' questo valore and b.stelle=\"3\"

$query="select a.id, a.albergo, a.titolo, DATE_FORMAT(a.valida_da, '%d/%m') AS valida_da,
 DATE_FORMAT(a.valida_fino, '%d/%m/%Y') AS valida_fino, a.price, a.prezzo_come,
 a.arrangiamento, a.notti, a.scadenza, a.pacchetto, b.hotel, b.stelle, b.comune from offerte a, 
albergo b where from_unixtime(scadenza, '%Y-%m-%d') >= CURDATE() and b.stelle=\"3\" "; 

Perdonami, ma non capisco se ci sei o ci fai!
Ti ho detto che se stelle è di tipo int NON ci vogliono gli apici. Ti ho anche postato come modificare la query! Cos'è quell'obbrobrio che hai scritto?

$query="select a.id, a.albergo, a.titolo, DATE_FORMAT(a.valida_da, '%d/%m') AS valida_da,
 DATE_FORMAT(a.valida_fino, '%d/%m/%Y') AS valida_fino, a.price, a.prezzo_come,
 a.arrangiamento, a.notti, a.scadenza, a.pacchetto, b.hotel, b.stelle, b.comune from offerte a, 
albergo b where from_unixtime(scadenza, '%Y-%m-%d') >= CURDATE() and b.stelle=3"; 

[allin81]

il problema non era negli apici, ho risolto.
In pratica mancava questo nella query

and a.albergo = b.hotel 

ora funziona