minor #5355 OrderedTraitsFixer - mark as risky (SpacePossum)

This PR was merged into the 2.17 branch.

Discussion
----------

OrderedTraitsFixer - mark as risky

closes #5350

Commits
-------

c8ea464 OrderedTraitsFixer - mark as risky

Author: SpacePossum

doc/rules/class_notation/ordered_traits.rst

@@ -4,6 +4,10 @@ Rule ``ordered_traits``
 
 Trait ``use`` statements must be sorted alphabetically.
 
+.. warning:: Using this rule is risky.
+
+   Risky when depending on order of the imports.
+
 Examples
 --------
 

doc/rules/index.rst

@@ -127,7 +127,7 @@ Class Notation
     Orders the elements of classes/interfaces/traits.
 - `ordered_interfaces <./class_notation/ordered_interfaces.rst>`_ *(risky)*
     Orders the interfaces in an ``implements`` or ``interface extends`` clause.
-- `ordered_traits <./class_notation/ordered_traits.rst>`_
+- `ordered_traits <./class_notation/ordered_traits.rst>`_ *(risky)*
     Trait ``use`` statements must be sorted alphabetically.
 - `protected_to_private <./class_notation/protected_to_private.rst>`_
     Converts ``protected`` variables and methods to ``private`` where possible.

src/Fixer/ClassNotation/OrderedTraitsFixer.php

@@ -29,7 +29,9 @@ public function getDefinition()
             'Trait `use` statements must be sorted alphabetically.',
             [
                 new CodeSample("<?php class Foo { \nuse Z; use A; }\n"),
-            ]
+            ],
+            null,
+            'Risky when depending on order of the imports.'
         );
     }
 
@@ -41,6 +43,14 @@ public function isCandidate(Tokens $tokens)
         return $tokens->isTokenKindFound(CT::T_USE_TRAIT);
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    public function isRisky()
+    {
+        return true;
+    }
+
     /**
      * {@inheritdoc}
      */