]> git.mxchange.org Git - friendica.git/blobdiff - src/Core/Hook.php
in "getidforurl" "no update" is now "update"
[friendica.git] / src / Core / Hook.php
index 2d6c945067965263e45f6a2b879602615e76facd..8fdadd666a9c769da520feff285f8cc32d21c2d6 100644 (file)
@@ -1,17 +1,35 @@
 <?php
 /**
- * @file src/Core/Hook.php
+ * @copyright Copyright (C) 2020, Friendica
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
  */
+
 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
@@ -44,13 +62,13 @@ 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.
         *
-        * @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)
        {
@@ -61,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.
         *
@@ -70,10 +88,11 @@ class Hook extends BaseObject
         * @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)) {
@@ -92,10 +111,11 @@ class Hook extends BaseObject
         * @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];
@@ -124,13 +144,14 @@ class Hook extends BaseObject
        }
 
        /**
-        * @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)
        {
@@ -144,7 +165,7 @@ class Hook extends BaseObject
                                                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']) {
@@ -158,30 +179,32 @@ 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.
         *
-        * @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)
        {
@@ -210,6 +233,8 @@ class Hook extends BaseObject
         */
        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') {