Fixed bug squizlabs#473 : Writing a report for an empty folder to exi…

…sting file includes the existing contents
gimler · Feb 3, 2015 · cc19ee1 · cc19ee1
1 parent 2f75556
commit cc19ee1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CodeSniffer/Reporting.php b/CodeSniffer/Reporting.php
@@ -230,7 +230,9 @@ public function printReport(
             $filename = $reportFile;
             $toScreen = false;
 
-            if (file_exists($filename) === true) {
+            if (file_exists($filename) === true
+                && isset($this->_cachedReports[$report]) === true
+            ) {
                 $reportCache = file_get_contents($filename);
             } else {
                 $reportCache = '';

diff --git a/package.xml b/package.xml
@@ -36,6 +36,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Fixed bug #453 : PSR2 standard does not allow closing tag for mixed PHP/HTML files
   - Fixed bug #457 : FunctionCallSignature sniffs do not support here/nowdoc syntax and can cause syntax error when fixing
   - Fixed bug #466 : PropertyLabelSpacing JS fixer issue when there is no space after colon
+  - Fixed bug #473 : Writing a report for an empty folder to existing file includes the existing contents
  </notes>
  <contents>
   <dir name="/">