notamedia / console-jedi

src/Agent/AgentHelper.php

Spacing +2 added lines, -2 removed lines

@@ -36,11 +36,11 @@
                     throw new ArgumentTypeException('callChain', 'array');
                 }
 
-                $chain .= '->' . $method . '(' . static::convertArgsToString($methodArgs) . ')';
+                $chain .= '->'.$method.'('.static::convertArgsToString($methodArgs).')';
             }
         }
 
-        return '\\' . $className . '::agent(' . static::convertArgsToString($args) . ')' . $chain . ';';
+        return '\\'.$className.'::agent('.static::convertArgsToString($args).')'.$chain.';';
     }
 
     protected static function convertArgsToString(array $args)

src/Agent/AgentTask.php

Spacing +5 added lines, -5 removed lines

@@ -117,7 +117,7 @@ 
      */
     public function setInterval($seconds)
     {
-        $this->interval = (int)$seconds;
+        $this->interval = (int) $seconds;
 
         return $this;
     }
@@ -131,7 +131,7 @@ 
      */
     public function setPeriodically($periodically)
     {
-        $this->periodically = (bool)$periodically;
+        $this->periodically = (bool) $periodically;
 
         return $this;
     }
@@ -145,7 +145,7 @@ 
      */
     public function setActive($active)
     {
-        $this->active = (bool)$active;
+        $this->active = (bool) $active;
 
         return $this;
     }
@@ -173,7 +173,7 @@ 
      */
     public function setSort($sort)
     {
-        $this->sort = (int)$sort;
+        $this->sort = (int) $sort;
 
         return $this;
     }
@@ -187,7 +187,7 @@ 
      */
     public function setUserId($userId)
     {
-        $this->userId = (int)$userId;
+        $this->userId = (int) $userId;
 
         return $this;
     }

src/Application/Command/InitCommand.php

Spacing +16 added lines, -16 removed lines

@@ -84,7 +84,7 @@ 
      */
     protected function initialize(InputInterface $input, OutputInterface $output)
     {
-        $this->tmplDir = __DIR__ . '/../../../tmpl';
+        $this->tmplDir = __DIR__.'/../../../tmpl';
         $this->questionHelper = $this->getHelper('question');
 
         parent::initialize($input, $output);
@@ -100,7 +100,7 @@ 
         $this->createEnvironmentsDir($input, $output);
         $this->createConfiguration($input, $output);
 
-        $output->writeln('<info>' . static::COMPLETED_LOGO . '</info>');
+        $output->writeln('<info>'.static::COMPLETED_LOGO.'</info>');
     }
 
     /**
@@ -111,14 +111,14 @@ 
      */
     protected function createEnvironmentsDir(InputInterface $input, OutputInterface $output)
     {
-        $targetDir = getcwd() . '/' . $this->envDir;
-        $tmplDir = $this->tmplDir . '/environments';
+        $targetDir = getcwd().'/'.$this->envDir;
+        $tmplDir = $this->tmplDir.'/environments';
 
         $output->writeln('  - Environment settings');
 
         if (file_exists($targetDir)) {
             $question = new ConfirmationQuestion(
-                '    <error>Directory ' . $targetDir . ' already exists</error>' . PHP_EOL
+                '    <error>Directory '.$targetDir.' already exists</error>'.PHP_EOL
                 . '    <info>Overwrite? [Y/n]</info> ',
                 true,
                 '/^(y|j)/i'
@@ -134,7 +134,7 @@ 
         $iterator = new \RecursiveIteratorIterator($tmplIterator, \RecursiveIteratorIterator::SELF_FIRST);
 
         foreach ($iterator as $item) {
-            $itemPath = $targetDir . '/' . $iterator->getSubPathName();
+            $itemPath = $targetDir.'/'.$iterator->getSubPathName();
 
             if ($item->isDir()) {
                 $fs->mkdir($itemPath);
@@ -143,7 +143,7 @@ 
             }
         }
 
-        $output->writeln('    Created directory settings of environments: <comment>' . $targetDir . '</comment>');
+        $output->writeln('    Created directory settings of environments: <comment>'.$targetDir.'</comment>');
     }
 
     /**
@@ -154,13 +154,13 @@ 
      */
     protected function createConfiguration(InputInterface $input, OutputInterface $output)
     {
-        $path = $this->getApplication()->getRoot() . '/.jedi.php';
+        $path = $this->getApplication()->getRoot().'/.jedi.php';
 
         $output->writeln('  - Configuration');
 
         if (file_exists($path)) {
             $question = new ConfirmationQuestion(
-                '    <error>Configuration file ' . $path . ' already exists</error>' . PHP_EOL
+                '    <error>Configuration file '.$path.' already exists</error>'.PHP_EOL
                 . '    <info>Overwrite? [Y/n]</info> ',
                 true,
                 '/^(y|j)/i'
@@ -174,20 +174,20 @@ 
         $fs = new Filesystem();
 
         $question = new Question('    <info>Enter path to web directory relative to '
-            . $this->getApplication()->getRoot() . ':</info> ' . PHP_EOL
-            . '    (or do not specify if you are already in the web directory)' . PHP_EOL);
+            . $this->getApplication()->getRoot().':</info> '.PHP_EOL
+            . '    (or do not specify if you are already in the web directory)'.PHP_EOL);
 
-        $question->setValidator(function ($answer) use ($fs) {
+        $question->setValidator(function($answer) use ($fs) {
             $path = $answer;
 
             if ($answer === null) {
                 $path = $this->getApplication()->getRoot();
             } elseif (!$fs->isAbsolutePath($answer)) {
-                $path = $this->getApplication()->getRoot() . '/' . $answer;
+                $path = $this->getApplication()->getRoot().'/'.$answer;
             }
 
             if (!is_dir($path)) {
-                throw new \RuntimeException('Directory "' . $path . '" is missing');
+                throw new \RuntimeException('Directory "'.$path.'" is missing');
             }
 
             return $answer;
@@ -195,7 +195,7 @@ 
 
         $webDir = $this->questionHelper->ask($input, $output, $question);
 
-        $content = file_get_contents($this->tmplDir . '/.jedi.php');
+        $content = file_get_contents($this->tmplDir.'/.jedi.php');
         $content = str_replace(
             ['%web-dir%', '%env-dir%'],
             [addslashes($webDir), addslashes($this->envDir)],
@@ -203,6 +203,6 @@ 
         );
         $fs->dumpFile($path, $content);
 
-        $output->writeln('    Created configuration file of application <comment>' . $path . '</comment>');
+        $output->writeln('    Created configuration file of application <comment>'.$path.'</comment>');
     }
 }
\ No newline at end of file

src/Application/Exception/BitrixException.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         }
 
         if ($ex = $APPLICATION->GetException()) {
-            throw new static($message ? $message . ': ' . $ex->GetString() : $ex->GetString());
+            throw new static($message ? $message.': '.$ex->GetString() : $ex->GetString());
         } else {
             throw new static($message ? $message : 'Unknown exception');
         }

src/Application/Application.php

Spacing +15 added lines, -15 removed lines

@@ -101,22 +101,22 @@ 
         $exitCode = parent::doRun($input, $output);
 
         if ($this->getConfiguration() === null) {
-            $output->writeln(PHP_EOL . '<error>No configuration loaded.</error> '
+            $output->writeln(PHP_EOL.'<error>No configuration loaded.</error> '
                 . 'Please run <info>init</info> command first');
         } else {
             switch ($this->getBitrixStatus()) {
                 case static::BITRIX_STATUS_UNAVAILABLE:
-                    $output->writeln(PHP_EOL . sprintf('<error>No Bitrix kernel found in %s.</error> '
+                    $output->writeln(PHP_EOL.sprintf('<error>No Bitrix kernel found in %s.</error> '
                             . 'Please run <info>env:init</info> command to configure', $this->getDocumentRoot()));
                     break;
 
                 case static::BITRIX_STATUS_NO_DB_CONNECTION:
-                    $output->writeln(PHP_EOL . '<error>Bitrix database connection is unavailable.</error>');
+                    $output->writeln(PHP_EOL.'<error>Bitrix database connection is unavailable.</error>');
                     break;
 
                 case static::BITRIX_STATUS_COMPLETE:
                     if ($this->getCommandName($input) === null) {
-                        $output->writeln(PHP_EOL . sprintf('Using Bitrix <info>kernel v%s</info>.</info>', SM_VERSION),
+                        $output->writeln(PHP_EOL.sprintf('Using Bitrix <info>kernel v%s</info>.</info>', SM_VERSION),
                             OutputInterface::VERBOSITY_VERY_VERBOSE);
                     }
                     break;
@@ -168,7 +168,7 @@ 
         $commands = [];
 
         foreach (ModuleManager::getInstalledModules() as $module) {
-            $cliFile = getLocalPath('modules/' . $module['ID'] . '/cli.php');
+            $cliFile = getLocalPath('modules/'.$module['ID'].'/cli.php');
 
             if ($cliFile === false) {
                 continue;
@@ -176,7 +176,7 @@ 
                 continue;
             }
 
-            $config = include_once $this->getDocumentRoot() . $cliFile;
+            $config = include_once $this->getDocumentRoot().$cliFile;
 
             if (isset($config['commands']) && is_array($config['commands'])) {
                 $commands = array_merge($commands, $config['commands']);
@@ -204,7 +204,7 @@ 
         $this->configuration = include $path;
 
         if (!is_array($this->configuration)) {
-            throw new ConfigurationException('Configuration file ' . $path . ' must return an array');
+            throw new ConfigurationException('Configuration file '.$path.' must return an array');
         }
 
         $filesystem = new Filesystem();
@@ -212,7 +212,7 @@ 
         if ($filesystem->isAbsolutePath($this->configuration['web-dir'])) {
             $this->setDocumentRoot($this->configuration['web-dir']);
         } else {
-            $this->setDocumentRoot($this->getRoot() . '/' . $this->configuration['web-dir']);
+            $this->setDocumentRoot($this->getRoot().'/'.$this->configuration['web-dir']);
         }
 
         if (!is_dir($_SERVER['DOCUMENT_ROOT'])) {
@@ -258,7 +258,7 @@ 
             /** @noinspection PhpUnusedLocalVariableInspection */
             $DB, $DBType, $DBHost, $DBLogin, $DBPassword, $DBName, $DBDebug, $DBDebugToFile, $APPLICATION, $USER, $DBSQLServerType;
 
-            require_once $_SERVER['DOCUMENT_ROOT'] . '/bitrix/modules/main/include/prolog_before.php';
+            require_once $_SERVER['DOCUMENT_ROOT'].'/bitrix/modules/main/include/prolog_before.php';
 
             if (defined('B_PROLOG_INCLUDED') && B_PROLOG_INCLUDED === true) {
                 $this->bitrixStatus = static::BITRIX_STATUS_COMPLETE;
@@ -278,8 +278,8 @@ 
     public function checkBitrix()
     {
         if (
-            !is_file($_SERVER['DOCUMENT_ROOT'] . '/bitrix/.settings.php')
-            && !is_file($_SERVER['DOCUMENT_ROOT'] . '/bitrix/.settings_extra.php')
+            !is_file($_SERVER['DOCUMENT_ROOT'].'/bitrix/.settings.php')
+            && !is_file($_SERVER['DOCUMENT_ROOT'].'/bitrix/.settings_extra.php')
         ) {
             return false;
         }
@@ -323,7 +323,7 @@ 
 
         $this->initializeBitrix();
 
-        spl_autoload_register(function ($className) {
+        spl_autoload_register(function($className) {
             $file = ltrim($className, "\\");
             $file = strtr($file, Loader::ALPHA_UPPER, Loader::ALPHA_LOWER);
             $file = str_replace('\\', '/', $file);
@@ -342,16 +342,16 @@ 
                 return false;
             }
 
-            $module = array_shift($arFile) . '.' . array_shift($arFile);
+            $module = array_shift($arFile).'.'.array_shift($arFile);
 
             if (!Loader::includeModule($module)) {
                 return false;
             }
 
-            $path = getLocalPath('/modules/' . $module . '/' . implode('/', $arFile) . '.php');
+            $path = getLocalPath('/modules/'.$module.'/'.implode('/', $arFile).'.php');
 
             if ($path !== false) {
-                include_once $this->getDocumentRoot() . $path;
+                include_once $this->getDocumentRoot().$path;
             }
         });
     }

src/Environment/Command/InitCommand.php

Spacing +19 added lines, -19 removed lines

@@ -73,13 +73,13 @@ 
             throw new \Exception('Config env-dir is missing');
         }
 
-        $dir = $this->getApplication()->getRoot() . '/' . $this->getApplication()->getConfiguration()['env-dir'];
+        $dir = $this->getApplication()->getRoot().'/'.$this->getApplication()->getConfiguration()['env-dir'];
 
         if (!is_dir($dir)) {
-            throw new \Exception('Directory ' . $dir . ' is missing');
+            throw new \Exception('Directory '.$dir.' is missing');
         }
 
-        $environments = include $dir . '/index.php';
+        $environments = include $dir.'/index.php';
 
         if (!is_array($environments)) {
             throw new \Exception('File with description of environments is missing');
@@ -107,8 +107,8 @@ 
             throw new \Exception('Environment path not found!');
         }
 
-        $this->dir = $dir . '/' . $environments[$code]['path'];
-        $this->config = include $this->dir . '/config.php';
+        $this->dir = $dir.'/'.$environments[$code]['path'];
+        $this->config = include $this->dir.'/config.php';
     }
 
     /**
@@ -121,10 +121,10 @@ 
         }
 
         foreach ($this->config as $config => $settings) {
-            $method = 'configure' . ucfirst($config);
+            $method = 'configure'.ucfirst($config);
 
             if (!in_array($method, $this->bootstrap) && method_exists($this, $method)) {
-                $output->writeln('<comment>Setup "' . $config . '"</comment>');
+                $output->writeln('<comment>Setup "'.$config.'"</comment>');
                 $this->$method($input, $output, $settings);
             }
         }
@@ -150,7 +150,7 @@ 
                 continue;
             }
 
-            $itemPath = $this->getApplication()->getRoot() . '/' . $iterator->getSubPathName();
+            $itemPath = $this->getApplication()->getRoot().'/'.$iterator->getSubPathName();
 
             if ($item->isDir()) {
                 $fs->mkdir($itemPath);
@@ -158,7 +158,7 @@ 
                 $fs->copy($item, $itemPath, true);
             }
 
-            $output->writeln('   ' . $itemPath);
+            $output->writeln('   '.$itemPath);
         }
     }
 
@@ -186,8 +186,8 @@ 
             throw new \InvalidArgumentException('Config "licenseKey" must be string type.');
         }
 
-        $licenseFileContent = "<" . "? $" . "LICENSE_KEY = \"" . EscapePHPString($licenseKey) . "\"; ?" . ">";
-        File::putFileContents(Application::getDocumentRoot() . BX_ROOT . '/license_key.php', $licenseFileContent);
+        $licenseFileContent = "<"."? $"."LICENSE_KEY = \"".EscapePHPString($licenseKey)."\"; ?".">";
+        File::putFileContents(Application::getDocumentRoot().BX_ROOT.'/license_key.php', $licenseFileContent);
     }
 
     /**
@@ -225,7 +225,7 @@ 
         $bar->setFormat('verbose');
 
         foreach ($modules as $moduleName) {
-            $message = "\r" . '   module:load ' . $moduleName . ': ';
+            $message = "\r".'   module:load '.$moduleName.': ';
 
             try {
                 if (isset($bar)) {
@@ -236,15 +236,15 @@ 
                 (new Module($moduleName))->load()->register();
 
                 $bar->clear();
-                $output->writeln($message . '<info>ok</info>');
+                $output->writeln($message.'<info>ok</info>');
             } catch (ModuleInstallException $e) {
                 $bar->clear();
                 $output->writeln($e->getMessage(), OutputInterface::VERBOSITY_VERBOSE);
-                $output->writeln($message . '<comment>not registered</comment> (install it in admin panel)');
+                $output->writeln($message.'<comment>not registered</comment> (install it in admin panel)');
             } catch (ModuleException $e) {
                 $bar->clear();
                 $output->writeln($e->getMessage(), OutputInterface::VERBOSITY_VERBOSE);
-                $output->writeln($message . '<error>FAILED</error>');
+                $output->writeln($message.'<error>FAILED</error>');
             }
             $bar->advance();
         }
@@ -309,13 +309,13 @@ 
                     $memcache->Resume($serverId);
                 } else {
                     $exception = $APPLICATION->GetException();
-                    $message = 'Invalid memcache config with index ' . $index;
+                    $message = 'Invalid memcache config with index '.$index;
 
                     if ($exception->GetString()) {
                         $message = str_replace('<br>', "\n", $exception->GetString());
                     }
 
-                    $output->writeln('<error>' . $message . '</error>');
+                    $output->writeln('<error>'.$message.'</error>');
                 }
             }
         }
@@ -344,12 +344,12 @@ 
                     if (isset($value['value']) && isset($value['siteId'])) {
                         Option::set($module, $code, $value['value'], $value['siteId']);
                     } else {
-                        $output->writeln('<error>Invalid option for module "' . $module . '" with code "' . $code . '"</error>');
+                        $output->writeln('<error>Invalid option for module "'.$module.'" with code "'.$code.'"</error>');
                     }
                 } else {
                     Option::set($module, $code, $value);
 
-                    $output->writeln('   option: "' . $code . '", module: "' . $module . '"');
+                    $output->writeln('   option: "'.$code.'", module: "'.$module.'"');
                 }
             }
         }

src/Module/Module.php

Spacing +14 added lines, -14 removed lines

@@ -108,7 +108,7 @@ 
     public function register()
     {
         if (!$this->isRegistered()) {
-            $moduleObject =& $this->getObject();
+            $moduleObject = & $this->getObject();
 
             /**
              * It's important to check if module class defines InstallDB method (it must register module)
@@ -123,7 +123,7 @@ 
 
             if (!$moduleObject->InstallDB() && BitrixException::hasException()) {
                 throw new Exception\ModuleInstallException(
-                    get_class($moduleObject) . '::InstallDB() returned false',
+                    get_class($moduleObject).'::InstallDB() returned false',
                     $this->name
                 );
             }
@@ -133,7 +133,7 @@ 
             /** @noinspection PhpVoidFunctionResultUsedInspection */
             if (!$moduleObject->InstallFiles() && BitrixException::hasException()) {
                 throw new Exception\ModuleInstallException(
-                    get_class($moduleObject) . '::InstallFiles() returned false',
+                    get_class($moduleObject).'::InstallFiles() returned false',
                     $this->name
                 );
             }
@@ -158,7 +158,7 @@ 
     {
         if (!$this->isRegistered()) {
             if (!$this->exist()) {
-                require_once($_SERVER["DOCUMENT_ROOT"] . '/bitrix/modules/main/classes/general/update_client_partner.php');
+                require_once($_SERVER["DOCUMENT_ROOT"].'/bitrix/modules/main/classes/general/update_client_partner.php');
 
                 if (!\CUpdateClientPartner::LoadModuleNoDemand(
                     $this->getName(),
@@ -199,7 +199,7 @@ 
             /** @noinspection PhpVoidFunctionResultUsedInspection */
             if (!$moduleObject->UnInstallFiles() && BitrixException::hasException()) {
                 throw new Exception\ModuleUninstallException(
-                    get_class($moduleObject) . '::UnInstallFiles() returned false',
+                    get_class($moduleObject).'::UnInstallFiles() returned false',
                     $this->name
                 );
             }
@@ -209,7 +209,7 @@ 
             /** @noinspection PhpVoidFunctionResultUsedInspection */
             if (!$moduleObject->UnInstallDB() && BitrixException::hasException()) {
                 throw new Exception\ModuleUninstallException(
-                    get_class($moduleObject) . '::UnInstallFiles() returned false',
+                    get_class($moduleObject).'::UnInstallFiles() returned false',
                     $this->name
                 );
             }
@@ -231,10 +231,10 @@ 
             $this->unRegister();
         }
 
-        $path = getLocalPath('modules/' . $this->getName());
+        $path = getLocalPath('modules/'.$this->getName());
 
         if ($path) {
-            (new Filesystem())->remove($_SERVER['DOCUMENT_ROOT'] . $path);
+            (new Filesystem())->remove($_SERVER['DOCUMENT_ROOT'].$path);
         }
 
         unset($this->object);
@@ -253,7 +253,7 @@ 
      */
     public function update(&$modulesUpdated = null)
     {
-        require_once($_SERVER["DOCUMENT_ROOT"] . '/bitrix/modules/main/classes/general/update_client_partner.php');
+        require_once($_SERVER["DOCUMENT_ROOT"].'/bitrix/modules/main/classes/general/update_client_partner.php');
 
         if (!$this->isThirdParty()) {
             throw new Exception\ModuleUpdateException('Kernel module updates are currently not supported.',
@@ -294,14 +294,14 @@ 
         $updateDir = null;
 
         if (!\CUpdateClientPartner::UnGzipArchive($updateDir, $errorMessage, true)) {
-            throw new Exception\ModuleUpdateException('[CL02] UnGzipArchive failed. ' . $errorMessage,
+            throw new Exception\ModuleUpdateException('[CL02] UnGzipArchive failed. '.$errorMessage,
                 $this->getName());
         }
 
         $this->validateUpdate($updateDir);
 
         if (isset($updateDescription["DATA"]["#"]["NOUPDATES"])) {
-            \CUpdateClientPartner::ClearUpdateFolder($_SERVER["DOCUMENT_ROOT"] . "/bitrix/updates/" . $updateDir);
+            \CUpdateClientPartner::ClearUpdateFolder($_SERVER["DOCUMENT_ROOT"]."/bitrix/updates/".$updateDir);
             return false;
         }
 
@@ -320,7 +320,7 @@ 
                 }
             }
         } else {
-            throw new Exception\ModuleUpdateException('[CL04] UpdateStepModules failed. ' . $errorMessage,
+            throw new Exception\ModuleUpdateException('[CL04] UpdateStepModules failed. '.$errorMessage,
                 $this->getName());
         }
 
@@ -336,14 +336,14 @@ 
     {
         $errorMessage = null;
         if (!\CUpdateClientPartner::CheckUpdatability($updateDir, $errorMessage)) {
-            throw new Exception\ModuleUpdateException('[CL03] CheckUpdatability failed. ' . $errorMessage,
+            throw new Exception\ModuleUpdateException('[CL03] CheckUpdatability failed. '.$errorMessage,
                 $this->getName());
         }
 
         if (isset($updateDescription["DATA"]["#"]["ERROR"])) {
             $errorMessage = "";
             foreach ($updateDescription["DATA"]["#"]["ERROR"] as $errorDescription) {
-                $errorMessage .= "[" . $errorDescription["@"]["TYPE"] . "] " . $errorDescription["#"];
+                $errorMessage .= "[".$errorDescription["@"]["TYPE"]."] ".$errorDescription["#"];
             }
             throw new Exception\ModuleUpdateException($errorMessage, $this->getName());
         }

src/Module/Command/ModuleCommand.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         if (in_array($this->getName(), ['module:register', 'module:unregister'])
             && $module->isThirdParty() && !$input->getOption('confirm-thirdparty')
         ) {
-            $output->writeln($module->isThirdParty() . ' is not a kernel module. Correct operation cannot be guaranteed for third-party modules!');
+            $output->writeln($module->isThirdParty().' is not a kernel module. Correct operation cannot be guaranteed for third-party modules!');
         }
     }
 

src/Module/Exception/ModuleException.php

Spacing +1 added lines, -1 removed lines

@@ -17,6 +17,6 @@
      */
     public function __construct($message, $moduleName, \Exception $previous = null)
     {
-        parent::__construct("[$moduleName] " . $message, 0, $previous);
+        parent::__construct("[$moduleName] ".$message, 0, $previous);
     }
 }
\ No newline at end of file