Counter mit MySQL

[Matze]

Hallo,

[...]
  $sql = "select * from nacms_news where id='".$_GET['id']."'";
    $result = mysql_query($sql);
    $news = mysql_fetch_assoc($result);
    $id = $news['id'];
[...]
$user = mysql_query("SELECT Clicksun FROM nacms_news WHERE id = $id");
  $user+=1;
  echo"$user";
  mysql_query("UPDATE nacms_news SET Clicks = $user WHERE id = $id");
[...]

Ich wundere mich über deinen Programmierstil.
Einmal verwendest du

$sql = "select * from nacms_news where id='".$_GET['id']."'";
    $result = mysql_query($sql);

dann

$user = mysql_query("SELECT Clicksun FROM nacms_news WHERE id = $id");

Warum nicht beide male den Query-String extra _oder_ in der Funktion?
Dann schreibst du den String einmal "select... where id='".$_GET['id']."'" und einmal "SELECT... WHERE id = $id".
Irgendwas stimmt da doch nicht. Dann solltest du die $_GET-Variable vorher escapen. mysql_real_escape_string()

Grüße, Matze