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
$stmt = DBA::select('hook', ['hook', 'file', 'function'], [], ['order' => ['priority' => 'desc', 'file']]);
while ($hook = DBA::fetch($stmt)) {
- if (!array_key_exists($hook['hook'], self::$hooks)) {
- self::$hooks[$hook['hook']] = [];
- }
- self::$hooks[$hook['hook']][] = [$hook['file'], $hook['function']];
+ self::add($hook['hook'], $hook['file'], $hook['function']);
}
DBA::close($stmt);
}
+ /**
+ * 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 string $hook
+ * @param string $file
+ * @param string $function
+ */
+ public static function add($hook, $file, $function)
+ {
+ if (!array_key_exists($hook, self::$hooks)) {
+ self::$hooks[$hook] = [];
+ }
+ self::$hooks[$hook][] = [$file, $function];
+ }
+
/**
* 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 $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 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];
/**
* Returns the list of callbacks for a single hook
*
- * @param string $name
+ * @param string $name Name of the hook
* @return array
*/
public static function getByName($name)
}
/**
- * @brief Forks a hook.
+ * Forks a hook.
*
* Use this function when you want to fork a hook via the worker.
*
- * @param string $name of the hook to call
- * @param string|array $data to transmit to the callback handler
+ * @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
+ 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(DI::app(), 'hook_fork', $fork_hook, $hookdata);
+ }
+
+ 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)
{
}
/**
- * check if an app_menu hook exist for addon $name.
+ * Checks if an app_menu hook exist for the provided addon name.
* Return true if the addon is an app
+ *
+ * @param string $name Name of the addon
+ * @return boolean
*/
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') {