trivago / chapi

src/Component/Http/HttpGuzzlClient.php

Braces +1 added lines, -2 removed lines

@@ -54,8 +54,7 @@
         {
             $_oResponse = $this->oGuzzelClient->request('GET', $sUrl, $_aRequestOptions);
             return new HttpGuzzlResponse($_oResponse);
-        }
-        catch (\Exception $oException)
+        } catch (\Exception $oException)
         {
             throw new HttpConnectionException(
                 sprintf('Can\'t get response from "%s"', $this->oGuzzelClient->getConfig('base_uri') . $sUrl),

src/BusinessCase/JobManagement/MarathonStoreJobBusinessCase.php

Braces +7 added lines, -10 removed lines

@@ -100,8 +100,7 @@ 
                         ));
                         return false;
                     }
-                }
-                catch(\Exception $e)
+                } catch(\Exception $e)
                 {
                     $this->oLogger->error(sprintf(
                         "Job %s cannot be added to remote : %s",$sAppId, $e->getMessage()
@@ -268,8 +267,7 @@ 
         if (empty($aJobNames))
         {
             $_aApps = $this->oJobRepositoryRemote->getJobs();
-        }
-        else
+        } else
         {
             $_aApps = [];
             foreach ($aJobNames as $_sAppName)
@@ -282,11 +280,12 @@ 
         foreach ($_aApps as $_oAppRemote)
         {
             $_oAppLocal = $this->oJobRepositoryLocal->getJob($_oAppRemote->getKey());
-            if (null == $_oAppLocal) // add
+            if (null == $_oAppLocal) {
+                // add
             {
                 $this->addJobInLocalRepository($_oAppRemote);
             }
-            else // update
+            } else // update
             {
                 $this->updateJobInLocalRepository($_oAppRemote, $bForceOverwrite);
             }
@@ -301,8 +300,7 @@ 
                 'App %s stored in local repository',
                 $oAppRemote->getKey()
             ));
-        }
-        else {
+        } else {
             $this->oLogger->error(sprintf(
                 'Failed to store %s in local repository',
                 $oAppRemote->getKey()
@@ -331,8 +329,7 @@ 
                     'App %s is updated in local repository',
                     $oAppRemote->getKey()
                 ));
-            }
-            else {
+            } else {
                 $this->oLogger->error(sprintf(
                     'Failed to update app %s in local repository',
                     $oAppRemote->getKey()

src/BusinessCase/JobManagement/ChronosStoreJobBusinessCase.php

Braces +4 added lines, -8 removed lines

@@ -101,8 +101,7 @@ 
         if (empty($aJobNames))
         {
             $_aChronosJobs = $this->oJobRepositoryChronos->getJobs();
-        }
-        else
+        } else
         {
             $_aChronosJobs = [];
             foreach ($aJobNames as $_sJobName)
@@ -124,8 +123,7 @@ 
                         'Job "%s" successfully stored in local repository',
                         $_oJobEntity->name
                     ));
-                }
-                else
+                } else
                 {
                     $this->oLogger->error(sprintf(
                         'Failed to store job "%s" in local repository',
@@ -156,8 +154,7 @@ 
                         'Job "%s" successfully updated in local repository',
                         $_oJobEntity->name
                     ));
-                }
-                else
+                } else
                 {
                     $this->oLogger->error(sprintf(
                         'Failed to update job "%s" in local repository',
@@ -245,8 +242,7 @@ 
             if ($this->oJobComparisonBusinessCase->hasSameJobType($_oJobEntityLocal, $_oJobEntityChronos))
             {
                 $_bHasUpdatedJob = $this->oJobRepositoryChronos->updateJob($_oJobEntityLocal);
-            }
-            else
+            } else
             {
                 $_bHasUpdatedJob = (
                     $this->oJobRepositoryChronos->removeJob($_oJobEntityChronos->name)

src/BusinessCase/Comparison/ChronosJobComparisonBusinessCase.php

Braces +1 added lines, -2 removed lines

@@ -317,8 +317,7 @@
         {
             $_oDateTime = new \DateTime(str_replace('Z', '', $_oIso8601Entity->sStartTime));
             $_oDateTime->setTimezone(new \DateTimeZone($sTimeZone));
-        }
-        else
+        } else
         {
             $_oDateTime = new \DateTime($_oIso8601Entity->sStartTime);
         }

src/BusinessCase/Comparison/MarathonJobComparisonBusinessCase.php

Braces +3 added lines, -6 removed lines

@@ -250,12 +250,10 @@ 
     {
         if (is_array($mValueA) && is_array($mValueB)) {
             return $this->isArrayEqual($mValueA, $mValueB);
-        }
-        elseif (is_object($mValueA) && is_object($mValueB))
+        } elseif (is_object($mValueA) && is_object($mValueB))
         {
             return $this->isArrayEqual(get_object_vars($mValueA), get_object_vars($mValueB));
-        }
-        elseif ((is_scalar($mValueA) && is_scalar($mValueB)) || (is_null($mValueA) && is_null($mValueB)))
+        } elseif ((is_scalar($mValueA) && is_scalar($mValueB)) || (is_null($mValueA) && is_null($mValueB)))
         {
             return $mValueA == $mValueB;
         }
@@ -288,8 +286,7 @@ 
                 {
                     return false;
                 }
-            }
-            else
+            } else
             {
                 foreach ($aValuesB as $_mValueB)
                 {

src/Service/JobRepository/BridgeFileSystem.php

Braces +9 added lines, -8 removed lines

@@ -155,8 +155,7 @@ 
                 DIRECTORY_SEPARATOR,
                 $oJobEntity->getKey()
             );
-        }
-        else
+        } else
         {
             $_sJobPath = $oJobEntity->getKey();
         }
@@ -268,15 +267,19 @@ 
 
             if ($_aTemp)
             {
-                if (property_exists($_aTemp, "name")) // chronos
+                if (property_exists($_aTemp, "name")) {
+                    // chronos
                 {
                     $_aJobEntities[] = new ChronosJobEntity($_aTemp);
+                }
 
-                } else if (property_exists($_aTemp, "id")) //marathon
+                } else if (property_exists($_aTemp, "id")) {
+                    //marathon
                 {
                     foreach ($this->getMarathonEntitiesForConfig($_aTemp) as $_oApp)
                     {
                         $_aJobEntities[] = $_oApp;
+                }
                     }
                 } else {
                     throw new JobLoadException(
@@ -297,8 +300,7 @@ 
                     $_aJobs[] = $_oJobEntity;
                 }
 
-            }
-            else
+            } else
             {
                 throw new JobLoadException(
                     sprintf('Unable to load json job data from "%s". Please check if the json is valid.', $_sJobFilePath),
@@ -323,8 +325,7 @@ 
                 $this->aGroupedApps[] = $_oApp->id;
                 $_aRet[] = $_oGroupEntity;
             }
-        }
-        else
+        } else
         {
             $_aRet[] = new MarathonAppEntity($aEntityData);
         }

src/Service/JobRepository/JobRepository.php

Braces +3 added lines, -1 removed lines

@@ -99,10 +99,12 @@
     {
         if ($this->oRepositoryBridge->addJob($oJobEntity))
         {
-            if (!is_null($this->oJobCollection)) // if no collection inited the new job will init by chronos request
+            if (!is_null($this->oJobCollection)) {
+                // if no collection inited the new job will init by chronos request
             {
                 $this->oJobCollection->offsetSet($oJobEntity->getKey(), $oJobEntity);
             }
+            }
 
             return true;
         }

src/Commands/AbstractCommand.php

Braces +3 added lines, -1 removed lines

@@ -116,12 +116,14 @@
         if (
             !file_exists($this->getHomeDir() . DIRECTORY_SEPARATOR . 'parameters.yml')
             && !file_exists($this->getWorkingDir() . DIRECTORY_SEPARATOR . '.chapiconfig')
-        ) // one file have to exist
+        ) {
+            // one file have to exist
         {
             $this->oOutput->writeln(sprintf(
                 '<error>%s</error>',
                 'No parameter file found. Please run "configure" command for initial setup or add a local `.chapiconfig` to your working directory.'
             ));
+        }
             return false;
         }
 

src/Commands/InfoCommand.php

Braces +1 added lines, -2 removed lines

@@ -55,8 +55,7 @@
                 $_mValue = (!empty($_mValue))
                     ? json_encode($_mValue, JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES)
                     : $_sEmptyString;
-            }
-            elseif (is_bool($_mValue))
+            } elseif (is_bool($_mValue))
             {
                 $_mValue = (true === $_mValue)
                     ? 'true'