<?php
/**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2022, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
/**
* Load hooks
+ *
+ * @return void
*/
public static function loadHooks()
{
* @param string $hook
* @param string $file
* @param string $function
+ * @return void
*/
- public static function add($hook, $file, $function)
+ public static function add(string $hook, string $file, string $function)
{
if (!array_key_exists($hook, self::$hooks)) {
self::$hooks[$hook] = [];
* @return mixed|bool
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
- public static function register($hook, $file, $function, $priority = 0)
+ public static function register(string $hook, string $file, string $function, int $priority = 0)
{
$file = str_replace(DI::app()->getBasePath() . DIRECTORY_SEPARATOR, '', $file);
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]);
}
/**
* @return boolean
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
- public static function unregister($hook, $file, $function)
+ public static function unregister(string $hook, string $file, string $function): bool
{
$relative_file = str_replace(DI::app()->getBasePath() . DIRECTORY_SEPARATOR, '', $file);
// 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);
- return $result;
+
+ return self::delete($condition);
}
/**
* @param string $name Name of the hook
* @return array
*/
- public static function getByName($name)
+ public static function getByName(string $name): array
{
$return = [];
* @param integer $priority of the hook
* @param string $name of the hook to call
* @param mixed $data to transmit to the callback handler
+ * @return void
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
- public static function fork($priority, $name, $data = null)
+ public static function fork(int $priority, string $name, $data = null)
{
if (array_key_exists($name, self::$hooks)) {
foreach (self::$hooks[$name] as $hook) {
*
* @param string $name of the hook to call
* @param string|array &$data to transmit to the callback handler
+ * @return void
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
- public static function callAll($name, &$data = null)
+ public static function callAll(string $name, &$data = null)
{
if (array_key_exists($name, self::$hooks)) {
foreach (self::$hooks[$name] as $hook) {
* @param string $name of the hook to call
* @param array $hook Hook data
* @param string|array &$data to transmit to the callback handler
+ * @return void
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
- public static function callSingle(App $a, $name, $hook, &$data = null)
+ public static function callSingle(App $a, string $name, array $hook, &$data = null)
{
// Don't run a theme's hook if the user isn't using the theme
if (strpos($hook[0], 'view/theme/') !== false && strpos($hook[0], 'view/theme/' . $a->getCurrentTheme()) === false) {
} else {
// remove orphan hooks
$condition = ['hook' => $name, 'file' => $hook[0], 'function' => $hook[1]];
- DBA::delete('hook', $condition, ['cascade' => false]);
+ self::delete($condition);
}
}
* @param string $name Name of the addon
* @return boolean
*/
- public static function isAddonApp($name)
+ public static function isAddonApp(string $name): bool
{
$name = Strings::sanitizeFilePathItem($name);
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): bool
+ {
+ $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): bool
+ {
+ $result = DBA::insert('hook', $condition);
+
+ if ($result) {
+ DI::cache()->delete('routerDispatchData');
+ }
+
+ return $result;
+ }
}