trivago / chapi

src/Entity/Marathon/AppEntity/IpAddress.php

Spacing +2 added lines, -2 removed lines

@@ -25,13 +25,13 @@
     {
         MarathonEntityUtils::setAllPossibleProperties($aData, $this);
 
-        if(isset($aData['groups']))
+        if (isset($aData['groups']))
         {
             $this->groups = $aData['groups'];
         }
         if (isset($aData['labels']))
         {
-            $this->labels = (object)$aData['labels'];
+            $this->labels = (object) $aData['labels'];
         }
     }
 }
\ No newline at end of file

src/BusinessCase/Comparison/MarathonJobComparisonBusinessCase.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $this->oDiffCompare = $oDiffCompare;
     }
 
-    protected function preCompareModifications(JobEntityInterface &$oLocalJob, JobEntityInterface &$oRemoteJob)
+    protected function preCompareModifications(JobEntityInterface & $oLocalJob, JobEntityInterface & $oRemoteJob)
     {
         if (
             !$oLocalJob instanceof MarathonAppEntity ||

src/BusinessCase/Comparison/ChronosJobComparisonBusinessCase.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     }
 
 
-    protected function preCompareModifications(JobEntityInterface &$oLocalJob, JobEntityInterface &$oRemoteJob)
+    protected function preCompareModifications(JobEntityInterface & $oLocalJob, JobEntityInterface & $oRemoteJob)
     {
         // no modification needed
         return;

src/BusinessCase/Comparison/AbstractJobComparisionBusinessCase.php

Spacing +2 added lines, -2 removed lines

@@ -166,7 +166,7 @@ 
             $_aDifferences[$_sProperty] = $this->oDiffCompare->compare(
                 $_oRemoteEntity->{$_sProperty},
                 $_oLocalEntity->{$_sProperty}
-            ) ;
+            );
         }
 
         return $_aDifferences;
@@ -219,7 +219,7 @@ 
      * @param JobEntityInterface $oRemoteJob
      * @return null
      */
-    abstract protected function preCompareModifications(JobEntityInterface &$oLocalJob, JobEntityInterface &$oRemoteJob);
+    abstract protected function preCompareModifications(JobEntityInterface & $oLocalJob, JobEntityInterface & $oRemoteJob);
 
     /**
      * Gets entity for each system with defaults set

src/BusinessCase/JobManagement/MarathonStoreJobBusinessCase.php

Spacing +5 added lines, -5 removed lines

@@ -72,7 +72,7 @@ 
             }
 
             // check if dependency is satisfied
-            if ( $_oJobEntityLocal->isDependencyJob())
+            if ($_oJobEntityLocal->isDependencyJob())
             {
                 try {
                     $circular = $this->isDependencyCircular($_oJobEntityLocal, count($_oJobEntityLocal->dependencies));
@@ -84,10 +84,10 @@ 
                         return false;
                     }
                 }
-                catch(\Exception $e)
+                catch (\Exception $e)
                 {
                     $this->oLogger->error(sprintf(
-                        'Job %s cannot be added to remote : %s',$sAppId, $e->getMessage()
+                        'Job %s cannot be added to remote : %s', $sAppId, $e->getMessage()
                     ));
                     return false;
                 }
@@ -133,7 +133,7 @@ 
         return !(count($arr) == count(array_unique($arr)));
     }
 
-    private function isDependencyCircular(MarathonAppEntity $oEntity, $iImmediateChildren, &$path=[])
+    private function isDependencyCircular(MarathonAppEntity $oEntity, $iImmediateChildren, &$path = [])
     {
         // Invariant: path will not have duplicates for acyclic dependency tree
         if ($this->hasDuplicates($path))
@@ -186,7 +186,7 @@ 
             // for B intermediate Child will be 2.
             // when we process D, it will be reduced to 1 and with C to 0
             // then we will pop B to generate path [A, E] when we reach E.
-            $iImmediateChildren = $iImmediateChildren -1;
+            $iImmediateChildren = $iImmediateChildren - 1;
             if ($iImmediateChildren == 0)
             {
                 array_pop($path);

src/Commands/ConfigureCommand.php

Spacing +2 added lines, -2 removed lines

@@ -165,7 +165,7 @@ 
         $_sYaml = $_oDumper->dump(array('parameters' => $aToStore), 2);
 
         $_oFileSystem = new Filesystem();
-        $_oFileSystem->dumpFile($this->getHomeDir() . '/parameters.yml', $_sYaml);
+        $_oFileSystem->dumpFile($this->getHomeDir().'/parameters.yml', $_sYaml);
     }
 
     /**
@@ -194,7 +194,7 @@ 
     private function getParameterValue($sKey, $mDefaultValue = null)
     {
         $_oParser = new Parser();
-        $_sParameterFile = $this->getHomeDir() . '/parameters.yml';
+        $_sParameterFile = $this->getHomeDir().'/parameters.yml';
 
         if (file_exists($_sParameterFile))
         {

src/BusinessCase/Comparison/CompositeJobComparisonBusinessCase.php

Spacing +7 added lines, -7 removed lines

@@ -34,10 +34,10 @@ 
     public function getLocalMissingJobs()
     {
         $_aMissingJobs = array();
-        foreach($this->aComposites as $jobComparers)
+        foreach ($this->aComposites as $jobComparers)
         {
             $missing = $jobComparers->getLocalMissingJobs();
-            $_aMissingJobs = array_merge($_aMissingJobs,  $missing);
+            $_aMissingJobs = array_merge($_aMissingJobs, $missing);
         }
         return $_aMissingJobs;
     }
@@ -48,7 +48,7 @@ 
     public function getRemoteMissingJobs()
     {
         $_aChronosMissingJobs = array();
-        foreach($this->aComposites as $jobComparers)
+        foreach ($this->aComposites as $jobComparers)
         {
             $_aChronosMissingJobs = array_merge($_aChronosMissingJobs, $jobComparers->getRemoteMissingJobs());
         }
@@ -61,7 +61,7 @@ 
     public function getLocalJobUpdates()
     {
         $_aLocalJobUpdates = array();
-        foreach($this->aComposites as $jobComparers)
+        foreach ($this->aComposites as $jobComparers)
         {
             $_aLocalJobUpdates = array_merge($_aLocalJobUpdates, $jobComparers->getLocalJobUpdates());
         }
@@ -75,7 +75,7 @@ 
     public function getJobDiff($sJobName)
     {
         $_aJobDiffs = array();
-        foreach($this->aComposites as $jobComparers)
+        foreach ($this->aComposites as $jobComparers)
         {
             // assuming same name won't be in all subsystems.
             // TODO: add support to handle the duplicate names in different subsystems
@@ -101,7 +101,7 @@ 
         }
         /** @var JobComparisonInterface $comparer */
         $comparer = null;
-        foreach($this->aComposites as $child)
+        foreach ($this->aComposites as $child)
         {
             if ($child->isJobAvailable($oJobEntityA->getKey()))
             {
@@ -125,7 +125,7 @@ 
      */
     public function isJobAvailable($sJobName)
     {
-        foreach($this->aComposites as $child)
+        foreach ($this->aComposites as $child)
         {
             if ($child->isJobAvailable($sJobName))
             {

src/Component/Http/HttpGuzzlClient.php

Spacing +6 added lines, -6 removed lines

@@ -63,7 +63,7 @@ 
         catch (ClientException $oException) // 400 level errors
         {
             throw new HttpConnectionException(
-                sprintf('Client error: Calling %s returned %d', $this->oGuzzelClient->getConfig('base_uri') . $sUrl, $oException->getCode()),
+                sprintf('Client error: Calling %s returned %d', $this->oGuzzelClient->getConfig('base_uri').$sUrl, $oException->getCode()),
                 $oException->getCode(),
                 $oException
             );
@@ -71,7 +71,7 @@ 
         catch (ServerException $oException) // 500 level errors
         {
             throw new HttpConnectionException(
-                sprintf('Server error: Calling %s returned %d', $this->oGuzzelClient->getConfig('base_uri') . $sUrl, $oException->getCode()),
+                sprintf('Server error: Calling %s returned %d', $this->oGuzzelClient->getConfig('base_uri').$sUrl, $oException->getCode()),
                 $oException->getCode(),
                 $oException
             );
@@ -79,7 +79,7 @@ 
         catch (TooManyRedirectsException $oException) // too many redirects to follow
         {
             throw new HttpConnectionException(
-                sprintf('Request to %s failed due to too many redirects', $this->oGuzzelClient->getConfig('base_uri') . $sUrl),
+                sprintf('Request to %s failed due to too many redirects', $this->oGuzzelClient->getConfig('base_uri').$sUrl),
                 HttpConnectionException::ERROR_CODE_TOO_MANY_REDIRECT_EXCEPTION,
                 $oException
             );
@@ -87,7 +87,7 @@ 
         catch (ConnectException $oException) // networking error
         {
             throw new HttpConnectionException(
-                sprintf('Cannot connect to %s due to some networking error', $this->oGuzzelClient->getConfig('base_uri') . $sUrl),
+                sprintf('Cannot connect to %s due to some networking error', $this->oGuzzelClient->getConfig('base_uri').$sUrl),
                 HttpConnectionException::ERROR_CODE_CONNECT_EXCEPTION,
                 $oException
             );
@@ -95,7 +95,7 @@ 
         catch (RequestException $oException) // networking error (connection timeout, DNS errors, etc.)
         {
             throw new HttpConnectionException(
-                sprintf('Cannot connect to %s due to networking error', $this->oGuzzelClient->getConfig('base_uri') . $sUrl),
+                sprintf('Cannot connect to %s due to networking error', $this->oGuzzelClient->getConfig('base_uri').$sUrl),
                 HttpConnectionException::ERROR_CODE_REQUEST_EXCEPTION,
                 $oException
             );
@@ -103,7 +103,7 @@ 
         catch (\Exception $oException)
         {
             throw new HttpConnectionException(
-                sprintf('Can\'t get response from "%s"', $this->oGuzzelClient->getConfig('base_uri') . $sUrl),
+                sprintf('Can\'t get response from "%s"', $this->oGuzzelClient->getConfig('base_uri').$sUrl),
                 HttpConnectionException::ERROR_CODE_UNKNOWN,
                 $oException
             );

src/Entity/Marathon/MarathonAppEntity.php

Spacing +11 added lines, -11 removed lines

@@ -51,7 +51,7 @@ 
     /**
      * @var array
      */
-    public $constraints  = [];
+    public $constraints = [];
 
 
     public $acceptedResourceRoles = null;
@@ -103,47 +103,47 @@ 
         {
             foreach ($aData['portDefinitions'] as $portDefinition)
             {
-                $this->portDefinitions[] = new PortDefinition((array)$portDefinition);
+                $this->portDefinitions[] = new PortDefinition((array) $portDefinition);
             }
         }
 
         if (isset($aData['container']))
         {
-            $this->container = new Container((array)$aData['container']);
+            $this->container = new Container((array) $aData['container']);
         }
 
         if (isset($aData['healthChecks']))
         {
-            foreach($aData['healthChecks'] as $healthCheck)
+            foreach ($aData['healthChecks'] as $healthCheck)
             {
-                $this->healthChecks[] = new HealthCheck((array)$healthCheck);
+                $this->healthChecks[] = new HealthCheck((array) $healthCheck);
             }
         }
 
         if (isset($aData['upgradeStrategy']))
         {
-            $this->upgradeStrategy = new UpgradeStrategy((array)$aData['upgradeStrategy']);
+            $this->upgradeStrategy = new UpgradeStrategy((array) $aData['upgradeStrategy']);
         } else {
             $this->upgradeStrategy = new UpgradeStrategy();
         }
 
         if (isset($aData['ipAddress']))
         {
-            $this->ipAddress = new IpAddress((array)$aData['ipAddress']);
+            $this->ipAddress = new IpAddress((array) $aData['ipAddress']);
         }
 
         if (isset($aData['env']))
         {
-            $this->env =  (object) $aData['env'];
+            $this->env = (object) $aData['env'];
         } else {
-            $this->env = (object)[];
+            $this->env = (object) [];
         }
 
         if (isset($aData['labels']))
         {
             $this->labels = (object) $aData['labels'];
         } else {
-            $this->labels = (object)[];
+            $this->labels = (object) [];
         }
         MarathonEntityUtils::setPropertyIfExist($aData, $this, 'constraints');
         MarathonEntityUtils::setPropertyIfExist($aData, $this, 'args');
@@ -161,7 +161,7 @@ 
         $_aRet = (array) $this;
         $_aRet = array_filter($_aRet, function($v, $k) {
             return !is_null($v) || empty($v);
-        }, ARRAY_FILTER_USE_BOTH );
+        }, ARRAY_FILTER_USE_BOTH);
         return $_aRet;
     }