itrukhin / bxconsole

src/Cron.php

Spacing +25 added lines, -25 removed lines

@@ -36,7 +36,7 @@ 
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         $logger = EnvHelper::getLogger('bx_cron');
-        if($logger) {
+        if ($logger) {
             $this->setLogger($logger);
         }
 
@@ -50,20 +50,20 @@ 
 
         $workedJobs = [];
 
-        if(!empty($jobs)) {
+        if (!empty($jobs)) {
 
             $lockStore = new FlockStore(pathinfo(EnvHelper::getCrontabFile(), PATHINFO_DIRNAME));
             $lockFactory = new LockFactory($lockStore);
-            if($this->logger) {
+            if ($this->logger) {
                 $lockFactory->setLogger($this->logger);
             }
 
-            foreach($jobs as $cmd => $job) {
+            foreach ($jobs as $cmd => $job) {
 
-                if($this->isActualJob($job)) {
+                if ($this->isActualJob($job)) {
 
                     $lock = $lockFactory->createLock($this->getLockName($cmd), self::EXEC_TIMEOUT);
-                    if($lock->acquire()) {
+                    if ($lock->acquire()) {
 
                         $workedJobs[$cmd] = $job;
 
@@ -74,11 +74,11 @@ 
                             $timeStart = microtime(true);
                             $returnCode = $command->run($cmdInput, $output);
 
-                            if(!$returnCode) {
+                            if (!$returnCode) {
 
                                 $workedJobs[$cmd]['status'] = self::EXEC_STATUS_SUCCESS;
                                 $msg = sprintf("%s: SUCCESS [%.2f s]", $cmd, microtime(true) - $timeStart);
-                                if($this->logger) {
+                                if ($this->logger) {
                                     $this->logger->alert($msg);
                                 }
                                 $output->writeln(PHP_EOL . $msg);
@@ -88,7 +88,7 @@ 
                                 $workedJobs[$cmd]['status'] = self::EXEC_STATUS_ERROR;
                                 $workedJobs[$cmd]['error_code'] = $returnCode;
                                 $msg = sprintf("%s: ERROR [%.2f s]", $cmd, microtime(true) - $timeStart);
-                                if($this->logger) {
+                                if ($this->logger) {
                                     $this->logger->alert($msg);
                                 }
                                 $output->writeln(PHP_EOL . $msg);
@@ -98,7 +98,7 @@ 
 
                             $workedJobs[$cmd]['status'] = self::EXEC_STATUS_ERROR;
                             $workedJobs[$cmd]['error'] = $e->getMessage();
-                            if($this->logger) {
+                            if ($this->logger) {
                                 $this->logger->error($e, ['command' => $cmd]);
                             }
                             $output->writeln(PHP_EOL . 'ERROR: ' . $e->getMessage());
@@ -117,7 +117,7 @@ 
                         break;
 
                     } else {
-                        if($this->logger) {
+                        if ($this->logger) {
                             $this->logger->warning($cmd . " is locked");
                         }
                     }
@@ -137,15 +137,15 @@ 
 
         $period = intval($job['period']);
 
-        if($period > 0) {
-            if($period < $this->minAgentPeriod) {
+        if ($period > 0) {
+            if ($period < $this->minAgentPeriod) {
                 $job['orig_period'] = $period;
                 $period = $job['period'] = $this->minAgentPeriod;
             }
-            if(time() - $job['last_exec'] >= $period) {
+            if (time() - $job['last_exec'] >= $period) {
                 return true;
             }
-        } else if(!empty($job['times'])) {
+        } else if (!empty($job['times'])) {
             //TODO:
         }
 
@@ -160,9 +160,9 @@ 
         $commands = $app->all();
 
         $selfCommands = [];
-        foreach($commands as $command) {
+        foreach ($commands as $command) {
             /** @var BxCommand $command */
-            if($command instanceof BxCommand) {
+            if ($command instanceof BxCommand) {
                 $name = $command->getName();
                 $selfCommands[$name] = [
                     'object' => $command,
@@ -172,12 +172,12 @@ 
 
         $agents = [];
         $reader = new AnnotationReader();
-        foreach($selfCommands as $cmd => $selfCommand) {
+        foreach ($selfCommands as $cmd => $selfCommand) {
             $reflectionClass = new \ReflectionClass($selfCommand['object']);
             $annotations = $reader->getClassAnnotations($reflectionClass);
 
-            foreach($annotations as $annotation) {
-                if($annotation instanceof Agent) {
+            foreach ($annotations as $annotation) {
+                if ($annotation instanceof Agent) {
                     $agents[$cmd] = $annotation->toArray();
                 }
             }
@@ -185,9 +185,9 @@ 
 
         $crontab = $this->getCronTab();
 
-        if(is_array($crontab)) {
-            foreach($crontab as $cmd => $job) {
-                if(is_array($job) && isset($agents[$cmd])) {
+        if (is_array($crontab)) {
+            foreach ($crontab as $cmd => $job) {
+                if (is_array($job) && isset($agents[$cmd])) {
                     $agents[$cmd] = array_merge($job, $agents[$cmd]);
                 }
             }
@@ -215,7 +215,7 @@ 
         $fh = fopen($filename, 'c');
         if (flock($fh, LOCK_EX)) {
             ftruncate($fh, 0);
-            if(!fwrite($fh, json_encode($agents, JSON_PRETTY_PRINT))) {
+            if (!fwrite($fh, json_encode($agents, JSON_PRETTY_PRINT))) {
                 throw new \Exception('Unable to write BX_CRONTAB : ' . $filename);
             }
         }
@@ -233,7 +233,7 @@ 
         $filename = EnvHelper::getCrontabFile();
 
         $fh = fopen($filename, 'r');
-        if(flock($fh, LOCK_SH)) {
+        if (flock($fh, LOCK_SH)) {
             $data = @fread($fh, filesize($filename));
             $cronTab = json_decode($data, true);
         }