review fix

src/linter/quickfix.go

@@ -65,8 +65,6 @@ func (g *QuickFixGenerator) NullableType(param ir.Node) quickfix.TextEdit {
 			EndPos:   v.Position.EndPos,
 		}
 		value = v.Value
-	default:
-		panic("unexpected type")
 	}
 
 	return quickfix.TextEdit{

src/linter/report.go

@@ -29,7 +29,7 @@ func addBuiltinCheckers(reg *CheckersRegistry) {
 			Name:     "nullableType",
 			Default:  true,
 			Quickfix: true,
-			Comment:  "Report not nullable string can be null.",
+			Comment:  "Report not nullable param can be null.",
 			Before:   `function f(string $str = null);`,
 			After:    `function f(?string $str = null);`,
 		},

src/tests/checkers/nullable_types_test.go

@@ -97,3 +97,30 @@ class MyClass2 {
 	}
 	test.RunAndMatch()
 }
+
+func TestNullableMultipleArgs(t *testing.T) {
+	test := linttest.NewSuite(t)
+	test.AddFile(`<?php
+function multipleArgsExample(?string $a, ?int $b = null, ?bool $c = null) {
+	return 0;
+}
+`)
+
+	test.RunAndMatch()
+}
+
+func TestNotNullableMultipleArgs(t *testing.T) {
+	test := linttest.NewSuite(t)
+	test.AddFile(`<?php
+function multipleArgsExample(string $a, int $b = null, bool $c = null) {
+	return 0;
+}
+`)
+
+	test.Expect = []string{
+		"parameter with null default value should be explicitly nullable",
+		"parameter with null default value should be explicitly nullable",
+	}
+
+	test.RunAndMatch()
+}

src/tests/golden/testdata/quickfix/nullableTypes.php

@@ -1,6 +1,6 @@
 <?php
 
-function f(string $filed = null) {
+function notNullableString(string $filed = null) {
     return 1;
 }
 
@@ -19,4 +19,4 @@ public function myMethod(MyClass1 $a = null) {
 
 function nullableArray(array $a = null) {
 	return 0;
-}
+}

src/tests/golden/testdata/quickfix/nullableTypes.php.fix.expected

@@ -1,6 +1,6 @@
 <?php
 
-function f(?string $filed = null) {
+function notNullableString(?string $filed = null) {
     return 1;
 }
 
@@ -19,4 +19,4 @@ class MyClass2 {
 
 function nullableArray(?array $a = null) {
 	return 0;
-}
+}