+global\auth::onMessage()

Complexity

Conditions	53
Paths	2171

Size

Total Lines	157
Code Lines	106

Duplication

Lines	28
Ratio	17.83 %

Importance

Changes

0

<?php
    public function onMessage($msgData, $message)
    {
        $channelID = (int) $msgData['message']['channelID'];

        if (in_array($channelID, $this->excludeChannel, true)) {
            return null;
        }

        $this->message = $message;
        $userID = $msgData['message']['fromID'];
        $userName = $msgData['message']['from'];
        $message = $msgData['message']['message'];
        $channelInfo = $this->message->channel;
        $guildID = $channelInfo[@guild_id];
        $data = command($message, $this->information()['trigger'], $this->config['bot']['trigger']);
        if (isset($data['trigger'])) {
            if (isset($this->config['bot']['primary'])) {
                if ($guildID != $this->config['bot']['primary']) {
                    $this->message->reply('**Failure:** The auth code your attempting to use is for another discord server');
                    return null;
                }

            }
            // If config is outdated
            if (null === $this->authGroups) {
                $this->message->reply('**Failure:** Please update the bots config to the latest version.');
                return null;
            }

            $code = $data['messageString'];
            $result = selectPending($this->db, $this->dbUser, $this->dbPass, $this->dbName, $code);

            if (strlen($code) < 12) {
                $this->message->reply('Invalid Code, check ' . $this->config['bot']['trigger'] . 'help auth for more info.');
                return null;
            }

            while ($rows = $result->fetch_assoc()) {
                $charID = (int) $rows['characterID'];
                $corpID = (int) $rows['corporationID'];
                $allianceID = (int) $rows['allianceID'];

                //If corp is new store in DB
                $corpInfo = getCorpInfo($corpID);

Are you sure the assignment to $corpInfo is correct as getCorpInfo($corpID) (which targets getCorpInfo()) seems to always return null.

                if (null === $corpInfo) {
                    $corpDetails = corpDetails($corpID);
                    if (null === $corpDetails) { // Make sure it's always set.
                        $this->message->reply('**Failure:** Unable to auth at this time, ESI is down. Please try again later.');
                        return null;
                    }
                    $corpTicker = $corpDetails['ticker'];
                    $corpName = (string)$corpDetails['corporation_name'];
                    if (null !== $corpTicker) {
                        addCorpInfo($corpID, $corpTicker, $corpName);
                    }
                } else {
                    $corpTicker = $corpInfo['corpTicker'];
                }

                //Add corp ticker to name
                if ($this->corpTickers === 'true') {
                    $setTicker = 1;
                }

                //Set eve name if nameCheck is true
                if ($this->nameEnforce === 'true') {
                    $nameEnforce = 1;
                }
                $role = null;

                $roles = @$this->message->channel->guild->roles;
                $member = @$this->message->channel->guild->members->get('id', $userID);
                $eveName = characterName($charID);
                if (null === $eveName) {
                    $this->message->reply('**Failure:** Unable to auth at this time, ESI is down. Please try again later.');
                    return null;
                }
                foreach ($this->authGroups as $authGroup) {
                    //Check if corpID matches
                    if ($corpID === $authGroup['corpID']) {
                        foreach ($roles as $role) {

This code seems to be duplicated across your project.

                            if ((string)$role->name === (string)$authGroup['corpMemberRole']) {
                                $member->addRole($role);
                                $role = 'corp';
                            }
                        }
                    }
                    //Check if allianceID matches
                    if ($allianceID === $authGroup['allianceID'] && $authGroup['allianceID'] != 0) {
                        foreach ($roles as $role) {

This code seems to be duplicated across your project.

                            if ((string)$role->name === (string)$authGroup['allyMemberRole']) {
                                $member->addRole($role);
                                $role = 'ally';
                            }
                        }
                    }
                    //check for standings based roles
                    if ($this->standingsBased === 'true' && $role === null) {
                        $allianceContacts = getContacts($allianceID);

Are you sure the assignment to $allianceContacts is correct as getContacts($allianceID) (which targets getContacts()) seems to always return null.

                        $corpContacts = getContacts($corpID);

Are you sure the assignment to $corpContacts is correct as getContacts($corpID) (which targets getContacts()) seems to always return null.

                        foreach ($roles as $role) {
                            if ((@(int)$allianceContacts['standings'] === 5 || @(int)$corpContacts['standings'] === 5) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['plus5Role']) {

This code seems to be duplicated across your project.

                                $member->addRole($role);
                                $role = 'blue';
                            }
                            if ((@(int)$allianceContacts['standings'] === 10 || @(int)$corpContacts['standings'] === 10) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['plus10Role']) {

This code seems to be duplicated across your project.

                                $member->addRole($role);
                                $role = 'blue';
                            }
                            if ((@(int)$allianceContacts['standings'] === 0 || @(int)$corpContacts['standings'] === 0 || (@(int)$allianceContacts['standings'] && @(int)$corpContacts['standings'] === null || '')) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['neutralRole']) {
                                $member->addRole($role);
                                $role = 'neut';
                            }
                            if ((@(int)$allianceContacts['standings'] === -5 || @(int)$corpContacts['standings'] === -5) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['minus5Role']) {

This code seems to be duplicated across your project.

                                $member->addRole($role);
                                $role = 'red';
                            }
                            if ((@(int)$allianceContacts['standings'] === -10 || @(int)$corpContacts['standings'] === -10) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['minus10Role']) {

This code seems to be duplicated across your project.

                                $member->addRole($role);
                                $role = 'red';
                            }
                        }
                    }
                    if (null !== $role) {
                        $guild = $this->discord->guilds->get('id', $guildID);
                        insertUser($this->db, $this->dbUser, $this->dbPass, $this->dbName, $userID, $charID, $eveName, $role);
                        disableReg($this->db, $this->dbUser, $this->dbPass, $this->dbName, $code);
                        $msg = ":white_check_mark: **Success:** {$userName} has been successfully authed.";
                        $this->logger->addInfo("auth: {$eveName} authed");
                        $this->message->reply($msg);
                        //Add ticker if set and change name if nameEnforce is on
                        if (isset($setTicker) || isset($nameEnforce)) {
                            if (isset($setTicker) && isset($nameEnforce)) {
                                $nick = "[{$corpTicker}] {$eveName}";
                            } elseif (null === $setTicker && isset($nameEnforce)) {
                                $nick = "{$eveName}";
                            } elseif (isset($setTicker) && !isset($nameEnforce)) {
                                $nick = "[{$corpTicker}] {$userName}";
                            }
                        }
                        if (null !== $nick) {
                            queueRename($userID, $nick, $this->guild);

The variable $nick does not seem to be defined for all execution paths leading up to this point.

                        }
                        $guild->members->save($member);
                        return null;
                    }
                }
                $this->message->reply('**Failure:** There are no roles available for your corp/alliance.');
                $this->logger->addInfo('Auth: User was denied due to not being in the correct corp or alliance ' . $eveName);
                return null;
            }
            $this->message->reply('**Failure:** There was an issue with your code.');
            $this->logger->addInfo('Auth: User was denied due to the code being invalid ' . $userName);
            return null;
        }
        return null;
    }