SuperdeskWebPublisher / SWPUpdaterBundle

Client/GuzzleClient.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         } catch (\Exception $e) {
             throw new ServiceUnavailableHttpException(
                 null,
-                'Could not resolve host: '.$this->baseUri,
+                'Could not resolve host: ' . $this->baseUri,
                 $e,
                 $e->getCode()
             );

DependencyInjection/SWPUpdaterExtension.php

Spacing +11 added lines, -11 removed lines

@@ -34,7 +34,7 @@ 
     {
         $this->container = $container;
         $config = $this->processConfiguration(new Configuration(), $configs);
-        $loader = new Loader\YamlFileLoader($this->container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($this->container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('services.yml');
 
         $this->checkAndSetSupportedClass($config);
@@ -45,24 +45,24 @@ 
                 $clientConfig[$key] = $config['client'][$key];
             }
 
-            $this->container->setParameter($this->getAlias().'.client', $clientConfig);
+            $this->container->setParameter($this->getAlias() . '.client', $clientConfig);
         }
 
         $options = array();
-        if ($this->container->hasParameter($this->getAlias().'.client.options')) {
-            $options = $this->container->getParameter($this->getAlias().'.client.options');
+        if ($this->container->hasParameter($this->getAlias() . '.client.options')) {
+            $options = $this->container->getParameter($this->getAlias() . '.client.options');
         }
 
-        $this->container->setParameter($this->getAlias().'.client.options', $options);
+        $this->container->setParameter($this->getAlias() . '.client.options', $options);
 
         if (!empty($config['version_class'])) {
-            $this->container->setParameter($this->getAlias().'.version_class', $config['version_class']);
+            $this->container->setParameter($this->getAlias() . '.version_class', $config['version_class']);
         }
 
         $this->setDirectories($config);
 
         if (true === $config['monolog_channel']) {
-            $this->container->setParameter($this->getAlias().'.monolog_channel', true);
+            $this->container->setParameter($this->getAlias() . '.monolog_channel', true);
         }
     }
 
@@ -90,12 +90,12 @@ 
         foreach (array('temp_dir', 'target_dir') as $value) {
             if ($this->isDefault($config[$value])) {
                 $this->container->setParameter(
-                    $this->getAlias().'.'.$value,
+                    $this->getAlias() . '.' . $value,
                     $this->checkDirType($value)
                 );
             } else {
                 $this->container->setParameter(
-                    $this->getAlias().'.'.$value,
+                    $this->getAlias() . '.' . $value,
                     $this->checkNotDefaultDirType($value, $config[$value])
                 );
             }
@@ -108,7 +108,7 @@ 
             return $this->container->getParameter('kernel.cache_dir');
         }
 
-        return $this->container->getParameter('kernel.root_dir').'/../';
+        return $this->container->getParameter('kernel.root_dir') . '/../';
     }
 
     private function checkNotDefaultDirType($dir, $configDir)
@@ -117,7 +117,7 @@ 
             return $configDir;
         }
 
-        return $this->container->getParameter('kernel.root_dir').'/'.$configDir;
+        return $this->container->getParameter('kernel.root_dir') . '/' . $configDir;
     }
 
     private function isDefault($dir)

Manager/AbstractManager.php

Spacing +4 added lines, -4 removed lines

@@ -132,17 +132,17 @@ 
     protected function copyRemoteFile($fromUrl, $name)
     {
         try {
-            $filePath = $this->tempDir.'/'.$name.'.zip';
+            $filePath = $this->tempDir . '/' . $name . '.zip';
             if (!file_exists($filePath)) {
                 $this->client->saveFile($fromUrl, $filePath);
 
-                $this->addLogInfo('Successfully downloaded update file: '.$filePath);
+                $this->addLogInfo('Successfully downloaded update file: ' . $filePath);
 
                 return true;
             }
         } catch (\Exception $e) {
             throw new NotFoundHttpException(
-                'Could not find file at the specified path: '.$fromUrl,
+                'Could not find file at the specified path: ' . $fromUrl,
                 $e,
                 $e->getCode()
             );
@@ -157,7 +157,7 @@ 
      */
     protected function sortPackagesByVersion(array $array = array())
     {
-        usort($array, function ($first, $second) {
+        usort($array, function($first, $second) {
             if ($first instanceof UpdatePackage && $second instanceof UpdatePackage) {
                 return version::compare($first->getVersion(), $second->getVersion());
             }

Manager/UpdateManager.php

Spacing +2 added lines, -2 removed lines

@@ -180,8 +180,8 @@
     {
         $sortedVersions = $this->sortPackagesByVersion($this->availableUpdates['core']);
         foreach ($sortedVersions as $update) {
-            $packageName = $update->getVersion().'.zip';
-            $packagePath = $this->tempDir.'/'.$packageName;
+            $packageName = $update->getVersion() . '.zip';
+            $packagePath = $this->tempDir . '/' . $packageName;
             $this->addLogInfo('Started updating application\'s core...');
             if (!file_exists($packagePath)) {
                 throw new NotFoundHttpException(sprintf(

Tests/DependencyInjection/Compiler/MonologCompilerPassTest.php

Indentation +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
      */
     public function testNoConfig()
     {
-          $this->container->expects($this->once())
+            $this->container->expects($this->once())
             ->method('hasParameter')
             ->with('swp_updater.monolog_channel')
             ->will($this->returnValue(false));

Tests/DependencyInjection/SWPUpdaterExtensionTest.php

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
         $this->assertEquals($curlOptions, $container->getParameter('swp_updater.client.options'));
         $this->assertEquals('Acme\DemoBundle\Version\Version', $container->getParameter('swp_updater.version.class'));
         $this->assertEquals($container->getParameter('kernel.cache_dir'), $container->getParameter('swp_updater.temp_dir'));
-        $this->assertEquals($container->getParameter('kernel.root_dir').'/../', $container->getParameter('swp_updater.target_dir'));
+        $this->assertEquals($container->getParameter('kernel.root_dir') . '/../', $container->getParameter('swp_updater.target_dir'));
         $this->assertFalse($container->hasParameter('swp_updater.monolog_channel'));
     }
 
@@ -70,7 +70,7 @@ 
         $config = $this->getConfig();
         $loader->load(array(array_merge($config, $directories)), $container);
         $this->assertEquals(
-            $container->getParameter('kernel.root_dir').'/some/temp/dir',
+            $container->getParameter('kernel.root_dir') . '/some/temp/dir',
             $container->getParameter('swp_updater.temp_dir')
         );
 

Tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-$file = __DIR__.'/../vendor/autoload.php';
+$file = __DIR__ . '/../vendor/autoload.php';
 if (!file_exists($file)) {
     throw new RuntimeException('Install dependencies using composer to run the test suite.');
 }

spec/SWP/UpdaterBundle/Client/DefaultClientSpec.php

Indentation +2 added lines, -2 removed lines

@@ -25,8 +25,8 @@
 
     function let()
     {
-    	$config = array('base_uri' => 'http://httpbin.org');
-    	$this->beConstructedWith($config);
+        $config = array('base_uri' => 'http://httpbin.org');
+        $this->beConstructedWith($config);
     }
 
     function it_should_make_a_call_to_remote_server()

Spacing +2 added lines, -2 removed lines

@@ -107,14 +107,14 @@
 
     function it_should_save_the_file()
     {
-        $filePath = __DIR__.'/../../../file.png';
+        $filePath = __DIR__ . '/../../../file.png';
         $response = $this->saveFile('http://httpbin.org/image/png', $filePath);
         $response->shouldBe(true);
     }
 
     function it_should_not_save_the_file()
     {
-        $response = $this->saveFile('fake_url', __DIR__.'/../../../file.png');
+        $response = $this->saveFile('fake_url', __DIR__ . '/../../../file.png');
         $response->shouldBe(false);
     }
 }

spec/SWP/UpdaterBundle/Client/GuzzleClientSpec.php

Spacing +2 added lines, -2 removed lines

@@ -107,14 +107,14 @@
 
     function it_should_save_the_file()
     {
-        $filePath = __DIR__.'/../../../file.png';
+        $filePath = __DIR__ . '/../../../file.png';
         $response = $this->saveFile('http://httpbin.org/image/png', $filePath);
         $response->shouldBe(true);
     }
 
     function it_should_not_save_the_file()
     {
-        $response = $this->saveFile('fake_url', __DIR__.'/../../../file.png');
+        $response = $this->saveFile('fake_url', __DIR__ . '/../../../file.png');
         $response->shouldBe(false);
     }
 }