A lot fixes to templates and missing functions added, more rewrites

[mailer.git] / inc / modules / member / what-themes.php

diff --git a/inc/modules/member/what-themes.php b/inc/modules/member/what-themes.php
index 17708e14d7e278ac6a43e7f2cd8f483121722c77..d238500bb59edb567c8a6d2b9c4f440514c15907 100644 (file)
--- a/inc/modules/member/what-themes.php
+++ b/inc/modules/member/what-themes.php
@@ -38,26 +38,26 @@ if (!defined('__SECURITY')) {
 } elseif (!IS_MEMBER()) {
        LOAD_URL("modules.php?module=index");
 } elseif ((!EXT_IS_ACTIVE("theme")) && (!IS_ADMIN())) {
-       ADD_FATAL(EXTENSION_PROBLEM_EXT_INACTIVE, "theme");
+       addFatalMessage(__FILE__, __LINE__, getMessage('EXTENSION_PROBLEM_EXT_INACTIVE'), "theme");
        return;
 }
 
 // Add description as navigation point
 ADD_DESCR("member", __FILE__);
 
-if (!empty($_POST['member_theme'])) {
+if (REQUEST_ISSET_POST(('member_theme'))) {
        // Save theme to member's profile
-       SQL_QUERY_ESC("UPDATE `"._MYSQL_PREFIX."_user_data` SET curr_theme='%s' WHERE userid=%s LIMIT 1",
-               array($_POST['member_theme'], $GLOBALS['userid']), __FILE__, __LINE__);
+       SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_user_data` SET curr_theme='%s' WHERE userid=%s LIMIT 1",
+               array(REQUEST_POST('member_theme'), $GLOBALS['userid']), __FILE__, __LINE__);
 
        // Set new theme for guests
-       $newTheme = SQL_ESCAPE($_POST['member_theme']);
+       $newTheme = SQL_ESCAPE(REQUEST_POST('member_theme'));
 
        // Change to new theme
        set_session('mxchange_theme', $newTheme);
 
        // Theme saved!
-       LOAD_TEMPLATE("admin_settings_saved", false, MEMBER_THEME_SAVED);
+       LOAD_TEMPLATE("admin_settings_saved", false, getMessage('MEMBER_THEME_SAVED'));
 }
 
 // Initialize array
@@ -71,14 +71,14 @@ $THEMES = array(
 );
 
 // Read directory "themes"
-$handle = opendir(PATH."theme/") or mxchange_die("Cannot read themes dir!");
+$handle = opendir(constant('PATH')."theme/") or mxchange_die("Cannot read themes dir!");
 while ($entry = readdir($handle)) {
        // Construct absolute theme.php file name
-       $theme = sprintf("%stheme/%s/theme.php", PATH, $entry);
+       $INC = sprintf("theme/%s/theme.php", $entry);
 
-       if (($entry != ".") && ($entry != "..") && (FILE_READABLE($theme)) && (THEME_IS_ACTIVE($entry))) {
+       if (($entry != ".") && ($entry != "..") && (INCLUDE_READABLE($INC)) && (THEME_IS_ACTIVE($entry))) {
                // Found a valid directory so let's load it's theme.php file
-               include($theme);
+               LOAD_INC($INC);
 
                // Add found theme to array
                $THEMES['theme_unix'][]   = $entry;
@@ -100,32 +100,32 @@ array_pk_sort($THEMES, array("theme_name"));
 $OUT = ""; $SW = 2;
 foreach ($THEMES['theme_unix'] as $key => $unix) {
        $default = "";
-       if (get_session('mxchange_theme') == $unix) $default = " checked selected";
+       if (get_session('mxchange_theme') == $unix) $default = " selected=\"selected\"";
 
        // Add row
-       $OUT .= "<TR>
-  <TD class=\"switch_sw".$SW." bottom2 right2\" align=\"center\" height=\"30\">
-    <INPUT type=\"radio\" name=\"member_theme\" class=\"member_normal\" value=\"".$unix."\"".$default.">
-  </TD>
-  <TD class=\"switch_sw".$SW." bottom2 right2\" align=\"center\">".$THEMES['theme_name'][$key]."</TD>
-  <TD class=\"switch_sw".$SW." bottom2 right2\" align=\"center\">
-    <A href=\"mailto:".$THEMES['theme_email'][$key]."?Subject=[Theme:] ".$THEMES['theme_name'][$key]." (".$unix.")"."\">".$THEMES['theme_author'][$key]."</A>
-  </TD>
-  <TD class=\"switch_sw".$SW." bottom2 right2\" align=\"center\">
-    <A href=\"".DEREFERER($THEMES['theme_url'][$key])."\" target=\"_blank\">".$THEMES['theme_url'][$key]."</A>
-  </TD>
-  <TD class=\"switch_sw".$SW." bottom2\" align=\"center\">v".$THEMES['theme_ver'][$key]."</TD>
-</TR>\n";
+       $OUT .= "<tr>
+  <td class=\"switch_sw".$SW." bottom2 right2\" align=\"center\" height=\"30\">
+    <input type=\"radio\" name=\"member_theme\" class=\"member_normal\" value=\"".$unix."\"".$default." />
+  </td>
+  <td class=\"switch_sw".$SW." bottom2 right2\" align=\"center\">".$THEMES['theme_name'][$key]."</td>
+  <td class=\"switch_sw".$SW." bottom2 right2\" align=\"center\">
+    <a href=\"mailto:".$THEMES['theme_email'][$key]."?Subject=[Theme:] ".$THEMES['theme_name'][$key]." (".$unix.")"."\">".$THEMES['theme_author'][$key]."</a>
+  </td>
+  <td class=\"switch_sw".$SW." bottom2 right2\" align=\"center\">
+    <a href=\"".DEREFERER($THEMES['theme_url'][$key])."\" target=\"_blank\">".$THEMES['theme_url'][$key]."</a>
+  </td>
+  <td class=\"switch_sw".$SW." bottom2\" align=\"center\">v".$THEMES['theme_ver'][$key]."</td>
+</tr>\n";
        $SW = 3 - $SW;
 }
 
 if (empty($OUT)) {
        // No themes found???
-       $OUT = "<TR>
-  <TD colspan=\"5\" class=\"bottom2\" height=\"80\">
-    ".LOAD_TEMPLATE("admin_settings_saved", true, MEMBER_NO_THEMES_FOUND)."
-  </TD>
-</TR>\n";
+       $OUT = "<tr>
+  <td colspan=\"5\" class=\"bottom2\" height=\"80\">
+    ".LOAD_TEMPLATE("admin_settings_saved", true, getMessage('MEMBER_NO_THEMES_FOUND'))."
+  </td>
+</tr>\n";
 }
 define('__THEME_LIST', $OUT);