]> git.mxchange.org Git - friendica.git/blobdiff - src/Core/Addon.php
Check for bad worker priorities
[friendica.git] / src / Core / Addon.php
index 59e1ec7b96d2b2f9ad4819604848376c37abdc09..511364b8acb0cff9bb6180e97d6dd8441bea288b 100644 (file)
@@ -1,6 +1,22 @@
 <?php
 /**
- * @file src/Core/Addon.php
+ * @copyright Copyright (C) 2020, Friendica
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
  */
 
 namespace Friendica\Core;
@@ -45,7 +61,7 @@ class Addon
                                        list($tmp, $addon) = array_map('trim', explode('/', $file));
                                        $info = self::getInfo($addon);
 
-                                       if (Config::get('system', 'show_unsupported_addons')
+                                       if (DI::config()->get('system', 'show_unsupported_addons')
                                                || strtolower($info['status']) != 'unsupported'
                                                || self::isEnabled($addon)
                                        ) {
@@ -83,7 +99,7 @@ class Addon
 
 
        /**
-        * @brief Synchronize addons:
+        * Synchronize addons:
         *
         * system.addon contains a comma-separated list of names
         * of addons which are used on this system.
@@ -96,41 +112,12 @@ class Addon
         */
        public static function loadAddons()
        {
-               $installed_addons = [];
-
-               $r = DBA::select('addon', [], ['installed' => 1]);
-               if (DBA::isResult($r)) {
-                       $installed_addons = DBA::toArray($r);
-               }
-
-               $addons = Config::get('system', 'addon');
-               $addons_arr = [];
-
-               if ($addons) {
-                       $addons_arr = explode(',', str_replace(' ', '', $addons));
-               }
-
-               self::$addons = $addons_arr;
-
-               $installed_arr = [];
-
-               foreach ($installed_addons as $addon) {
-                       if (!self::isEnabled($addon['name'])) {
-                               self::uninstall($addon['name']);
-                       } else {
-                               $installed_arr[] = $addon['name'];
-                       }
-               }
-
-               foreach (self::$addons as $p) {
-                       if (!in_array($p, $installed_arr)) {
-                               self::install($p);
-                       }
-               }
+               $installed_addons = DBA::selectToArray('addon', ['name'], ['installed' => true]);
+               self::$addons = array_column($installed_addons, 'name');
        }
 
        /**
-        * @brief uninstalls an addon.
+        * uninstalls an addon.
         *
         * @param string $addon name of the addon
         * @return void
@@ -149,15 +136,13 @@ class Addon
                        $func();
                }
 
-               DBA::delete('hook', ['file' => 'addon/' . $addon . '/' . $addon . '.php']);
+               Hook::delete(['file' => 'addon/' . $addon . '/' . $addon . '.php']);
 
                unset(self::$addons[array_search($addon, self::$addons)]);
-
-               Addon::saveEnabledList();
        }
 
        /**
-        * @brief installs an addon.
+        * installs an addon.
         *
         * @param string $addon name of the addon
         * @return bool
@@ -178,31 +163,21 @@ class Addon
                if (function_exists($addon . '_install')) {
                        $func = $addon . '_install';
                        $func(DI::app());
+               }
 
-                       $addon_admin = (function_exists($addon . "_addon_admin") ? 1 : 0);
-
-                       DBA::insert('addon', ['name' => $addon, 'installed' => true,
-                               'timestamp' => $t, 'plugin_admin' => $addon_admin]);
-
-                       // we can add the following with the previous SQL
-                       // once most site tables have been updated.
-                       // This way the system won't fall over dead during the update.
-
-                       if (file_exists('addon/' . $addon . '/.hidden')) {
-                               DBA::update('addon', ['hidden' => true], ['name' => $addon]);
-                       }
-
-                       if (!self::isEnabled($addon)) {
-                               self::$addons[] = $addon;
-                       }
-
-                       Addon::saveEnabledList();
-
-                       return true;
-               } else {
-                       Logger::error("Addon {addon}: {action} failed", ['action' => 'install', 'addon' => $addon]);
-                       return false;
+               DBA::insert('addon', [
+                       'name' => $addon,
+                       'installed' => true,
+                       'timestamp' => $t,
+                       'plugin_admin' => function_exists($addon . '_addon_admin'),
+                       'hidden' => file_exists('addon/' . $addon . '/.hidden')
+               ]);
+
+               if (!self::isEnabled($addon)) {
+                       self::$addons[] = $addon;
                }
+
+               return true;
        }
 
        /**
@@ -210,46 +185,25 @@ class Addon
         */
        public static function reload()
        {
-               $addons = Config::get('system', 'addon');
-               if (strlen($addons)) {
-                       $r = DBA::select('addon', [], ['installed' => 1]);
-                       if (DBA::isResult($r)) {
-                               $installed = DBA::toArray($r);
-                       } else {
-                               $installed = [];
+               $addons = DBA::selectToArray('addon', [], ['installed' => true]);
+
+               foreach ($addons as $addon) {
+                       $addonname = Strings::sanitizeFilePathItem(trim($addon['name']));
+                       $fname = 'addon/' . $addonname . '/' . $addonname . '.php';
+                       $t = @filemtime($fname);
+                       if (!file_exists($fname) || ($addon['timestamp'] == $t)) {
+                               continue;
                        }
 
-                       $addon_list = explode(',', $addons);
-
-                       foreach ($addon_list as $addon) {
-                               $addon = Strings::sanitizeFilePathItem(trim($addon));
-                               $fname = 'addon/' . $addon . '/' . $addon . '.php';
-                               if (file_exists($fname)) {
-                                       $t = @filemtime($fname);
-                                       foreach ($installed as $i) {
-                                               if (($i['name'] == $addon) && ($i['timestamp'] != $t)) {
-
-                                                       Logger::notice("Addon {addon}: {action}", ['action' => 'reload', 'addon' => $i['name']]);
-                                                       @include_once($fname);
-
-                                                       if (function_exists($addon . '_uninstall')) {
-                                                               $func = $addon . '_uninstall';
-                                                               $func(DI::app());
-                                                       }
-                                                       if (function_exists($addon . '_install')) {
-                                                               $func = $addon . '_install';
-                                                               $func(DI::app());
-                                                       }
-                                                       DBA::update('addon', ['timestamp' => $t], ['id' => $i['id']]);
-                                               }
-                                       }
-                               }
-                       }
+                       Logger::notice("Addon {addon}: {action}", ['action' => 'reload', 'addon' => $addon['name']]);
+
+                       self::uninstall($fname);
+                       self::install($fname);
                }
        }
 
        /**
-        * @brief Parse addon comment in search of addon infos.
+        * Parse addon comment in search of addon infos.
         *
         * like
         * \code
@@ -267,8 +221,6 @@ class Addon
         */
        public static function getInfo($addon)
        {
-               $a = DI::app();
-
                $addon = Strings::sanitizeFilePathItem($addon);
 
                $info = [
@@ -286,7 +238,7 @@ class Addon
 
                $stamp1 = microtime(true);
                $f = file_get_contents("addon/$addon/$addon.php");
-               $a->getProfiler()->saveTimestamp($stamp1, "file", System::callstack());
+               DI::profiler()->saveTimestamp($stamp1, "file");
 
                $r = preg_match("|/\*.*\*/|msU", $f, $m);
 
@@ -341,16 +293,6 @@ class Addon
                return self::$addons;
        }
 
-       /**
-        * Saves the current enabled addon list in the system.addon config key
-        *
-        * @return boolean
-        */
-       public static function saveEnabledList()
-       {
-               return Config::set('system', 'addon', implode(',', self::$addons));
-       }
-
        /**
         * Returns the list of non-hidden enabled addon names
         *