A lot while() conditions rewritten to SQL_FETCHARRAY(), see bug #107, @TODO tags...

[mailer.git] / inc / modules / admin / what-theme_edit.php

diff --git a/inc/modules/admin/what-theme_edit.php b/inc/modules/admin/what-theme_edit.php
index 28bf8512e9e7e048f29e109bcedd856b5bbacd17..e211ca41e4a221f6fd03cffcca2ce25c8583641c 100644 (file)
--- a/inc/modules/admin/what-theme_edit.php
+++ b/inc/modules/admin/what-theme_edit.php
@@ -99,31 +99,29 @@ $THEME_MODE = "test";
 $OUT = ""; $SW = 2;
 $result = SQL_QUERY("SELECT id, theme_path, theme_active, theme_ver, theme_name FROM `{!_MYSQL_PREFIX!}_themes` ORDER BY theme_path", __FILE__, __LINE__);
 if (SQL_NUMROWS($result) > 0) {
-       while (list($id, $unix, $active, $ver, $name) = SQL_FETCHROW($result)) {
+       while ($content = SQL_FETCHARRAY($result)) {
                // Construct IFN
-               $INC = sprintf("theme/%s/theme.php",
-                       $unix
-               );
+               $INC = sprintf("theme/%s/theme.php", $content['theme_path']);
 
                // Load theme in test mode
                LOAD_INC($INC);
 
                // Is the loaded theme name != current theme name?
-               $LINK = $unix;
-               if ($unix != GET_CURR_THEME()) $LINK = "<a href=\"{!URL!}/modules.php?module=admin&amp;what=theme_edit&amp;default_theme=".$unix."\" title=\"{--ADMIN_SET_AS_NEW_THEME--}\">".$unix."</a>";
+               $LINK = $content['theme_path'];
+               if ($content['theme_path'] != GET_CURR_THEME()) $LINK = "<a href=\"{!URL!}/modules.php?module=admin&amp;what=theme_edit&amp;default_theme=".$content['theme_path']."\" title=\"{--ADMIN_SET_AS_NEW_THEME--}\">".$content['theme_path']."</a>";
 
                // Prepare data for the row template
                $content = array(
                        'sw'        => $SW,
-                       'id'        => $id,
-                       'active'    => $active,
+                       'id'        => $content['id'],
+                       'active'    => $content['theme_active'],
                        'link'      => $LINK,
-                       'name'      => $name,
-                       'is_act'    => TRANSLATE_YESNO($active),
-                       'email'     => "<a href=\"mailto:".$THEME_EMAIL."?Subject=[Theme:] ".$THEME_NAME." (".$unix.")"."\">".$THEME_AUTHOR."</a>",
+                       'name'      => $content['theme_name'],
+                       'is_act'    => TRANSLATE_YESNO($content['theme_active']),
+                       'email'     => "<a href=\"mailto:".$THEME_EMAIL."?Subject=[Theme:] ".$THEME_NAME." (".$content['theme_path'].")"."\">".$THEME_AUTHOR."</a>",
                        'url_link'  => DEREFERER($THEME_URL),
                        'url_title' => $THEME_URL,
-                       'ver'       => $ver,
+                       'ver'       => $content['theme_ver'],
                );
 
                // Load row template and switch color