[MySQL] UPDATE SET non fa

[shadyz]

PREMESSA: Questo codice non aggiorna..

pagina 1

        // ESEMPIO DI PAGINA [B]pagina.php?id=2[/B]

    $db = mysql_connect($db_host, $db_user, $db_password)
        or die ("Impossibile connettersi: " . mysql_error());
    mysql_select_db($db_name, $db)
        or die ("Errore nella selezione del database.");
    
    
    $select = mysql_query ("SELECT * FROM tabella WHERE id = '$id'", $db);
    $record = mysql_fetch_array ($select);
    
    <form method="post" action="update.php" name="upload">
        <input type="text" disabled="disabled" size="2" name="id" value="{$record['id']}" />
        <input type="text" size="80" name="titolo" value="{$record['titolo']}" />
        <input type="text" size="80" name="testo" value="{$record['testo']}" />
    </form> 

pagina 2

    $db = mysql_connect($db_host, $db_user, $db_password)
        or die ("Impossibile connettersi: " . mysql_error());
    mysql_select_db($db_name, $db)
        or die ("Errore nella selezione del database.");
    
    $id = $_REQUEST['id'];
    
    $update = mysql_query ("UPDATE tabella SET
                        id = '$id' ,
                        titolo = '$titolo' ,
                        testo = '$testo'
                        
                        WHERE id = '$id'
                        ", $db); 

[digitos]

<form method="post" action="update.php" name="upload">
        <input type="hidden" name="id" value="<?php echo $record['id']?>" />
        <input type="text" style="width:80px;" name="titolo" value="<?php echo $record['titolo']?>" />
        <input type="text" style="width:80px" name="testo" value="<?php echo $record['testo']?>" />
    </form>

pure accessibile

[shadyz]

non è quello il problema. Quella parte è già in PHP, ho usato un heredoc.

[digitos]

Sono query con le quali lavoro tutti i giorni. No vedo errori a meno che la query sia errata.

Facciamo un controllata...
anche se non dovrebbe cambiare nulla, prova a sostituire $_REQUEST con $_POST

poi controlliamo che la query sia corretta quindi

$update = mysql_query ("UPDATE tabella SET id = '$id' ,titolo = '$titolo' ,testo = '$testo' WHERE id = '$id'", $db) or die(mysql_error()); 

e già che ci siamo mettici anche un bel


error_reporting(E_ALL);

all'inizio della pagina di update.php così vediamo tutti gli errori.

ciao

[rasega]

Non è che il campo id della tabella è autoincrement ??
Forse non ti funziona la query per quello, non dovrebbe essere settabile se è autoincrement...

[luca200]

1) "Non funziona" non vuol dire niente. Impara a spiegare qual è l'errore

2) "$id = $_REQUEST['id']" è inutile se hai i register_globals a on. Se invece li hai a off, lo devi fare non solo su $id, ma su tutti i campi in input. Qual è il tuo caso?

Poi ci sarebbe da chiedersi che senso ha mettere il campo ID nella SET, ma temo che andiamo su livelli troppo alti.......

[digitos]

Originariamente inviato da luca200

Poi ci sarebbe da chiedersi che senso ha mettere il campo ID nella SET, ma temo che andiamo su livelli troppo alti.......

infatti non ho tirato fuori questa questione altrimenti ci sarebbe da aprire un altro capitolo

[shadyz]

Ho scoperto che il problema è dato da questa stringa

<input type="text" disabled="disabled" size="2" name="id" value="{$record['id']}" /> 

ed in particolar modo da disabled="disabled".
Ho risolto per ora togliendo il disabled. Ma come posso ottenere lo stesso risultato? Hidden non ha lo stesso effetto...

[digitos]

Originariamente inviato da shadyz
Ho scoperto che il problema è dato da questa stringa

<input type="text" disabled="disabled" size="2" name="id" value="{$record['id']}" /> 

ed in particolar modo da disabled="disabled".
Ho risolto per ora togliendo il disabled. Ma come posso ottenere lo stesso risultato? Hidden non ha lo stesso effetto...

Appunto in questi casi è concettualmente sbagliato quello che stai facendo.
Come ti ho fatto vedere prima, il campo id dev'essere semplicemente passato alla pagina update. Non c'è necessità che questo valore venga visualizzato onde per cui si utilizza hidden. In questi casi si UTILIZZA SEMPRE hidden.

Come ti hanno detto già altri in questo thread, non avresti nemmeno bisogno di passare il valore id. Basterebbe assegnare in mysql al campo id l'attributo "autoincrement".

Usando disable=disable questo attributo non è "valido" per una corretta accessibilità ed usabilità della pagina.

[rasega]

Siccome ho avuto un problema simile aggiungo che volendo mantenere la select per mostrare il valore selezionato, ma non poterlo modificare, puoi comunque renderla disabled ed aggiungere effettivamente un campo hidden per passare in post l'id.
Dal punto di vista della renderizzazione grafica della pagina non si ottengono gli stessi effetti su browser differenti, se non sbaglio IE7 non rende grigia la select disabilitata che dovrai di conseguenza gestire tramite gli stili.

Ciau !!