Niktux / karma

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)
         ));
     }

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

@@ -92,7 +92,7 @@
         $targetContent = $this->injectValues($file, $content, $environment);
         
         $targetFile = substr($file, 0, strlen($this->suffix) * -1);
-        $this->debug("Write $targetFile");
+        $this->debug("write $targetFile");
 
         if($this->dryRun === false)
         {

src/Karma/Command.php

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

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

Indentation +2 added lines, -2 removed lines

@@ -102,8 +102,8 @@
     private function printHeader()
     {
         $this->output->writeln(sprintf(
-           '<comment>%s</comment>',
-           $this->getLogo()
+            '<comment>%s</comment>',
+            $this->getLogo()
         ));
     }
     

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/Configuration/AbstractReader.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
 
     public function setDefaultEnvironment($environment)
     {
-        if(! empty($environment) && is_string($environment))
+        if( ! empty($environment) && is_string($environment))
         {
             $this->defaultEnvironment = $environment;
         }
@@ -88,7 +88,7 @@ 
 
     private function handleCustomData($value)
     {
-        if(! is_string($value))
+        if( ! is_string($value))
         {
             return $value;
         }

src/Karma/Configuration/FilterInputVariable.php

Spacing +2 added lines, -2 removed lines

@@ -8,7 +8,7 @@ 
     {
         if(is_array($value))
         {
-            return array_map(function ($item) {
+            return array_map(function($item) {
                 return $this->filterOneValue($item);
             }, $value);
         }
@@ -18,7 +18,7 @@ 
 
     private function filterOneValue($value)
     {
-        if(! is_string($value))
+        if( ! is_string($value))
         {
             return $value;
         }

src/Karma/Configuration/Parser.php

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)
         {

Indentation +3 added lines, -3 removed lines

@@ -221,9 +221,9 @@
         foreach($files as $file => $status)
         {
             $message = sprintf(
-               'External file %s was %s',
-               $file,
-               $status['found'] ? 'found' : '<options=bold>not found</options=bold>'
+                'External file %s was %s',
+                $file,
+                $status['found'] ? 'found' : '<options=bold>not found</options=bold>'
             );
             
             $this->warning($message);

Doc Comments +6 added lines

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

src/Karma/Configuration/Parser/GroupParser.php

Indentation +1 added lines, -1 removed lines

@@ -153,7 +153,7 @@
         if(! empty($errors))
         {
             throw new \RuntimeException(sprintf(
-               'Error : a group can not be part of another group (%s)',
+                'Error : a group can not be part of another group (%s)',
                 implode(', ', $errors)
             ));
         }

Spacing +2 added lines, -2 removed lines

@@ -104,7 +104,7 @@ 
 
     private function hasDuplicatedValues(array $values)
     {
-        $duplicatedValues = array_filter(array_count_values($values), function ($counter) {
+        $duplicatedValues = array_filter(array_count_values($values), function($counter) {
             return $counter !== 1;
         });
 
@@ -150,7 +150,7 @@ 
 
         $errors = array_intersect($allEnvironments, array_keys($this->groups));
 
-        if(! empty($errors))
+        if( ! empty($errors))
         {
             throw new \RuntimeException(sprintf(
                'Error : a group can not be part of another group (%s)',

src/Karma/Configuration/Parser/IncludeParser.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
     private function checkFilenameIsValid($filename)
     {
-        if(! preg_match('~.*\.conf$~', $filename))
+        if( ! preg_match('~.*\.conf$~', $filename))
         {
             $this->triggerError("$filename is not a valid file name", 'Invalid dependency');
         }