Changelog + unit tests for PR #1011

Author: gsherwood

CodeSniffer/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.inc

@@ -134,6 +134,7 @@ switch ($var) {
 }
 
 defined('FOO') or die('error');
+defined('FOO') || die('error');
 interface myInterface {
     function myFunc();
 }

CodeSniffer/Standards/Squiz/Tests/PHP/NonExecutableCodeUnitTest.php

@@ -76,16 +76,16 @@ public function getWarningList()
                 95  => 1,
                 105 => 1,
                 123 => 1,
-                146 => 1,
-                149 => 1,
-                152 => 1,
-                165 => 1,
-                179 => 1,
-                221 => 1,
+                147 => 1,
+                150 => 1,
+                153 => 1,
+                166 => 1,
+                180 => 1,
                 222 => 1,
                 223 => 1,
-                224 => 2,
-                228 => 1,
+                224 => 1,
+                225 => 2,
+                229 => 1,
                );
 
     }//end getWarningList()

package.xml

@@ -46,7 +46,9 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Fixed bug #973 : Anonymous class declaration and PSR1.Files.SideEffects.FoundWithSymbols
   - Fixed bug #974 : Error when file ends with "function"
   - Fixed bug #979 : Anonymous function with return type hint is not refactored as expected
-  - Fixed bug #893 : Squiz.WhiteSpace.MemberVarSpacing.AfterComment fails to fix error when comment is not a docblock
+  - Fixed bug #983 : Squiz.WhiteSpace.MemberVarSpacing.AfterComment fails to fix error when comment is not a docblock
+  - Fixed bug #1010 : Squiz NonExectuableCode sniff does not detect boolean OR
+    -- Thanks to Derek Henderson for the patch
  </notes>
  <contents>
   <dir name="/">