Rename confusing PHPStan rule name

it never occurred to me that this was misleading until I read some Devin documentation,
noticed that Devin misunderstood was the class was for, and then realized actually
Devin understood correctly, and it was the name of the class that was wrong. Funny
how that happens...

phpstan.neon.dist

@@ -15,7 +15,7 @@ rules:
 	- pocketmine\phpstan\rules\DisallowEnumComparisonRule
 	- pocketmine\phpstan\rules\DisallowForeachByReferenceRule
 	- pocketmine\phpstan\rules\ExplodeLimitRule
-	- pocketmine\phpstan\rules\UnsafeForeachArrayOfStringRule
+	- pocketmine\phpstan\rules\UnsafeForeachArrayWithStringKeysRule
 #	- pocketmine\phpstan\rules\ThreadedSupportedTypesRule
 
 parameters:

tests/phpstan/configs/phpstan-bugs.neon

@@ -256,5 +256,5 @@ parameters:
 			message: '#^Strict comparison using \=\=\= between 0 and 0 will always evaluate to true\.$#'
 			identifier: identical.alwaysTrue
 			count: 1
-			path: ../rules/UnsafeForeachArrayOfStringRule.php
+			path: ../rules/UnsafeForeachArrayWithStringKeysRule.php
 

tests/phpstan/rules/UnsafeForeachArrayWithStringKeysRule.php

@@ -41,7 +41,7 @@ use function sprintf;
 /**
  * @implements Rule<Foreach_>
  */
-final class UnsafeForeachArrayOfStringRule implements Rule{
+final class UnsafeForeachArrayWithStringKeysRule implements Rule{
 
 	public function getNodeType() : string{
 		return Foreach_::class;