shibdib / Dramiel

src/plugins/onMessage/charInfo.php

<?php
/**
 * The MIT License (MIT)
 *
 * Copyright (c) 2016 Robert Sardinia
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in all
 * copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 * SOFTWARE.
 */

use Discord\Discord;

/**
 * @property  message
 */
class charInfo

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public $config;
    public $discord;
    public $logger;
    private $excludeChannel;
    private $message;
    private $triggers;

    /**
     * @param $config
     * @param $discord
     * @param $logger
     */
    public function init($config, $discord, $logger)

This method seems to be duplicated in your project.

    {
        $this->config = $config;
        $this->discord = $discord;
        $this->logger = $logger;
        $this->excludeChannel = $this->config['bot']['restrictedChannels'];
        $this->triggers[] = $this->config['bot']['trigger'] . 'char';
        $this->triggers[] = $this->config['bot']['trigger'] . 'Char';
    }

    /**
     * @param $msgData
     * @param $message
     * @return null
     */
    public function onMessage($msgData, $message)
    {
        $channelID = (int) $msgData['message']['channelID'];

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

        $this->message = $message;

        $message = $msgData['message']['message'];
        $user = $msgData['message']['from'];

        $data = command($message, $this->information()['trigger'], $this->config['bot']['trigger']);
        if (isset($data['trigger'])) {

            // Most EVE players on Discord use their ingame name, so lets support @highlights
            $messageString = strstr($data['messageString'], '@') ? str_replace('<@', '', str_replace('>', '', $data['messageString'])) : $data['messageString'];
            if (is_numeric($messageString)) {
                // The person used @highlighting, so now we got a discord id, lets map that to a name
                $messageString = dbQueryField('SELECT name FROM usersSeen WHERE id = :id', 'name', array(':id' => $messageString));
            }

            $cleanString = urlencode($messageString);
            $characterID = urlencode(characterID($cleanString));

            if (empty($characterID)) {
                return $this->message->reply('**Error:** no data available');
            }

            //Get details
            $characterDetails = characterDetails($characterID);
            if (null === $characterDetails) {
                return $this->message->reply('**Error:** ESI is down. Try again later.');
            }
            $corporationID = $characterDetails['corporation_id'];
            $corporationName = corpName($corporationID);
            if (null === $corporationName) {
                return $this->message->reply('**Error:** ESI is down. Try again later.');
            }
            $corporationDetails = corpDetails($corporationID);
            $allianceID = @$corporationDetails['alliance_id'];
            if (null !== $allianceID) {
                $allianceName = allianceName($allianceID);
            } else {
                $allianceName = '';
            }
            $characterName = $characterDetails['name'];
            $dateOfBirth = $characterDetails['birthday'];

            if ($characterName === null || $characterName === '') {
                return $this->message->reply('**Error:** No character found.');
            }

            //ZKill lookup
            $url = "https://zkillboard.com/api/orderDirection/desc/limit/1/no-items/characterID/{$characterID}/xml/";
            $xml = makeApiRequest($url);
            if (empty($xml)) {
                return $this->message->reply('**Error:** ZKill is down. Try again later.');
            }
            foreach ($xml->result->rowset->row as $kill) {
                $lastSeenSystemID = @$kill->attributes()->solarSystemID;
                $lastSeenDate = @$kill->attributes()->killTime;
                if (null === $lastSeenSystemID || null === $lastSeenDate) {
                    $lastSeenSystem = 'No activity reported.';
                    $lastSeenDate = 'No activity reported.';
                } else {
                    $lastSeenSystem = getSystemName($lastSeenSystemID);
                }
            }
            foreach ($xml->result->rowset->row->rowset->row as $attacker) {
                if ($attacker->attributes()->characterID == $characterID) {
                    $lastSeenShipID = $attacker->attributes()->shipTypeID;
                    $lastSeenShip = getTypeName($lastSeenShipID);
                } else {
                    $lastSeenShip = 'No activity reported.';
                }
            }
            $url = "https://zkillboard.com/character/{$characterID}/";

            $msg = "```Name: {$characterName}
DOB: {$dateOfBirth}
			
Corporation Name: {$corporationName}
Alliance Name: {$allianceName}

Last Seen In System: {$lastSeenSystem}

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

Last Seen Flying a: {$lastSeenShip}

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

Last Seen On: {$lastSeenDate}```

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

For more info, visit: $url";

            $this->logger->addInfo("charInfo: Sending character info to {$user}");
            $this->message->reply($msg);
        }
        return null;
    }

    /**
     * @return array
     */
    public function information()
    {
        return array(
            'name' => 'char',
            'trigger' => $this->triggers,
            'information' => 'Returns basic EVE Online data about a character. To use simply type !char character_name'
        );
    }

}