DX: do not override getConfigurationDefinition

Author: keradus

src/Fixer/Operator/OperatorLinebreakFixer.php

@@ -78,23 +78,6 @@ function foo() {
         );
     }
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getConfigurationDefinition()
-    {
-        return new FixerConfigurationResolver([
-            (new FixerOptionBuilder('only_booleans', 'whether to limit operators to only boolean ones'))
-                ->setAllowedTypes(['bool'])
-                ->setDefault(false)
-                ->getOption(),
-            (new FixerOptionBuilder('position', 'whether to place operators at the beginning or at the end of the line'))
-                ->setAllowedValues(['beginning', 'end'])
-                ->setDefault($this->position)
-                ->getOption(),
-        ]);
-    }
-
     /**
      * {@inheritdoc}
      */
@@ -121,6 +104,23 @@ public function isCandidate(Tokens $tokens)
         return true;
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    protected function createConfigurationDefinition()
+    {
+        return new FixerConfigurationResolver([
+            (new FixerOptionBuilder('only_booleans', 'whether to limit operators to only boolean ones'))
+                ->setAllowedTypes(['bool'])
+                ->setDefault(false)
+                ->getOption(),
+            (new FixerOptionBuilder('position', 'whether to place operators at the beginning or at the end of the line'))
+                ->setAllowedValues(['beginning', 'end'])
+                ->setDefault($this->position)
+                ->getOption(),
+        ]);
+    }
+
     /**
      * {@inheritdoc}
      */

src/Fixer/Whitespace/NoExtraConsecutiveBlankLinesFixer.php

@@ -41,11 +41,6 @@ public function configure(array $configuration = null)
         $this->configuration = $configuration;
     }
 
-    public function getConfigurationDefinition()
-    {
-        return $this->getFixer()->getConfigurationDefinition();
-    }
-
     /**
      * {@inheritdoc}
      */
@@ -54,6 +49,11 @@ public function getSuccessorsNames()
         return array_keys($this->proxyFixers);
     }
 
+    protected function createConfigurationDefinition()
+    {
+        return $this->getFixer()->createConfigurationDefinition();
+    }
+
     /**
      * {@inheritdoc}
      */