X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FCore%2FHook.php;h=81c5d882dc1b6688219e4e057e3d2229491c39f9;hb=06284e60073f374c1bd411e0bba6474a13c14f10;hp=959ed9fa16e3193069c9684ea75f97e927b7f4c5;hpb=1de3f186d79f3703850524c1d58c237be3eb25f3;p=friendica.git diff --git a/src/Core/Hook.php b/src/Core/Hook.php index 959ed9fa16..81c5d882dc 100644 --- a/src/Core/Hook.php +++ b/src/Core/Hook.php @@ -1,11 +1,27 @@ . + * */ + namespace Friendica\Core; use Friendica\App; -use Friendica\BaseObject; use Friendica\Database\DBA; use Friendica\DI; use Friendica\Util\Strings; @@ -13,7 +29,7 @@ use Friendica\Util\Strings; /** * Some functions to handle hooks */ -class Hook extends BaseObject +class Hook { /** * Array of registered hooks @@ -46,7 +62,7 @@ class Hook extends BaseObject } /** - * @brief Adds a new hook to the hooks array. + * Adds a new hook to the hooks array. * * This function is meant to be called by modules on each page load as it works after loadHooks has been called. * @@ -63,7 +79,7 @@ class Hook extends BaseObject } /** - * @brief Registers a hook. + * Registers a hook. * * This function is meant to be called once when an addon is enabled for example as it doesn't add to the current hooks. * @@ -83,9 +99,7 @@ class Hook extends BaseObject return true; } - $result = DBA::insert('hook', ['hook' => $hook, 'file' => $file, 'function' => $function, 'priority' => $priority]); - - return $result; + return self::insert(['hook' => $hook, 'file' => $file, 'function' => $function, 'priority' => $priority]); } /** @@ -103,10 +117,10 @@ class Hook extends BaseObject // This here is only needed for fixing a problem that existed on the develop branch $condition = ['hook' => $hook, 'file' => $file, 'function' => $function]; - DBA::delete('hook', $condition); + self::delete($condition); $condition = ['hook' => $hook, 'file' => $relative_file, 'function' => $function]; - $result = DBA::delete('hook', $condition); + $result = self::delete($condition); return $result; } @@ -128,7 +142,7 @@ class Hook extends BaseObject } /** - * @brief Forks a hook. + * Forks a hook. * * Use this function when you want to fork a hook via the worker. * @@ -163,7 +177,7 @@ class Hook extends BaseObject } /** - * @brief Calls a hook. + * Calls a hook. * * Use this function when you want to be able to allow a hook to manipulate * the provided data. @@ -182,7 +196,7 @@ class Hook extends BaseObject } /** - * @brief Calls a single hook. + * Calls a single hook. * * @param App $a * @param string $name of the hook to call @@ -204,7 +218,7 @@ class Hook extends BaseObject } else { // remove orphan hooks $condition = ['hook' => $name, 'file' => $hook[0], 'function' => $hook[1]]; - DBA::delete('hook', $condition, ['cascade' => false]); + self::delete($condition); } } @@ -229,4 +243,44 @@ class Hook extends BaseObject return false; } + + /** + * Deletes one or more hook records + * + * We have to clear the cached routerDispatchData because addons can provide routes + * + * @param array $condition + * @return bool + * @throws \Exception + */ + public static function delete(array $condition) + { + $result = DBA::delete('hook', $condition); + + if ($result) { + DI::cache()->delete('routerDispatchData'); + } + + return $result; + } + + /** + * Inserts a hook record + * + * We have to clear the cached routerDispatchData because addons can provide routes + * + * @param array $condition + * @return bool + * @throws \Exception + */ + private static function insert(array $condition) + { + $result = DBA::insert('hook', $condition); + + if ($result) { + DI::cache()->delete('routerDispatchData'); + } + + return $result; + } }