getKillmails::getBigKM() - Code Metrics - Inspection of "Daily Inspection: Merge pull request #236 from shi..." - shibdib/Dramiel - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Branch — dev (952e73)

unknown

created 2021-05-17 18:22 UTC

getKillmails::getBigKM() C

↳ Parent: getKillmails

Complexity

Conditions	13
Paths	20

Size

Total Lines

Duplication

Lines	8
Ratio	13.11 %

Importance

Changes

Metric	Value
cc	13
nc	20
nop	0
dl	8
loc	61
rs	6.6166
c	0
b	0
f	0

How to fix Long Method Complexity

<?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;

/**
 * Class getKillmails
 */
class getKillmails
{
    public $config;
    public $db;
    public $discord;
    public $channelConfig;
    public $lastCheck = 0;
    public $logger;
    public $groupConfig;

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

    {
        $this->config = $config;
        $this->discord = $discord;
        $this->logger = $logger;
        $this->groupConfig = $config['plugins']['getKillmails']['groupConfig'];
        $this->guild = $config['bot']['guild'];
class MyClass { }

$x = new MyClass();
$x->foo = true;
        //Refresh check at bot start
        setPermCache('killmailCheck', time() - 5);
    }

    /**
     *
     */
    public function tick()
    {
        // What was the servers last reported state
        $lastStatus = getPermCache('serverState');
        if ($lastStatus === 'online') {
            $lastChecked = getPermCache('killmailCheck');
            if ($lastChecked <= time()) {
                //check if user is still using the old config
                if (null === $this->groupConfig) {

                    $this->logger->addError('Killmails: UPDATE YOUR CONFIG TO RECEIVE KILLMAILS');
                    setPermCache('killmailCheck', time() + 600);
                    return null;
                }
                $this->logger->addInfo('Killmails: Checking for new killmails.');
                $this->getKM();
                setPermCache('killmailCheck', time() + 600);
                if (@$this->config['plugins']['getKillmails']['bigKills']['shareBigKills'] === 'true') {
                    $this->logger->addInfo('Killmails: Checking for 10b+ kills.');
                    $this->getBigKM();
                }
            }
        }
    }

    private function getKM()
    {
        foreach ($this->groupConfig as $kmGroup) {
            $SavedKillID = getPermCache("{$kmGroup['name']}newestKillmailID");
            //check if start id is greater than current id and if it is set
            if (null !== $SavedKillID && preg_match('/[a-z]/i', $SavedKillID)) {
                getStartMail($kmGroup);
                $SavedKillID = getPermCache("{$kmGroup['name']}newestKillmailID");
            }
            if ($kmGroup['startMail'] > $SavedKillID || null === $SavedKillID) {
                $SavedKillID = $kmGroup['startMail'];
            }
            if ((string)$kmGroup['allianceID'] === '0' & $kmGroup['lossMails'] === 'true') {
// Returns always int(0).
return 0 === $foo & 4;
return (0 === $foo) & 4;

// More likely intended return: true/false
return 0 === ($foo & 4);
                $url = "https://zkillboard.com/api/corporationID/{$kmGroup['corpID']}/no-attackers/no-items/orderDirection/asc/pastSeconds/10800/";
            }
            if ((string)$kmGroup['allianceID'] === '0' & $kmGroup['lossMails'] === 'false') {
// Returns always int(0).
return 0 === $foo & 4;
return (0 === $foo) & 4;

// More likely intended return: true/false
return 0 === ($foo & 4);
                $url = "https://zkillboard.com/api/corporationID/{$kmGroup['corpID']}/no-attackers/no-items/kills/orderDirection/asc/pastSeconds/10800/";
            }
            if ((string)$kmGroup['allianceID'] !== '0' & $kmGroup['lossMails'] === 'true') {
// Returns always int(0).
return 0 === $foo & 4;
return (0 === $foo) & 4;

// More likely intended return: true/false
return 0 === ($foo & 4);
                $url = "https://zkillboard.com/api/allianceID/{$kmGroup['allianceID']}/no-attackers/no-items/orderDirection/asc/pastSeconds/10800/";
            }
            if ((string)$kmGroup['allianceID'] !== '0' & $kmGroup['lossMails'] === 'false') {
// Returns always int(0).
return 0 === $foo & 4;
return (0 === $foo) & 4;

// More likely intended return: true/false
return 0 === ($foo & 4);
                $url = "https://zkillboard.com/api/allianceID/{$kmGroup['allianceID']}/no-attackers/no-items/kills/orderDirection/asc/pastSeconds/10800/";
            }

            if (!isset($url)) { // Make sure it's always set.
                $this->logger->addInfo('Killmails: ERROR - Ensure your config file is setup correctly for killmails.');
                return null;
            }

            $kills = json_decode(downloadData($url), true);
//            $i = 0;
            if (isset($kills)) {
                foreach ($kills as $kill) {
 //                   if ($i < 10) {
                        //if big kill isn't set, disable it
                        if (!array_key_exists('bigKill', $kmGroup)) {
                            $kmGroup['bigKill'] = 99999999999999999999999999;
                        }
                        $killID = $kill['killID'];
                        if ($killID <= $SavedKillID) {
//                            $i++;
                            continue;
                        }
                        $channelID = $kmGroup['channel'];
                        $solarSystemID = $kill['solarSystemID'];
                        $systemName = getSystemName($solarSystemID);
                        $killTime = $kill['killTime'];
                        $victimAllianceName = '';
                    if ($kill['victim']['allianceName'] !== null && $kill['victim']['allianceName'] !== '') {

                            $victimAllianceName = "|{$kill['victim']['allianceName']}";
                        }
                        $victimName = $kill['victim']['characterName'];
                        $victimCorpName = $kill['victim']['corporationName'];
                        $victimShipID = $kill['victim']['shipTypeID'];
                        $shipName = getTypeName($victimShipID);
                        $rawValue = $kill['zkb']['totalValue'];
                        //Check if killmail meets minimum value and if it meets lost minimum value
                        if (isset($kmGroup['minimumValue']) && isset($kmGroup['minimumlossValue'])) {
                            if ($rawValue < $kmGroup['minimumValue']) {
                                if ($kill['victim']['corporationID'] == $kmGroup['corpID'] && $rawValue > $kmGroup['minimumlossValue'] ||
                                $kill['victim']['allianceID'] == $kmGroup['allianceID'] && $rawValue > $kmGroup['minimumlossValue'])
                                {
                                    $this->logger->addInfo("Killmails: Mail {$killID} posted because it meet minimum loss value required.");
                                } else {
                                    $this->logger->addInfo("Killmails: Mail {$killID} ignored for not meeting the minimum value required.");
                                    setPermCache("{$kmGroup['name']}newestKillmailID", $killID);
                                    continue;
                                }
                            }
                        }
                        $totalValue = number_format($kill['zkb']['totalValue']);
                        // Check if it's a structure
                        if ($victimName !== '' || ' ' || null) {

                            if ($kmGroup['bigKill'] != null && $rawValue >= $kmGroup['bigKill']) {
                                $channelID = $kmGroup['bigKillChannel'];
                                $msg = "@here \n :warning:***Expensive Killmail***:warning: \n **{$killTime}**\n\n**{$shipName}** worth **{$totalValue} ISK** flown by **{$victimName}** of (***{$victimCorpName}|{$victimAllianceName}***) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
                            } elseif ($kmGroup['bigKill'] == null || $rawValue <= $kmGroup['bigKill']) {
                                $msg = "**{$killTime}**\n\n**{$shipName}** worth **{$totalValue} ISK** flown by **{$victimName}** of (***{$victimCorpName}|{$victimAllianceName}***) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
                            }
                        } elseif ($victimName === '') {
                            $msg = "**{$killTime}**\n\n**{$shipName}** worth **{$totalValue} ISK** owned by (***{$victimCorpName}|{$victimAllianceName}***) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
                        }

                        if (!isset($msg)) { // Make sure it's always set.
                            return null;
                        }
                        queueMessage($msg, $channelID, $this->guild);
                        $this->logger->addInfo("Killmails: Mail {$killID} queued.");

//                        $i++;
/*                    } else {
                        $updatedID = getPermCache("{$kmGroup['name']}newestKillmailID");
                        $this->logger->addInfo("Killmails: Kill posting cap reached, newest kill id for {$kmGroup['name']} is {$updatedID}");
                        break;
                    }*/
                }
                if (!isset($killID)) {
                    $killID = $SavedKillID;
                }
                setPermCache("{$kmGroup['name']}newestKillmailID", $killID);
            }
            $updatedID = getPermCache("{$kmGroup['name']}newestKillmailID");
            $this->logger->addInfo("Killmails: All kills posted, newest kill id for {$kmGroup['name']} is {$updatedID}");
            continue;
        }
    }

    private function getBigKM()
    {
        $oldID = getPermCache('bigKillNewestKillmailID');
        if (null === $oldID || preg_match('/[a-z]/i', $oldID)) {
            getStartBigMail();
            $oldID = getPermCache('bigKillNewestKillmailID');
        }

        $url = 'https://zkillboard.com/api/kills/orderDirection/desc/iskValue/10000000000/limit/10/';

        $kills = json_decode(downloadData($url), true);
        $i = 0;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        if (isset($kills)) {
            foreach ($kills as $kill) {
                $cacheID = getPermCache('bigKillNewestKillmailID');
 //               if ($i < 10) {
                    $killID = $kill['killID'];
                    //check if mail is old
                    if ((int)$killID <= (int)$oldID) {
                        continue;
                    }
                    //save highest killID for cache
                    if ($killID > $cacheID) {
                        setPermCache('bigKillNewestKillmailID', $killID);
                    }
                    $channelID = $this->config['plugins']['getKillmails']['bigKills']['bigKillChannel'];
                    $solarSystemID = $kill['solarSystemID'];
                    $systemName = getSystemName($solarSystemID);
                    $killTime = $kill['killTime'];
                $victimAllianceName = '';
                if ($kill['victim']['allianceName'] !== null && $kill['victim']['allianceName'] !== '') {

                    $victimAllianceName = "|{$kill['victim']['allianceName']}";
                }
                    $victimName = $kill['victim']['characterName'];
                    $victimCorpName = $kill['victim']['corporationName'];
                    $victimShipID = $kill['victim']['shipTypeID'];
                    $shipName = getTypeName($victimShipID);
                    $totalValue = number_format($kill['zkb']['totalValue']);
                    // Check if it's a structure
                    if ($victimName !== '' || ' ' || null) {

                        $msg = "**10b+ Kill Reported**\n\n**{$killTime}**\n\n**{$shipName}** worth **{$totalValue} ISK** flown by **{$victimName}** of (***{$victimCorpName}|{$victimAllianceName}***) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
                    } else {
                        $msg = "**10b+ Kill Reported**\n\n**{$killTime}**\n\n**{$shipName}** worth **{$totalValue} ISK** owned by (***{$victimCorpName}|{$victimAllianceName}***) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
                    }
                    if (!isset($msg)) { // Make sure it's always set.
                        return null;
                    }
                    queueMessage($msg, $channelID, $this->guild);
                    $this->logger->addInfo("Killmails: Mail {$killID} queued.");

/*                    $i++;
                } else {
                    $cacheID = getPermCache('bigKillNewestKillmailID');
                    $this->logger->addInfo("Killmails: bigKill posting cap reached, newest kill id is {$cacheID}");
                    break;
                }*/
            }
        }
        $updatedID = getPermCache('bigKillNewestKillmailID');
        $this->logger->addInfo("Killmails: All bigKills posted, newest kill id is {$updatedID}");
    }
}


1		<?php
2		/**
3		* The MIT License (MIT)
4		*
5		* Copyright (c) 2016 Robert Sardinia
6		*
7		* Permission is hereby granted, free of charge, to any person obtaining a copy
8		* of this software and associated documentation files (the "Software"), to deal
9		* in the Software without restriction, including without limitation the rights
10		* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
11		* copies of the Software, and to permit persons to whom the Software is
12		* furnished to do so, subject to the following conditions:
13		*
14		* The above copyright notice and this permission notice shall be included in all
15		* copies or substantial portions of the Software.
16		*
17		* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
18		* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
19		* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
20		* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
21		* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
22		* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
23		* SOFTWARE.
24		*/
25
26		use discord\discord;
27
28		/**
29		* Class getKillmails
30		*/
31		class getKillmails
32		{
33		public $config;
34		public $db;
35		public $discord;
36		public $channelConfig;
37		public $lastCheck = 0;
38		public $logger;
39		public $groupConfig;
40
41		/**
42		* @param $config
43		* @param $discord
44		* @param $logger
45		*/
46	View Code Duplication	public function init($config, $discord, $logger)
		0 ignored issues – show Duplication introduced 2017-08-09 18:57 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
47		{
48		$this->config = $config;
49		$this->discord = $discord;
50		$this->logger = $logger;
51		$this->groupConfig = $config['plugins']['getKillmails']['groupConfig'];
52		$this->guild = $config['bot']['guild'];
		0 ignored issues – show Bug introduced 2016-12-15 18:49 UTC by Report Bug Copy Issue Report The property `guild` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
53		//Refresh check at bot start
54		setPermCache('killmailCheck', time() - 5);
55		}
56
57		/**
58		*
59		*/
60		public function tick()
61		{
62		// What was the servers last reported state
63		$lastStatus = getPermCache('serverState');
64		if ($lastStatus === 'online') {
65		$lastChecked = getPermCache('killmailCheck');
66		if ($lastChecked <= time()) {
67		//check if user is still using the old config
68	View Code Duplication	if (null === $this->groupConfig) {
		0 ignored issues – show Duplication introduced 2016-12-15 18:49 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
69		$this->logger->addError('Killmails: UPDATE YOUR CONFIG TO RECEIVE KILLMAILS');
70		setPermCache('killmailCheck', time() + 600);
71		return null;
72		}
73		$this->logger->addInfo('Killmails: Checking for new killmails.');
74		$this->getKM();
75		setPermCache('killmailCheck', time() + 600);
76		if (@$this->config['plugins']['getKillmails']['bigKills']['shareBigKills'] === 'true') {
77		$this->logger->addInfo('Killmails: Checking for 10b+ kills.');
78		$this->getBigKM();
79		}
80		}
81		}
82		}
83
84		private function getKM()
85		{
86		foreach ($this->groupConfig as $kmGroup) {
87		$SavedKillID = getPermCache("{$kmGroup['name']}newestKillmailID");
88		//check if start id is greater than current id and if it is set
89		if (null !== $SavedKillID && preg_match('/[a-z]/i', $SavedKillID)) {
90		getStartMail($kmGroup);
91		$SavedKillID = getPermCache("{$kmGroup['name']}newestKillmailID");
92		}
93		if ($kmGroup['startMail'] > $SavedKillID \|\| null === $SavedKillID) {
94		$SavedKillID = $kmGroup['startMail'];
95		}
96	View Code Duplication	if ((string)$kmGroup['allianceID'] === '0' & $kmGroup['lossMails'] === 'true') {
		0 ignored issues – show Comprehensibility introduced 2017-02-16 18:36 UTC by Report Bug Copy Issue Report Consider adding parentheses for clarity. Current Interpretation: `((string) $kmGroup['alli...'lossMails'] === 'true'`, Probably Intended Meaning: `(string) $kmGroup['allia...lossMails'] === 'true')` When comparing the result of a bit operation, we suggest to add explicit parenthesis and not to rely on PHP’s built-in operator precedence to ensure the code behaves as intended and to make it more readable. Let’s take a look at these examples: // Returns always int(0). return 0 === $foo & 4; return (0 === $foo) & 4; // More likely intended return: true/false return 0 === ($foo & 4); Loading history... Duplication introduced 2016-12-15 18:49 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
97		$url = "https://zkillboard.com/api/corporationID/{$kmGroup['corpID']}/no-attackers/no-items/orderDirection/asc/pastSeconds/10800/";
98		}
99	View Code Duplication	if ((string)$kmGroup['allianceID'] === '0' & $kmGroup['lossMails'] === 'false') {
		0 ignored issues – show Comprehensibility introduced 2017-02-16 18:36 UTC by Report Bug Copy Issue Report Consider adding parentheses for clarity. Current Interpretation: `((string) $kmGroup['alli...lossMails'] === 'false'`, Probably Intended Meaning: `(string) $kmGroup['allia...ossMails'] === 'false')` When comparing the result of a bit operation, we suggest to add explicit parenthesis and not to rely on PHP’s built-in operator precedence to ensure the code behaves as intended and to make it more readable. Let’s take a look at these examples: // Returns always int(0). return 0 === $foo & 4; return (0 === $foo) & 4; // More likely intended return: true/false return 0 === ($foo & 4); Loading history... Duplication introduced 2016-11-04 18:22 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
100		$url = "https://zkillboard.com/api/corporationID/{$kmGroup['corpID']}/no-attackers/no-items/kills/orderDirection/asc/pastSeconds/10800/";
101		}
102	View Code Duplication	if ((string)$kmGroup['allianceID'] !== '0' & $kmGroup['lossMails'] === 'true') {
		0 ignored issues – show Comprehensibility introduced 2017-02-16 18:36 UTC by Report Bug Copy Issue Report Consider adding parentheses for clarity. Current Interpretation: `((string) $kmGroup['alli...'lossMails'] === 'true'`, Probably Intended Meaning: `(string) $kmGroup['allia...lossMails'] === 'true')` When comparing the result of a bit operation, we suggest to add explicit parenthesis and not to rely on PHP’s built-in operator precedence to ensure the code behaves as intended and to make it more readable. Let’s take a look at these examples: // Returns always int(0). return 0 === $foo & 4; return (0 === $foo) & 4; // More likely intended return: true/false return 0 === ($foo & 4); Loading history... Duplication introduced 2016-11-04 18:22 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
103		$url = "https://zkillboard.com/api/allianceID/{$kmGroup['allianceID']}/no-attackers/no-items/orderDirection/asc/pastSeconds/10800/";
104		}
105	View Code Duplication	if ((string)$kmGroup['allianceID'] !== '0' & $kmGroup['lossMails'] === 'false') {
		0 ignored issues – show Comprehensibility introduced 2017-02-16 18:36 UTC by Report Bug Copy Issue Report Consider adding parentheses for clarity. Current Interpretation: `((string) $kmGroup['alli...lossMails'] === 'false'`, Probably Intended Meaning: `(string) $kmGroup['allia...ossMails'] === 'false')` When comparing the result of a bit operation, we suggest to add explicit parenthesis and not to rely on PHP’s built-in operator precedence to ensure the code behaves as intended and to make it more readable. Let’s take a look at these examples: // Returns always int(0). return 0 === $foo & 4; return (0 === $foo) & 4; // More likely intended return: true/false return 0 === ($foo & 4); Loading history... Duplication introduced 2016-12-08 16:13 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
106		$url = "https://zkillboard.com/api/allianceID/{$kmGroup['allianceID']}/no-attackers/no-items/kills/orderDirection/asc/pastSeconds/10800/";
107		}
108
109		if (!isset($url)) { // Make sure it's always set.
110		$this->logger->addInfo('Killmails: ERROR - Ensure your config file is setup correctly for killmails.');
111		return null;
112		}
113
114		$kills = json_decode(downloadData($url), true);
115		// $i = 0;
116		if (isset($kills)) {
117		foreach ($kills as $kill) {
118		// if ($i < 10) {
119		//if big kill isn't set, disable it
120		if (!array_key_exists('bigKill', $kmGroup)) {
121		$kmGroup['bigKill'] = 99999999999999999999999999;
122		}
123		$killID = $kill['killID'];
124		if ($killID <= $SavedKillID) {
125		// $i++;
126		continue;
127		}
128		$channelID = $kmGroup['channel'];
129		$solarSystemID = $kill['solarSystemID'];
130		$systemName = getSystemName($solarSystemID);
131		$killTime = $kill['killTime'];
132		$victimAllianceName = '';
133	View Code Duplication	if ($kill['victim']['allianceName'] !== null && $kill['victim']['allianceName'] !== '') {
		0 ignored issues – show Duplication introduced 2016-12-08 16:13 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
134		$victimAllianceName = "\|{$kill['victim']['allianceName']}";
135		}
136		$victimName = $kill['victim']['characterName'];
137		$victimCorpName = $kill['victim']['corporationName'];
138		$victimShipID = $kill['victim']['shipTypeID'];
139		$shipName = getTypeName($victimShipID);
140		$rawValue = $kill['zkb']['totalValue'];
141		//Check if killmail meets minimum value and if it meets lost minimum value
142		if (isset($kmGroup['minimumValue']) && isset($kmGroup['minimumlossValue'])) {
143		if ($rawValue < $kmGroup['minimumValue']) {
144		if ($kill['victim']['corporationID'] == $kmGroup['corpID'] && $rawValue > $kmGroup['minimumlossValue'] \|\|
145		$kill['victim']['allianceID'] == $kmGroup['allianceID'] && $rawValue > $kmGroup['minimumlossValue'])
146		{
147		$this->logger->addInfo("Killmails: Mail {$killID} posted because it meet minimum loss value required.");
148		} else {
149		$this->logger->addInfo("Killmails: Mail {$killID} ignored for not meeting the minimum value required.");
150		setPermCache("{$kmGroup['name']}newestKillmailID", $killID);
151		continue;
152		}
153		}
154		}
155		$totalValue = number_format($kill['zkb']['totalValue']);
156		// Check if it's a structure
157	View Code Duplication	if ($victimName !== '' \|\| ' ' \|\| null) {
		0 ignored issues – show Duplication introduced 2016-07-16 19:16 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
158		if ($kmGroup['bigKill'] != null && $rawValue >= $kmGroup['bigKill']) {
159		$channelID = $kmGroup['bigKillChannel'];
160		$msg = "@here \n :warning:*Expensive Killmail:warning: \n {$killTime}\n\n{$shipName}* worth {$totalValue} ISK flown by {$victimName} of (*{$victimCorpName}\|{$victimAllianceName}*) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
161		} elseif ($kmGroup['bigKill'] == null \|\| $rawValue <= $kmGroup['bigKill']) {
162		$msg = "{$killTime}\n\n{$shipName} worth {$totalValue} ISK flown by {$victimName} of (*{$victimCorpName}\|{$victimAllianceName}*) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
163		}
164		} elseif ($victimName === '') {
165		$msg = "{$killTime}\n\n{$shipName} worth {$totalValue} ISK owned by (*{$victimCorpName}\|{$victimAllianceName}*) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
166		}
167
168		if (!isset($msg)) { // Make sure it's always set.
169		return null;
170		}
171		queueMessage($msg, $channelID, $this->guild);
172		$this->logger->addInfo("Killmails: Mail {$killID} queued.");
173
174		// $i++;
175		/* } else {
176		$updatedID = getPermCache("{$kmGroup['name']}newestKillmailID");
177		$this->logger->addInfo("Killmails: Kill posting cap reached, newest kill id for {$kmGroup['name']} is {$updatedID}");
178		break;
179		}*/
180		}
181		if (!isset($killID)) {
182		$killID = $SavedKillID;
183		}
184		setPermCache("{$kmGroup['name']}newestKillmailID", $killID);
185		}
186		$updatedID = getPermCache("{$kmGroup['name']}newestKillmailID");
187		$this->logger->addInfo("Killmails: All kills posted, newest kill id for {$kmGroup['name']} is {$updatedID}");
188		continue;
189		}
190		}
191
192		private function getBigKM()
193		{
194		$oldID = getPermCache('bigKillNewestKillmailID');
195		if (null === $oldID \|\| preg_match('/[a-z]/i', $oldID)) {
196		getStartBigMail();
197		$oldID = getPermCache('bigKillNewestKillmailID');
198		}
199
200		$url = 'https://zkillboard.com/api/kills/orderDirection/desc/iskValue/10000000000/limit/10/';
201
202		$kills = json_decode(downloadData($url), true);
203		$i = 0;
		0 ignored issues – show Unused Code introduced 2017-03-14 20:18 UTC by Report Bug Copy Issue Report `$i` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
204		if (isset($kills)) {
205		foreach ($kills as $kill) {
206		$cacheID = getPermCache('bigKillNewestKillmailID');
207		// if ($i < 10) {
208		$killID = $kill['killID'];
209		//check if mail is old
210		if ((int)$killID <= (int)$oldID) {
211		continue;
212		}
213		//save highest killID for cache
214		if ($killID > $cacheID) {
215		setPermCache('bigKillNewestKillmailID', $killID);
216		}
217		$channelID = $this->config['plugins']['getKillmails']['bigKills']['bigKillChannel'];
218		$solarSystemID = $kill['solarSystemID'];
219		$systemName = getSystemName($solarSystemID);
220		$killTime = $kill['killTime'];
221		$victimAllianceName = '';
222	View Code Duplication	if ($kill['victim']['allianceName'] !== null && $kill['victim']['allianceName'] !== '') {
		0 ignored issues – show Duplication introduced 2016-12-15 18:49 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
223		$victimAllianceName = "\|{$kill['victim']['allianceName']}";
224		}
225		$victimName = $kill['victim']['characterName'];
226		$victimCorpName = $kill['victim']['corporationName'];
227		$victimShipID = $kill['victim']['shipTypeID'];
228		$shipName = getTypeName($victimShipID);
229		$totalValue = number_format($kill['zkb']['totalValue']);
230		// Check if it's a structure
231	View Code Duplication	if ($victimName !== '' \|\| ' ' \|\| null) {
		0 ignored issues – show Duplication introduced 2016-12-15 18:49 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
232		$msg = "10b+ Kill Reported\n\n{$killTime}\n\n{$shipName} worth {$totalValue} ISK flown by {$victimName} of (*{$victimCorpName}\|{$victimAllianceName}*) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
233		} else {
234		$msg = "10b+ Kill Reported\n\n{$killTime}\n\n{$shipName} worth {$totalValue} ISK owned by (*{$victimCorpName}\|{$victimAllianceName}*) killed in {$systemName}\nhttps://zkillboard.com/kill/{$killID}/";
235		}
236		if (!isset($msg)) { // Make sure it's always set.
237		return null;
238		}
239		queueMessage($msg, $channelID, $this->guild);
240		$this->logger->addInfo("Killmails: Mail {$killID} queued.");
241
242		/* $i++;
243		} else {
244		$cacheID = getPermCache('bigKillNewestKillmailID');
245		$this->logger->addInfo("Killmails: bigKill posting cap reached, newest kill id is {$cacheID}");
246		break;
247		}*/
248		}
249		}
250		$updatedID = getPermCache('bigKillNewestKillmailID');
251		$this->logger->addInfo("Killmails: All bigKills posted, newest kill id is {$updatedID}");
252		}
253		}
254

shibdib / Dramiel

Branch — dev (952e73)

getKillmails::getBigKM() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like