Completed
Pull Request — master (#397)
by De Cramer
02:34
created
src/eXpansion/Framework/Core/Plugins/AutomaticMatchSettingsSave.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
             return;
55 55
         }
56 56
 
57
-        $path = 'MatchSettings/' . $this->fileName->get();
57
+        $path = 'MatchSettings/'.$this->fileName->get();
58 58
 
59 59
         try {
60 60
             $this->connectionFactory->getConnection()->saveMatchSettings($path);
Please login to merge, or discard this patch.
src/eXpansion/Framework/Core/Plugins/AutomaticServerSettingsSave.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
             return;
58 58
         }
59 59
 
60
-        $path = 'Config/' . $this->fileName->get();
60
+        $path = 'Config/'.$this->fileName->get();
61 61
         $fileSystem = $this->fileSystem->getUserData();
62 62
 
63 63
         if (!$fileSystem->has($path)) {
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
             }
112 112
             $oldXml->server_options->{$search}[0] = $out;
113 113
         }
114
-        $this->logger->info('Saving server settings to : ' . $path);
114
+        $this->logger->info('Saving server settings to : '.$path);
115 115
         $xml = $oldXml->asXML();
116 116
 
117 117
         $fileSystem->update($path, $xml);
Please login to merge, or discard this patch.
eXpansion/Framework/AdminGroups/DependencyInjection/Compiler/ConfigPass.php 2 patches
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -49,11 +49,11 @@
 block discarded – undo
49 49
     }
50 50
 
51 51
         /**
52
-     * Create the config services.
53
-     *
54
-     * @param $groups
55
-     * @param ContainerBuilder $container
56
-     */
52
+         * Create the config services.
53
+         *
54
+         * @param $groups
55
+         * @param ContainerBuilder $container
56
+         */
57 57
     protected function createConfigs($groups, $permissions, ContainerBuilder $container)
58 58
     {
59 59
         $configManager = $container->findDefinition(ConfigManagerInterface::class);
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -60,15 +60,15 @@  discard block
 block discarded – undo
60 60
 
61 61
         foreach ($groups as $groupCode => $group)
62 62
         {
63
-            $pathPrefix = $container->getParameter('expansion.admin_groups.config.path') . "/$groupCode";
63
+            $pathPrefix = $container->getParameter('expansion.admin_groups.config.path')."/$groupCode";
64 64
 
65
-            $id = 'expansion.admin_groups.config.label.' . $groupCode;
65
+            $id = 'expansion.admin_groups.config.label.'.$groupCode;
66 66
             $container->setDefinition($id, new ChildDefinition('expansion.admin_groups.config.label.abstract'))
67 67
                 ->replaceArgument('$path', "$pathPrefix/label")
68 68
                 ->replaceArgument('$defaultValue', $group['label']);
69 69
             $configManager->addMethodCall('registerConfig', [new Reference($id), $id]);
70 70
 
71
-            $id = 'expansion.admin_groups.config.logins.' . $groupCode;
71
+            $id = 'expansion.admin_groups.config.logins.'.$groupCode;
72 72
             $container->setDefinition($id, new ChildDefinition('expansion.admin_groups.config.logins.abstract'))
73 73
                 ->setArgument('$path', "$pathPrefix/logins")
74 74
                 ->setArgument('$defaultValue', $group['logins']);
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 
77 77
             if ($groupCode != "master_admin") {
78 78
                 foreach ($permissions as $permission) {
79
-                    $id = 'expansion.admin_groups.config.permissions.' . $groupCode . ".$permission";
79
+                    $id = 'expansion.admin_groups.config.permissions.'.$groupCode.".$permission";
80 80
                     $container->setDefinition($id, new ChildDefinition('expansion.admin_groups.config.permissions.abstract'))
81 81
                         ->setArgument('$path', "$pathPrefix/perm_$permission")
82 82
                         ->setArgument('$defaultValue', in_array($permission, $group['permissions']))
Please login to merge, or discard this patch.
src/eXpansion/Framework/Core/Services/DedicatedConnection/Factory.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
                 $this->console->getSfStyleOutput()->error(
102 102
                     [
103 103
                         "Looks like your Dedicated server is either offline or has wrong config settings",
104
-                        "Error message: " . $lastExcelption->getMessage()
104
+                        "Error message: ".$lastExcelption->getMessage()
105 105
                     ]
106 106
                 );
107 107
                 $this->logger->error(
@@ -171,14 +171,14 @@  discard block
 block discarded – undo
171 171
                     ]
172 172
                 );
173 173
                 $this->console->getSfStyleOutput()->block([
174
-                    "Host : " . $this->host,
175
-                    "Port : " . $this->port,
176
-                    "Timeout : " . $this->timeout,
177
-                    "User : " . $this->user,
174
+                    "Host : ".$this->host,
175
+                    "Port : ".$this->port,
176
+                    "Timeout : ".$this->timeout,
177
+                    "User : ".$this->user,
178 178
                     "Password : ****",
179 179
                 ]);
180 180
             }
181
-        } while($attempts < $maxAttempts && !is_null($lastExcelption));
181
+        } while ($attempts < $maxAttempts && !is_null($lastExcelption));
182 182
 
183 183
         return $lastExcelption;
184 184
     }
Please login to merge, or discard this patch.
src/eXpansion/Bundle/InfoMessages/Plugins/InfoMessagesPlugin.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use eXpansion\Bundle\InfoMessages\DependencyInjection\InfoMessagesExtension;
6 6
 use eXpansion\Framework\Config\Model\IntegerConfig;
7
-use eXpansion\Framework\Config\Model\TextListConfig;
8 7
 use eXpansion\Framework\Config\Services\ConfigManagerInterface;
9 8
 use eXpansion\Framework\Core\DataProviders\Listener\ListenerInterfaceExpTimer;
10 9
 use eXpansion\Framework\Core\Helpers\Translations;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -81,7 +81,7 @@
 block discarded – undo
81 81
             // Ideally the list of configs should have been passed in the construct, to achieve this we would need
82 82
             // to inject the config services throught the Extension. We have chose not to do this to simply simplify
83 83
             // the code to make it a bit easier for people that are new to symfony to understand.
84
-            $values = $this->configManager->get(InfoMessagesExtension::CONFIG_PATH_PREFIX . $locale);
84
+            $values = $this->configManager->get(InfoMessagesExtension::CONFIG_PATH_PREFIX.$locale);
85 85
 
86 86
             if (count($values) > 0) {
87 87
                 $messagesPerLocale[] = [
Please login to merge, or discard this patch.
src/eXpansion/Framework/Core/Plugins/Analytics.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,6 @@
 block discarded – undo
7 7
 use eXpansion\Framework\Core\Helpers\Http;
8 8
 use eXpansion\Framework\Core\Helpers\Structures\HttpResult;
9 9
 use eXpansion\Framework\Core\Helpers\Version;
10
-use eXpansion\Framework\Core\Services\Application;
11 10
 use eXpansion\Framework\Core\Storage\GameDataStorage;
12 11
 use eXpansion\Framework\Core\Storage\PlayerStorage;
13 12
 use Psr\Log\LoggerInterface;
Please login to merge, or discard this patch.
src/eXpansion/Framework/Core/Plugins/StartGreeter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
         $this->chatNotification->sendMessage('$z$fff$s       -  -   –    –     —     —    –    –   -   -');
50 50
         $this->chatNotification->sendMessage('$z$s$w$i$fff       e X p a n s i o n ²', null, []);
51 51
         $this->chatNotification->sendMessage(
52
-            '$z$o$000                version$o '. $this->version->getExpansionVersion(), null, []);
52
+            '$z$o$000                version$o '.$this->version->getExpansionVersion(), null, []);
53 53
 
54 54
         $this->chatNotification->sendMessage('$z$s$fff                 —————————— ');
55 55
         $this->chatNotification->sendMessage("");
Please login to merge, or discard this patch.
src/eXpansion/Framework/Core/Services/Application.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
         $this->console->writeln('$fff `"Ybbd8"\'  8P        Y8$fff  88`YbbdP"\'  $0d0  88888888888');
39 39
         $this->console->writeln('$fff                        $fff  88          $0d0                ');
40 40
         $this->console->writeln('$777                        $fff  88          $0d0               ');
41
-        $this->console->writeln('$777  eXpansion v' . $this->version->getExpansionVersion());
41
+        $this->console->writeln('$777  eXpansion v'.$this->version->getExpansionVersion());
42 42
 
43 43
         return parent::init($console);
44 44
     }
Please login to merge, or discard this patch.
eXpansion/Bundle/InfoMessages/DependencyInjection/InfoMessagesExtension.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
             $id = "eXpansion.info_messages.config.messages.$locale";
36 36
 
37 37
             $service = $container->setDefinition($id, new ChildDefinition(self::ABSTRACT_SERVICE_DEFINITION_ID))
38
-                ->replaceArgument('$path', self::CONFIG_PATH_PREFIX . "$locale")
38
+                ->replaceArgument('$path', self::CONFIG_PATH_PREFIX."$locale")
39 39
                 ->replaceArgument('$name', "expansion_info_messages.config.messages.$locale.name")
40 40
                 ->addTag("expansion.config");
41 41
 
Please login to merge, or discard this patch.