X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FCore%2FAddon.php;h=ef2289ca598e6794b096e0c444dbfa1c69613bf7;hb=4e4eab7548c6c7bb7f096beb39419fef276af500;hp=2ec46b71c1835b2792841c4d418a1c2e03ac5ff6;hpb=c1f99c70b1c7d62120723f3b142e843ba25ab338;p=friendica.git diff --git a/src/Core/Addon.php b/src/Core/Addon.php index 2ec46b71c1..ef2289ca59 100644 --- a/src/Core/Addon.php +++ b/src/Core/Addon.php @@ -1,18 +1,34 @@ . + * */ namespace Friendica\Core; -use Friendica\BaseObject; -use Friendica\Database\DBA; +use Friendica\DI; +use Friendica\Model\Contact; use Friendica\Util\Strings; /** * Some functions to handle addons */ -class Addon extends BaseObject +class Addon { /** * The addon sub-directory @@ -45,7 +61,7 @@ class Addon extends BaseObject 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) ) { @@ -68,22 +84,26 @@ class Addon extends BaseObject public static function getAdminList() { $addons_admin = []; - $addonsAdminStmt = DBA::select('addon', ['name'], ['plugin_admin' => 1], ['order' => ['name']]); - while ($addon = DBA::fetch($addonsAdminStmt)) { - $addons_admin[$addon['name']] = [ - 'url' => 'admin/addons/' . $addon['name'], - 'name' => $addon['name'], + $addons = DI::config()->get('addons'); + ksort($addons); + foreach ($addons as $name => $data) { + if (empty($data['admin'])) { + continue; + } + + $addons_admin[$name] = [ + 'url' => 'admin/addons/' . $name, + 'name' => $name, 'class' => 'addon' ]; } - DBA::close($addonsAdminStmt); return $addons_admin; } /** - * @brief Synchronize addons: + * Synchronize addons: * * system.addon contains a comma-separated list of names * of addons which are used on this system. @@ -96,52 +116,22 @@ class Addon extends BaseObject */ 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); - } - } + self::$addons = array_keys(DI::config()->get('addons') ?? []); } /** - * @brief uninstalls an addon. + * uninstalls an addon. * * @param string $addon name of the addon * @return void * @throws \Exception */ - public static function uninstall($addon) + public static function uninstall(string $addon) { $addon = Strings::sanitizeFilePathItem($addon); - Logger::notice("Addon {addon}: {action}", ['action' => 'uninstall', 'addon' => $addon]); - DBA::delete('addon', ['name' => $addon]); + Logger::debug("Addon {addon}: {action}", ['action' => 'uninstall', 'addon' => $addon]); + DI::config()->delete('addons', $addon); @include_once('addon/' . $addon . '/' . $addon . '.php'); if (function_exists($addon . '_uninstall')) { @@ -149,107 +139,75 @@ class Addon extends BaseObject $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 * @throws \Exception */ - public static function install($addon) + public static function install(string $addon): bool { $addon = Strings::sanitizeFilePathItem($addon); + $addon_file_path = 'addon/' . $addon . '/' . $addon . '.php'; + // silently fail if addon was removed of if $addon is funky - if (!file_exists('addon/' . $addon . '/' . $addon . '.php')) { + if (!file_exists($addon_file_path)) { return false; } - Logger::notice("Addon {addon}: {action}", ['action' => 'install', 'addon' => $addon]); - $t = @filemtime('addon/' . $addon . '/' . $addon . '.php'); - @include_once('addon/' . $addon . '/' . $addon . '.php'); + Logger::debug("Addon {addon}: {action}", ['action' => 'install', 'addon' => $addon]); + $t = @filemtime($addon_file_path); + @include_once($addon_file_path); if (function_exists($addon . '_install')) { $func = $addon . '_install'; - $func(self::getApp()); - - $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; - } + $func(DI::app()); + } - Addon::saveEnabledList(); + DI::config()->set('addons', $addon, [ + 'last_update' => $t, + 'admin' => function_exists($addon . '_addon_admin'), + ]); - return true; - } else { - Logger::error("Addon {addon}: {action} failed", ['action' => 'install', 'addon' => $addon]); - return false; + if (!self::isEnabled($addon)) { + self::$addons[] = $addon; } + + return true; } /** * reload all updated addons + * + * @return void */ 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 = DI::config()->get('addons'); + + foreach ($addons as $name => $data) { + $addonname = Strings::sanitizeFilePathItem(trim($name)); + $addon_file_path = 'addon/' . $addonname . '/' . $addonname . '.php'; + if (file_exists($addon_file_path) && $data['last_update'] == filemtime($addon_file_path)) { + // Addon unmodified, skipping + continue; } - $addon_list = explode(',', $addons); + Logger::debug("Addon {addon}: {action}", ['action' => 'reload', 'addon' => $name]); - 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(self::getApp()); - } - if (function_exists($addon . '_install')) { - $func = $addon . '_install'; - $func(self::getApp()); - } - DBA::update('addon', ['timestamp' => $t], ['id' => $i['id']]); - } - } - } - } + self::uninstall($name); + self::install($name); } } /** - * @brief Parse addon comment in search of addon infos. + * Parse addon comment in search of addon infos. * * like * \code @@ -265,10 +223,8 @@ class Addon extends BaseObject * @return array with the addon information * @throws \Exception */ - public static function getInfo($addon) + public static function getInfo(string $addon): array { - $a = self::getApp(); - $addon = Strings::sanitizeFilePathItem($addon); $info = [ @@ -284,9 +240,9 @@ class Addon extends BaseObject return $info; } - $stamp1 = microtime(true); + DI::profiler()->startRecording('file'); $f = file_get_contents("addon/$addon/$addon.php"); - $a->getProfiler()->saveTimestamp($stamp1, "file", System::callstack()); + DI::profiler()->stopRecording(); $r = preg_match("|/\*.*\*/|msU", $f, $m); @@ -305,6 +261,12 @@ class Addon extends BaseObject if ($type == "author" || $type == "maintainer") { $r = preg_match("|([^<]+)<([^>]+)>|", $v, $m); if ($r) { + if (!empty($m[2]) && empty(parse_url($m[2], PHP_URL_SCHEME))) { + $contact = Contact::getByURL($m[2], false); + if (!empty($contact['url'])) { + $m[2] = $contact['url']; + } + } $info[$type][] = ['name' => $m[1], 'link' => $m[2]]; } else { $info[$type][] = ['name' => $v]; @@ -326,7 +288,7 @@ class Addon extends BaseObject * @param string $addon * @return boolean */ - public static function isEnabled($addon) + public static function isEnabled(string $addon): bool { return in_array($addon, self::$addons); } @@ -336,84 +298,25 @@ class Addon extends BaseObject * * @return array */ - public static function getEnabledList() + public static function getEnabledList(): array { 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 * * @return array * @throws \Exception */ - public static function getVisibleList() + public static function getVisibleList(): array { $visible_addons = []; - $stmt = DBA::select('addon', ['name'], ['hidden' => false, 'installed' => true]); - if (DBA::isResult($stmt)) { - foreach (DBA::toArray($stmt) as $addon) { - $visible_addons[] = $addon['name']; - } + $addons = DI::config()->get('addons'); + foreach ($addons as $name => $data) { + $visible_addons[] = $name; } return $visible_addons; } - - /** - * Shim of Hook::register left for backward compatibility purpose. - * - * @see Hook::register - * @deprecated since version 2018.12 - * @param string $hook the name of the hook - * @param string $file the name of the file that hooks into - * @param string $function the name of the function that the hook will call - * @param int $priority A priority (defaults to 0) - * @return mixed|bool - * @throws \Friendica\Network\HTTPException\InternalServerErrorException - */ - public static function registerHook($hook, $file, $function, $priority = 0) - { - return Hook::register($hook, $file, $function, $priority); - } - - /** - * Shim of Hook::unregister left for backward compatibility purpose. - * - * @see Hook::unregister - * @deprecated since version 2018.12 - * @param string $hook the name of the hook - * @param string $file the name of the file that hooks into - * @param string $function the name of the function that the hook called - * @return boolean - * @throws \Friendica\Network\HTTPException\InternalServerErrorException - */ - public static function unregisterHook($hook, $file, $function) - { - return Hook::unregister($hook, $file, $function); - } - - /** - * Shim of Hook::callAll left for backward-compatibility purpose. - * - * @see Hook::callAll - * @deprecated since version 2018.12 - * @param string $name of the hook to call - * @param string|array &$data to transmit to the callback handler - * @throws \Friendica\Network\HTTPException\InternalServerErrorException - */ - public static function callHooks($name, &$data = null) - { - Hook::callAll($name, $data); - } }