Output warnings for deprecated rules and options

PHP-CS-Fixer · Nov 9, 2019 · d3ed10a · d3ed10a
1 parent f2928db
commit d3ed10a
Show file tree

Hide file tree

Showing 8 changed files with 64 additions and 10 deletions.
diff --git a/php-cs-fixer b/php-cs-fixer
@@ -85,6 +85,7 @@ $xdebug->check();
 unset($xdebug);
 
 $application = new Application();
+$application->enableDeprecationWarnings();
 $application->run();
 
 __HALT_COMPILER();
diff --git a/src/AbstractFixer.php b/src/AbstractFixer.php
@@ -124,7 +124,7 @@ public function configure(array $configuration = null)
                 throw new \InvalidArgumentException("{$message} This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
             }
 
-            @trigger_error($message, E_USER_DEPRECATED);
+            Application::triggerDeprecation($message);
 
             $configuration = [];
         }
@@ -148,7 +148,7 @@ public function configure(array $configuration = null)
                     throw new \InvalidArgumentException("{$message} This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
                 }
 
-                @trigger_error($message, E_USER_DEPRECATED);
+                Application::triggerDeprecation($message);
             }
         }
 

diff --git a/src/Console/Application.php b/src/Console/Application.php
@@ -43,6 +43,12 @@ final class Application extends BaseApplication
      */
     private $toolInfo;
 
+    private $deprecationWarningsEnabled = false;
+
+    private $customErrorHandlerRegistered = false;
+
+    private static $printCurrentDeprecationNotice = false;
+
     public function __construct()
     {
         if (!getenv('PHP_CS_FIXER_FUTURE_MODE')) {
@@ -63,6 +69,21 @@ public function __construct()
         ));
     }
 
+    public function enableDeprecationWarnings()
+    {
+        $this->deprecationWarningsEnabled = true;
+    }
+
+    /**
+     * @param string $message
+     */
+    public static function triggerDeprecation($message)
+    {
+        self::$printCurrentDeprecationNotice = true;
+        @trigger_error($message, E_USER_DEPRECATED);
+        self::$printCurrentDeprecationNotice = false;
+    }
+
     /**
      * {@inheritdoc}
      */
@@ -72,7 +93,24 @@ public function doRun(InputInterface $input, OutputInterface $output)
             ? $output->getErrorOutput()
             : ($input->hasParameterOption('--format', true) && 'txt' !== $input->getParameterOption('--format', null, true) ? null : $output)
         ;
+
+        $previousErrorHandler = null;
+
         if (null !== $stdErr) {
+            if ($this->deprecationWarningsEnabled && !$this->customErrorHandlerRegistered) {
+                $previousErrorHandler = set_error_handler(function ($severity, $message, $file, $line) use (&$previousErrorHandler, $stdErr) {
+                    if (self::$printCurrentDeprecationNotice && $severity & E_USER_DEPRECATED) {
+                        $stdErr->writeln("<bg=yellow;fg=black;>{$message}</>");
+                    }
+
+                    if (\is_callable($previousErrorHandler)) {
+                        $previousErrorHandler($severity, $message, $file, $line);
+                    }
+                });
+
+                $this->customErrorHandlerRegistered = true;
+            }
+
             $warningsDetector = new WarningsDetector($this->toolInfo);
             $warningsDetector->detectOldVendor();
             $warningsDetector->detectOldMajor();
@@ -81,7 +119,17 @@ public function doRun(InputInterface $input, OutputInterface $output)
             }
         }
 
-        return parent::doRun($input, $output);
+        $exitCode = parent::doRun($input, $output);
+
+        if (null !== $previousErrorHandler) {
+            set_error_handler($previousErrorHandler);
+        } else {
+            restore_error_handler();
+        }
+
+        $this->customErrorHandlerRegistered = false;
+
+        return $exitCode;
     }
 
     /**

diff --git a/src/Console/ConfigurationResolver.php b/src/Console/ConfigurationResolver.php
@@ -430,7 +430,7 @@ public function getProgress()
                         throw new \InvalidArgumentException("{$message} This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
                     }
 
-                    @trigger_error($message, E_USER_DEPRECATED);
+                    Application::triggerDeprecation($message);
                 }
 
                 $this->progress = $progressType;
@@ -767,7 +767,7 @@ private function validateRules(array $rules)
                     throw new \RuntimeException("{$message} This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
                 }
 
-                @trigger_error($message, E_USER_DEPRECATED);
+                Application::triggerDeprecation($message);
             }
         }
     }
@@ -919,7 +919,7 @@ private function resolveOptionBooleanValue($optionName)
             throw new InvalidConfigurationException("{$message} This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
         }
 
-        @trigger_error($message, E_USER_DEPRECATED);
+        Application::triggerDeprecation($message);
 
         return false;
     }

diff --git a/src/Fixer/Operator/BinaryOperatorSpacesFixer.php b/src/Fixer/Operator/BinaryOperatorSpacesFixer.php
@@ -14,6 +14,7 @@
 
 use PhpCsFixer\AbstractFixer;
 use PhpCsFixer\ConfigurationException\InvalidFixerConfigurationException;
+use PhpCsFixer\Console\Application;
 use PhpCsFixer\Console\Command\HelpCommand;
 use PhpCsFixer\Fixer\ConfigurationDefinitionFixerInterface;
 use PhpCsFixer\FixerConfiguration\FixerConfigurationResolver;
@@ -524,7 +525,7 @@ private function resolveOldConfig(array $configuration)
             throw new InvalidFixerConfigurationException($this->getName(), "{$message} This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
         }
 
-        @trigger_error($message, E_USER_DEPRECATED);
+        Application::triggerDeprecation($message);
 
         return $newConfig;
     }

diff --git a/src/Fixer/Whitespace/NoExtraBlankLinesFixer.php b/src/Fixer/Whitespace/NoExtraBlankLinesFixer.php
@@ -14,6 +14,7 @@
 
 use PhpCsFixer\AbstractFixer;
 use PhpCsFixer\ConfigurationException\InvalidConfigurationException;
+use PhpCsFixer\Console\Application;
 use PhpCsFixer\Fixer\ConfigurationDefinitionFixerInterface;
 use PhpCsFixer\Fixer\WhitespacesAwareFixerInterface;
 use PhpCsFixer\FixerConfiguration\AllowedValueSubset;
@@ -323,7 +324,7 @@ protected function createConfigurationDefinition()
                                 throw new InvalidConfigurationException("{$message} This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
                             }
 
-                            @trigger_error($message, E_USER_DEPRECATED);
+                            Application::triggerDeprecation($message);
                             $token = 'use_trait';
 
                             break;

diff --git a/src/FixerConfiguration/FixerConfigurationResolver.php b/src/FixerConfiguration/FixerConfigurationResolver.php
@@ -12,6 +12,7 @@
 
 namespace PhpCsFixer\FixerConfiguration;
 
+use PhpCsFixer\Console\Application;
 use Symfony\Component\OptionsResolver\Exception\InvalidOptionsException;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 
@@ -67,7 +68,7 @@ public function resolve(array $options)
                         throw new InvalidOptionsException(sprintf('Aliased option %s/%s is passed multiple times.', $name, $alias));
                     }
 
-                    @trigger_error(sprintf('Option "%s" is deprecated, use "%s" instead.', $alias, $name), E_USER_DEPRECATED);
+                    Application::triggerDeprecation(sprintf('Option "%s" is deprecated, use "%s" instead.', $alias, $name));
 
                     $options[$name] = $options[$alias];
                     unset($options[$alias]);

diff --git a/src/FixerConfiguration/FixerConfigurationResolverRootless.php b/src/FixerConfiguration/FixerConfigurationResolverRootless.php
@@ -12,6 +12,8 @@
 
 namespace PhpCsFixer\FixerConfiguration;
 
+use PhpCsFixer\Console\Application;
+
 /**
  * @internal
  *
@@ -88,7 +90,7 @@ function (FixerOptionInterface $option) {
                     throw new \RuntimeException("{$message}. This check was performed as `PHP_CS_FIXER_FUTURE_MODE` env var is set.");
                 }
 
-                @trigger_error($message, E_USER_DEPRECATED);
+                Application::triggerDeprecation($message);
 
                 $options = [$this->root => $options];
             }