UPDATE record

Tadam sono tornato dagli inferi a rompere le P***E,
Oggi ho un piccolo problema insolito visto che ha sempre funzionato.

Praticamente devo fare l'update di UN record che tramite un link utilizza il metodo GET per prelevare l'id del record e cambiarne in questo caso il valore.

Allora per esempio ho un articolo già scritto e devo clickare un bottone per pubblicarlo allora nel link ho fatto:

Codice PHP:

---

echo "<a href='pubblica.php?id=$id'>".$titolo."</a>";


---

in pubblica.php ho questo:

Codice PHP:

---

<?php
$myconn = mysql_connect('localhost', 'root', '') or die('Errore connessione'); //DATI CONNESSIONE AL SERVER
mysql_select_db('articoli', $myconn) or die('Errore connessione database'); //CONNESSIONE AL DATABASE

if(isset($_GET['id'])&&(is_numeric($_GET['id']))){
// valorizziamo la variabile relativa all'id dell'articolo e includiamo il file di configurazione
$id = $_GET['id'];
$stato = "pubblicato";

// selezioniamo dalla tabella i dati relativi all'articolo
$sql = "UPDATE art SET stato = '$stato'";
$query = @mysql_query($sql) or die (mysql_error());

// se per quell'id esiste un articolo..
header('Location: index.php');
} else echo "C'è qualcosa che non va";
?>

---

Però purtroppo quando clicko nel bottone mi pubblica anche gli altri articoli, dov'è il problema? :shock:

SET senza WHERE, si c'è qualcosa che non va...

Citazione:

---

echo "<a href='pubblica.php?id=$id'>$titolo</a>";

---

^___ qui concat non serve, puoi scrivere direttamente cosi'

Codice PHP:

---

<?php
$myconn = mysql_connect("localhost");
mysql_select_db('articoli', $myconn);
if( !isset($_GET['id'])
|| !is_numeric($_GET['id'])) die("C'è qualcosa che non va");
$id = $_GET['id'];
$sql = "UPDATE art SET stato = 'pubblicato' WHERE id = '$id'";
$query = @mysql_query($sql, $myconn);
header('Location: index.php');
?>

---

E' sempre stato così >.<

ri-leggi :)

da sapere: senza una clausola where, update interessa tutti i records di una tabella

Ora se clicko su pubblica non fa niente, aggiorna la pagina e basta


Edit: Scherzo ho sistemato, grazie mille. Ascolta non vorrei essere invadente ma non è che hai skype o qualcosa del genere?

se non ti piace il nuovo codice puoi usare il vecchio ma devi imporre WHERE il_campo_id_dell_articolo = id_ricevuto_con_get
ho supposto che id articolo si chiami id, se si chiama diversamente devi cambiarlo:mrgreen:

Ri leggi :D

no, non usi altri mezzi 'social' :mrgreen: ma sul mio foro ho una chatroom, se fissi un appuntamento prima

Desidero appuntamento allora, devo chiedere due cosette diciamo Off Topic da altervista. Grazie