Changelog + unit tests for #2368

squizlabs · Apr 3, 2019 · dd3f6e0 · dd3f6e0
1 parent 89888ba
commit dd3f6e0
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/package.xml b/package.xml
@@ -27,6 +27,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
  <license uri="https://github.com/squizlabs/PHP_CodeSniffer/blob/master/licence.txt">BSD 3-Clause License</license>
  <notes>
   - Squiz.Arrays.ArrayDeclaration now has improved handling of syntax errors
+  - Fixed bug #2368 : MySource.PHP.AjaxNullComparison throws error when first function has no doc comment 
   - Fixed bug #2414 : Indention false positive in switch/case/if combination
   - Fixed bug #2423 : Squiz.Formatting.OperatorBracket.MissingBrackets error with static
   - Fixed bug #2450 : Indentation false positive when closure containing nested IF conditions used as function argument

diff --git a/src/Standards/MySource/Tests/PHP/AjaxNullComparisonUnitTest.inc b/src/Standards/MySource/Tests/PHP/AjaxNullComparisonUnitTest.inc
@@ -1,4 +1,8 @@
 <?php
+
+public static function foo() {
+}
+
 /**
  * Adds a new issue.
  *
@@ -176,4 +180,3 @@ public static function addIssue(
     }
 
 }//end addIssue()
-?>
diff --git a/src/Standards/MySource/Tests/PHP/AjaxNullComparisonUnitTest.php b/src/Standards/MySource/Tests/PHP/AjaxNullComparisonUnitTest.php
@@ -41,12 +41,12 @@ public function getErrorList()
     public function getWarningList()
     {
         return [
-            37  => 1,
-            49  => 1,
-            60  => 1,
-            73  => 1,
-            88  => 1,
-            118 => 1,
+            41  => 1,
+            53  => 1,
+            64  => 1,
+            77  => 1,
+            92  => 1,
+            122 => 1,
         ];
 
     }//end getWarningList()