[DeadCode] Skip static property in RemoveDefaultValueFromAssignedPropertyRector

[TomasVotruba]

A static property is shared per class, not per instance. Removing its default value makes it uninitialized until the first constructor call, so any read before that (or without ever instantiating the class) throws Error: Typed static property must not be accessed before initialization.

ConstructorAssignDetector matches StaticPropertyFetch as well as PropertyFetch, so self::$value = $value; in the constructor was treated as "always assigned" and the default was dropped.

 final class SkipStaticProperty
 {
-    private static ?int $value = null;
+    private static ?int $value;

     public function __construct(int $value)
     {
         self::$value = $value;
     }

     public static function getValue(): ?int
     {
         return self::$value;
     }
 }

SkipStaticProperty::getValue() throws after the change, without ever constructing the class.

Static properties are now skipped.

[TomasVotruba]

Packagist is down, but feature is correct 👍