Method createConnectionSettings has 30 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function createConnectionSettings($gatewayId, $gatewaySecret)
    {
        /** @var XenForo_Options $xenOptions */
        $xenOptions = XenForo_Application::getOptions();

Function ReceiveCallback has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

var ReceiveCallback = (function () {
    'use strict';
    var me = {};
    var $inputElem;
    var $hiddenElem;

Method checkForReceiverCode has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function checkForReceiverCode(ThreemaGateway_Handler_Action_Callback $handler,
                                        Threema\MsgApi\Helpers\ReceiveMessageResult $receiveResult,
                                        Threema\MsgApi\Messages\ThreemaMessage $threemaMsg,
                                        &$output,
                                        &$saveMessage,

Method _getExistingData has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function _getExistingData($data)
    {
        // try primary key first
        /** @var string $requestId */
        if ($requestId = $this->_getExistingPrimaryKey($data, 'request_id')) {

Method queryCredits has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected static function queryCredits(array &$status, array &$extraError)
    {
        /** @var ThreemaGateway_Handler_Permissions $permissions */
        $permissions = ThreemaGateway_Handler_Permissions::getInstance();

Method checkTecRequire has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected static function checkTecRequire(array &$status, array &$extraError)
    {
        // optional check: HTTPS
        if (!XenForo_Application::$secure) {
            $extraError[]['text'] = new XenForo_Phrase('option_threema_gateway_status_no_https');

Function getMessageMetaData has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function getMessageMetaData($groupById = false, $ignoreInvalid = true)
    {
        /** @var array $limitOptions */
        $limitOptions = $this->prepareLimitFetchOptions($this->fetchOptions);

• Read up
  Read up

Function processPending has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function processPending(array $processOptions = [])
    {
        if (!$this->pendingRequests) {
            $this->preProcessPending();
        }

• Read up
  Read up

Function getRandomBytes has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public static function getRandomBytes($number)
    {
        /** @var string $output */
        $output = '';

• Read up
  Read up

Method validatePreConditions has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function validatePreConditions(&$errorString)
    {
        /** @var XenForo_Options $options */
        $options = XenForo_Application::getOptions();

Method validateRequest has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function validateRequest(&$errorString)
    {
        // access token validation (authentication of Gateway server)
        /** @var XenForo_Options $options */
        $options = XenForo_Application::getOptions();

Method uninstall has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function uninstall()
    {
        // delete throttle table
        /** @var ThreemaGateway_Installer_ActionThrottle $throttleInstaller */
        $throttleInstaller = new ThreemaGateway_Installer_ActionThrottle;

Consider simplifying this complex logical expression.

Open

        if (!$this->input->inRequest('from') ||
            !$this->input->inRequest('to') ||
            !$this->input->inRequest('messageId') ||
            !$this->input->inRequest('date') ||
            !$this->input->inRequest('nonce') ||

Function actionTwoStepEnable has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function actionTwoStepEnable()
    {
        /** @var XenForo_ControllerResponse_View $parent */
        $parent = parent::actionTwoStepEnable();

• Read up
  Read up

Function triggerVerification has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    public function triggerVerification($context, array $user, $userIp, array &$providerData)
    {
        parent::triggerVerification($context, $user, $userIp, $providerData);

        // this 2FA mode requires end-to-end encryption

• Read up
  Read up

Function _getExistingData has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    protected function _getExistingData($data)
    {
        // try primary key first
        /** @var string $requestId */
        if ($requestId = $this->_getExistingPrimaryKey($data, 'request_id')) {

• Read up
  Read up

Method testCallbackPreSave has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public static function testCallbackPreSave(ThreemaGateway_Handler_Action_Callback $handler,
                                        Threema\MsgApi\Helpers\ReceiveMessageResult $receiveResult,
                                        Threema\MsgApi\Messages\ThreemaMessage $threemaMsg,
                                        &$output,
                                        &$saveMessage,

Method testCallbackPostSave has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public static function testCallbackPostSave(ThreemaGateway_Handler_Action_Callback $handler,
                                        Threema\MsgApi\Helpers\ReceiveMessageResult $receiveResult,
                                        Threema\MsgApi\Messages\ThreemaMessage $threemaMsg,
                                        &$output,
                                        $messageSaved,

Function getDefaultThreemaId has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    final protected function getDefaultThreemaId(array $user)
    {
        /** @var XenForo_Options $options */
        $options = XenForo_Application::getOptions();
        // check for custom user field

• Read up
  Read up

Method __construct has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function __construct(ThreemaGateway_Handler_Action_Callback $handler,
                                Threema\MsgApi\Helpers\ReceiveMessageResult $receiveResult,
                                Threema\MsgApi\Messages\ThreemaMessage $threemaMsg,
                                &$output,
                                &$saveMessage,