fix for issue #3963

.automation/test/php/.mega-linter.yml.sample

@@ -0,0 +1,15 @@
+ENABLE:
+  - PHP
+DISABLE_LINTERS:
+  - PHP_PSALM
+  - PHP_PHPLINT
+  - PHP_PHPCS
+  - PHP_PHPSTAN
+LOG_LEVEL: debug
+PARALLEL: false
+UPDATED_SOURCES_REPORTER: true
+APPLY_FIXES: PHP_PHPCSFIXER
+PHP_PHPCSFIXER_CONFIG_FILE: .php-cs-fixer.risky.php
+PHP_PHPCSFIXER_ARGUMENTS:
+  - "--allow-risky=yes"
+  - "--diff"

.automation/test/php/.php-cs-fixer.risky.php

@@ -0,0 +1,13 @@
+<?php
+
+$finder = (new PhpCsFixer\Finder())
+    ->in('.')
+;
+
+return (new PhpCsFixer\Config())
+    ->setRules([
+        '@PER-CS' => true,
+        '@PhpCsFixer:risky' => true,
+    ])
+    ->setFinder($finder)
+;

.automation/test/php/php_fix_1.php

@@ -0,0 +1,3 @@
+<?php
+
+$x[] = $y;

megalinter/Linter.py

@@ -256,6 +256,8 @@ def __init__(self, params=None, linter_config=None):
             # No fixing config on linter descriptor
             if self.cli_lint_fix_arg_name is None:
                 self.apply_fixes = False
+            if len(self.cli_lint_fix_remove_args) == 0:
+                self.apply_fixes = False
             # APPLY_FIXES is "all"
             elif param_apply_fixes == "all" or (
                 isinstance(param_apply_fixes, bool) and param_apply_fixes is True
@@ -1260,12 +1262,14 @@ def build_lint_command(self, file=None) -> list:
         # Add fix argument if defined
         if self.apply_fixes is True and (
             self.cli_lint_fix_arg_name is not None
+            or len(self.cli_lint_fix_remove_args) > 0
             or str(self.cli_executable_fix) != str(self.cli_executable)
         ):
             args_pos = len(self.cli_executable)
             cmd = cmd[args_pos:]  # Remove executable elements
             cmd = self.cli_executable_fix + cmd
-            cmd += [self.cli_lint_fix_arg_name]
+            if self.cli_lint_fix_arg_name is not None:
+                cmd += [self.cli_lint_fix_arg_name]
             self.try_fix = True
 
         # Add user-defined extra arguments if defined