A lot while() conditions rewritten to SQL_FETCHARRAY(), see bug #107, @TODO tags...
[mailer.git] / inc / modules / admin / what-config_home.php
index 3f3f8ec..3a1af21 100644 (file)
@@ -40,20 +40,17 @@ if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
 // Add description as navigation point
 ADD_DESCR("admin", __FILE__);
 
+// Init "sub" value
 $sub = "";
 if (REQUEST_ISSET_GET(('sub'))) $sub = REQUEST_GET('sub');
 
 switch ($sub)
 {
-case "": // Output selection table
-       LOAD_TEMPLATE("admin_config_home_main");
-       break;
-
 case "settings": // Settings related to the index page
        if (IS_FORM_SENT()) {
                // Save settings
                ADMIN_SAVE_SETTINGS_POST();
-               OUTPUT_HTML("<a href=\"{!URL!}/modules.php?module=admin&amp;what=config_home&amp;sub=settings\">".ADMIN_CONTINUE_TO_CONFIG."</a>");
+               OUTPUT_HTML("<a href=\"{!URL!}/modules.php?module=admin&amp;what=config_home&amp;sub=settings\">{--ADMIN_CONTINUE_TO_CONFIG--}</a>");
        } else {
                // Prepare data for the template
                $content = array_fill(-1, 60, "");
@@ -76,28 +73,32 @@ FROM `{!_MYSQL_PREFIX!}_guest_menu`
 WHERE `action`='main' AND `what` != '' AND `what` IS NOT NULL AND `locked`='N'
 ORDER BY `sort` ASC", __FILE__, __LINE__);
 
-       $SW = 2; $OUT = ""; $atitle = "";
-       while (list($id, $wht, $wtitle) = SQL_FETCHROW($result)) {
-               $LINK = "<a href=\"{!URL!}/modules.php?module=admin&amp;what=config_home&amp;sub=target&amp;home=".$wht."\">"._IS_NEW_HOME."</a>";
-               if ($wht == getConfig('index_home')) {
+       // Init variables
+       $SW = 2;
+       $OUT = "";
+
+       // Process all entries
+       while ($content = SQL_FETCHARRAY($result)) {
+               // Link for setting 'what' as new "home"
+               $LINK = "<a href=\"{!URL!}/modules.php?module=admin&amp;what=config_home&amp;sub=target&amp;home=".$content['what']."\">{--_IS_NEW_HOME--}</a>";
+
+               // Is this current home?
+               if ($content['what'] == getConfig('index_home')) {
                        // Is current home
-                       $LINK   = "<div class=\"admin_note\">{!_IS_CURRENT_HOME!}</div>";
-                       $atitle = "<div class=\"admin_note\">".$atitle."</div>";
-                       $wtitle = "<div class=\"admin_note\">".$wtitle."</div>";
-               }
-
-               // Add row...
-               $OUT .= "<tr>
-         <td class=\"switch_sw".$SW." bottom right\">&nbsp;".$id."</td>
-         <td class=\"switch_sw".$SW." bottom right\">&nbsp;".$wtitle."</td>
-         <td class=\"switch_sw".$SW." bottom\" align=\"center\">
-           ".$LINK."
-         </td>
-       </tr>\n";
+                       $LINK = "<div class=\"admin_note\">{!_IS_CURRENT_HOME!}</div>";
+                       $content['title'] = "<div class=\"admin_note\">".$content['title']."</div>";
+               } // END - if
+
+               // Prepare content
+               $content['sw'] = $SW;
+               $content['link'] = $LINK;
+
+               // Load row template
+               $OUT .= LOAD_TEMPLATE("admin_config_home_row", true, $content);
 
                // Switch color
                $SW = 3 - $SW;
-       }
+       } // END - while
 
        // Free memory
        SQL_FREERESULT($result);
@@ -106,6 +107,11 @@ ORDER BY `sort` ASC", __FILE__, __LINE__);
        // Load template
        LOAD_TEMPLATE("admin_config_home");
        break;
+
+default: // Output selection table
+       LOAD_TEMPLATE("admin_config_home_main");
+       break;
+
 }
 
 //