hej!

napisałem proste wybieranie działów z pliku lub z bazy danych, za pomoca ID działu lub jego nazwy

wywoływanie strony index.php?content=NAZWA_LUB_ID_DZIALU

w bazie sa zapisane informacje o dziala w where_c jesli 1 to dzial na pliku jesli 0 to tresc dzialu w bazie.

[PHP] pobierz, plaintext 
<?php
 
 
if(empty($_GET[&#092;"content\"])) {
    //content selection
    include(&#092;"content/news.php\");
}
else {
    //else content selection
    $sql = 'SELECT * FROM contents WHERE name_c = \"'.$_GET[&#092;"content\"].'\" OR id_c = \"'.$_GET[\"content\"].'\"';
    $res = $db -> sql_query($sql);
    $row = $db -> sql_fetchrow($res);
    if($num = $db -> sql_numrows($res) > 0) {
        // if something is in database
        if($row[&#092;"where_c\"] == 1 & file_exists(\"content/\".$row[\"name_c\"].\".php\")) {
            // if it's a file
            include(&#092;"content/\".$row[\"name_c\"].\".php\");
        }
        elseif($row[&#092;"where_c\"] == 0) {
            //if it's in database
            include(&#092;"content/database_content.php\");
        }
        else {
            // if not exist
            error_code(&#092;"BRAK_DZIALU\");
        }
        // end if it's file
    }
    else {
        // if not exist
        error_code(&#092;"BRAK_DZIALU_DUZE\");
    }
    // end checking in database
}
//end content selection
 
 
?>
[PHP] pobierz, plaintext 

i pytanko, ja to w.g was wyglada, dobrze, lamersko? jakies inne propozycje na zorganizowanie takiego wyboru.

Warning: security vulnerability

ta zmiana wystarczy?

[PHP] pobierz, plaintext 
<?php
 
$sql = 'SELECT * FROM contents WHERE name_c = \"'.mysql_escape_string($_GET[&#092;"content\"]).'\" OR id_c = \"'.mysql_escape_string ($_GET[\"content\"]).'\"';
 
?>
[PHP] pobierz, plaintext