rugk / xenforo-threema-gateway

src/library/ThreemaGateway/Handler/Action/TfaCallback/TextMessage.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      * Returns "false" if the process should be canceled. Otherwise "true".
      *
      * @throws XenForo_Exception
-     * @return bool
+     * @return false|null
      */
     protected function preProcessPending()
     {

src/library/ThreemaGateway/Model/Messages.php

Doc Comments +7 added lines, -8 removed lines

@@ -84,8 +84,8 @@ 
      * option call it this way: injectFetchOption('where', []).
      *
      * @param string $option The option name to inject
-     * @param mixed  $value  The value of the option to set.
-     * @param mixed  $append If set to true, the value is not overriden, but
+     * @param string  $value  The value of the option to set.
+     * @param boolean  $append If set to true, the value is not overriden, but
      *                       just appended as an array. (default: false)
      */
     public function injectFetchOption($option, $value, $append = false)
@@ -147,7 +147,7 @@ 
      * If you want to limit the types you want to query this method would be a
      * good way for you to use.
      *
-     * @param string $typeCode one (string) or more (array) mtype codes
+     * @param string $typeCodes one (string) or more (array) mtype codes
      */
     public function setTypeCode($typeCodes)
     {
@@ -197,7 +197,6 @@ 
      * Limit the result to a number of datasets.
      *
      * @param int $limit    oldest date of messages
-     * @param int $date_max latest date of messages (optional)
      */
     public function setResultLimit($limit)
     {
@@ -518,9 +517,9 @@ 
      * submit additional conditions with the first parameter.
      * Attention: This ignores the limit/offset clause for simplicity.
      *
-     * @param array $additionalConditions Add additional where conditions if
+     * @param string[] $additionalConditions Add additional where conditions if
      *                                    neccessary.
-     * @param array $removeOnlyField      When set only the passed fields are
+     * @param string[] $removeOnlyField      When set only the passed fields are
      *                                    updated to "null" rather than deleting
      *                                    the whole record.
      */
@@ -582,7 +581,7 @@ 
      *
      * @param array $baseArray  the main array, where the key/value pairs get to
      * @param array $subArray   the array, which keys should be removed
-     * @param array $removeKeys an array of keys, which should be removed
+     * @param string[] $removeKeys an array of keys, which should be removed
      *
      * @throws XenForo_Exception
      * @return false|array
@@ -611,7 +610,7 @@ 
      * Groups an array by using the value of a specific index in it.
      *
      * @param array      $array       the array, which is sued as the base
-     * @param string|int $indexKey    the value of the key, which should be used
+     * @param string $indexKey    the value of the key, which should be used
      *                                for indexing
      * @param bool       $ignoreIndex Set to true to ignore multiple values in
      *                                $array. If activated only the last key of

src/library/ThreemaGateway/Handler/Action/TfaCallback/DeliveryReceipt.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      * Returns "false" if the process should be canceled. Otherwise "true".
      *
      * @throws XenForo_Exception
-     * @return bool
+     * @return false|null
      */
     protected function preProcessPending()
     {

src/library/ThreemaGateway/Installer/TfaProvider.php

Indentation +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
         $db->query('INSERT INTO `xf_tfa_provider`
                   (`provider_id`, `provider_class`, `priority`, `active`)
                   VALUES (?, ?, ?, ?)',
-                  [$this->tfaId, $this->tfaClass, $this->tfaPriority, (int) $enabled]);
+                    [$this->tfaId, $this->tfaClass, $this->tfaPriority, (int) $enabled]);
     }
 
     /**

src/library/ThreemaGateway/Helper/Message.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
     public static function getOldestPossibleReplayAttackDate()
     {
         /** @var XenForo_Options $options */
-        $options   = XenForo_Application::getOptions();
+        $options = XenForo_Application::getOptions();
         /** @var int $rejectOlDefault the default maximum age of a message according*/
         $rejectOlDefault = strtotime('-14 days', XenForo_Application::$time);
         /* @var int $rejectOld the maximum age of a message according to the options */

src/library/ThreemaGateway/Handler/PhpSdk.php

Doc Comments +1 added lines

@@ -65,6 +65,7 @@
      * Initiate PHP-SDK.
      *
      * @param ThreemaGateway_Handler_Settings
+     * @param ThreemaGateway_Handler_Settings|null $settings
      * @throws XenForo_Exception
      */
     private function __construct($settings)

src/library/ThreemaGateway/Handler/Action/Callback.php

Doc Comments +1 added lines, -1 removed lines

@@ -358,7 +358,7 @@
      * In case nothing could be found, this returns "null".
      *
      * @param  string            $key
-     * @return string|array|null
+     * @return string
      */
     public function getRequest($key = null)
     {

src/library/ThreemaGateway/Handler/Action/TfaCallback/Abstract.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      * Returns "false" if the process should be canceled. Otherwise "true".
      *
      * @throws XenForo_Exception
-     * @return bool
+     * @return false|null
      */
     protected function preProcessPending()
     {

src/library/ThreemaGateway/Helper/Key.php

Doc Comments -1 removed lines

@@ -60,7 +60,6 @@
      *
      * XenForo template helper: threemaisvalidpubkey.
      *
-     * @param  string $threemaid Threema ID
      * @return bool
      */
     public static function checkPublic($pubKey)