Script funktioniert nicht mehr

[AmicaNoctis]

029| if(!isset($site))
030| {
031|   $site="index";
032| }
033| if($site == index)
034| { 

Völlig klar, weil du anscheinend ignorierst, was dir gesagt wurde: $site gibt es nicht. Nimm stattdessen $_GET["site"]! Überall!

[Murderdeath]

also überall wo $site steht soll ich mit $_GET["site"] ersetzen? Sorry das ich nochmals nachfrage...

[AmicaNoctis]

Geeenau.

[Murderdeath]

super, vielen, vielen Dank!! Es klappt jetzt erstmal alles wieder...

Jetzt noch die Frage zu den Sicherheitslücken? habe ich die behoben dadurch, dass ich jetzt mit mysql_real_escape_string arbeite?

[AmicaNoctis]

SQL Injection ist (soweit ich das sehen kann) nicht mehr möglich. Ob es noch weitere Lücken gibt und gab, weiß ich jetzt nicht.

[onemorenerd]

Das hättest du auch einfacher haben können.

$site = isset($_GET['site']) ? $_GET['site'] : 'index'; 

ganz am Anfang des Scripts hätte genügt.