From: Brion Vibber <brion@pobox.com>
Date: Mon, 13 Sep 2010 20:37:13 +0000 (-0700)
Subject: Ticket #2701: old color settings now get reset when changing themes, so you don't... 
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=ead2f3375ad8f623b6b15aa4c7194224575b3c41;p=quix0rs-gnu-social.git

Ticket #2701: old color settings now get reset when changing themes, so you don't end up with the previous theme's colors.
---

diff --git a/actions/designadminpanel.php b/actions/designadminpanel.php
index 763737175b..e3d772f7d1 100644
--- a/actions/designadminpanel.php
+++ b/actions/designadminpanel.php
@@ -154,9 +154,22 @@ class DesignadminpanelAction extends AdminPanelAction
 
         $config->query('BEGIN');
 
-        // Only update colors if the theme has not changed.
-
-        if (!$themeChanged) {
+        if ($themeChanged) {
+            // If the theme has changed, reset custom colors and let them pick
+            // up the new theme's defaults.
+            $colors = array('background', 'content', 'sidebar', 'text', 'link');
+            foreach ($colors as $colorKey) {
+                // Clear from global config so we see defaults on this page...
+                $GLOBALS['config']['design'][$colorKey . 'color'] = false;
+
+                // And remove old settings from DB...
+                $this->deleteSetting('design', $colorKey . 'color');
+            }
+        } else {
+            // Only save colors from the form if the theme has not changed.
+            //
+            // @fixme a future more ajaxy form should allow theme switch
+            // and color customization in one step.
 
             $bgcolor = new WebColor($this->trimmed('design_background'));
             $ccolor  = new WebColor($this->trimmed('design_content'));