@@ -84,10 +84,12 @@ |
||
| 84 | 84 | { |
| 85 | 85 | if ($this->oRepositoryBridge->addJob($oJobEntity)) |
| 86 | 86 | { |
| 87 | - if (!is_null($this->oJobCollection)) // if no collection inited the new job will init by chronos request |
|
| 87 | + if (!is_null($this->oJobCollection)) { |
|
| 88 | + // if no collection inited the new job will init by chronos request |
|
| 88 | 89 | { |
| 89 | 90 | $this->oJobCollection->offsetSet($oJobEntity->name, $oJobEntity); |
| 90 | 91 | } |
| 92 | + } |
|
| 91 | 93 | |
| 92 | 94 | return true; |
| 93 | 95 | } |
@@ -49,8 +49,7 @@ |
||
| 49 | 49 | if (JobUtils::isWildcard($aJobNames)) |
| 50 | 50 | { |
| 51 | 51 | $_oJobIndexService->resetJobIndex(); |
| 52 | - } |
|
| 53 | - else |
|
| 52 | + } else |
|
| 54 | 53 | { |
| 55 | 54 | $_oJobIndexService->removeJobs($aJobNames); |
| 56 | 55 | } |
@@ -230,8 +230,7 @@ |
||
| 230 | 230 | // set path to job file map |
| 231 | 231 | $this->setJobFileToMap($_oJobEntity->name, $_sJobFilePath); |
| 232 | 232 | } |
| 233 | - } |
|
| 234 | - else |
|
| 233 | + } else |
|
| 235 | 234 | { |
| 236 | 235 | throw new JobLoadException( |
| 237 | 236 | sprintf('Unable to load json job data from "%s". Please check if the json is valid.', $_sJobFilePath), |
@@ -125,12 +125,14 @@ |
||
| 125 | 125 | if ( |
| 126 | 126 | !file_exists($this->getHomeDir() . DIRECTORY_SEPARATOR . 'parameters.yml') |
| 127 | 127 | AND !file_exists(getcwd() . DIRECTORY_SEPARATOR . '.chapiconfig') |
| 128 | - ) // one file have to exist |
|
| 128 | + ) { |
|
| 129 | + // one file have to exist |
|
| 129 | 130 | { |
| 130 | 131 | $this->oOutput->writeln(sprintf( |
| 131 | 132 | '<error>%s</error>', |
| 132 | 133 | 'No parameter file found. Please run "configure" command for initial setup or add a local `.chapiconfig` to your working directory.' |
| 133 | 134 | )); |
| 135 | + } |
|
| 134 | 136 | return false; |
| 135 | 137 | } |
| 136 | 138 | |