netresearch / kite

src/Task/ComposerTask.php

Doc Comments +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
     /**
      * Execute the command
      *
-     * @return mixed
+     * @return null|string
      */
     protected function executeCommand()
     {

Spacing +5 added lines, -5 removed lines

@@ -57,7 +57,7 @@ 
     {
         if ($name === 'command') {
             parent::offsetSet('composerCommand', $value);
-            $value = 'composer ' .  $value;
+            $value = 'composer '.$value;
         }
         parent::offsetSet($name, $value);
     }
@@ -84,19 +84,19 @@ 
     protected function getCommand()
     {
         $cmd = $this->get('command');
-        $argv = substr(parent::getCommand(), strlen($cmd)) . ' ';
+        $argv = substr(parent::getCommand(), strlen($cmd)).' ';
         if (strpos($argv, ' --no-ansi ') === false
             && strpos($argv, ' --ansi ') === false
             && $this->console->getOutput()->isDecorated()
         ) {
-            $argv = ' --ansi' . $argv;
+            $argv = ' --ansi'.$argv;
         }
         if (!$this->shouldExecute()
             && strpos($argv, ' --dry-run ') === false
         ) {
-            $argv = ' --dry-run' . $argv;
+            $argv = ' --dry-run'.$argv;
         }
-        return $cmd . rtrim($argv);
+        return $cmd.rtrim($argv);
     }
 }
 ?>

src/Workflow/RenderReference.php

Spacing +19 added lines, -19 removed lines

@@ -60,7 +60,7 @@ 
     public function assemble()
     {
         $this->callback(
-            function () {
+            function() {
                 $this->findLoaders();
                 $file = $this->get('file');
                 $this->console->getFilesystem()->ensureDirectoryExists(dirname($file));
@@ -109,7 +109,7 @@ 
 
         foreach (['task', 'workflow'] as $type) {
             $lines[] = '';
-            $lines[] = ucfirst($type) . 's';
+            $lines[] = ucfirst($type).'s';
             $lines[] = str_repeat('=', strlen($type) + 1);
             $lines[] = '';
             $taskObjects = $this->loadTaskObjects($type);
@@ -139,7 +139,7 @@ 
                     $lines[] = 'Options';
                     $lines[] = '```````';
                     $lines[] = '';
-                    $this->renderVariables($lines, $taskVariables, $type . '-' . $name);
+                    $this->renderVariables($lines, $taskVariables, $type.'-'.$name);
                 }
                 if ($taskCommonVariables) {
                     $lines[] = 'Common options';
@@ -175,7 +175,7 @@ 
         $keys = ['type', 'default', 'required', 'label'];
         $lines[] = '   * - Name';
         foreach ($keys as $key) {
-            $lines[] = '     - ' . ucfirst($key);
+            $lines[] = '     - '.ucfirst($key);
         }
 
         foreach ($variables as $configName => $config) {
@@ -184,11 +184,11 @@ 
             }
             $lines[] = '   * - ';
             $lines[] = '';
-            $lines[] = '       .. |' . $anchorPrefix . '-' . $configName . '| replace:: ' . $configName;
+            $lines[] = '       .. |'.$anchorPrefix.'-'.$configName.'| replace:: '.$configName;
             $lines[] = '';
-            $lines[] = '       .. _' . $anchorPrefix . '-' . $configName . ':';
+            $lines[] = '       .. _'.$anchorPrefix.'-'.$configName.':';
             $lines[] = '';
-            $lines[] = '       ' . $configName;
+            $lines[] = '       '.$configName;
             $lines[] = '';
             foreach ($keys as $key) {
                 if (!array_key_exists($key, $config)) {
@@ -197,7 +197,7 @@ 
                     $v = $config[$key];
                     $value = $v === null ? 'null' : ($v === true ? 'true' : ($v === false ? 'false' : $v));
                     if (is_string($value)) {
-                        $value = ':code:`' . $value . '`';
+                        $value = ':code:`'.$value.'`';
                     }
                 } elseif ($key === 'required') {
                     $value = $config[$key] === true ? 'X' : $config[$key];
@@ -205,12 +205,12 @@ 
                     $value = $config[$key];
                 }
                 if (is_array($value)) {
-                    $value = "\n\n       .. code::php\n\n           " . str_replace("\n", "\n\n           ", call_user_func('print' . '_r', $value, true)) . "\n\n";
+                    $value = "\n\n       .. code::php\n\n           ".str_replace("\n", "\n\n           ", call_user_func('print'.'_r', $value, true))."\n\n";
                 } else {
                     $value = str_replace("\n", "\n\n       ", $value);
                 }
-                $value = preg_replace('/\{@see\s+(' . implode('|', array_keys($variables)) . ')\}/', '|' . $anchorPrefix . '-$1|_', $value);
-                $lines[] = '     - ' . $value;
+                $value = preg_replace('/\{@see\s+('.implode('|', array_keys($variables)).')\}/', '|'.$anchorPrefix.'-$1|_', $value);
+                $lines[] = '     - '.$value;
             }
         }
         $lines[] = '';
@@ -225,8 +225,8 @@ 
     {
         static $commonVars;
         if (!is_array($commonVars)) {
-            $className = 'NetresearchKiteTask' . uniqid();
-            eval('class ' . $className . ' extends \\Netresearch\\Kite\\Task {}');
+            $className = 'NetresearchKiteTask'.uniqid();
+            eval('class '.$className.' extends \\Netresearch\\Kite\\Task {}');
             $instance = new $className($this);
             $commonVars = $instance->get('_variableConfiguration');
         }
@@ -243,7 +243,7 @@ 
     protected function loadTaskObjects($type)
     {
         $objects = [];
-        $requiredType = 'Netresearch\\Kite\\' . ucfirst($type);
+        $requiredType = 'Netresearch\\Kite\\'.ucfirst($type);
         foreach ($this->factory->getNamespaces($type) as $namespace) {
             $namespaceLength = strlen($namespace);
             if ($dir = $this->findDirectoryForNamespace($namespace)) {
@@ -251,7 +251,7 @@ 
                     if (substr($file, -4) !== '.php') {
                         continue;
                     }
-                    $class = $namespace . '\\' . substr(strtr($file, '/', '\\'), 0, -4);
+                    $class = $namespace.'\\'.substr(strtr($file, '/', '\\'), 0, -4);
                     $reflectionClass = new \ReflectionClass($class);
                     if (!$reflectionClass->isSubclassOf($requiredType) || !$reflectionClass->isInstantiable()) {
                         continue;
@@ -338,7 +338,7 @@ 
                     continue;
                 }
                 foreach ($dirs as $dir) {
-                    if (file_exists($file = $dir . DIRECTORY_SEPARATOR . substr($logicalPathPsr4, $length))) {
+                    if (file_exists($file = $dir.DIRECTORY_SEPARATOR.substr($logicalPathPsr4, $length))) {
                         return $file;
                     }
                 }
@@ -346,7 +346,7 @@ 
 
             // PSR-4 fallback dirs
             foreach ($loader->getFallbackDirsPsr4() as $dir) {
-                if (file_exists($file = $dir . DIRECTORY_SEPARATOR . $logicalPathPsr4)) {
+                if (file_exists($file = $dir.DIRECTORY_SEPARATOR.$logicalPathPsr4)) {
                     return $file;
                 }
             }
@@ -364,7 +364,7 @@ 
             foreach ($loader->getPrefixes() as $prefix => $dirs) {
                 if (0 === strpos($namespace, $prefix)) {
                     foreach ($dirs as $dir) {
-                        if (file_exists($file = $dir . DIRECTORY_SEPARATOR . $logicalPathPsr0)) {
+                        if (file_exists($file = $dir.DIRECTORY_SEPARATOR.$logicalPathPsr0)) {
                             return $file;
                         }
                     }
@@ -373,7 +373,7 @@ 
 
             // PSR-0 fallback dirs
             foreach ($loader->getFallbackDirs() as $dir) {
-                if (file_exists($file = $dir . DIRECTORY_SEPARATOR . $logicalPathPsr0)) {
+                if (file_exists($file = $dir.DIRECTORY_SEPARATOR.$logicalPathPsr0)) {
                     return $file;
                 }
             }

src/Workflow/Composer/Checkout.php

Spacing +3 added lines, -3 removed lines

@@ -66,7 +66,7 @@ 
     public function assemble()
     {
         $this->callback(
-            function () {
+            function() {
                 $this->checkoutPackages(
                     (array) $this->get('branch'),
                     $this->get('merge'),
@@ -109,9 +109,9 @@ 
             $lastBranch = array_pop($branches);
             $message = 'Could not find branch ';
             if ($branches) {
-                $message .= implode(', ', $branches) . ' or ';
+                $message .= implode(', ', $branches).' or ';
             }
-            $message .= $lastBranch . ' in any installed package';
+            $message .= $lastBranch.' in any installed package';
             $this->console->output("<warning>$message</warning>");
             return;
         }

src/Workflow/ClearCodeCaches.php

Spacing +7 added lines, -7 removed lines

@@ -57,12 +57,12 @@ 
     public function assemble()
     {
         $this->callback(
-            function () {
+            function() {
                 $scriptPath = $this->createScript();
                 if ($webUrl = $this->get('webUrl')) {
                     $this->callScript($webUrl, $scriptPath);
                 } else {
-                    $this->scp($scriptPath, '{node}:{node.webRoot}/' . $scriptPath);
+                    $this->scp($scriptPath, '{node}:{node.webRoot}/'.$scriptPath);
                     foreach ($this->get('nodes') as $node) {
                         /* @var \Netresearch\Kite\Node $node */
                         if ($webUrl = $node->get('webUrl', null)) {
@@ -83,10 +83,10 @@ 
      */
     protected function createScript()
     {
-        $scriptPath = $this->get('baseDir') . '/ccc-' . uniqid() . '.php';
+        $scriptPath = $this->get('baseDir').'/ccc-'.uniqid().'.php';
         file_put_contents(
             $scriptPath,
-            '<?' . "php
+            '<?'."php
             if (function_exists('clearstatcache')) {
                 echo 'statcache|';
                 clearstatcache(true);
@@ -116,9 +116,9 @@ 
      */
     protected function callScript($baseUrl, $scriptPath)
     {
-        $url = rtrim($baseUrl, '/') . '/' . ltrim($scriptPath, '/');
+        $url = rtrim($baseUrl, '/').'/'.ltrim($scriptPath, '/');
 
-        $this->console->output('Calling ' . $url, OutputInterface::VERBOSITY_DEBUG);
+        $this->console->output('Calling '.$url, OutputInterface::VERBOSITY_DEBUG);
         $ch = curl_init();
         curl_setopt($ch, CURLOPT_URL, $url);
         curl_setopt($ch, CURLOPT_HEADER, 1);
@@ -134,7 +134,7 @@ 
             $this->console->outdent();
         } else {
             if ($caches) {
-                $this->console->output('Cleared code caches (' . implode(', ', $caches) . ')');
+                $this->console->output('Cleared code caches ('.implode(', ', $caches).')');
             } else {
                 $this->console->output('No code caches found');
             }

src/Service/Config.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
      */
     public function loadPreset($name)
     {
-        include dirname(dirname(__DIR__)) . '/presets/' . $name . '.php';
+        include dirname(dirname(__DIR__)).'/presets/'.$name.'.php';
     }
 
     /**

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
     {
         $absPath = stream_resolve_include_path($path);
         if ($absPath === false) {
-            throw new Exception("File $path could not be found");
+            throw new Exception("file $path could not be found");
         }
         include $absPath;
     }
@@ -76,7 +76,7 @@ 
     {
         if ($job) {
             if (!array_key_exists($job, $this['jobs'])) {
-                throw new Exception("Job $job is not configured");
+                throw new Exception("job $job is not configured");
             }
             return $this['jobs'][$job];
         }

src/Service/Factory.php

Spacing +4 added lines, -4 removed lines

@@ -171,15 +171,15 @@
         $ucType = ucfirst($type);
         if (!strpos($definition, '\\')) {
             $taskClass = str_replace(' ', '\\', ucwords(str_replace('-', ' ', $definition)));
-            $taskClass = $taskClass . ($postfixType ? $ucType : '');
+            $taskClass = $taskClass.($postfixType ? $ucType : '');
             foreach ($this->namespaces[$type] as $namespace) {
-                $potentialClass = '\\' . $namespace . '\\' . $taskClass;
+                $potentialClass = '\\'.$namespace.'\\'.$taskClass;
                 if (class_exists($potentialClass)) {
                     return $potentialClass;
                 }
             }
-        } elseif (!is_subclass_of(ltrim($definition, '\\'), 'Netresearch\\Kite\\' . $ucType)) {
-            throw new Exception($definition .' must extend Netresearch\\Kite\\' . $ucType);
+        } elseif (!is_subclass_of(ltrim($definition, '\\'), 'Netresearch\\Kite\\'.$ucType)) {
+            throw new Exception($definition.' must extend Netresearch\\Kite\\'.$ucType);
         }
         return $definition;
     }

src/Service/Filesystem.php

Spacing +16 added lines, -16 removed lines

@@ -84,7 +84,7 @@ 
     {
         $dir = rtrim($dir, '/\\');
 
-        return count(glob($dir . '/*') ?: array()) === 0 && count(glob($dir . '/.*') ?: array()) === 2;
+        return count(glob($dir.'/*') ?: array()) === 0 && count(glob($dir.'/.*') ?: array()) === 2;
     }
 
     /**
@@ -104,7 +104,7 @@ 
         }
 
         if (preg_match('{^(?:[a-z]:)?[/\\\\]+$}i', $directory)) {
-            throw new \RuntimeException('Aborting an attempted deletion of ' . $directory . ', this was probably not intended, if it is a real use case please report it.');
+            throw new \RuntimeException('Aborting an attempted deletion of '.$directory.', this was probably not intended, if it is a real use case please report it.');
         }
 
         if (!function_exists('proc_open')) {
@@ -161,12 +161,12 @@ 
         if (!is_dir($directory)) {
             if (file_exists($directory)) {
                 throw new \RuntimeException(
-                    $directory . ' exists and is not a directory.'
+                    $directory.' exists and is not a directory.'
                 );
             }
             if (!@mkdir($directory, 0777, true)) {
                 throw new \RuntimeException(
-                    $directory . ' does not exist and could not be created.'
+                    $directory.' does not exist and could not be created.'
                 );
             }
         }
@@ -207,7 +207,7 @@ 
             $this->ensureDirectoryExists($target);
 
             foreach ($ri as $file) {
-                $targetPath = $target . DIRECTORY_SEPARATOR . $ri->getSubPathName();
+                $targetPath = $target.DIRECTORY_SEPARATOR.$ri->getSubPathName();
                 if ($file->isDir()) {
                     $this->ensureDirectoryExists($targetPath);
                 } else {
@@ -293,11 +293,11 @@ 
         }
 
         if (dirname($from) === dirname($to)) {
-            return './' . basename($to);
+            return './'.basename($to);
         }
 
         $commonPath = $to;
-        while (strpos($from . '/', $commonPath . '/') !== 0 && '/' !== $commonPath && !preg_match('{^[a-z]:/?$}i', $commonPath)) {
+        while (strpos($from.'/', $commonPath.'/') !== 0 && '/' !== $commonPath && !preg_match('{^[a-z]:/?$}i', $commonPath)) {
             $commonPath = strtr(dirname($commonPath), '\\', '/');
         }
 
@@ -305,11 +305,11 @@ 
             return $to;
         }
 
-        $commonPath = rtrim($commonPath, '/') . '/';
+        $commonPath = rtrim($commonPath, '/').'/';
         $sourcePathDepth = substr_count(substr($from, strlen($commonPath)), '/');
         $commonPathCode = str_repeat('../', $sourcePathDepth);
 
-        return ($commonPathCode . substr($to, strlen($commonPath))) ?: './';
+        return ($commonPathCode.substr($to, strlen($commonPath))) ?: './';
     }
 
     /**
@@ -337,7 +337,7 @@ 
         }
 
         $commonPath = $to;
-        while (strpos($from . '/', $commonPath . '/') !== 0 && '/' !== $commonPath && !preg_match('{^[a-z]:/?$}i', $commonPath) && '.' !== $commonPath) {
+        while (strpos($from.'/', $commonPath.'/') !== 0 && '/' !== $commonPath && !preg_match('{^[a-z]:/?$}i', $commonPath) && '.' !== $commonPath) {
             $commonPath = strtr(dirname($commonPath), '\\', '/');
         }
 
@@ -345,15 +345,15 @@ 
             return var_export($to, true);
         }
 
-        $commonPath = rtrim($commonPath, '/') . '/';
-        if (strpos($to, $from . '/') === 0) {
-            return '__DIR__ . ' . var_export(substr($to, strlen($from)), true);
+        $commonPath = rtrim($commonPath, '/').'/';
+        if (strpos($to, $from.'/') === 0) {
+            return '__DIR__ . '.var_export(substr($to, strlen($from)), true);
         }
         $sourcePathDepth = substr_count(substr($from, strlen($commonPath)), '/') + $directories;
-        $commonPathCode = str_repeat('dirname(', $sourcePathDepth) . '__DIR__' . str_repeat(')', $sourcePathDepth);
+        $commonPathCode = str_repeat('dirname(', $sourcePathDepth).'__DIR__'.str_repeat(')', $sourcePathDepth);
         $relTarget = substr($to, strlen($commonPath));
 
-        return $commonPathCode . (strlen($relTarget) ? '.' . var_export('/' . $relTarget, true) : '');
+        return $commonPathCode.(strlen($relTarget) ? '.'.var_export('/'.$relTarget, true) : '');
     }
 
     /**
@@ -426,7 +426,7 @@ 
             }
         }
 
-        return $prefix . ($absolute ? '/' : '') . implode('/', $parts);
+        return $prefix.($absolute ? '/' : '').implode('/', $parts);
     }
 
     /**

src/Workflow.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
     public function offsetSet($name, $value)
     {
         if (in_array($name, ['workflow', 'script', 'task', 'tasks'], true)) {
-            throw new Exception($name . ' not allowed on workflows');
+            throw new Exception($name.' not allowed on workflows');
         }
         parent::offsetSet($name, $value);
     }

src/Job.php

Spacing +3 added lines, -3 removed lines

@@ -125,7 +125,7 @@ 
         if ($name === 'options' || $name === 'arguments') {
             $type = substr($name, 0, -1);
             if (!is_array($value)) {
-                throw new Exception($name . ' must be array');
+                throw new Exception($name.' must be array');
             }
             foreach ($value as $variable => $config) {
                 $from = $this->camelCaseToLowerCaseDashed($variable);
@@ -165,9 +165,9 @@ 
                     if (($value === null || is_array($value) && !isset($value[0]) && count($value) <= 1)
                         && preg_match('/(^|\|)bool(ean)?($|\|)/', $config['type']) && strpos($config['type'], '|')
                     ) {
-                        if ($input->hasParameterOption($opt = '--' . $from)
+                        if ($input->hasParameterOption($opt = '--'.$from)
                             || array_key_exists('shortcut', $config)
-                            && $input->hasParameterOption($opt = '-' . $config['shortcut'])
+                            && $input->hasParameterOption($opt = '-'.$config['shortcut'])
                         ) {
                             // Set value to true, when option has no value
                             $value = true;