Completed
Push — 16-vcs ( 37de27 )
by Nicolas
28:38 queued 23:50
created
src/Karma/Configuration/ValueFilterIterator.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -14,6 +14,9 @@
 block discarded – undo
14 14
         $isRegex,
15 15
         $filter;
16 16
 
17
+    /**
18
+     * @param string $filter
19
+     */
17 20
     public function __construct($filter, \Iterator $iterator)
18 21
     {
19 22
         parent::__construct($iterator);
Please login to merge, or discard this patch.
src/Karma/Command/Display.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
         foreach($values as $variable => $value)
62 62
         {
63 63
             $this->output->writeln(sprintf(
64
-               '<fg=cyan>%s</fg=cyan> = %s',
64
+                '<fg=cyan>%s</fg=cyan> = %s',
65 65
                 $variable,
66 66
                 $this->formatValue($value)
67 67
             ));
Please login to merge, or discard this patch.
src/Karma/Configuration/Parser.php 4 patches
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -178,7 +178,7 @@
 block discarded – undo
178 178
 
179 179
     private function changeCurrentFile($filePath)
180 180
     {
181
-        $this->info("Reading $filePath");
181
+        $this->info("reading $filePath");
182 182
 
183 183
         foreach($this->parsers as $parser)
184 184
         {
Please login to merge, or discard this patch.
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -65,6 +65,9 @@  discard block
 block discarded – undo
65 65
         return $this;
66 66
     }
67 67
     
68
+    /**
69
+     * @param string $masterFilePath
70
+     */
68 71
     public function parse($masterFilePath)
69 72
     {
70 73
         try
@@ -182,6 +185,9 @@  discard block
 block discarded – undo
182 185
         return $groupName;
183 186
     }
184 187
     
188
+    /**
189
+     * @param string $groupName
190
+     */
185 191
     private function switchGroupParser($groupName)
186 192
     {
187 193
         if(! isset($this->parsers[$groupName]))
Please login to merge, or discard this patch.
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -221,9 +221,9 @@
 block discarded – undo
221 221
         foreach($files as $file => $status)
222 222
         {
223 223
             $message = sprintf(
224
-               'External file %s was %s',
225
-               $file,
226
-               $status['found'] ? 'found' : '<options=bold>not found</options=bold>'
224
+                'External file %s was %s',
225
+                $file,
226
+                $status['found'] ? 'found' : '<options=bold>not found</options=bold>'
227 227
             );
228 228
             
229 229
             $this->warning($message);
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     
48 48
     public function enableIncludeSupport()
49 49
     {
50
-        if(! isset($this->parsers[self::INCLUDES]))
50
+        if( ! isset($this->parsers[self::INCLUDES]))
51 51
         {
52 52
             $this->parsers[self::INCLUDES] = new IncludeParser();
53 53
         }
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
     
58 58
     public function enableExternalSupport()
59 59
     {
60
-        if(! isset($this->parsers[self::EXTERNALS]))
60
+        if( ! isset($this->parsers[self::EXTERNALS]))
61 61
         {
62 62
             $this->parsers[self::EXTERNALS] = new ExternalParser(new Parser($this->fs));
63 63
         }
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
     {
88 88
         $files = array($masterFilePath);
89 89
         
90
-        while(! empty($files))
90
+        while( ! empty($files))
91 91
         {
92 92
             foreach($files as $file)
93 93
             {
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
     
134 134
     private function extractLines($filePath)
135 135
     {
136
-        if(! $this->fs->has($filePath))
136
+        if( ! $this->fs->has($filePath))
137 137
         {
138 138
             throw new \RuntimeException("$filePath does not exist");
139 139
         }
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
     
185 185
     private function switchGroupParser($groupName)
186 186
     {
187
-        if(! isset($this->parsers[$groupName]))
187
+        if( ! isset($this->parsers[$groupName]))
188 188
         {
189 189
             throw new \RuntimeException('Unknown group name ' . $groupName);
190 190
         }
Please login to merge, or discard this patch.
src/Karma/Configuration/Parser/IncludeParser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
 
27 27
     private function checkFilenameIsValid($filename)
28 28
     {
29
-        if(! preg_match('~.*\.conf$~', $filename))
29
+        if( ! preg_match('~.*\.conf$~', $filename))
30 30
         {
31 31
             $this->triggerError("$filename is not a valid file name", 'Invalid dependency');
32 32
         }
Please login to merge, or discard this patch.
src/Karma/Hydrator.php 4 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@
 block discarded – undo
99 99
         }
100 100
 
101 101
         $this->info(sprintf(
102
-           '%d files generated',
102
+            '%d files generated',
103 103
             count($distFiles)
104 104
         ));
105 105
     }
Please login to merge, or discard this patch.
Doc Comments   +8 added lines patch added patch discarded remove patch
@@ -101,6 +101,11 @@  discard block
 block discarded – undo
101 101
         }
102 102
     }
103 103
     
104
+    /**
105
+     * @param string|boolean $content
106
+     *
107
+     * @return string
108
+     */
104 109
     private function injectValues($sourceFile, $content, $environment)
105 110
     {
106 111
         $formatter = $this->formatterProvider->getFormatter();
@@ -118,6 +123,9 @@  discard block
 block discarded – undo
118 123
         return $targetContent;
119 124
     }
120 125
     
126
+    /**
127
+     * @param string $targetFile
128
+     */
121 129
     private function backupFile($targetFile)
122 130
     {
123 131
         if($this->enableBackup === true)
Please login to merge, or discard this patch.
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@
 block discarded – undo
92 92
         $targetContent = $this->injectValues($file, $content, $environment);
93 93
         
94 94
         $targetFile = substr($file, 0, strlen($this->suffix) * -1);
95
-        $this->debug("Write $targetFile");
95
+        $this->debug("write $targetFile");
96 96
 
97 97
         if($this->dryRun === false)
98 98
         {
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@
 block discarded – undo
88 88
     
89 89
     private function injectValues($sourceFile, $content, $environment)
90 90
     {
91
-        $targetContent = preg_replace_callback('~<%(?P<variableName>[A-Za-z0-9_\.]+)%>~', function(array $matches) use($environment){
91
+        $targetContent = preg_replace_callback('~<%(?P<variableName>[A-Za-z0-9_\.]+)%>~', function(array $matches) use($environment) {
92 92
             return $this->reader->read($matches['variableName'], $environment);
93 93
         }, $content, -1, $count);
94 94
         
Please login to merge, or discard this patch.
src/Karma/Logging/OutputAware.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
         {
43 43
             if($verbosity <= $this->output->getVerbosity())
44 44
             {
45
-                if(! is_array($messages))
45
+                if( ! is_array($messages))
46 46
                 {
47 47
                     $messages = array($messages);
48 48
                 }
Please login to merge, or discard this patch.
src/Karma/ProfileReader.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@
 block discarded – undo
56 56
         catch(ParseException $e)
57 57
         {
58 58
             throw new \RuntimeException(sprintf(
59
-               'Error while parsing profile : %s',
59
+                'Error while parsing profile : %s',
60 60
                 $e->getMessage()
61 61
             ));
62 62
         }
Please login to merge, or discard this patch.
src/Karma/VCS/Git.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -123,7 +123,7 @@
 block discarded – undo
123 123
             $titleLineNumber = count($lines);
124 124
 
125 125
             // ensure an empty line preceeds comment
126
-            if(! empty($lines[$titleLineNumber - 1]))
126
+            if( ! empty($lines[$titleLineNumber - 1]))
127 127
             {
128 128
                 $lines[$titleLineNumber] = '';
129 129
                 $titleLineNumber++;
Please login to merge, or discard this patch.
src/Karma/VcsHandler.php 1 patch
Upper-Lower-Casing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,13 +42,13 @@
 block discarded – undo
42 42
 
43 43
             if($this->vcs->isTracked($targetFile))
44 44
             {
45
-                $this->logger->info("Untrack $targetFile");
45
+                $this->logger->info("untrack $targetFile");
46 46
                 $this->vcs->untrackFile($targetFile);
47 47
             }
48 48
 
49 49
             if($this->vcs->isIgnored($targetFile) === false)
50 50
             {
51
-                $this->logger->info("Ignore $targetFile");
51
+                $this->logger->info("ignore $targetFile");
52 52
                 $this->vcs->ignoreFile($targetFile);
53 53
             }
54 54
         }
Please login to merge, or discard this patch.