From 02cd847adc95a8a48d5d6e2e4b68ac30c71520ae Mon Sep 17 00:00:00 2001
From: Geert Eltink <geert.eltink@gmail.com>
Date: Sun, 25 Oct 2020 07:05:06 +0100
Subject: [PATCH] fix: disable automatic missing native typehint fixing

Fixes #47

Signed-off-by: Geert Eltink <geert.eltink@gmail.com>
---
 src/LaminasCodingStandard/ruleset.xml     |  3 +++
 test/expected-report.txt                  | 10 +++++-----
 test/fixable/4.1.ExtendsAndImplements.php |  5 +++++
 test/fixable/7.Closures.php               |  8 ++++++--
 test/fixed/4.1.ExtendsAndImplements.php   |  6 ++++++
 test/fixed/4.4.MethodsAndFunctions.php    |  2 +-
 test/fixed/7.Closures.php                 |  6 +++++-
 7 files changed, 31 insertions(+), 9 deletions(-)

diff --git a/src/LaminasCodingStandard/ruleset.xml b/src/LaminasCodingStandard/ruleset.xml
index ac1786d..5399a3e 100644
--- a/src/LaminasCodingStandard/ruleset.xml
+++ b/src/LaminasCodingStandard/ruleset.xml
@@ -808,6 +808,9 @@
         </properties>
     </rule>
     <rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHint"/>
+    <rule ref="SlevomatCodingStandard.TypeHints.ReturnTypeHint">
+        <exclude name="SlevomatCodingStandard.TypeHints.ReturnTypeHint.MissingNativeTypeHint"/>
+    </rule>
     <rule ref="SlevomatCodingStandard.TypeHints.UselessConstantTypeHint"/>
 
     <!-- The asterisks in a DocBlock should align, and there should be one
diff --git a/test/expected-report.txt b/test/expected-report.txt
index 17a0e2b..76bff45 100644
--- a/test/expected-report.txt
+++ b/test/expected-report.txt
@@ -11,10 +11,10 @@ test/fixable/2.5.KeywordsAndTypes.php                 8       0
 test/fixable/2.6.Variables.php                        3       0
 test/fixable/2.7.Arrays.php                           9       0
 test/fixable/3.DeclareNamespaceAndImport.php          12      0
-test/fixable/4.1.ExtendsAndImplements.php             5       0
+test/fixable/4.1.ExtendsAndImplements.php             6       0
 test/fixable/4.2.UsingTraits.php                      5       0
 test/fixable/4.3.PropertiesAndConstants.php           7       0
-test/fixable/4.4.MethodsAndFunctions.php              27      0
+test/fixable/4.4.MethodsAndFunctions.php              26      0
 test/fixable/4.5.MethodAndFunctionArguments.php       66      0
 test/fixable/4.6.AbstractFinalAndStatic.php           5       0
 test/fixable/4.7.MethodAndFunctionCalls.php           11      0
@@ -30,14 +30,14 @@ test/fixable/6.1.UnaryOperators.php                   6       0
 test/fixable/6.2.BinaryOperators.php                  35      0
 test/fixable/6.3.TernaryOperators.php                 13      0
 test/fixable/6.Operators.php                          97      0
-test/fixable/7.Closures.php                           4       0
+test/fixable/7.Closures.php                           2       0
 test/fixable/8.AnonymousClasses.php                   1       0
 test/fixable/9.CommentingAndDocBlocks.php             17      0
 test/fixable/9.GenericTypeHintSyntax.php              4       0
 ----------------------------------------------------------------------
-A TOTAL OF 568 ERRORS AND 2 WARNINGS WERE FOUND IN 31 FILES
+A TOTAL OF 566 ERRORS AND 2 WARNINGS WERE FOUND IN 31 FILES
 ----------------------------------------------------------------------
-PHPCBF CAN FIX 484 OF THESE SNIFF VIOLATIONS AUTOMATICALLY
+PHPCBF CAN FIX 483 OF THESE SNIFF VIOLATIONS AUTOMATICALLY
 ----------------------------------------------------------------------
 
 
diff --git a/test/fixable/4.1.ExtendsAndImplements.php b/test/fixable/4.1.ExtendsAndImplements.php
index 46dede2..d4168d8 100644
--- a/test/fixable/4.1.ExtendsAndImplements.php
+++ b/test/fixable/4.1.ExtendsAndImplements.php
@@ -16,6 +16,11 @@ class ExtendsAndImplements
     ArrayAccess,
     Countable,
     Serializable {
+    public function testImplementedMethodWithoutTypeHint()
+    {
+        // This method should not have the missing return type added after
+        // running the fixer.
+    }
     public function testClassDeclaration(): void
     {
         // The extends and implements keywords MUST be declared on the same line
diff --git a/test/fixable/7.Closures.php b/test/fixable/7.Closures.php
index 96fd3b4..5902b44 100644
--- a/test/fixable/7.Closures.php
+++ b/test/fixable/7.Closures.php
@@ -41,6 +41,10 @@ public function testClosures(): void
             echo "$arg1, $arg2";
         };
 
+        $closureWithoutReturnType = function ($arg1, $arg2) {
+            echo "$arg1, $arg2";
+        };
+
         $closureWithArgsAndVars = function ($arg1, $arg2) use ($var1, $var2): void {
             echo "$arg1, $arg2, $var1, $var2";
         };
@@ -49,7 +53,7 @@ public function testClosures(): void
             return $arg1 === $arg2 && $var1 === $var2;
         };
 
-        $multiLineCLosre = function (
+        $multiLineClosure = function (
             $arg1,
             $arg2
         ) use (
@@ -65,7 +69,7 @@ public function testInheritedVariablesMustBeUsed(): void
         // Inherited variables passed via `use` MUST be used in closures.
 
         $message = 'world';
-        $example = function ($arg) use ($message, $extraVar) {
+        $example = function ($arg) use ($message, $extraVar): void {
             echo "$arg $message";
         };
         $example('hello');
diff --git a/test/fixed/4.1.ExtendsAndImplements.php b/test/fixed/4.1.ExtendsAndImplements.php
index 6981cbe..00b5635 100644
--- a/test/fixed/4.1.ExtendsAndImplements.php
+++ b/test/fixed/4.1.ExtendsAndImplements.php
@@ -14,6 +14,12 @@ class ExtendsAndImplements extends ParentClass implements
     Countable,
     Serializable
 {
+    public function testImplementedMethodWithoutTypeHint()
+    {
+        // This method should not have the missing return type added after
+        // running the fixer.
+    }
+
     public function testClassDeclaration(): void
     {
         // The extends and implements keywords MUST be declared on the same line
diff --git a/test/fixed/4.4.MethodsAndFunctions.php b/test/fixed/4.4.MethodsAndFunctions.php
index fb3b7a1..32d6a56 100644
--- a/test/fixed/4.4.MethodsAndFunctions.php
+++ b/test/fixed/4.4.MethodsAndFunctions.php
@@ -6,7 +6,7 @@
 
 class MethodsAndFunctions
 {
-    public function testMethodsAndFunctions($arg1, &$arg2, $arg3 = []): void
+    public function testMethodsAndFunctions($arg1, &$arg2, $arg3 = [])
     {
         // Visibility MUST be declared on all methods.
         //
diff --git a/test/fixed/7.Closures.php b/test/fixed/7.Closures.php
index 966f2ed..995aa75 100644
--- a/test/fixed/7.Closures.php
+++ b/test/fixed/7.Closures.php
@@ -41,6 +41,10 @@ public function testClosures(): void
             echo "$arg1, $arg2";
         };
 
+        $closureWithoutReturnType = function ($arg1, $arg2) {
+            echo "$arg1, $arg2";
+        };
+
         $closureWithArgsAndVars = function ($arg1, $arg2) use ($var1, $var2): void {
             echo "$arg1, $arg2, $var1, $var2";
         };
@@ -49,7 +53,7 @@ public function testClosures(): void
             return $arg1 === $arg2 && $var1 === $var2;
         };
 
-        $multiLineCLosre = function (
+        $multiLineClosure = function (
             $arg1,
             $arg2
         ) use (