Passaggio php4 -> php5

[amadeus1982]

ciao a tutti!

il server di un sito a cui sto lavorando ha pensato bene di trasferirsi notte tempo, cambiando la versione PHP da 4 a 5.

manco a dirlo, lo script che prima andava perfettamente, ora non va più un kaiser.

se mi dessi una mano mi faresi un grosso favore...

questo è il codice maledetto

<?php include('../common/menugenerale.php'); ?>
    <?php include('menusezione.php'); 
    include ('config.inc.php'); 
    $db = mysql_connect($db_host, $db_user, $db_password);
    if ($db == FALSE)
    die ("Errore nella connessione. Verificare i parametri nel file config.inc.php");
    mysql_select_db($db_name, $db)
    or die ("Errore nella selezione del database. Verificare i parametri nel file config.inc.php");
    
    if ($id == 'tutto') {
        $query = "SELECT * FROM iconografia WHERE ico_hs = '' ORDER BY ico_nomefile";
    }
    else
        $query = "SELECT * FROM iconografia where ico_hs = $id ORDER BY ico_nomefile";
    $result = mysql_query($query, $db) or die(mysql_error());?>

e questo è il messaggio di errore che ricevo:

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near 'ORDER BY ico_nomefile' at line 1

qualcuno mi può aiutare pliz?!?!? non ci capisco più nulla, sto fondendo!!!

[thepooh]

A prima vista gli errori possono essere due:
-il campo ico_nomefile non esiste, ma mi sembra ti dia un'altro tipo di errore
-$id è nullo, quindi la query di risultato sarebbe
WHERE ico_hs = ORDER BY ico_nomefile
Fatti una echo e vedi che query mandi sul db

echo $query;
//  $result = mysql_query($query, $db) or die(mysql_error());
?> 

Comunque manca la parentesi
}
della chiusura della if!

[amadeus1982]

ho risolto sostituendo $id con $_GET[id]

non chiedetemi il perchè

ma funziona

grazie mille!

[thepooh]

Originariamente inviato da thepooh
-$id è nullo, quindi la query di risultato sarebbe
WHERE ico_hs = ORDER BY ico_nomefile
Fatti una echo e vedi che query mandi sul db

[nik600]

a quanto pare prima avevi i register_globals a 1...

questo significa che ogni variabile, get o post, viene automaticamnte resa disponibile senza dovervi accedere rispettivamente con $_GET[] o $_POST[]

[13manuel84]

Originariamente inviato da amadeus1982
ho risolto sostituendo $id con $_GET[id]

non chiedetemi il perchè

ma funziona

grazie mille!

beh ora però devi cambiare ogni variabile del sito che ricevi da get o da post

sempre detto che è meglio lavorare come se le register_globals fossero a Off, così va bene ovunque

[amadeus1982]

non ci metto molto.
faccio un bell'extended replace

grazie mille a tutti!