? If $site ? /(o)(o)\

[Schaffi89]

Hy Leute.

Ich arbeite viel mit Webspell.
Da gib es ein Script der stellt den inhalt der jeweiligen Seite dar.

if(!isset($site)) $site="news";
//Sichheitslücke beheben
$invalide = array('/','/\/',':','.');
$site = str_replace($invalide,' ',$site);
if(!file_exists($site.".php")){ echo'<h2>Error 404</h2><br>Die von Ihnen angeforderte Datei namens <b>'.$site.'.php</b> ist nicht vorhanden.';
redirect("index.php?site=news",'',3);
}
include($site.".php");


Dies funktioniert auch, möchte aber gerne in meiner eigenden Homepage dies einbauen so das ich nur noch die index verändern muss um irgendwelche Stylesachen zu verändern.
Habe dieses Script auch ausprobiert kommt aber immer eine Fehlermeldung

Fatal error: Call to undefined function gettemplate()

Um z.B. auf die Seite Impressum zukommen muss man nur noch den Hyperlink so benennen "index.php?site=news", dies funktioniert aber nicht.


Danke schonmal im Vorraus

[szk]

Fatal error: Call to undefined function gettemplate()

Du solltest bei der Fehlermeldung auch eine Position
(welche php-Datei, welche Zeile) bekommen...
da scheint es den Funktionsaufruf gettemplate() zu
geben den PHP nicht kennt.

sk.

[feuervogel]

ich hab keine ahnung was dein problem ist, denn im dem von dir geposteten codeschnipsel kommt kein get_template() vor.

und bevor du irgendwelchen komplizierten code versuchst zu kopieren (was wahrscheinlich softwareklau ist) und ihn dir dann zurechtfrickelst, versuch doch lieber mal ihn komplett selbstzuschreiben.

bei problemen kannst du gerne hier fragen :o)

[Schaffi89]

Wie gesagt zurechtgefrickelst habe ich den code nicht der steht so bei Webspell.
Weißt du vielleicht wie ich den code schreiben könnte.
Ich bin ein absoluter Anfänger in PHP.

[feuervogel]

man kann es so machen:

PHP-Code:

switch( $_GET[ 'site' ] ) {

case 'news':

include( 'impressum.php' );

break;

default:

include( 'start.php' );

} 


wenn du nun also index.php?site=news aufrufst, wird die datei impressum.php eingebunden und sonst die datei start.php (natürlich müssen die beiden dateien existieren).

das ist so ungefähr das was dein codeschnipsel macht (nur etwas weniger aufwändig).

allerdings wird weiterhin die fehlermeldung auftauchen, da ja somit der aufruf "get_template()" nicht entfernt wurde und weiterhin zum fehler führt.

desweiteren: kauf dir lieber ein gescheites buch anstatt dir code von webspell zu übernehmen und dann wegen überforderung andere für dich programmieren zu lassen.

[Swoosh]

kannst es auch so machen

PHP-Code:

<?
$site = $_GET['site'];
$file = 'sites/'.$site.'.php';

if(!isset($site)) {
  header("Location: ?site=home");
}

if(file_exists($file)) { 
  echo 'Datei nicht gefunden!'; }
}
else {
  include($file);
}
?>

Oder das ganze zusammen gefasst

<?

PHP-Code:

$section = $_GET['section'];
$incsite = empty($section) ? "home.php" : (file_exists($section.".php") ? $section.".php" : "404.php"); 

include($incsite); 


?>

[Schaffi89]

Danke !!!!!!!

Funktioniert alles.
Habe mich mal bisschen belesen zu den If Anweisungen ist ja fast wie im VB.

Danke nochmal.

[Domi]

Zitat von Swoosh

Oder das ganze zusammen gefasst

<?

PHP-Code:

$section = $_GET['section'];
$incsite = empty($section) ? "home.php" : (file_exists($section.".php") ? $section.".php" : "404.php"); 

include($incsite); 


?>

auch nicht gut, du solltest die slashes und . und .. rausfiltern, oder einfach in nem array alle erlaubten seiten definieren und dann mit in_array() einfach abfragen, ob diese vorhanden ist.

gruss
Domi

[feuervogel]

Zitat von Domi

auch nicht gut, du solltest die slashes und . und .. rausfiltern, oder einfach in nem array alle erlaubten seiten definieren und dann mit in_array() einfach abfragen, ob diese vorhanden ist.

gruss
Domi

hm, eigentlich empfiehlt sich da ein switch-statement, weils übersichtlicher ist :-) und so wie ich es vorgeschlagen habe, kann eigentlich (fast) nichts passieren...