namespace Friendica\Core;
use Friendica\App;
-use Friendica\BaseObject;
use Friendica\Database\DBA;
+use Friendica\DI;
+use Friendica\Util\Strings;
/**
* Some functions to handle hooks
*/
-class Hook extends BaseObject
+class Hook
{
/**
* Array of registered hooks
}
/**
- * @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.
*
- * @param type $hook
- * @param type $file
- * @param type $function
+ * @param string $hook
+ * @param string $file
+ * @param string $function
*/
public static function add($hook, $file, $function)
{
}
/**
- * @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.
*
* @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 register($hook, $file, $function, $priority = 0)
{
- $file = str_replace(self::getApp()->getBasePath() . DIRECTORY_SEPARATOR, '', $file);
+ $file = str_replace(DI::app()->getBasePath() . DIRECTORY_SEPARATOR, '', $file);
$condition = ['hook' => $hook, 'file' => $file, 'function' => $function];
if (DBA::exists('hook', $condition)) {
* @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 unregister($hook, $file, $function)
{
- $relative_file = str_replace(self::getApp()->getBasePath() . DIRECTORY_SEPARATOR, '', $file);
+ $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];
}
/**
- * @brief Forks a hook.
+ * Forks a hook.
*
* Use this function when you want to fork a hook via the worker.
*
* @param integer $priority of the hook
* @param string $name of the hook to call
* @param mixed $data to transmit to the callback handler
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
public static function fork($priority, $name, $data = null)
{
if (array_key_exists($name, self::$hooks)) {
foreach (self::$hooks[$name] as $hook) {
// Call a hook to check if this hook call needs to be forked
- $hookdata = ['name' => $name, 'data' => $data, 'execute' => true];
-
if (array_key_exists('hook_fork', self::$hooks)) {
+ $hookdata = ['name' => $name, 'data' => $data, 'execute' => true];
+
foreach (self::$hooks['hook_fork'] as $fork_hook) {
if ($hook[0] != $fork_hook[0]) {
continue;
}
- self::callSingle(self::getApp(), 'hook_fork', $fork_hook, $hookdata);
+ self::callSingle(DI::app(), 'hook_fork', $fork_hook, $hookdata);
}
- }
- if (!$hookdata['execute']) {
- continue;
+ if (!$hookdata['execute']) {
+ continue;
+ }
}
Worker::add($priority, 'ForkHook', $name, $hook, $data);
}
/**
- * @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.
*
- * @param string $name of the hook to call
+ * @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 callAll($name, &$data = null)
{
if (array_key_exists($name, self::$hooks)) {
foreach (self::$hooks[$name] as $hook) {
- self::callSingle(self::getApp(), $name, $hook, $data);
+ self::callSingle(DI::app(), $name, $hook, $data);
}
}
}
/**
- * @brief Calls a single hook.
+ * Calls a single hook.
*
- * @param App $a
- * @param string $name of the hook to call
- * @param array $hook Hook data
+ * @param App $a
+ * @param string $name of the hook to call
+ * @param array $hook Hook data
* @param string|array &$data to transmit to the callback handler
+ * @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/
public static function callSingle(App $a, $name, $hook, &$data = null)
{
*/
public static function isAddonApp($name)
{
+ $name = Strings::sanitizeFilePathItem($name);
+
if (array_key_exists('app_menu', self::$hooks)) {
foreach (self::$hooks['app_menu'] as $hook) {
if ($hook[0] == 'addon/' . $name . '/' . $name . '.php') {