trivago / chapi

src/Entity/Marathon/AppEntity/IpAddress.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
         }
         MarathonEntityUtils::setAllPossibleProperties($oData, $this);
 
-        if(isset($oData["groups"]))
+        if (isset($oData["groups"]))
         {
             $this->groups = $oData["groups"];
         }

src/Entity/Marathon/MarathonEntityUtils.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public static function setAllPossibleProperties($oData, $oTarget)
     {
-        foreach($oData as $attrName => $attrValue)
+        foreach ($oData as $attrName => $attrValue)
         {
             // dont set array or objects.
             // Because this would need further type information to properly set.

src/Entity/Marathon/AppSubEntityFactory.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         if (is_array($mData))
         {
             $ret = [];
-            foreach($mData as $data)
+            foreach ($mData as $data)
             {
                 $ret[] = new self::$sSubEntityMap[$sName]($data);
             }

src/Entity/Marathon/MarathonAppEntity.php

Spacing +11 added lines, -11 removed lines

@@ -32,7 +32,7 @@ 
     /**
      * @var PortDefinition[]
      */
-    public $portDefinitions =[];
+    public $portDefinitions = [];
 
     public $requirePorts = false;
 
@@ -50,7 +50,7 @@ 
     /**
      * @var array
      */
-    public $constraints  = [];
+    public $constraints = [];
 
 
     public $acceptedResourceRoles = null;
@@ -103,44 +103,44 @@ 
 
         if (isset($aData["portDefinitions"])) {
             foreach ($aData["portDefinitions"] as $portDefinition) {
-                $this->portDefinitions[] = new DockerPortMapping((array)$portDefinition);
+                $this->portDefinitions[] = new DockerPortMapping((array) $portDefinition);
             }
         }
 
         if (isset($aData["container"])) {
-            $this->container = new Container((array)$aData["container"]);
+            $this->container = new Container((array) $aData["container"]);
         }
 
         if (isset($aData["fetch"]))
         {
-            foreach($aData["fetch"] as $fetch) {
-                $this->fetch[] = new FetchUrl((array)$fetch);
+            foreach ($aData["fetch"] as $fetch) {
+                $this->fetch[] = new FetchUrl((array) $fetch);
             }
         }
 
         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"];
         }
 
         if (isset($aData["labels"]))

src/Component/RemoteClients/ChronosApiClient.php

Spacing +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
      */
     public function removeJob($sJobName)
     {
-        $_oResponse = $this->oHttpClient->delete('/scheduler/job/' . $sJobName);
+        $_oResponse = $this->oHttpClient->delete('/scheduler/job/'.$sJobName);
         return ($_oResponse->getStatusCode() == 204);
     }
 
@@ -91,7 +91,7 @@ 
      */
     public function getJobStats($sJobName)
     {
-        return $this->sendGetJsonRequest('/scheduler/job/stat/' . $sJobName);
+        return $this->sendGetJsonRequest('/scheduler/job/stat/'.$sJobName);
     }
 
     /**

src/BusinessCase/JobManagement/MarathonStoreJobBusinessCase.php

Spacing +5 added lines, -5 removed lines

@@ -89,7 +89,7 @@ 
             $_oJobEntityLocal = $this->oJobRepositoryLocal->getJob($sAppId);
 
             // check if dependency is satisfied
-            if ( $_oJobEntityLocal->isDependencyJob())
+            if ($_oJobEntityLocal->isDependencyJob())
             {
                 try {
                     $circular = $this->isDependencyCircular($_oJobEntityLocal, count($_oJobEntityLocal->dependencies));
@@ -101,10 +101,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;
                 }
@@ -150,7 +150,7 @@ 
         return !(count($arr) == count(array_unique($arr)));
     }
 
-    private function isDependencyCircular(JobEntityInterface $oEntity, $immediateChildren, &$path=[])
+    private function isDependencyCircular(JobEntityInterface $oEntity, $immediateChildren, &$path = [])
     {
         // Invariant: path will not have duplicates for acyclic dependency tree
         if ($this->hasDuplicates($path))
@@ -196,7 +196,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.
-            $immediateChildren = $immediateChildren -1;
+            $immediateChildren = $immediateChildren - 1;
             if ($immediateChildren == 0)
             {
                 array_pop($path);

src/BusinessCase/Comparison/ChronosJobComparisonBusinessCase.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
      */
     public function getRemoteMissingJobs()
     {
-        $_ret =  $this->getMissingJobsInCollectionA(
+        $_ret = $this->getMissingJobsInCollectionA(
             $this->oJobRepositoryChronos->getJobs(),
             $this->oJobRepositoryLocalChronos->getJobs()
         );

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)
         {
             // NOTE: only gets the first one.
             // does it make sense?
@@ -99,7 +99,7 @@ 
         }
         /** @var JobComparisonInterface $comparer */
         $comparer = null;
-        foreach($this->aComposites as $child)
+        foreach ($this->aComposites as $child)
         {
             if ($child->isJobAvailable($oJobEntityA->getKey()))
             {
@@ -123,7 +123,7 @@ 
      */
     public function isJobAvailable($sJobName)
     {
-        foreach($this->aComposites as $child) {
+        foreach ($this->aComposites as $child) {
             if ($child->isJobAvailable($sJobName))
             {
                 return true;

src/BusinessCase/Comparison/MarathonJobComparisonBusinessCase.php

Spacing +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
         $_aLocalJobs = $this->oLocalRepository->getJobs();
 
         /** @var JobEntityInterface $_oLocalJob */
-        foreach($_aLocalJobs as $_oLocalJob)
+        foreach ($_aLocalJobs as $_oLocalJob)
         {
             $_oRemoteJob = $this->oRemoteRepository->getJob($_oLocalJob->getKey());
             if (!$_oRemoteJob)
@@ -138,7 +138,7 @@ 
             $_aDifferences[$_sProperty] = $this->oDiffCompare->compare(
                 $_oRemoteJob->{$_sProperty},
                 $_oLocalJob->{$_sProperty}
-            ) ;
+            );
         }
 
         return $_aDifferences;