trivago / chapi

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/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'

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/BusinessCase/JobManagement/MarathonStoreJobBusinessCase.php

Braces +1 added lines, -2 removed lines

@@ -83,8 +83,7 @@
                         ));
                         return false;
                     }
-                }
-                catch(\Exception $e)
+                } catch(\Exception $e)
                 {
                     $this->oLogger->error(sprintf(
                         'Job %s cannot be added to remote : %s',$sAppId, $e->getMessage()

src/BusinessCase/JobManagement/ChronosStoreJobBusinessCase.php

Braces +1 added lines, -2 removed lines

@@ -151,8 +151,7 @@
             if ($this->oJobComparisonBusinessCase->hasSameJobType($_oJobEntityLocal, $_oJobEntityChronos))
             {
                 $_bHasUpdatedJob = $this->oJobRepositoryRemote->updateJob($_oJobEntityLocal);
-            }
-            else
+            } else
             {
                 $_bHasUpdatedJob = (
                     $this->oJobRepositoryRemote->removeJob($_oJobEntityChronos->getKey())

src/BusinessCase/JobManagement/AbstractStoreJobBusinessCase.php

Braces +3 added lines, -6 removed lines

@@ -52,8 +52,7 @@ 
         if (empty($aEntityNames))
         {
             $_aRemoteEntities = $this->oJobRepositoryRemote->getJobs();
-        }
-        else
+        } else
         {
             $_aRemoteEntities = [];
             foreach ($aEntityNames as $_sJobName)
@@ -85,8 +84,7 @@ 
                 'Entity %s stored in local repository',
                 $oAppRemote->getKey()
             ));
-        }
-        else {
+        } else {
             $this->oLogger->error(sprintf(
                 'Failed to store %s in local repository',
                 $oAppRemote->getKey()
@@ -116,8 +114,7 @@ 
                     'Entity %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/Comparison/MarathonJobComparisonBusinessCase.php

Braces +3 added lines, -6 removed lines

@@ -104,12 +104,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;
         }
@@ -142,8 +140,7 @@ 
                 {
                     return false;
                 }
-            }
-            else
+            } else
             {
                 foreach ($aValuesB as $_mValueB)
                 {