Apply php-cs-fixer fix --rules nullable_type_declaration_for_default_null_value

nicolas-grekas · nicolas-grekas · commit ff424ed4736e · 2024-01-23T14:51:25.000+01:00
diff --git a/Definition.php b/Definition.php
@@ -32,7 +32,7 @@ final class Definition
      * @param Transition[]         $transitions
      * @param string|string[]|null $initialPlaces
      */
-    public function __construct(array $places, array $transitions, $initialPlaces = null, MetadataStoreInterface $metadataStore = null)
+    public function __construct(array $places, array $transitions, $initialPlaces = null, ?MetadataStoreInterface $metadataStore = null)
     {
         foreach ($places as $place) {
             $this->addPlace($place);
diff --git a/Dumper/DumperInterface.php b/Dumper/DumperInterface.php
@@ -27,5 +27,5 @@ interface DumperInterface
      *
      * @return string
      */
-    public function dump(Definition $definition, Marking $marking = null, array $options = []);
+    public function dump(Definition $definition, ?Marking $marking = null, array $options = []);
 }
diff --git a/Dumper/GraphvizDumper.php b/Dumper/GraphvizDumper.php
@@ -44,7 +44,7 @@ class GraphvizDumper implements DumperInterface
      *  * node: The default options for nodes (places + transitions)
      *  * edge: The default options for edges
      */
-    public function dump(Definition $definition, Marking $marking = null, array $options = [])
+    public function dump(Definition $definition, ?Marking $marking = null, array $options = [])
     {
         $places = $this->findPlaces($definition, $marking);
         $transitions = $this->findTransitions($definition);
@@ -62,7 +62,7 @@ public function dump(Definition $definition, Marking $marking = null, array $opt
     /**
      * @internal
      */
-    protected function findPlaces(Definition $definition, Marking $marking = null): array
+    protected function findPlaces(Definition $definition, ?Marking $marking = null): array
     {
         $workflowMetadata = $definition->getMetadataStore();
 
diff --git a/Dumper/MermaidDumper.php b/Dumper/MermaidDumper.php
@@ -66,7 +66,7 @@ public function __construct(string $transitionType, string $direction = self::DI
         $this->transitionType = $transitionType;
     }
 
-    public function dump(Definition $definition, Marking $marking = null, array $options = []): string
+    public function dump(Definition $definition, ?Marking $marking = null, array $options = []): string
     {
         $this->linkCount = 0;
         $placeNameMap = [];
diff --git a/Dumper/PlantUmlDumper.php b/Dumper/PlantUmlDumper.php
@@ -53,15 +53,15 @@ class PlantUmlDumper implements DumperInterface
 
     private $transitionType = self::STATEMACHINE_TRANSITION;
 
-    public function __construct(string $transitionType = null)
+    public function __construct(?string $transitionType = null)
     {
         if (!\in_array($transitionType, self::TRANSITION_TYPES, true)) {
             throw new \InvalidArgumentException("Transition type '$transitionType' does not exist.");
         }
         $this->transitionType = $transitionType;
     }
 
-    public function dump(Definition $definition, Marking $marking = null, array $options = []): string
+    public function dump(Definition $definition, ?Marking $marking = null, array $options = []): string
     {
         $options = array_replace_recursive(self::DEFAULT_OPTIONS, $options);
 
@@ -191,7 +191,7 @@ private function escape(string $string): string
         return '"'.str_replace('"', '', $string).'"';
     }
 
-    private function getState(string $place, Definition $definition, Marking $marking = null): string
+    private function getState(string $place, Definition $definition, ?Marking $marking = null): string
     {
         $workflowMetadata = $definition->getMetadataStore();
 
diff --git a/Dumper/StateMachineGraphvizDumper.php b/Dumper/StateMachineGraphvizDumper.php
@@ -27,7 +27,7 @@ class StateMachineGraphvizDumper extends GraphvizDumper
      *  * node: The default options for nodes (places)
      *  * edge: The default options for edges
      */
-    public function dump(Definition $definition, Marking $marking = null, array $options = [])
+    public function dump(Definition $definition, ?Marking $marking = null, array $options = [])
     {
         $places = $this->findPlaces($definition, $marking);
         $edges = $this->findEdges($definition);
diff --git a/Event/Event.php b/Event/Event.php
@@ -29,7 +29,7 @@ class Event extends BaseEvent
     private $transition;
     private $workflow;
 
-    public function __construct(object $subject, Marking $marking, Transition $transition = null, WorkflowInterface $workflow = null, array $context = [])
+    public function __construct(object $subject, Marking $marking, ?Transition $transition = null, ?WorkflowInterface $workflow = null, array $context = [])
     {
         $this->subject = $subject;
         $this->marking = $marking;
diff --git a/Event/GuardEvent.php b/Event/GuardEvent.php
@@ -28,7 +28,7 @@ final class GuardEvent extends Event
     /**
      * {@inheritdoc}
      */
-    public function __construct(object $subject, Marking $marking, Transition $transition, WorkflowInterface $workflow = null)
+    public function __construct(object $subject, Marking $marking, Transition $transition, ?WorkflowInterface $workflow = null)
     {
         parent::__construct($subject, $marking, $transition, $workflow);
 
@@ -45,7 +45,7 @@ public function isBlocked(): bool
         return !$this->transitionBlockerList->isEmpty();
     }
 
-    public function setBlocked(bool $blocked, string $message = null): void
+    public function setBlocked(bool $blocked, ?string $message = null): void
     {
         if (!$blocked) {
             $this->transitionBlockerList->clear();
diff --git a/EventListener/GuardListener.php b/EventListener/GuardListener.php
@@ -32,7 +32,7 @@ class GuardListener
     private $roleHierarchy;
     private $validator;
 
-    public function __construct(array $configuration, ExpressionLanguage $expressionLanguage, TokenStorageInterface $tokenStorage, AuthorizationCheckerInterface $authorizationChecker, AuthenticationTrustResolverInterface $trustResolver, RoleHierarchyInterface $roleHierarchy = null, ValidatorInterface $validator = null)
+    public function __construct(array $configuration, ExpressionLanguage $expressionLanguage, TokenStorageInterface $tokenStorage, AuthorizationCheckerInterface $authorizationChecker, AuthenticationTrustResolverInterface $trustResolver, ?RoleHierarchyInterface $roleHierarchy = null, ?ValidatorInterface $validator = null)
     {
         $this->configuration = $configuration;
         $this->expressionLanguage = $expressionLanguage;
diff --git a/Metadata/InMemoryMetadataStore.php b/Metadata/InMemoryMetadataStore.php
@@ -27,7 +27,7 @@ final class InMemoryMetadataStore implements MetadataStoreInterface
     /**
      * @param \SplObjectStorage<Transition, array>|null $transitionsMetadata
      */
-    public function __construct(array $workflowMetadata = [], array $placesMetadata = [], \SplObjectStorage $transitionsMetadata = null)
+    public function __construct(array $workflowMetadata = [], array $placesMetadata = [], ?\SplObjectStorage $transitionsMetadata = null)
     {
         $this->workflowMetadata = $workflowMetadata;
         $this->placesMetadata = $placesMetadata;
diff --git a/Registry.php b/Registry.php
@@ -27,7 +27,7 @@ public function addWorkflow(WorkflowInterface $workflow, WorkflowSupportStrategy
         $this->workflows[] = [$workflow, $supportStrategy];
     }
 
-    public function has(object $subject, string $workflowName = null): bool
+    public function has(object $subject, ?string $workflowName = null): bool
     {
         foreach ($this->workflows as [$workflow, $supportStrategy]) {
             if ($this->supports($workflow, $supportStrategy, $subject, $workflowName)) {
@@ -41,7 +41,7 @@ public function has(object $subject, string $workflowName = null): bool
     /**
      * @return Workflow
      */
-    public function get(object $subject, string $workflowName = null)
+    public function get(object $subject, ?string $workflowName = null)
     {
         $matched = [];
 
diff --git a/StateMachine.php b/StateMachine.php
@@ -20,7 +20,7 @@
  */
 class StateMachine extends Workflow
 {
-    public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, string $name = 'unnamed', array $eventsToDispatch = null)
+    public function __construct(Definition $definition, ?MarkingStoreInterface $markingStore = null, ?EventDispatcherInterface $dispatcher = null, string $name = 'unnamed', ?array $eventsToDispatch = null)
     {
         parent::__construct($definition, $markingStore ?? new MethodMarkingStore(true), $dispatcher, $name, $eventsToDispatch);
     }
diff --git a/Tests/EventListener/GuardListenerTest.php b/Tests/EventListener/GuardListenerTest.php
@@ -148,7 +148,7 @@ public function testGuardExpressionBlocks()
         $this->assertTrue($event->isBlocked());
     }
 
-    private function createEvent(Transition $transition = null)
+    private function createEvent(?Transition $transition = null)
     {
         $subject = new Subject();
         $transition = $transition ?? new Transition('name', 'from', 'to');
diff --git a/Tests/WorkflowTest.php b/Tests/WorkflowTest.php
@@ -791,7 +791,7 @@ class EventDispatcherMock implements \Symfony\Contracts\EventDispatcher\EventDis
 {
     public $dispatchedEvents = [];
 
-    public function dispatch($event, string $eventName = null): object
+    public function dispatch($event, ?string $eventName = null): object
     {
         $this->dispatchedEvents[] = $eventName;
 
diff --git a/TransitionBlocker.php b/TransitionBlocker.php
@@ -67,7 +67,7 @@ public static function createBlockedByExpressionGuardListener(string $expression
      * Creates a blocker that says the transition cannot be made because of an
      * unknown reason.
      */
-    public static function createUnknown(string $message = null, int $backtraceFrame = 2): self
+    public static function createUnknown(?string $message = null, int $backtraceFrame = 2): self
     {
         if (null !== $message) {
             return new static($message, self::UNKNOWN);
diff --git a/Workflow.php b/Workflow.php
@@ -67,7 +67,7 @@ class Workflow implements WorkflowInterface
      */
     private $eventsToDispatch = null;
 
-    public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, string $name = 'unnamed', array $eventsToDispatch = null)
+    public function __construct(Definition $definition, ?MarkingStoreInterface $markingStore = null, ?EventDispatcherInterface $dispatcher = null, string $name = 'unnamed', ?array $eventsToDispatch = null)
     {
         $this->definition = $definition;
         $this->markingStore = $markingStore ?? new MethodMarkingStore();

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ final class Definition`
`32`	`32`	`* @param Transition[] $transitions`
`33`	`33`	`* @param string\|string[]\|null $initialPlaces`
`34`	`34`	`*/`
`35`		`- public function __construct(array $places, array $transitions, $initialPlaces = null, MetadataStoreInterface $metadataStore = null)`
	`35`	`+ public function __construct(array $places, array $transitions, $initialPlaces = null, ?MetadataStoreInterface $metadataStore = null)`
`36`	`36`	`{`
`37`	`37`	`foreach ($places as $place) {`
`38`	`38`	`$this->addPlace($place);`
Original file line number	Diff line number	Diff line change
`@@ -27,5 +27,5 @@ interface DumperInterface`
`27`	`27`	`*`
`28`	`28`	`* @return string`
`29`	`29`	`*/`
`30`		`- public function dump(Definition $definition, Marking $marking = null, array $options = []);`
	`30`	`+ public function dump(Definition $definition, ?Marking $marking = null, array $options = []);`
`31`	`31`	`}`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public function __construct(string $transitionType, string $direction = self::DI`
`66`	`66`	`$this->transitionType = $transitionType;`
`67`	`67`	`}`
`68`	`68`
`69`		`- public function dump(Definition $definition, Marking $marking = null, array $options = []): string`
	`69`	`+ public function dump(Definition $definition, ?Marking $marking = null, array $options = []): string`
`70`	`70`	`{`
`71`	`71`	`$this->linkCount = 0;`
`72`	`72`	`$placeNameMap = [];`
Original file line number	Diff line number	Diff line change
`@@ -53,15 +53,15 @@ class PlantUmlDumper implements DumperInterface`
`53`	`53`
`54`	`54`	`private $transitionType = self::STATEMACHINE_TRANSITION;`
`55`	`55`
`56`		`- public function __construct(string $transitionType = null)`
	`56`	`+ public function __construct(?string $transitionType = null)`
`57`	`57`	`{`
`58`	`58`	`if (!\in_array($transitionType, self::TRANSITION_TYPES, true)) {`
`59`	`59`	`throw new \InvalidArgumentException("Transition type '$transitionType' does not exist.");`
`60`	`60`	`}`
`61`	`61`	`$this->transitionType = $transitionType;`
`62`	`62`	`}`
`63`	`63`
`64`		`- public function dump(Definition $definition, Marking $marking = null, array $options = []): string`
	`64`	`+ public function dump(Definition $definition, ?Marking $marking = null, array $options = []): string`
`65`	`65`	`{`
`66`	`66`	`$options = array_replace_recursive(self::DEFAULT_OPTIONS, $options);`
`67`	`67`
`@@ -191,7 +191,7 @@ private function escape(string $string): string`
`191`	`191`	`return '"'.str_replace('"', '', $string).'"';`
`192`	`192`	`}`
`193`	`193`
`194`		`- private function getState(string $place, Definition $definition, Marking $marking = null): string`
	`194`	`+ private function getState(string $place, Definition $definition, ?Marking $marking = null): string`
`195`	`195`	`{`
`196`	`196`	`$workflowMetadata = $definition->getMetadataStore();`
`197`	`197`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class StateMachineGraphvizDumper extends GraphvizDumper`
`27`	`27`	`* * node: The default options for nodes (places)`
`28`	`28`	`* * edge: The default options for edges`
`29`	`29`	`*/`
`30`		`- public function dump(Definition $definition, Marking $marking = null, array $options = [])`
	`30`	`+ public function dump(Definition $definition, ?Marking $marking = null, array $options = [])`
`31`	`31`	`{`
`32`	`32`	`$places = $this->findPlaces($definition, $marking);`
`33`	`33`	`$edges = $this->findEdges($definition);`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ class Event extends BaseEvent`
`29`	`29`	`private $transition;`
`30`	`30`	`private $workflow;`
`31`	`31`
`32`		`- public function __construct(object $subject, Marking $marking, Transition $transition = null, WorkflowInterface $workflow = null, array $context = [])`
	`32`	`+ public function __construct(object $subject, Marking $marking, ?Transition $transition = null, ?WorkflowInterface $workflow = null, array $context = [])`
`33`	`33`	`{`
`34`	`34`	`$this->subject = $subject;`
`35`	`35`	`$this->marking = $marking;`
Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ final class GuardEvent extends Event`
`28`	`28`	`/**`
`29`	`29`	`* {@inheritdoc}`
`30`	`30`	`*/`
`31`		`- public function __construct(object $subject, Marking $marking, Transition $transition, WorkflowInterface $workflow = null)`
	`31`	`+ public function __construct(object $subject, Marking $marking, Transition $transition, ?WorkflowInterface $workflow = null)`
`32`	`32`	`{`
`33`	`33`	`parent::__construct($subject, $marking, $transition, $workflow);`
`34`	`34`
`@@ -45,7 +45,7 @@ public function isBlocked(): bool`
`45`	`45`	`return !$this->transitionBlockerList->isEmpty();`
`46`	`46`	`}`
`47`	`47`
`48`		`- public function setBlocked(bool $blocked, string $message = null): void`
	`48`	`+ public function setBlocked(bool $blocked, ?string $message = null): void`
`49`	`49`	`{`
`50`	`50`	`if (!$blocked) {`
`51`	`51`	`$this->transitionBlockerList->clear();`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ class GuardListener`
`32`	`32`	`private $roleHierarchy;`
`33`	`33`	`private $validator;`
`34`	`34`
`35`		`- public function __construct(array $configuration, ExpressionLanguage $expressionLanguage, TokenStorageInterface $tokenStorage, AuthorizationCheckerInterface $authorizationChecker, AuthenticationTrustResolverInterface $trustResolver, RoleHierarchyInterface $roleHierarchy = null, ValidatorInterface $validator = null)`
	`35`	`+ public function __construct(array $configuration, ExpressionLanguage $expressionLanguage, TokenStorageInterface $tokenStorage, AuthorizationCheckerInterface $authorizationChecker, AuthenticationTrustResolverInterface $trustResolver, ?RoleHierarchyInterface $roleHierarchy = null, ?ValidatorInterface $validator = null)`
`36`	`36`	`{`
`37`	`37`	`$this->configuration = $configuration;`
`38`	`38`	`$this->expressionLanguage = $expressionLanguage;`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ final class InMemoryMetadataStore implements MetadataStoreInterface`
`27`	`27`	`/**`
`28`	`28`	`* @param \SplObjectStorage<Transition, array>\|null $transitionsMetadata`
`29`	`29`	`*/`
`30`		`- public function __construct(array $workflowMetadata = [], array $placesMetadata = [], \SplObjectStorage $transitionsMetadata = null)`
	`30`	`+ public function __construct(array $workflowMetadata = [], array $placesMetadata = [], ?\SplObjectStorage $transitionsMetadata = null)`
`31`	`31`	`{`
`32`	`32`	`$this->workflowMetadata = $workflowMetadata;`
`33`	`33`	`$this->placesMetadata = $placesMetadata;`