rugk / xenforo-threema-gateway

src/library/ThreemaGateway/Installer/Permissions.php

Doc Comments +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      *                                   2 = registered
      *                                   3 = administrator
      *                                   4 = moderator
-     * @param string $applyPermissionId  The permission id
+     * @param string $permissionId  The permission id
      * @param string $permissionValue    (optional) The value of the permission: allow, deny
      *                                   use_int
      * @param int    $permissionValueInt (optional) When using an integer (use_int)

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/Option/ReceiveCallback.php

Doc Comments -1 removed lines

@@ -50,7 +50,6 @@
     /**
      * Verifies whether the dir of the file is valid (can be created) and is writable.
      *
-     * @param string             $filepath  Input
      * @param XenForo_DataWriter $dw
      * @param string             $fieldName Name of field/option
      *

src/library/ThreemaGateway/Option/ThreemaGatewaySecret.php

Doc Comments -1 removed lines

@@ -53,7 +53,6 @@
     /**
      * Verifies the Threema Gateway Secret format.
      *
-     * @param string             $threemaid Input threema ID
      * @param XenForo_DataWriter $dw
      * @param string             $fieldName Name of field/option
      *

src/library/ThreemaGateway/Tfa/AbstractProvider.php

Doc Comments -4 removed lines

@@ -207,9 +207,7 @@ 
      * Currently this is not correctly implemented in XenForo.
      * See {@link https://xenforo.com/community/threads/1-5-documentation-for-two-step-authentication.102846/#post-1031047}
      *
-     * @param XenForo_Input $input
      * @param array         $user
-     * @param array         $error
      *
      * @return string HTML code
      */
@@ -392,8 +390,6 @@ 
     /**
      * Called when verifying displaying the choose 2FA mode.
      *
-     * @param array  $user
-     * @param object $error
      *
      * @return bool
      */

Indentation +2 added lines, -2 removed lines

@@ -422,8 +422,8 @@
     abstract protected function generateDefaultData();
 
     /**
-    * Adjust the view params for managing the 2FA mode, e.g. add special
-    * params needed by your template.
+     * Adjust the view params for managing the 2FA mode, e.g. add special
+     * params needed by your template.
      *
      * @param array  $viewParams
      * @param string $context

Spacing +1 added lines, -1 removed lines

@@ -726,7 +726,7 @@
             $displayTime = floor($hours / 24) . ' ' . new XenForo_Phrase('threemagw_days');
         }
 
-        return (string)$displayTime;
+        return (string) $displayTime;
     }
 
     /**

src/library/ThreemaGateway/Tfa/Fast.php

Indentation +2 added lines, -2 removed lines

@@ -340,8 +340,8 @@
     }
 
     /**
-    * Adjust the view params for managing the 2FA mode, e.g. add special
-    * params needed by your template.
+     * Adjust the view params for managing the 2FA mode, e.g. add special
+     * params needed by your template.
      *
      * @param array  $viewParams
      * @param string $context

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/Model/TfaPendingMessagesConfirmation.php

Indentation +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
         /** @var mixed $result result of SQL query */
         $result = $this->_getDb()->fetchRow('SELECT * FROM `' . self::DB_TABLE . '`
                   WHERE `request_id` = ?',
-                  $requestId);
+                    $requestId);
 
         if (!$result) {
             return null;
@@ -79,7 +79,7 @@ 
         /** @var mixed $result result of SQL query */
         $result = $this->fetchAllKeyed('SELECT * FROM `' . self::DB_TABLE . '`
                   WHERE ' . $this->getConditionsForClause($conditionsArray),
-                  'request_id', $paramsArray);
+                    'request_id', $paramsArray);
 
         if (!$result) {
             return null;

src/library/ThreemaGateway/Model/Keystore.php

Indentation +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
         /** @var mixed $result result of SQL query */
         $result = $this->_getDb()->fetchRow('SELECT * FROM `' . self::DB_TABLE . '`
                   WHERE `threema_id` = ?',
-                  $threemaId);
+                    $threemaId);
 
         if (is_array($result) && array_key_exists('public_key', $result)) {
             return (string) $result['public_key'];