Make fixer risky

Author: kubawerlos

README.md

@@ -168,6 +168,7 @@ Empty function body must be abbreviated as `{}` and placed on the same line as t
 
 #### ForeachUseValueFixer
 Value from `foreach` must not be used if possible.
+  *Risky: when the value is re-used or being sorted.*
 ```diff
  <?php
  foreach ($elements as $key => $value) {

src/Fixer/ForeachUseValueFixer.php

@@ -57,6 +57,7 @@ public function getDefinition(): FixerDefinitionInterface
                     PHP,
             )],
             '',
+            'when the value is re-used or being sorted',
         );
     }
 
@@ -72,7 +73,7 @@ public function isCandidate(Tokens $tokens): bool
 
     public function isRisky(): bool
     {
-        return false;
+        return true;
     }
 
     public function fix(\SplFileInfo $file, Tokens $tokens): void
@@ -155,11 +156,7 @@ private function fixForeachBody(
         string $variableName
     ): void {
         $sequence = [
-            $arrayToken,
-            '[',
-            [\T_VARIABLE, $keyName],
-            ']',
-        ];
+            $arrayToken, '[', [\T_VARIABLE, $keyName], ']'];
 
         $index = $openBraceIndex;
         while (($found = $tokens->findSequence($sequence, $index, $closeBraceIndex)) !== null) {

tests/Fixer/ForeachUseValueFixerTest.php

@@ -20,7 +20,7 @@ final class ForeachUseValueFixerTest extends AbstractFixerTestCase
 {
     public function testIsRisky(): void
     {
-        self::assertRiskiness(false);
+        self::assertRiskiness(true);
     }
 
     /**