Issues in Dramiel.php - All Issues - Inspection of "fix for chat commands not loading" - shibdib/Dramiel - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( b8e5e4...9393d7 )

by Bob

created 2016-09-07 17:16 UTC

Dramiel.php (4 issues)

Labels

Severity

Minor 4

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?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.
 */

// Require the vendor stuff
require_once(__DIR__ . "/vendor/autoload.php");

// Setup logger
use Monolog\Logger;
use Monolog\Handler\StreamHandler;

// More memory allowance
ini_set("memory_limit", "1024M");

// Just incase we get launched from somewhere else
chdir(__DIR__);

// Enable garbage collection
gc_enable();

// When the bot started
$startTime = time();

// create a log channel
$logger = new Logger('Dramiel');
$logger->pushHandler(new StreamHandler(__DIR__ . '/log/dramielLog.log', Logger::DEBUG));
$logger->addInfo('Logger Initiated');

GLOBAL $logger;
function myFunction($a, $b) {
    // Do something
}

// Require the config
if (file_exists("config/config.php")) {
    require_once("config/config.php");
} else {
    $logger->error("config.php not found (you might wanna start by editing and renaming config_new.php)");
    die();
}

// Load the library files (Probably a prettier way to do this that i haven't thought up yet)
foreach (glob(__DIR__ . "/src/lib/*.php") as $lib) {
    require_once($lib);
}

//Startup DB Check
updateDramielDB($logger);
updateCCPData($logger);

// Init Discord
use Discord\Discord;
use Discord\Parts\User\Game;
use Discord\WebSockets\Event;
use Discord\WebSockets\WebSocket;

$discord = new Discord(["token" => $config["bot"]["token"]]);

// Load tick plugins
$pluginDirs = array("src/plugins/onTick/*.php");
$logger->info("Loading background plugins");
$plugins = array();
foreach ($pluginDirs as $dir) {
    foreach (glob($dir) as $plugin) {
        // Only load the plugins we want to load, according to the config
        if (!in_array(str_replace(".php", "", basename($plugin)), $config["enabledPlugins"])) {
            continue;
        }

        require_once($plugin);
        $fileName = str_replace(".php", "", basename($plugin));
        $p = new $fileName();
        $p->init($config, $discord, $logger);
        $pluginsT[] = $p;
    }
}
// Number of plugins loaded
$logger->info("Loaded: " . count($pluginsT) . " background plugins");

// Load chat plugins
$pluginDirs = array("src/plugins/onMessage/*.php", "src/plugins/admin/*.php");
$adminPlugins = array("setNickname","updateBot","holder");
$logger->addInfo("Loading in chat plugins");
$plugins = array();
foreach ($pluginDirs as $dir) {
    foreach (glob($dir) as $plugin) {
		var_dump($plugin);
		var_dump($adminPlugins);
        // Only load the plugins we want to load, according to the config
        if (!in_array(str_replace(".php", "", basename($plugin)), $config["enabledPlugins"]) && !in_array(str_replace(".php", "", basename($plugin)), $adminPlugins)) {
            continue;
        }

        require_once($plugin);
        $fileName = str_replace(".php", "", basename($plugin));
        $p = new $fileName();
        $p->init($config, $discord, $logger);
        $plugins[] = $p;
    }
}

// Number of chat plugins loaded
$logger->addInfo("Loaded: " . count($plugins) . " chat plugins");

$discord->on(
    'ready',
    function($discord) use ($logger, $config, $plugins, $pluginsT) {
        // In here we can access any of the WebSocket events.
        //
        // There is a list of event constants that you can
        // find here: https://teamreflex.github.io/DiscordPHP/classes/Discord.WebSockets.Event.html
        //
        // We will echo to the console that the WebSocket is ready.
        $logger->addInfo('Discord WebSocket is ready!' . PHP_EOL);
        //$game = new Game(array('name' => $config["bot"]["game"], 'url' => null, 'type' => null), true);

        //$discord->updatePresence($game, false);

        // $ws->setNickname($config["bot"]["name"]); //not in yet


        // Database check
        $discord->loop->addPeriodicTimer(86400, function() use ($logger) {
            updateDramielDB($logger);
            updateCCPData($logger);
        });

        // Run the Tick plugins
        $discord->loop->addPeriodicTimer(5, function() use ($pluginsT) {
            foreach ($pluginsT as $plugin)
                $plugin->tick();
        });

        // Mem cleanup every 30 minutes
        $discord->loop->addPeriodicTimer(1800, function() use ($logger) {
            $logger->addInfo("Memory in use: " . memory_get_usage() / 1024 / 1024 . "MB");
            gc_collect_cycles(); // Collect garbage
            $logger->addInfo("Memory in use after garbage collection: " . memory_get_usage() / 1024 / 1024 . "MB");
        });

        $discord->on(
            Event::MESSAGE_CREATE,
            function($message) use ($logger, $config, $plugins) {

                $msgData = array(
                    "message" => array(
                        "timestamp" => $message->timestamp,
                        "id" => $message->id,
                        "message" => $message->content,
                        "channelID" => $message->channel_id,
                        "from" => $message->author->username,
                        "fromID" => $message->author->id,
                        "fromDiscriminator" => $message->author->discriminator,
                        "fromAvatar" => $message->author->avatar
                    )
                );

                if ($message->content == '(╯°□°）╯︵ ┻━┻') {
                    $message->reply('┬─┬﻿ ノ( ゜-゜ノ)');
                }

                // We are just checking if the message equals to ping and replying to the user with a pong!
                if ($message->content == 'ping') {
                    $message->reply('pong!');
                }
                // Check for plugins
                if (isset($message->content[0])) {
                    if ($message->content[0] == $config["bot"]["trigger"]) {
                        foreach ($plugins as $plugin) {
                            try {
                                $plugin->onMessage($msgData, $message);
                            } catch (Exception $e) {
                                $logger->addError("Error: " . $e->getMessage());
                            }
                        }
                    }
                }
            }
        );
    }
);
$discord->on(
    'error',
    function($error) use ($logger) {
        $logger->addError($error);
        exit(1);
    }
);
$discord->on(
    'reconnecting',
    function() use ($logger) {
        $logger->addInfo('Websocket is reconnecting..');
    });
$discord->on(
    'reconnected',
    function() use ($logger) {
        $logger->addInfo('Websocket was reconnected..');
    });
// Now we will run the ReactPHP Event Loop!
$discord->run();



Push — master ( b8e5e4...9393d7 )

Dramiel.php (4 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

1. Pass all data via parameters

2. Create a class that maintains your state

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			// Require the vendor stuff
27			require_once(__DIR__ . "/vendor/autoload.php");
28
29			// Setup logger
30			use Monolog\Logger;
31			use Monolog\Handler\StreamHandler;
32
33			// More memory allowance
34			ini_set("memory_limit", "1024M");
35
36			// Just incase we get launched from somewhere else
37			chdir(__DIR__);
38
39			// Enable garbage collection
40			gc_enable();
41
42			// When the bot started
43			$startTime = time();
44
45			// create a log channel
46			$logger = new Logger('Dramiel');
47			$logger->pushHandler(new StreamHandler(__DIR__ . '/log/dramielLog.log', Logger::DEBUG));
48			$logger->addInfo('Logger Initiated');
49
50			GLOBAL $logger;
			0 ignored issues – show Compatibility Best Practice introduced 2016-07-16 19:16 UTC by Report Bug Copy Issue Report Show Similar Issues like this Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
51
52			// Require the config
53			if (file_exists("config/config.php")) {
54			require_once("config/config.php");
55			} else {
56			$logger->error("config.php not found (you might wanna start by editing and renaming config_new.php)");
57			die();
58			}
59
60			// Load the library files (Probably a prettier way to do this that i haven't thought up yet)
61			foreach (glob(__DIR__ . "/src/lib/*.php") as $lib) {
62			require_once($lib);
63			}
64
65			//Startup DB Check
66			updateDramielDB($logger);
67			updateCCPData($logger);
68
69			// Init Discord
70			use Discord\Discord;
71			use Discord\Parts\User\Game;
72			use Discord\WebSockets\Event;
73			use Discord\WebSockets\WebSocket;
74
75			$discord = new Discord(["token" => $config["bot"]["token"]]);
76
77			// Load tick plugins
78			$pluginDirs = array("src/plugins/onTick/*.php");
79			$logger->info("Loading background plugins");
80			$plugins = array();
81			foreach ($pluginDirs as $dir) {
82			foreach (glob($dir) as $plugin) {
83			// Only load the plugins we want to load, according to the config
84			if (!in_array(str_replace(".php", "", basename($plugin)), $config["enabledPlugins"])) {
85			continue;
86			}
87
88			require_once($plugin);
89			$fileName = str_replace(".php", "", basename($plugin));
90			$p = new $fileName();
91			$p->init($config, $discord, $logger);
92			$pluginsT[] = $p;
93			}
94			}
95			// Number of plugins loaded
96			$logger->info("Loaded: " . count($pluginsT) . " background plugins");
97
98			// Load chat plugins
99			$pluginDirs = array("src/plugins/onMessage/.php", "src/plugins/admin/.php");
100			$adminPlugins = array("setNickname","updateBot","holder");
101			$logger->addInfo("Loading in chat plugins");
102			$plugins = array();
103			foreach ($pluginDirs as $dir) {
104			foreach (glob($dir) as $plugin) {
105			var_dump($plugin);
106			var_dump($adminPlugins);
107			// Only load the plugins we want to load, according to the config
108			if (!in_array(str_replace(".php", "", basename($plugin)), $config["enabledPlugins"]) && !in_array(str_replace(".php", "", basename($plugin)), $adminPlugins)) {
109			continue;
110			}
111
112			require_once($plugin);
113			$fileName = str_replace(".php", "", basename($plugin));
114			$p = new $fileName();
115			$p->init($config, $discord, $logger);
116			$plugins[] = $p;
117			}
118			}
119
120			// Number of chat plugins loaded
121			$logger->addInfo("Loaded: " . count($plugins) . " chat plugins");
122
123			$discord->on(
124			'ready',
125			function($discord) use ($logger, $config, $plugins, $pluginsT) {
126			// In here we can access any of the WebSocket events.
127			//
128			// There is a list of event constants that you can
129			// find here: https://teamreflex.github.io/DiscordPHP/classes/Discord.WebSockets.Event.html
130			//
131			// We will echo to the console that the WebSocket is ready.
132			$logger->addInfo('Discord WebSocket is ready!' . PHP_EOL);
133			//$game = new Game(array('name' => $config["bot"]["game"], 'url' => null, 'type' => null), true);
			0 ignored issues – show Unused Code Comprehensibility introduced 2016-08-29 11:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this `61%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
134			//$discord->updatePresence($game, false);
			0 ignored issues – show Unused Code Comprehensibility introduced 2016-08-29 11:14 UTC by Report Bug Copy Issue Report Show Similar Issues like this `80%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
135			// $ws->setNickname($config["bot"]["name"]); //not in yet
			0 ignored issues – show Unused Code Comprehensibility introduced 2016-07-16 19:16 UTC by Report Bug Copy Issue Report Show Similar Issues like this `80%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
136
137			// Database check
138			$discord->loop->addPeriodicTimer(86400, function() use ($logger) {
139			updateDramielDB($logger);
140			updateCCPData($logger);
141			});
142
143			// Run the Tick plugins
144			$discord->loop->addPeriodicTimer(5, function() use ($pluginsT) {
145			foreach ($pluginsT as $plugin)
146			$plugin->tick();
147			});
148
149			// Mem cleanup every 30 minutes
150			$discord->loop->addPeriodicTimer(1800, function() use ($logger) {
151			$logger->addInfo("Memory in use: " . memory_get_usage() / 1024 / 1024 . "MB");
152			gc_collect_cycles(); // Collect garbage
153			$logger->addInfo("Memory in use after garbage collection: " . memory_get_usage() / 1024 / 1024 . "MB");
154			});
155
156			$discord->on(
157			Event::MESSAGE_CREATE,
158			function($message) use ($logger, $config, $plugins) {
159
160			$msgData = array(
161			"message" => array(
162			"timestamp" => $message->timestamp,
163			"id" => $message->id,
164			"message" => $message->content,
165			"channelID" => $message->channel_id,
166			"from" => $message->author->username,
167			"fromID" => $message->author->id,
168			"fromDiscriminator" => $message->author->discriminator,
169			"fromAvatar" => $message->author->avatar
170			)
171			);
172
173			if ($message->content == '(╯°□°）╯︵ ┻━┻') {
174			$message->reply('┬─┬ ノ( ゜-゜ノ)');
175			}
176
177			// We are just checking if the message equals to ping and replying to the user with a pong!
178			if ($message->content == 'ping') {
179			$message->reply('pong!');
180			}
181			// Check for plugins
182			if (isset($message->content[0])) {
183			if ($message->content[0] == $config["bot"]["trigger"]) {
184			foreach ($plugins as $plugin) {
185			try {
186			$plugin->onMessage($msgData, $message);
187			} catch (Exception $e) {
188			$logger->addError("Error: " . $e->getMessage());
189			}
190			}
191			}
192			}
193			}
194			);
195			}
196			);
197			$discord->on(
198			'error',
199			function($error) use ($logger) {
200			$logger->addError($error);
201			exit(1);
202			}
203			);
204			$discord->on(
205			'reconnecting',
206			function() use ($logger) {
207			$logger->addInfo('Websocket is reconnecting..');
208			});
209			$discord->on(
210			'reconnected',
211			function() use ($logger) {
212			$logger->addInfo('Websocket was reconnected..');
213			});
214			// Now we will run the ReactPHP Event Loop!
215			$discord->run();
216
217

shibdib / Dramiel

Push — master ( b8e5e4...9393d7 )

Dramiel.php (4 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like