Ik ben bezig met site te maken en gebruik hierbij smarty.
Nu genereer ik wel html in de smarty code, ipv in de template...
omdat dit volgens mij toch veel simpeler is dan arrays uitlezen in de template file ???

Mijn vraag is, wat vind je van dit stukje code, k ben een junior en leer graag bij :)

PHP Code:

<?php
/* page menu generator :
 * ---------------------
 * Retrieve the pagenames from the database depending on language and show them as menu.
 *  
 * Pages are sorted by page_weight and categorized by page_position_name 
 * 
 * Automatically creates smarty objects using the page_position_name as keyname
 * -->You can easily add new menus in the admin.
 * -->Show them on your site by including  the following in the .tpl file
 *    
 * smarty : {$page_position_name}   
 *  fe: <ul>{$header_menu}</ul>  will show the header menu
 *  fe: <ul>{$footer_menu}</ul>  will show the footer menu 
 *      
*/
mysql_select_db($dbname, $con);
$result = mysql_query("           
          SELECT page_position.page_position_name, page_trans.page_title
          FROM page_position

          INNER JOIN pages
          ON page_position.page_position_id=pages.page_position
          
          
          INNER JOIN page_trans
          ON pages.page_id = page_trans.page_id
          
                    
          WHERE page_trans.language_id='".$_COOKIE['lang']."'
          ORDER BY pages.page_weight ASC
          ");

  while($row = mysql_fetch_array($result))
  {
   $menu[$row[0]] .= '<li><a href=?page='.$row[1].'" title="'.$row[1].'" >'.$row[1].'</a></li>';
   $smarty -> assign ($row[0], $menu[$row[0]]);   
  }
?>

de value van de cookie 'lang' wordt in één van de 1e regels van het script gecheck op het bestaan ervan, ... staat deze niet in de db (of is deze niet actief) dan stopt het script ...

Zou er wel even htmlentities() overheen gooien, de titel en dergelijke dan :)

idd ben ik vergeten... is wel zo dat dit enkel pagina's zijn die door de admin worden toegevoegd niet door de users :) maar beetje voorzichtigheid kan nooit kwaad :)

tnx voor de tip

EDIT : heb nu in de ouput de ' vervangen door " omdat ik \n wou gebruiken (wat voor een of andere reden niet werkt met ' ) maar toch zet hij de <li> elementen niet onder elkaar ... ??
omdat ik geen echo gebruik maar alles opsla in de array ? hoe kan ik het toch laten werken?
iemand een idee?

$menu[$row[0]] .= "<li><a href=?page=".$row[1]."'' title='".$row[1]."'' >".$row[1]."</a></li>\n";
$smarty -> assign ($row[0], $menu[$row[0]]);