Fixed bug #20139 : No report file generated on success

squizlabs · Dec 4, 2013 · f08443f · f08443f
1 parent 61917c2
commit f08443f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/CodeSniffer/Reporting.php b/CodeSniffer/Reporting.php
@@ -179,11 +179,6 @@ public function printReport(
         $reportFile='',
         $reportWidth=80
     ) {
-        if ($this->totalFiles === 0) {
-            // No files generated errors.
-            return 0;
-        }
-
         $reportClass = self::factory($report);
 
         if ($reportFile !== null) {

diff --git a/package.xml b/package.xml
@@ -39,6 +39,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Fixed bug #20136 : Squiz_Sniffs_WhiteSpace_ScopeKeywordSpacingSniff and Traits
   - Fixed bug #20138 : Protected property underscore and camel caps issue (in trait with Zend)
     -- Thanks to Gaetan Rousseau for the patch
+  - Fixed bug #20139 : No report file generated on success
  </notes>
  <contents>
   <dir name="/">