Changeset 5382 for trunk/admin/include/themes.class.php

Timestamp:

Mar 26, 2010, 3:16:54 PM (14 years ago)

Author:

plg

Message:

feature 1514: make the "deactivate" action inactive if there is no active
theme left.

bug fixed: when setting the default theme, make sure at least one user will
be updated.

File:

• trunk/admin/include/themes.class.php (modified) (3 diffs)

trunk/admin/include/themes.class.php

@@ -118 +118 @@
         }
 
+        // you can't deactivate the last theme
+        if (count($this->db_themes_by_id) <= 1)
+        {
+          array_push(
+            $errors,
+            l10n('Impossible to deactivate this theme, you need at least one theme.')
+            );
+          break;
+        }
+
         if ($theme_id == get_default_theme())
         {
@@ -228 +238 @@
   function set_default_theme($theme_id)
   {
+    global $conf;
+    
     // first we need to know which users are using the current default theme
     $default_theme = get_default_theme();
@@ -237 +249 @@
   WHERE theme = "'.$default_theme.'"
 ;';
-    $user_ids = array_from_query($query, 'user_id');
+    $user_ids = array_unique(
+      array_merge(
+        array_from_query($query, 'user_id'),
+        array($conf['guest_id'], $conf['default_user_id'])
+        )
+      );
 
     // $user_ids can't be empty, at least the default user has the default