é corretta questa modifica?

[fdr77]

riguardo questa mod il top menu
http://custom.simplemachines.org/mods/index.php?mod=2564
volevo sapere se ho fatto bene
Nel tema in uso ho già una cosa simile ma questa è più carina...quindi volevo sapere se cambiare da così

Code Select Expand

function template_body_above()
{
global $context, $settings, $options, $scripturl, $txt, $modSettings;

echo '
<div id="topbar">
<div id="header" class="wrapper">
<div id="toplinks">
<ul>';
if (!empty($settings['show_date_time']))
echo '
<li class="time">', $context['current_time'],'</li>';

if(!empty($settings['toplink_but1']))
echo '
<li><a href="', $settings['toplink_url1'] , '">' , $settings['toplink_but1'] , '</a></li>';

if(!empty($settings['toplink_but2']))
echo '
<li><a href="', $settings['toplink_url2'] , '">' , $settings['toplink_but2'] , '</a></li>';

if(!empty($settings['toplink_but3']))
echo '
<li><a href="', $settings['toplink_url3'] , '">' , $settings['toplink_but3'] , '</a></li>';

echo'
</ul>
</div>';
if(!empty($settings['show_search']))
{
echo '
<div id="searchbar">
<form action="', $scripturl, '?action=search2" method="post" accept-charset="', $context['character_set'], '">
<input class="inputbox" type="text" name="search" value="', $txt['forum_search'], '" onfocus="this.value = \'\';" onblur="if(this.value==\'\') this.value=\'', $txt['forum_search'], '\';" />';

// Search within current topic?

a così

Code Select Expand

function template_body_above()
{
global $context, $settings, $options, $scripturl, $txt, $modSettings;

    echo '
<div id="topbar">
<div id="header" class="wrapper">
<div id="site_menu" class="align_right" style="padding-top: 9px; font-size: 16px;">
<ul class="dropmenu" id="site_nav">';
if (!empty($settings['show_date_time']))
echo '
<li class="time">', $context['current_time'],'</li>';
                        for ($i=1; $i<6; $i++)
      {
         if (!empty($modSettings['tab_url' . $i]) && !empty($modSettings['tab_title' . $i]))
         {
            echo '
               <li id="button', $i, '">
                  <a class="firstlevel" href="http://www.discusrewind.net/', $modSettings['tab_url' . $i],'"';
if(!empty($modSettings['newpage' . $i]))
echo ' TARGET="_blank"';
echo '>
<span class="firstlevel">', $modSettings['tab_title' . $i], '</span>
                  </a>
               </li>';
         }
      }

echo'
</ul>
</div>';
if(!empty($settings['show_search']))
{
echo '
<div id="searchbar">
<form action="', $scripturl, '?action=search2" method="post" accept-charset="', $context['character_set'], '">
<input class="inputbox" type="text" name="search" value="', $txt['forum_search'], '" onfocus="this.value = \'\';" onblur="if(this.value==\'\') this.value=\'', $txt['forum_search'], '\';" />';

// Search within current topic?

é corretto.
In pratica ho rimosso quello originale e sostituito con questo ma volevo sapere se la punteggiatura(' ;" etc etc) è messa in modo corretto

La modifica dice di inserire questa parte nell'indextemplate

Code Select Expand


// The code for the menu you installed dont you remember it?
   echo '
<div id="site_menu" class="align_right" style="padding-top:9px;">
<ul class="dropmenu" id="site_nav">';

   for ($i=1; $i<6; $i++)
      {
         if (!empty($modSettings['tab_url' . $i]) && !empty($modSettings['tab_title' . $i]))
         {
            echo '
               <li id="button', $i, '">
                  <a class="firstlevel" href="', $modSettings['tab_url' . $i],'"';
if(!empty($modSettings['newpage' . $i]))
echo ' TARGET="_blank"';
echo '>
<span class="firstlevel">', $modSettings['tab_title' . $i], '</span>
                  </a>
               </li>';
         }
      }
   echo'
         </ul>
            </div>

[emanuele]

Così ad occhio parrebbe sia tutto corretto...comunque gli errori di sintassi raramente vengono perdonati (certo, se diventano errori di logica vengono php non se ne può accorgere) e se c'è qualcosa di sbagliato ti viene restituito un errore.

[fdr77]

Ok grazie mille