Replace BaseObject class with DI::* calls

[friendica.git] / src / Core / Theme.php

diff --git a/src/Core/Theme.php b/src/Core/Theme.php
index c972e09fc71a659c085a7a0288c7b69f3391d19c..dea3e20e647b2ed0740a6535d135c2b80863b2b3 100644 (file)
--- a/src/Core/Theme.php
+++ b/src/Core/Theme.php
@@ -6,7 +6,7 @@
 
 namespace Friendica\Core;
 
-use Friendica\BaseObject;
+use Friendica\DI;
 use Friendica\Model\Profile;
 use Friendica\Util\Strings;
 
@@ -20,7 +20,7 @@ class Theme
        public static function getAllowedList()
        {
                $allowed_themes_str = Config::get('system', 'allowed_themes');
-               $allowed_themes_raw = explode(',', $allowed_themes_str);
+               $allowed_themes_raw = explode(',', str_replace(' ', '', $allowed_themes_str));
                $allowed_themes = [];
                if (count($allowed_themes_raw)) {
                        foreach ($allowed_themes_raw as $theme) {
@@ -31,7 +31,12 @@ class Theme
                        }
                }
 
-               return $allowed_themes;
+               return array_unique($allowed_themes);
+       }
+
+       public static function setAllowedList(array $allowed_themes)
+       {
+               Config::set('system', 'allowed_themes', implode(',', array_unique($allowed_themes)));
        }
 
        /**
@@ -133,13 +138,20 @@ class Theme
 
                // silently fail if theme was removed or if $theme is funky
                if (file_exists("view/theme/$theme/theme.php")) {
-                       Logger::log("Addons: uninstalling theme " . $theme);
+                       include_once "view/theme/$theme/theme.php";
 
-                       if (function_exists("{$theme}_uninstall")) {
-                               $func = "{$theme}_uninstall";
+                       $func = "{$theme}_uninstall";
+                       if (function_exists($func)) {
                                $func();
                        }
                }
+
+               $allowed_themes = Theme::getAllowedList();
+               $key = array_search($theme, $allowed_themes);
+               if ($key !== false) {
+                       unset($allowed_themes[$key]);
+                       Theme::setAllowedList($allowed_themes);
+               }
        }
 
        public static function install($theme)
@@ -151,16 +163,21 @@ class Theme
                        return false;
                }
 
-               Logger::log("Addons: installing theme $theme");
+               try {
+                       include_once "view/theme/$theme/theme.php";
 
-               include_once "view/theme/$theme/theme.php";
-
-               if (function_exists("{$theme}_install")) {
                        $func = "{$theme}_install";
-                       $func();
+                       if (function_exists($func)) {
+                               $func();
+                       }
+
+                       $allowed_themes = Theme::getAllowedList();
+                       $allowed_themes[] = $theme;
+                       Theme::setAllowedList($allowed_themes);
+
                        return true;
-               } else {
-                       Logger::log("Addons: FAILED installing theme $theme");
+               } catch (\Exception $e) {
+                       Logger::error('Theme installation failed', ['theme' => $theme, 'error' => $e->getMessage()]);
                        return false;
                }
        }
@@ -168,45 +185,33 @@ class Theme
        /**
         * @brief Get the full path to relevant theme files by filename
         *
-        * This function search in the theme directory (and if not present in global theme directory)
-        * if there is a directory with the file extension and  for a file with the given
-        * filename.
+        * This function searches in order in the current theme directory, in the current theme parent directory, and lastly
+        * in the base view/ folder.
         *
         * @param string $file Filename
-        * @param string $root Full root path
         * @return string Path to the file or empty string if the file isn't found
-        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \Exception
         */
-       public static function getPathForFile($file, $root = '')
+       public static function getPathForFile($file)
        {
-               $file = basename($file);
+               $a = DI::app();
+
+               $theme = $a->getCurrentTheme();
+
+               $parent = Strings::sanitizeFilePathItem($a->theme_info['extends'] ?? $theme);
 
-               // Make sure $root ends with a slash / if it's not blank
-               if ($root !== '' && $root[strlen($root) - 1] !== '/') {
-                       $root = $root . '/';
-               }
-               $theme_info = \get_app()->theme_info;
-               if (is_array($theme_info) && array_key_exists('extends', $theme_info)) {
-                       $parent = $theme_info['extends'];
-               } else {
-                       $parent = 'NOPATH';
-               }
-               $theme = \get_app()->getCurrentTheme();
-               $parent = Strings::sanitizeFilePathItem($parent);
-               $ext = substr($file, strrpos($file, '.') + 1);
                $paths = [
-                       "{$root}view/theme/$theme/$ext/$file",
-                       "{$root}view/theme/$parent/$ext/$file",
-                       "{$root}view/$ext/$file",
+                       "view/theme/$theme/$file",
+                       "view/theme/$parent/$file",
+                       "view/$file",
                ];
-               foreach ($paths as $p) {
-                       // strpos() is faster than strstr when checking if one string is in another (http://php.net/manual/en/function.strstr.php)
-                       if (strpos($p, 'NOPATH') !== false) {
-                               continue;
-                       } elseif (file_exists($p)) {
-                               return $p;
+
+               foreach ($paths as $path) {
+                       if (file_exists($path)) {
+                               return $path;
                        }
                }
+
                return '';
        }
 
@@ -227,7 +232,7 @@ class Theme
                        return 'view/theme/' . $theme . '/style.css';
                }
 
-               $a = BaseObject::getApp();
+               $a = DI::app();
 
                $query_params = [];