Niktux / karma

src/Karma/Command/ConfigureActionCommand.php

Doc Comments +12 added lines

@@ -30,6 +30,11 @@ 
     protected
         $environment;
 
+    /**
+     * @param string $name
+     * @param string $description
+     * @param string $outputTitle
+     */
     public function __construct(Application $app, $name, $description, $outputTitle)
     {
         $this->name = $name;
@@ -128,6 +133,10 @@ 
     }
 
     abstract protected function launchConfigurationAction(ConfigurableProcessor $processor);
+
+    /**
+     * @return ConfigurableProcessor
+     */
     abstract protected function getProcessor();
 
     private function configureProcessor(ConfigurableProcessor $processor)
@@ -153,6 +162,9 @@ 
         }
     }
 
+    /**
+     * @param string $optionName
+     */
     private function parseOptionWithAssignments(InputInterface $input, $optionName)
     {
         $strings = $input->getOption($optionName);

Indentation +7 added lines, -7 removed lines

@@ -146,7 +146,7 @@ 
         {
             $processor->setSystemEnvironment($this->systemEnvironment);
 
-             $this->app['logger']->info(sprintf(
+                $this->app['logger']->info(sprintf(
                 'Hydrate <important>system</important> variables with <important>%s</important> values',
                 $this->systemEnvironment
             ));
@@ -197,9 +197,9 @@ 
         foreach($overrides as $variable => $value)
         {
             $logger->info(sprintf(
-               'Override <important>%s</important> with value <important>%s</important>',
-               $variable,
-               $value
+                'Override <important>%s</important> with value <important>%s</important>',
+                $variable,
+                $value
             ));
 
             $value = $this->parseList($value);
@@ -216,9 +216,9 @@ 
         foreach($data as $variable => $value)
         {
             $logger->info(sprintf(
-               'Set custom data <important>%s</important> with value <important>%s</important>',
-               $variable,
-               $value
+                'Set custom data <important>%s</important> with value <important>%s</important>',
+                $variable,
+                $value
             ));
 
             $reader->setCustomData($variable, $this->filterValue($value));

Spacing +2 added lines, -2 removed lines

@@ -104,7 +104,7 @@ 
             $sourcePath = $profile->getSourcePath();
         }
 
-        if(! is_array($sourcePath))
+        if( ! is_array($sourcePath))
         {
             $sourcePath = array($sourcePath);
         }
@@ -157,7 +157,7 @@ 
     {
         $strings = $input->getOption($optionName);
 
-        if(! is_array($strings))
+        if( ! is_array($strings))
         {
             $strings = array($strings);
         }

src/Karma/Configuration/Parser.php

Doc Comments +3 added lines

@@ -199,6 +199,9 @@
         return $sectionName;
     }
 
+    /**
+     * @param string $sectionName
+     */
     private function switchSectionParser($sectionName)
     {
         if(! isset($this->parsers[$sectionName]))

Indentation +3 added lines, -3 removed lines

@@ -240,8 +240,8 @@ 
             if($status['found'] === false)
             {
                 $this->warning(sprintf(
-                   'External file %s was not found',
-                   $file
+                    'External file %s was not found',
+                    $file
                 ));
             }
         }
@@ -294,7 +294,7 @@ 
 
     public function getDefaultEnvironmentsForGroups()
     {
-         $defaultEnvironments = array();
+            $defaultEnvironments = array();
 
         if(isset($this->parsers[self::GROUPS]))
         {

Spacing +6 added lines, -6 removed lines

@@ -49,7 +49,7 @@ 
 
     public function enableIncludeSupport()
     {
-        if(! isset($this->parsers[self::INCLUDES]))
+        if( ! isset($this->parsers[self::INCLUDES]))
         {
             $this->parsers[self::INCLUDES] = new IncludeParser();
         }
@@ -59,7 +59,7 @@ 
 
     public function enableExternalSupport()
     {
-        if(! isset($this->parsers[self::EXTERNALS]))
+        if( ! isset($this->parsers[self::EXTERNALS]))
         {
             $this->parsers[self::EXTERNALS] = new ExternalParser(new Parser($this->fs));
         }
@@ -69,7 +69,7 @@ 
 
     public function enableGroupSupport()
     {
-        if(! isset($this->parsers[self::GROUPS]))
+        if( ! isset($this->parsers[self::GROUPS]))
         {
             $this->parsers[self::GROUPS] = new GroupParser();
         }
@@ -101,7 +101,7 @@ 
     {
         $files = array($masterFilePath);
 
-        while(! empty($files))
+        while( ! empty($files))
         {
             foreach($files as $file)
             {
@@ -151,7 +151,7 @@ 
 
     private function extractLines($filePath)
     {
-        if(! $this->fs->has($filePath))
+        if( ! $this->fs->has($filePath))
         {
             throw new \RuntimeException("$filePath does not exist");
         }
@@ -201,7 +201,7 @@ 
 
     private function switchSectionParser($sectionName)
     {
-        if(! isset($this->parsers[$sectionName]))
+        if( ! isset($this->parsers[$sectionName]))
         {
             throw new \RuntimeException('Unknown section name ' . $sectionName);
         }

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

@@ -178,7 +178,7 @@
 
     private function changeCurrentFile($filePath)
     {
-        $this->info("Reading $filePath");
+        $this->info("reading $filePath");
 
         foreach($this->parsers as $parser)
         {

src/Karma/Configuration/ValueFilterIterator.php

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
         $isRegex,
         $filter;
 
+    /**
+     * @param string $filter
+     */
     public function __construct($filter, \Iterator $iterator)
     {
         parent::__construct($iterator);

src/Karma/Hydrator.php

Doc Comments +9 added lines

@@ -127,6 +127,9 @@ 
         }
     }
 
+    /**
+     * @param string|boolean $fileContent
+     */
     private function parseFileDirectives($file, & $fileContent, $environment)
     {
         $this->currentFormatterName = null;
@@ -210,6 +213,9 @@ 
         return preg_replace('~(<%\s*karma:[^%]*%>\s*)~i', '', $fileContent);
     }
 
+    /**
+     * @return string
+     */
     private function injectValues($sourceFile, $content, $environment, $replacementCounter = 0)
     {
         $replacementCounter += $this->injectScalarValues($content, $environment);
@@ -334,6 +340,9 @@ 
         return "\n";
     }
 
+    /**
+     * @param string $targetFile
+     */
     private function backupFile($targetFile)
     {
         if($this->enableBackup === true)

Indentation +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
         }
 
         $this->info(sprintf(
-           '%d files generated',
+            '%d files generated',
             count($distFiles)
         ));
     }

Spacing +2 added lines, -2 removed lines

@@ -193,12 +193,12 @@
         $values = $this->readValueToInject($variable, $environment);
         $formatter = $this->getFormatterForCurrentTargetFile();
 
-        if(! is_array($values))
+        if( ! is_array($values))
         {
             $values = array($values);
         }
 
-        array_walk($values, function (& $value) use ($formatter) {
+        array_walk($values, function(& $value) use ($formatter) {
             $value = $formatter->format($value);
         });
 

Braces +1 added lines, -2 removed lines

@@ -266,8 +266,7 @@
     {
         $formatter = $this->getFormatterForCurrentTargetFile();
 
-        $content = preg_replace_callback(self::VARIABLE_REGEX, function(array $matches) use($environment, $formatter)
-        {
+        $content = preg_replace_callback(self::VARIABLE_REGEX, function(array $matches) use($environment, $formatter) {
             $value = $this->readValueToInject($matches['variableName'], $environment);
 
             if(is_array($value))

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

@@ -118,7 +118,7 @@
 
         $targetContent = $this->injectValues($file, $content, $environment, $replacementCounter);
 
-        $this->debug("Write $this->currentTargetFile");
+        $this->debug("write $this->currentTargetFile");
 
         if($this->dryRun === false)
         {

src/Karma/Application.php

Spacing +7 added lines, -7 removed lines

@@ -108,7 +108,7 @@ 
             
             $paths = $c['sources.path'];
             
-            if(! is_array($paths))
+            if( ! is_array($paths))
             {
                 $paths = array($paths);
             }
@@ -181,7 +181,7 @@ 
         $this['git'] = $git;
         $this['vcs'] = $git;
 
-        $this['vcsHandler'] = $this->protect(function (Vcs $vcs) {
+        $this['vcsHandler'] = $this->protect(function(Vcs $vcs) {
             $handler = new VcsHandler($vcs, $this['finder']);
 
             $handler->setLogger($this['logger'])
@@ -197,7 +197,7 @@ 
             return new \Psr\Log\NullLogger();
         });
 
-        $this['formatter.provider'] = $this->share(function ($c) {
+        $this['formatter.provider'] = $this->share(function($c) {
             return new ProfileProvider($c['profile']);
         });
 
@@ -210,19 +210,19 @@ 
             return $hydrator;
         };
 
-        $this['generator.nameTranslator'] = $this->share(function ($c) {
+        $this['generator.nameTranslator'] = $this->share(function($c) {
             $translator = new FilePrefixTranslator();
             $translator->changeMasterFile($c['configuration.masterFile']);
 
             return $translator;
         });
 
-        $this['generator.variableProvider'] = function ($c) {
+        $this['generator.variableProvider'] = function($c) {
             $provider = new VariableProvider($c['parser'], $c['configuration.masterFile']);
 
             $profile = $c['profile'];
             $options = $profile->getGeneratorOptions();
-            if(! isset($options['translator']) || $options['translator'] === 'prefix')
+            if( ! isset($options['translator']) || $options['translator'] === 'prefix')
             {
                 $provider->setNameTranslator($c['generator.nameTranslator']);
             }
@@ -230,7 +230,7 @@ 
             return $provider;
         };
 
-        $this['configurationFilesGenerator'] = $this->share(function ($c) {
+        $this['configurationFilesGenerator'] = $this->share(function($c) {
             return new YamlGenerator($c['sources.fileSystem'], $c['configuration'], $c['generator.variableProvider']);
         });
     }

src/Karma/Command.php

Indentation +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
         }
 
         $this->output->writeln(
-           $this->getLogo($this->output->isDecorated())
+            $this->getLogo($this->output->isDecorated())
         );
     }
 

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

@@ -130,7 +130,7 @@
 ...@@...@@..
 ...@@....@@.
 
-ASCIIART;
+asciiart;
 
         $background = 'fg=magenta';
         $text = 'fg=white';

src/Karma/Command/Diff.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         $table->enableFormattingTags()
             ->setHeaders(array($environment1, $environment2))
             ->displayKeys()
-            ->setValueRenderingFunction(function($value){
+            ->setValueRenderingFunction(function($value) {
                 return $this->formatValue($value);
             });
 

Braces +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         $table->enableFormattingTags()
             ->setHeaders(array($environment1, $environment2))
             ->displayKeys()
-            ->setValueRenderingFunction(function($value){
+            ->setValueRenderingFunction(function($value) {
                 return $this->formatValue($value);
             });
 

src/Karma/Command/Display.php

Indentation +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
         foreach($values as $variable => $value)
         {
             $this->output->writeln(sprintf(
-               '<fg=cyan>%s</fg=cyan> = %s',
+                '<fg=cyan>%s</fg=cyan> = %s',
                 $variable,
                 $this->formatValue($value)
             ));

src/Karma/Command/Hydrate.php

Spacing +3 added lines, -3 removed lines

@@ -34,11 +34,11 @@ 
     {
         $unusedVariables = $processor->getUnusedVariables();
 
-        if(! empty($unusedVariables))
+        if( ! empty($unusedVariables))
         {
             $logger = $this->app['logger'];
 
-            $logger->warning('You have unused variables : you should remove them or check if you have not mispelled them').
+            $logger->warning('You have unused variables : you should remove them or check if you have not mispelled them') .
             $logger->warning(sprintf(
                 'Unused variables : %s',
                 implode(', ', $unusedVariables)
@@ -50,7 +50,7 @@ 
     {
         $unvaluedVariables = $processor->getUnvaluedVariables();
 
-        if(! empty($unvaluedVariables))
+        if( ! empty($unvaluedVariables))
         {
             $logger = $this->app['logger'];