itrukhin / bxconsole

src/Bitrix/Loader.php

Spacing +2 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
      */
     public function initializeBitrix() {
 
-        if($this->checkBitrix()) {
+        if ($this->checkBitrix()) {
 
             /**
              * Declare global legacy variables
@@ -53,7 +53,7 @@ 
 
             $cliFile = getLocalPath('modules/' . $module['ID'] . '/cli.php');
 
-            if(!$cliFile) {
+            if (!$cliFile) {
                 continue;
             }
 

src/EnvHelper.php

Spacing +9 added lines, -9 removed lines

@@ -9,12 +9,12 @@ 
 
     public static function loadEnv() {
 
-        if(class_exists('\Symfony\Component\Dotenv\Dotenv')) {
+        if (class_exists('\Symfony\Component\Dotenv\Dotenv')) {
             $envFile = realpath(__DIR__ . '/../../../../.env');
-            if(!is_file($envFile)) {
+            if (!is_file($envFile)) {
                 $envFile = realpath(__DIR__ . '/../../../../../.env');
             }
-            if(is_file($envFile)) {
+            if (is_file($envFile)) {
                 try {
                     $env = new \Symfony\Component\Dotenv\Dotenv();
                     $env->load($envFile);
@@ -32,15 +32,15 @@ 
 
         $_SERVER['DOCUMENT_ROOT'] = realpath(__DIR__ . '/../../../../');
 
-        if(isset($_ENV['APP_DOCUMENT_ROOT']) && is_dir($_ENV['APP_DOCUMENT_ROOT'])) {
+        if (isset($_ENV['APP_DOCUMENT_ROOT']) && is_dir($_ENV['APP_DOCUMENT_ROOT'])) {
             $_SERVER['DOCUMENT_ROOT'] = $_ENV['APP_DOCUMENT_ROOT'];
             return $_SERVER['DOCUMENT_ROOT'];
         }
 
         $composerFile = realpath(__DIR__ . '/../../../../composer.json');
-        if(is_file($composerFile)) {
+        if (is_file($composerFile)) {
             $composerConfig = json_decode(file_get_contents($composerFile), true);
-            if(isset($composerConfig['extra']['document-root']) && is_dir($composerConfig['extra']['document-root'])) {
+            if (isset($composerConfig['extra']['document-root']) && is_dir($composerConfig['extra']['document-root'])) {
                 $_SERVER['DOCUMENT_ROOT'] = $composerConfig['extra']['document-root'];
                 return $_SERVER['DOCUMENT_ROOT'];
             }
@@ -55,10 +55,10 @@ 
      */
     public static function getLogger($channel) {
 
-        if(isset($_ENV['APP_LOG_CLASS']) && class_exists($_ENV['APP_LOG_CLASS'])) {
+        if (isset($_ENV['APP_LOG_CLASS']) && class_exists($_ENV['APP_LOG_CLASS'])) {
             $logClass = $_ENV['APP_LOG_CLASS'];
             $log = new $logClass($channel);
-            if($log instanceof LoggerInterface) {
+            if ($log instanceof LoggerInterface) {
                 return $log;
             }
         }
@@ -71,7 +71,7 @@ 
      */
     public static function getCrontabFile() {
 
-        if(isset($_ENV['BX_CRONTAB_FOLDER']) && $_ENV['BX_CRONTAB_FOLDER']) {
+        if (isset($_ENV['BX_CRONTAB_FOLDER']) && $_ENV['BX_CRONTAB_FOLDER']) {
             return rtrim($_ENV['BX_CRONTAB_FOLDER'], "/") . '/bx_crontab.json';
         }
 

src/Cron.php

Spacing +24 added lines, -24 removed lines

@@ -47,20 +47,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;
 
@@ -71,11 +71,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);
@@ -85,7 +85,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);
@@ -95,7 +95,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());
@@ -114,7 +114,7 @@ 
                         break;
 
                     } else {
-                        if($this->logger) {
+                        if ($this->logger) {
                             $this->logger->warning($cmd . " is locked");
                         }
                     }
@@ -134,15 +134,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:
         }
 
@@ -157,9 +157,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,
@@ -169,12 +169,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();
                 }
             }
@@ -182,9 +182,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]);
                 }
             }
@@ -212,7 +212,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);
             }
         }
@@ -230,7 +230,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);
         }

src/BxCommand.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         /** @var Command $annotation */
         $annotation = $this->getAnnotation(\App\BxConsole\Annotations\Command::class);
 
-        if($annotation) {
+        if ($annotation) {
             $this->setName($annotation->name);
             $this->setDescription($annotation->description);
             $this->setHelp($annotation->help);

src/Application.php

Spacing +3 added lines, -3 removed lines

@@ -25,8 +25,8 @@ 
 
         $this->isBitrixLoaded = $loader->initializeBitrix();
 
-        if($this->isBitrixLoaded) {
-            foreach($loader->getModulesCommands() as $command) {
+        if ($this->isBitrixLoaded) {
+            foreach ($loader->getModulesCommands() as $command) {
                 $this->add($command);
             }
         }
@@ -38,7 +38,7 @@ 
 
         $exitCode = parent::doRun($input, $output);
 
-        if($this->isBitrixLoaded) {
+        if ($this->isBitrixLoaded) {
             if ($this->getCommandName($input) === null) {
                 $output->writeln(PHP_EOL . sprintf('Using Bitrix <info>kernel v%s</info>.</info>', SM_VERSION),
                     OutputInterface::VERBOSITY_VERY_VERBOSE);