Refs matomo-org#3942

* Enabling one theme a time: automatically deactivates another theme if it was enabled (except Zeitgeist which is always on)
sabl0r · Jul 16, 2013 · 53654d5 · 53654d5
1 parent 7198b79
commit 53654d5
Showing 1 changed file with 35 additions and 6 deletions.
diff --git a/core/PluginsManager.php b/core/PluginsManager.php
@@ -192,10 +192,13 @@ protected function getPluginsDirectory()
      * Deactivate plugin
      *
      * @param string $pluginName  Name of plugin
+     * @param array $plugins Array of plugin names
      */
-    public function deactivatePlugin($pluginName)
+    public function deactivatePlugin($pluginName, $plugins = false)
     {
-        $plugins = $this->pluginsToLoad;
+        if(empty($plugins)) {
+            $plugins = $this->pluginsToLoad;
+        }
         $key = array_search($pluginName, $plugins);
 
         $plugin = $this->loadPlugin($pluginName);
@@ -219,6 +222,8 @@ public function deactivatePlugin($pluginName)
 
         Piwik_Config::getInstance()->forceSave();
         Piwik::deleteAllCacheOnUpdate();
+
+        return $plugins;
     }
 
     /**
@@ -267,9 +272,14 @@ public function activatePlugin($pluginName)
         // we add the plugin to the list of activated plugins
         if (!in_array($pluginName, $plugins)) {
             $plugins[] = $pluginName;
-        } else {
-            // clean up if we find a dupe
-            $plugins = array_unique($plugins);
+        }
+        $plugins = array_unique($plugins);
+
+        // Only one theme enabled at a time
+        $themeAlreadyEnabled = $this->getThemeEnabled();
+        if($plugin->isTheme()
+            && $themeAlreadyEnabled) {
+            $plugins = $this->deactivatePlugin( $themeAlreadyEnabled, $plugins );
         }
 
         // the config file will automatically be saved with the new plugin
@@ -279,6 +289,25 @@ public function activatePlugin($pluginName)
         Piwik::deleteAllCacheOnUpdate();
     }
 
+    /**
+     * Returns the name of the non default theme currently enabled.
+     * If Zeitgeist is enabled, returns false (nb: Zeitgeist cannot be disabled)
+     *
+     * @return string
+     */
+    protected function getThemeEnabled()
+    {
+        $plugins = $this->getLoadedPlugins();
+        foreach($plugins as $plugin) {
+            /* @var $plugin Piwik_Plugin */
+            if($plugin->isTheme()
+                && $plugin->getPluginName() != Piwik_Twig::DEFAULT_THEME) {
+                return $plugin->getPluginName();
+            }
+        }
+        return false;
+    }
+
     /**
      * Load the specified plugins
      *
@@ -356,7 +385,7 @@ public function getLoadedPluginsName()
      *        'UserSettings' => Piwik_Plugin $pluginObject,
      *    );
      *
-     * @return array
+     * @return array,Piwik_Plugin
      */
     public function getLoadedPlugins()
     {