dedlfix: vergiss das Escapen nicht!

Beitrag lesen

Tach!

ich habe es jetzt so umgesetzt. Gibt es daran etwas auszusetzen?

Nur eine Kleinigkeit, oder zwei.

$_SESSION['success_merken'] = 1;

Natürlich merkt man sich in Variablen (auch in Session-Variablen) Werte. Das ist so essenziell, dass man nicht dazuschreiben muss, dass man sich sich dort was merkt. Es reicht also, den Eintrag nur success zu benennen. Zudem sieht mir das nach einem booleschen Wert true/false aus und nicht danach, dass man 1 bis viele Status auseinanderhalten soll. Also nimm lieber ein true statt einer 1, das drückt die Intention besser aus als ein Zahlenwert.

header("Location: artikel.php?kat=$kategorie&artikel=$artikel#succes");

Die URL ist ein Kontext mit bestimmten Regeln. Will man dort Werte einfügen, muss man sich an die Regel halten, damit diese unbeschadet am Ende ankommen. $kategorie und $artikel müssen URL-kodiert werden. urlencode() oder rawurlencode() kannst du dafür nehmen (an der Stelle ist urlencode() richtiger, aber rawurlencode() passt da auch).

dedlfix.