diff --git a/application/configs/core.xml b/application/configs/core.xml deleted file mode 100644 index da62f13c..00000000 --- a/application/configs/core.xml +++ /dev/null @@ -1,15 +0,0 @@ - - - - - - 20 - - - - tracklist.txt - - - mx - - diff --git a/application/core/chat.php b/application/core/chat.php index 7042c504..aeeef77d 100644 --- a/application/core/chat.php +++ b/application/core/chat.php @@ -68,7 +68,7 @@ class Chat { * @return bool */ public function sendInformation($message, $login = null, $prefix = false) { - $format = $this->maniaControl->settingManager->getSetting($this, 'ErrorFormat', '$fff'); + $format = $this->maniaControl->settingManager->getSetting($this, 'InformationFormat', '$fff'); return $this->sendChat($format . $message, $login); } @@ -81,7 +81,7 @@ class Chat { * @return bool */ public function sendSuccess($message, $login = null, $prefix = false) { - $format = $this->maniaControl->settingManager->getSetting($this, 'ErrorFormat', '$0f0'); + $format = $this->maniaControl->settingManager->getSetting($this, 'SuccessFormat', '$0f0'); return $this->sendChat($format . $message, $login); } diff --git a/application/core/commands.php b/application/core/commands.php index 49b5b7bf..9f36fba8 100644 --- a/application/core/commands.php +++ b/application/core/commands.php @@ -8,40 +8,34 @@ namespace ManiaControl; * @author steeffeen */ class Commands { - + /** * Private properties */ - private $mc = null; - + private $maniaControl = null; private $config = null; - private $commandHandlers = array(); - private $openBills = array(); - private $serverShutdownTime = -1; - private $serverShutdownEmpty = false; /** * Construct commands handler */ - public function __construct($mc) { - $this->mc = $mc; + public function __construct(ManiaControl $maniaControl) { + $this->maniaControl = $maniaControl; // Load config $this->config = FileUtil::loadConfig('commands.xml'); // Register for callbacks - $this->mc->callbacks->registerCallbackHandler(Callbacks::CB_MC_5_SECOND, $this, 'each5Seconds'); - $this->mc->callbacks->registerCallbackHandler(Callbacks::CB_MP_BILLUPDATED, $this, 'handleBillUpdated'); - $this->mc->callbacks->registerCallbackHandler(Callbacks::CB_MP_PLAYERCHAT, $this, 'handleChatCallback'); + $this->maniaControl->callbacks->registerCallbackHandler(Callbacks::CB_MC_5_SECOND, $this, 'each5Seconds'); + $this->maniaControl->callbacks->registerCallbackHandler(Callbacks::CB_MP_BILLUPDATED, $this, 'handleBillUpdated'); + $this->maniaControl->callbacks->registerCallbackHandler(Callbacks::CB_MP_PLAYERCHAT, $this, 'handleChatCallback'); // Register basic commands - $commands = array('help', 'version', 'shutdown', 'shutdownserver', 'networkstats', 'systeminfo', 'getservername', - 'setservername', 'getplanets', 'donate', 'pay', 'kick', 'nextmap', 'restartmap', 'addmap', 'removemap', 'startwarmup', - 'stopwarmup'); + $commands = array('help', 'version', 'shutdown', 'shutdownserver', 'networkstats', 'systeminfo', 'setservername', 'getplanets', + 'donate', 'pay', 'kick', 'nextmap', 'restartmap', 'addmap', 'removemap', 'startwarmup', 'stopwarmup'); foreach ($commands as $command) { $this->registerCommandHandler($command, $this, 'command_' . $command); } @@ -53,12 +47,13 @@ class Commands { * @param string $commandName * @param object $handler * @param string $method + * @return bool */ public function registerCommandHandler($commandName, $handler, $method) { $command = strtolower($commandName); if (!is_object($handler) || !method_exists($handler, $method)) { - trigger_error("Given handler can't handle command '" . $command . "' (no method '" . $method . "')!"); - return; + trigger_error("Given handler can't handle command '{$command}' (no method '{$method}')!"); + return false; } if (!array_key_exists($command, $this->commandHandlers) || !is_array($this->commandHandlers[$command])) { // Init handlers array @@ -66,43 +61,57 @@ class Commands { } // Register command handler array_push($this->commandHandlers[$command], array($handler, $method)); + return true; } /** * Handle chat callback + * + * @param array $callback + * @return bool */ - public function handleChatCallback($callback) { + public function handleChatCallback(array $callback) { $chat = $callback[1]; // Check for command - if (!$chat[3]) return; + if (!$chat[3]) { + return false; + } // Check for valid player - if ($chat[0] <= 0 || strlen($chat[1]) <= 0) return; + if ($chat[0] <= 0 || strlen($chat[1]) <= 0) { + return false; + } // Handle command $command = explode(" ", substr($chat[2], 1)); $command = strtolower($command[0]); if (!array_key_exists($command, $this->commandHandlers) || !is_array($this->commandHandlers[$command])) { // No command handler registered - return; + return true; } // Inform command handlers foreach ($this->commandHandlers[$command] as $handler) { call_user_func(array($handler[0], $handler[1]), $callback); } + return true; } /** * Handle bill updated callback + * + * @param array $callback + * @return bool */ - public function handleBillUpdated($callback) { + public function handleBillUpdated(array $callback) { $bill = $callback[1]; - if (!array_key_exists($bill[0], $this->openBills)) return; + if (!array_key_exists($bill[0], $this->openBills)) { + return false; + } $login = $this->openBills[$bill[0]]; switch ($bill[1]) { case 4: { // Payed $message = 'Success! Thanks.'; - $this->mc->chat->sendSuccess($message, $login); + $this->maniaControl->chat->sendSuccess($message, $login); unset($this->openBills[$bill[0]]); break; } @@ -110,18 +119,19 @@ class Commands { { // Refused $message = 'Transaction cancelled.'; - $this->mc->chat->sendError($message, $login); + $this->maniaControl->chat->sendError($message, $login); unset($this->openBills[$bill[0]]); break; } case 6: { // Error - $this->mc->chat->sendError($bill[2], $login); + $this->maniaControl->chat->sendError($bill[2], $login); unset($this->openBills[$bill[0]]); break; } } + return true; } /** @@ -133,8 +143,10 @@ class Commands { */ private function getRightsLevel($commandName, $defaultLevel) { $command_rights = $this->config->xpath('//' . strtolower($commandName) . '/..'); - if (empty($command_rights)) return $defaultLevel; - $rights = $this->mc->authentication->RIGHTS_LEVELS; + if (empty($command_rights)) { + return $defaultLevel; + } + $rights = $this->maniaControl->authentication->RIGHTS_LEVELS; $highest_level = null; foreach ($command_rights as $right) { $levelName = $right->getName(); @@ -143,34 +155,41 @@ class Commands { $highest_level = $levelInt; } } - if ($highest_level === null || !array_key_exists($highest_level, $rights)) return $defaultLevel; + if ($highest_level === null || !array_key_exists($highest_level, $rights)) { + return $defaultLevel; + } return $rights[$highest_level]; } /** * Send ManiaControl version + * + * @param array $chat + * @return bool */ - private function command_version($chat) { + private function command_version(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('version', 'all'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('version', 'all'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; - } - if (!$this->mc->chat->sendInformation('This server is using ManiaControl v' . ManiaControl::VERSION . '!', $login)) { - trigger_error("Couldn't send version to '" . $login . "'. " . $this->mc->getClientErrorText()); + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } + $message = 'This server is using ManiaControl v' . ManiaControl::VERSION . '!'; + return $this->maniaControl->chat->sendInformation($message, $login); } /** * Send help list + * + * @param array $chat + * @return bool */ - private function command_help($chat) { + private function command_help(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('help', 'all'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('help', 'all'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } // TODO: improve help command // TODO: enable help for specific commands @@ -178,7 +197,7 @@ class Commands { $commands = array_keys($this->commandHandlers); $count = count($commands); for ($index = 0; $index < $count; $index++) { - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel($commands[$index], 'superadmin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel($commands[$index], 'superadmin'))) { unset($commands[$index]); } } @@ -191,93 +210,99 @@ class Commands { } $index++; } - if (!$this->mc->chat->sendInformation($list, $login)) { - trigger_error("Couldn't send help list to '" . $login . "'. " . $this->mc->getClientErrorText()); - } + return $this->maniaControl->chat->sendInformation($list, $login); } /** * Handle getplanets command + * + * @param array $chat + * @return bool */ - private function command_getplanets($chat) { + private function command_getplanets(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('getplanets', 'admin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('getplanets', 'admin'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } - if (!$this->mc->client->query('GetServerPlanets')) { - trigger_error("Couldn't retrieve server planets. " . $this->mc->getClientErrorText()); - } - else { - $planets = $this->mc->client->getResponse(); - if (!$this->mc->chat->sendInformation('This Server has ' . $planets . ' Planets!', $login)) { - trigger_error("Couldn't send server planets to '" . $login . "'. " . $this->mc->getClientErrorText()); - } + if (!$this->maniaControl->client->query('GetServerPlanets')) { + trigger_error("Couldn't retrieve server planets. " . $this->maniaControl->getClientErrorText()); + return false; } + $planets = $this->maniaControl->client->getResponse(); + $message = "This Server has {$planets} Planets!"; + return $this->maniaControl->chat->sendInformation($message, $login); } /** * Handle donate command + * + * @param array $chat + * @return bool */ - private function command_donate($chat) { + private function command_donate(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('donate', 'all'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('donate', 'all'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); + $this->maniaControl->authentication->sendNotAllowed($login); return; } $params = explode(' ', $chat[1][2]); if (count($params) < 2) { // TODO: send usage information - return; + return false; } $amount = (int) $params[1]; if (!$amount || $amount <= 0) { // TODO: send usage information - return; + return false; } if (count($params) >= 3) { $receiver = $params[2]; - $receiverPlayer = $this->mc->database->getPlayer($receiver); + $receiverPlayer = $this->maniaControl->database->getPlayer($receiver); $receiverName = ($receiverPlayer ? $receiverPlayer['NickName'] : $receiver); } else { $receiver = ''; - $receiverName = $this->mc->server->getName(); + $receiverName = $this->maniaControl->server->getName(); } $message = 'Donate ' . $amount . ' Planets to $<' . $receiverName . '$>?'; - if (!$this->mc->client->query('SendBill', $login, $amount, $message, $receiver)) { + if (!$this->maniaControl->client->query('SendBill', $login, $amount, $message, $receiver)) { trigger_error( - "Couldn't create donation of " . $amount . " planets from '" . $login . "' for '" . $receiver . "'. " . - $this->mc->getClientErrorText()); - $this->mc->chat->sendError("Creating donation failed.", $login); - } - else { - $bill = $this->mc->client->getResponse(); - $this->openBills[$bill] = $login; + "Couldn't create donation of {$amount} planets from '{$login}' for '{$receiver}'. " . + $this->maniaControl->getClientErrorText()); + $this->maniaControl->chat->sendError("Creating donation failed.", $login); + return false; } + + $bill = $this->maniaControl->client->getResponse(); + $this->openBills[$bill] = $login; + return true; } /** * Handle pay command + * + * @param array $chat + * @return bool */ - private function command_pay($chat) { + private function command_pay(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('pay', 'superadmin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('pay', 'superadmin'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } $params = explode(' ', $chat[1][2]); if (count($params) < 2) { // TODO: send usage information - return; + return false; } $amount = (int) $params[1]; if (!$amount || $amount <= 0) { // TODO: send usage information - return; + return false; } if (count($params) >= 3) { $receiver = $params[2]; @@ -285,114 +310,118 @@ class Commands { else { $receiver = $login; } - $message = 'Payout from $<' . $this->mc->server->getName() . '$>.'; - if (!$this->mc->client->query('Pay', $receiver, $amount, $message)) { + $message = 'Payout from $<' . $this->maniaControl->server->getName() . '$>.'; + if (!$this->maniaControl->client->query('Pay', $receiver, $amount, $message)) { trigger_error( - "Couldn't create payout of" . $amount . " planets by '" . $login . "' for '" . $receiver . "'. " . - $this->mc->getClientErrorText()); - $this->mc->chat->sendError("Creating payout failed.", $login); - } - else { - $bill = $this->mc->client->getResponse(); - $this->openBills[$bill] = $login; + "Couldn't create payout of {$amount} planets by '{$login}' for '{$receiver}'. " . + $this->maniaControl->getClientErrorText()); + $this->maniaControl->chat->sendError("Creating payout failed.", $login); + return false; } + $bill = $this->maniaControl->client->getResponse(); + $this->openBills[$bill] = $login; + return true; } /** * Handle networkstats command + * + * @param array $chat + * @return bool */ - private function command_networkstats($chat) { + private function command_networkstats(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('networkstats', 'superadmin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('networkstats', 'superadmin'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; - } - $networkStats = $this->mc->server->getNetworkStats(); - $message = 'NetworkStats: ' . 'uptime=' . $networkStats['Uptime'] . ', ' . 'nbConn=' . $networkStats['NbrConnection'] . ', ' . - 'recvRate=' . $networkStats['RecvNetRate'] . ', ' . 'sendRate=' . $networkStats['SendNetRate'] . ', ' . 'recvTotal=' . - $networkStats['SendNetRate'] . ', ' . 'sentTotal=' . $networkStats['SendNetRate']; - if (!$this->mc->chat->sendInformation($message, $login)) { - trigger_error("Couldn't send network stats to '" . $login . "'. " . $this->mc->getClientErrorText()); + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } + $networkStats = $this->maniaControl->server->getNetworkStats(); + $message = 'NetworkStats: uptime=' . $networkStats['Uptime'] . ', nbConn=' . $networkStats['NbrConnection'] . ', recvRate=' . + $networkStats['RecvNetRate'] . ', sendRate=' . $networkStats['SendNetRate'] . ', recvTotal=' . + $networkStats['SendNetRate'] . ', sentTotal=' . $networkStats['SendNetRate']; + return $this->maniaControl->chat->sendInformation($message, $login); } /** * Handle systeminfo command + * + * @param array $chat + * @return bool */ - private function command_systeminfo($chat) { + private function command_systeminfo(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('systeminfo', 'superadmin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('systeminfo', 'superadmin'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; - } - $systemInfo = $this->mc->server->getSystemInfo(); - $message = 'SystemInfo: ' . 'ip=' . $systemInfo['PublishedIp'] . ', ' . 'port=' . $systemInfo['Port'] . ', ' . 'p2pPort=' . - $systemInfo['P2PPort'] . ', ' . 'title=' . $systemInfo['TitleId'] . ', ' . 'login=' . $systemInfo['ServerLogin'] . ', '; - if (!$this->mc->chat->sendInformation($message, $login)) { - trigger_error("Couldn't send system info to '" . $login . "'. " . $this->mc->getClientErrorText()); + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } + $systemInfo = $this->maniaControl->server->getSystemInfo(); + $message = 'SystemInfo: ip=' . $systemInfo['PublishedIp'] . ', port=' . $systemInfo['Port'] . ', p2pPort=' . + $systemInfo['P2PPort'] . ', title=' . $systemInfo['TitleId'] . ', login=' . $systemInfo['ServerLogin'] . ', '; + return $this->maniaControl->chat->sendInformation($message, $login); } /** * Handle shutdown command + * + * @param array $chat + * @return bool */ - private function command_shutdown($chat) { + private function command_shutdown(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('shutdown', 'superadmin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('shutdown', 'superadmin'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } - $this->mc->quit("ManiaControl shutdown requested by '{$login}'"); + return $this->maniaControl->quit("ManiaControl shutdown requested by '{$login}'"); } /** * Handle startwarmup command + * + * @param array $chat + * @return bool */ - private function command_startwarmup($chat) { + private function command_startwarmup(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('startwarmup', 'operator'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('startwarmup', 'operator'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; - } - if (!$this->mc->client->query("SetWarmUp", true)) { - trigger_error("Couldn't start warmup. " . $this->mc->getClientErrorText()); - $player = $this->mc->database->getPlayer($login); - $this->mc->chat->sendInformation('$<' . ($player ? $player['NickName'] : $login) . '$> started WarmUp!'); + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } + return $this->maniaControl->client->query("SetWarmUp", true); } /** * Handle stopwarmup command + * + * @param array $chat + * @return bool */ - private function command_stopwarmup($chat) { + private function command_stopwarmup(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('stopwarmup', 'operator'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('stopwarmup', 'operator'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; - } - if (!$this->mc->client->query("SetWarmUp", false)) { - trigger_error("Couldn't stop warmup. " . $this->mc->getClientErrorText()); - } - else { - $player = $this->mc->database->getPlayer($login); - $this->mc->chat->sendInformation('$<' . ($player ? $player['NickName'] : $login) . '$> stopped WarmUp!'); + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } + return $this->maniaControl->client->query("SetWarmUp", false); } /** * Handle server shutdown command + * + * @param array $chat + * @return bool */ - private function command_shutdownserver($chat) { + private function command_shutdownserver(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('shutdownserver', 'superadmin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('shutdownserver', 'superadmin'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } // Check for delayed shutdown $params = explode(' ', $chat[1][2]); @@ -401,283 +430,280 @@ class Commands { if ($param == 'empty') { $this->serverShutdownEmpty = !$this->serverShutdownEmpty; if ($this->serverShutdownEmpty) { - $this->mc->chat->sendInformation("The server will shutdown as soon as it's empty!", $login); - } - else { - $this->mc->chat->sendInformation("Empty-shutdown cancelled!", $login); + $this->maniaControl->chat->sendInformation("The server will shutdown as soon as it's empty!", $login); + return true; } + $this->maniaControl->chat->sendInformation("Empty-shutdown cancelled!", $login); + return true; } - else { - $delay = (int) $param; - if ($delay <= 0) { - // Cancel shutdown - $this->serverShutdownTime = -1; - $this->mc->chat->sendInformation("Delayed shutdown cancelled!", $login); - } - else { - // Trigger delayed shutdown - $this->serverShutdownTime = time() + $delay * 60.; - $this->mc->chat->sendInformation("The server will shut down in " . $delay . " minutes!", $login); - } + $delay = (int) $param; + if ($delay <= 0) { + // Cancel shutdown + $this->serverShutdownTime = -1; + $this->maniaControl->chat->sendInformation("Delayed shutdown cancelled!", $login); + return true; } + // Trigger delayed shutdown + $this->serverShutdownTime = time() + $delay * 60.; + $this->maniaControl->chat->sendInformation("The server will shut down in " . $delay . " minutes!", $login); + return true; } - else { - $this->shutdownServer($login); - } + return $this->shutdownServer($login); } /** * Handle kick command + * + * @param array $chat + * @return bool */ - private function command_kick($chat) { + private function command_kick(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('kick', 'operator'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('kick', 'operator'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } $params = explode(' ', $chat[1][2], 3); if (count($params) < 2) { // TODO: show usage - return; + return false; } $target = $params[1]; - $players = $this->mc->server->getPlayers(); + $players = $this->maniaControl->server->getPlayers(); foreach ($players as $player) { - if ($player['Login'] != $target) continue; + if ($player['Login'] != $target) { + continue; + } // Kick player + $message = ''; if (isset($params[2])) { $message = $params[2]; } - else { - $message = ""; - } - if (!$this->mc->client->query('Kick', $target, $message)) { - trigger_error("Couldn't kick player '" . $target . "'! " . $this->mc->getClientErrorText()); - } - return; + return $this->maniaControl->client->query('Kick', $target, $message); } - $this->mc->chat->sendError("Invalid player login.", $login); + $this->maniaControl->chat->sendError("Invalid player login.", $login); + return false; } /** * Handle removemap command + * + * @param array $chat + * @return bool */ - private function command_removemap($chat) { + private function command_removemap(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('kick', 'operator'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('kick', 'operator'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } // TODO: allow params // Get map name - $map = $this->mc->server->getMap(); + $map = $this->maniaControl->server->getMap(); if (!$map) { - $this->mc->chat->sendError("Couldn't remove map.", $login); + $this->maniaControl->chat->sendError("Couldn't remove map.", $login); + return false; } - else { - $mapName = $map['FileName']; - - // Remove map - if (!$this->mc->client->query('RemoveMap', $mapName)) { - trigger_error("Couldn't remove current map. " . $this->mc->getClientErrorText()); - } - else { - $this->mc->chat->sendSuccess('Map removed.', $login); - } + $mapName = $map['FileName']; + // Remove map + if (!$this->maniaControl->client->query('RemoveMap', $mapName)) { + trigger_error("Couldn't remove current map. " . $this->maniaControl->getClientErrorText()); + return false; } + $this->maniaControl->chat->sendSuccess('Map removed.', $login); + return true; } /** * Handle addmap command + * + * @param array $chat + * @return bool */ - private function command_addmap($chat) { + private function command_addmap(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('addmap', 'operator'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('addmap', 'operator'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } $params = explode(' ', $chat[1][2], 2); if (count($params) < 2) { // TODO: show usage - return; + return false; } // Check if ManiaControl can even write to the maps dir - if (!$this->mc->client->query('GetMapsDirectory')) { - trigger_error("Couldn't get map directory. " . $this->mc->getClientErrorText()); - $this->mc->chat->sendError("ManiaControl couldn't retrieve the maps directory.", $login); - return; + if (!$this->maniaControl->client->query('GetMapsDirectory')) { + trigger_error("Couldn't get map directory. " . $this->maniaControl->getClientErrorText()); + $this->maniaControl->chat->sendError("ManiaControl couldn't retrieve the maps directory.", $login); + return false; } - else { - $mapDir = $this->mc->client->getResponse(); - if (!is_dir($mapDir)) { - trigger_error("ManiaControl doesn't have have access to the maps directory in '" . $mapDir . "'."); - $this->mc->chat->sendError("ManiaControl doesn't have access to the maps directory.", $login); - return; - } - $dlDir = (string) $this->mc->config->maps_dir; - // Create mx directory if necessary - if (!is_dir($mapDir . $dlDir) && !mkdir($mapDir . $dlDir)) { - trigger_error("ManiaControl doesn't have to rights to save maps in'" . $mapDir . $dlDir, "'."); - $this->mc->chat->sendError("ManiaControl doesn't have to rights to save maps.", $login); - return; - } - $mapDir .= $dlDir . '/'; - // Download the map - if (is_numeric($params[1])) { - $serverInfo = $this->mc->server->getSystemInfo(); - $title = strtolower(substr($serverInfo['TitleId'], 0, 2)); - // Check if map exists - $url = 'http://' . $title . '.mania-exchange.com/api/tracks/get_track_info/id/' . $params[1] . '?format=json'; - $mapInfo = FileUtil::loadFile($url); - if (!$mapInfo || strlen($mapInfo) <= 0) { - // Invalid id - $this->mc->chat->sendError('Invalid MX-Id!', $login); - return; - } - $mapInfo = json_decode($mapInfo, true); - $url = 'http://' . $title . '.mania-exchange.com/tracks/download/' . $params[1]; - $file = FileUtil::loadFile($url); - if (!$file) { - // Download error - $this->mc->chat->sendError('Download failed!', $login); - return; - } - // Save map - $fileName = $mapDir . $mapInfo['TrackID'] . '_' . $mapInfo['Name'] . '.Map.Gbx'; - if (!file_put_contents($fileName, $file)) { - // Save error - $this->mc->chat->sendError('Saving map failed!', $login); - return; - } - // Check for valid map - if (!$this->mc->client->query('CheckMapForCurrentServerParams', $fileName)) { - trigger_error("Couldn't check if map is valid. " . $this->mc->getClientErrorText()); - } - else { - $response = $this->mc->client->getResponse(); - if (!$response) { - // Inalid map type - $this->mc->chat->sendError("Invalid map type.", $login); - return; - } - } - // Add map to map list - if (!$this->mc->client->query('InsertMap', $fileName)) { - $this->mc->chat->sendError("Couldn't add map to match settings!", $login); - return; - } - $this->mc->chat->sendSuccess('Map $<' . $mapInfo['Name'] . '$> successfully added!'); - } - else { - // TODO: check if map exists locally - // TODO: load map from direct url - } + $mapDir = $this->maniaControl->client->getResponse(); + if (!is_dir($mapDir)) { + trigger_error("ManiaControl doesn't have have access to the maps directory in '{$mapDir}'."); + $this->maniaControl->chat->sendError("ManiaControl doesn't have access to the maps directory.", $login); + return false; } + $downloadDirectory = $this->maniaControl->settingManager->getSetting($this, 'MapDownloadDirectory', 'mx'); + // Create download directory if necessary + if (!is_dir($mapDir . $downloadDirectory) && !mkdir($mapDir . $downloadDirectory)) { + trigger_error("ManiaControl doesn't have to rights to save maps in '{$mapDir}{$downloadDirectory}'."); + $this->maniaControl->chat->sendError("ManiaControl doesn't have the rights to save maps.", $login); + return false; + } + $mapDir .= $downloadDirectory . '/'; + // Download the map + $mapId = $params[1]; + if (is_numeric($mapId)) { + // Load from MX + $serverInfo = $this->maniaControl->server->getSystemInfo(); + $title = strtolower(substr($serverInfo['TitleId'], 0, 2)); + // Check if map exists + $url = "http://{$title}.mania-exchange.com/api/tracks/get_track_info/id/{$mapId}?format=json"; + $mapInfo = FileUtil::loadFile($url); + if (!$mapInfo || strlen($mapInfo) <= 0) { + // Invalid id + $this->maniaControl->chat->sendError('Invalid MX-Id!', $login); + return false; + } + $mapInfo = json_decode($mapInfo, true); + $url = "http://{$title}.mania-exchange.com/tracks/download/{$mapId}"; + $file = FileUtil::loadFile($url); + if (!$file) { + // Download error + $this->maniaControl->chat->sendError('Download failed!', $login); + return false; + } + // Save map + $fileName = $mapDir . $mapInfo['TrackID'] . '_' . $mapInfo['Name'] . '.Map.Gbx'; + if (!file_put_contents($fileName, $file)) { + // Save error + $this->maniaControl->chat->sendError('Saving map failed!', $login); + return false; + } + // Check for valid map + if (!$this->maniaControl->client->query('CheckMapForCurrentServerParams', $fileName)) { + trigger_error("Couldn't check if map is valid. " . $this->maniaControl->getClientErrorText()); + $this->maniaControl->chat->sendError('Error checking map!', $login); + return false; + } + $response = $this->maniaControl->client->getResponse(); + if (!$response) { + // Inalid map type + $this->maniaControl->chat->sendError("Invalid map type.", $login); + return false; + } + // Add map to map list + if (!$this->maniaControl->client->query('InsertMap', $fileName)) { + $this->maniaControl->chat->sendError("Couldn't add map to match settings!", $login); + return false; + } + $this->maniaControl->chat->sendSuccess('Map $<' . $mapInfo['Name'] . '$> successfully added!'); + return true; + } + // TODO: add local map by filename + // TODO: load map from direct url } /** * Handle nextmap command + * + * @param array $chat + * @return bool */ - private function command_nextmap($chat) { + private function command_nextmap(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('nextmap', 'operator'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('nextmap', 'operator'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; - } - if (!$this->mc->client->query('NextMap')) { - trigger_error("Couldn't skip map. " . $this->mc->getClientErrorText()); + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } + return $this->maniaControl->client->query('NextMap'); } /** * Handle retartmap command + * + * @param array $chat + * @return bool */ - private function command_restartmap($chat) { + private function command_restartmap(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('restartmap', 'operator'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('restartmap', 'operator'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } - if (!$this->mc->client->query('RestartMap')) { - trigger_error("Couldn't restart map. " . $this->mc->getClientErrorText()); - } - } - - /** - * Handle getservername command - */ - private function command_getservername($chat) { - $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('getservername', 'operator'))) { - // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; - } - $serverName = $this->mc->server->getName(); - $this->mc->chat->sendInformation("Server Name: " . $serverName, $login); + return $this->maniaControl->client->query('RestartMap'); } /** * Handle setservername command + * + * @param array $chat + * @return bool */ - private function command_setservername($chat) { + private function command_setservername(array $chat) { $login = $chat[1][1]; - if (!$this->mc->authentication->checkRight($login, $this->getRightsLevel('setservername', 'admin'))) { + if (!$this->maniaControl->authentication->checkRight($login, $this->getRightsLevel('setservername', 'admin'))) { // Not allowed! - $this->mc->authentication->sendNotAllowed($login); - return; + $this->maniaControl->authentication->sendNotAllowed($login); + return false; } $params = explode(' ', $chat[1][2], 2); if (count($params) < 2) { // TODO: show usage - return; + return false; } $serverName = $params[1]; - if (!$this->mc->client->query('SetServerName', $serverName)) { - trigger_error("Couldn't set server name. " . $this->mc->getClientErrorText()); - $this->mc->chat->sendError("Error!"); - } - else { - $serverName = $this->mc->server->getName(); - $this->mc->chat->sendInformation("New Name: " . $serverName); + if (!$this->maniaControl->client->query('SetServerName', $serverName)) { + trigger_error("Couldn't set server name. " . $this->maniaControl->getClientErrorText()); + $this->maniaControl->chat->sendError("Error!", $login); + return false; } + $serverName = $this->maniaControl->server->getName(); + $this->maniaControl->chat->sendInformation("New Name: " . $serverName, $login); + return true; } /** * Check stuff each 5 seconds + * + * @param array $callback + * @return bool */ - public function each5Seconds() { + public function each5Seconds(array $callback) { // Empty shutdown if ($this->serverShutdownEmpty) { - $players = $this->mc->server->getPlayers(); + $players = $this->maniaControl->server->getPlayers(); if (count($players) <= 0) { - $this->shutdownServer('empty'); + return $this->shutdownServer('empty'); } } // Delayed shutdown if ($this->serverShutdownTime > 0) { if (time() >= $this->serverShutdownTime) { - $this->shutdownServer('delayed'); + return $this->shutdownServer('delayed'); } } } /** * Perform server shutdown + * + * @param string $login + * @return bool */ private function shutdownServer($login = '#') { - $this->mc->client->resetError(); - if (!$this->mc->client->query('StopServer') || $this->mc->client->isError()) { - trigger_error("Server shutdown command from '" . $login . "' failed. " . $this->mc->getClientErrorText()); - return; + if (!$this->maniaControl->client->query('StopServer')) { + trigger_error("Server shutdown command from '{$login}' failed. " . $this->maniaControl->getClientErrorText()); + return false; } - $this->mc->quit("Server shutdown requested by '" . $login . "'"); + $this->maniaControl->quit("Server shutdown requested by '{$login}'"); + return true; } } diff --git a/application/core/maniaControlClass.php b/application/core/maniaControlClass.php index 4d25e485..0de8a6c8 100644 --- a/application/core/maniaControlClass.php +++ b/application/core/maniaControlClass.php @@ -55,13 +55,11 @@ class ManiaControl { * Private properties */ private $shutdownRequested = false; - private $config = null; /** * Construct ManiaControl */ public function __construct() { - $this->config = FileUtil::loadConfig('core.xml'); $this->database = new Database($this); $this->settingManager = new SettingManager($this); $this->chat = new Chat($this); @@ -169,14 +167,11 @@ class ManiaControl { $port = $this->server->config->xpath('port'); if (!$host) trigger_error("Invalid server configuration (port).", E_USER_ERROR); $port = (string) $port[0]; - $timeout = $this->config->xpath('timeout'); - if (!$timeout) trigger_error("Invalid core configuration (timeout).", E_USER_ERROR); - $timeout = (int) $timeout[0]; error_log("Connecting to server at {$host}:{$port}..."); // Connect - if (!$this->client->InitWithIp($host, $port, $timeout)) { + if (!$this->client->InitWithIp($host, $port, 20)) { trigger_error("Couldn't connect to server! " . $this->getClientErrorText(), E_USER_ERROR); } diff --git a/application/core/server.php b/application/core/server.php index e4721bc9..4ad7e926 100644 --- a/application/core/server.php +++ b/application/core/server.php @@ -353,8 +353,7 @@ class Server { if ($response['Code'] === 4) return true; // Server not yet in given status -> Wait for it... $waitBegin = time(); - $timeoutTags = $this->mc->config->xpath('timeout'); - $maxWaitTime = (!empty($timeoutTags) ? (int) $timeoutTags[0] : 20); + $maxWaitTime = 20; $lastStatus = $response['Name']; error_log("Waiting for server to reach status " . $statusCode . "..."); error_log("Current Status: " . $lastStatus); diff --git a/application/readme.txt b/application/readme.txt index 26ab8d43..d74c4ef1 100644 --- a/application/readme.txt +++ b/application/readme.txt @@ -1,10 +1,13 @@ -********************************************** -* * -* mc ManiaPlanet Server Control * -* Written by steeffeen * -* Contact: mail@steeffeen.com * -* * -********************************************** +******************************************************* +* * +* ManiaControl - ManiaPlanet Server Controller * +* Written by steeffeen & kremsy * +* * +* Contact: * +* steff@maniacontrol.com, * +* lukas@maniacontrol.com * +* * +******************************************************* SETUP: @@ -12,32 +15,28 @@ SETUP: 2. Configure the needed settings: - 2.1 Open the file 'configs/server.mc.xml'. + 2.1 Open the file 'configs/server.xml'. Enter your maniaplanet server information. - 2.2 Open the file 'configs/database.mc.xml' - Enter your mysql server information or disable database usage if you don't have a mysql server available. + 2.2 Open the file 'configs/database.xml' + Enter your mysql server information. - 2.3 Open the file 'configs/authentication.mc.xml'. - Add the player logins who should have access to the commands of mc. + 2.3 Open the file 'configs/authentication.xml'. + Add the player logins who should have access to the commands of ManiaControl. -3. (Optional) Enable or disable the available plugins in the file 'configs/plugins.mc.xml'. +3. Run the tool via the shell script 'ManiaControl.sh' (UNIX) or the batch file 'ManiaControl.bat' (Windows) -4. (Optional) Edit the other config files in 'configs/' in order to customize your mc to fit your needs. - -5. Run the tool via the shell script 'mc.sh' (UNIX) or the batch file 'mc.bat' (Windows) - -6. Enjoy! +4. Enjoy! INFORMATION: -- mc is only tested on UNIX machines - - even though it might run properly on Windows I can't promise it will work all the time - - furthermore I can't promise that there won't be a feature in the future that makes it impossible to run mc under Windows - - in order to run mc under Windows you have to alter the file mc.bat and enter the path to your php.exe +- ManiaControl is only tested on UNIX machines + - even though it might run properly on Windows we can't promise it will work all the time + - furthermore we can't promise that there won't be a feature in the future that makes it impossible to run ManiaControl on Windows + - in order to run ManiaControl on Windows you have to alter the file ManiaControl.bat and enter the path to your php.exe - Tests were performed using PHP Version 5.4 - If you notice problems with other version please let me know -- Please report bugs by writing a mail to mail@steeffeen.com +- Please report bugs by writing a mail to bugs@maniacontrol.com