Replace more docblocks by type-hints

Author: nicolas-grekas

DependencyInjection/ValidateWorkflowsPass.php

@@ -24,7 +24,7 @@ class ValidateWorkflowsPass implements CompilerPassInterface
 {
     private $definitionTag;
 
-    public function __construct($definitionTag = 'workflow.definition')
+    public function __construct(string $definitionTag = 'workflow.definition')
     {
         $this->definitionTag = $definitionTag;
     }

Event/Event.php

@@ -32,7 +32,7 @@ class Event extends BaseEvent
      * @param Transition $transition
      * @param string     $workflowName
      */
-    public function __construct($subject, Marking $marking, Transition $transition, $workflowName = 'unnamed')
+    public function __construct($subject, Marking $marking, Transition $transition, string $workflowName = 'unnamed')
     {
         $this->subject = $subject;
         $this->marking = $marking;

EventListener/GuardListener.php

@@ -32,7 +32,7 @@ class GuardListener
     private $roleHierarchy;
     private $validator;
 
-    public function __construct($configuration, ExpressionLanguage $expressionLanguage, TokenStorageInterface $tokenStorage, AuthorizationCheckerInterface $authenticationChecker, AuthenticationTrustResolverInterface $trustResolver, RoleHierarchyInterface $roleHierarchy = null, ValidatorInterface $validator = null)
+    public function __construct(array $configuration, ExpressionLanguage $expressionLanguage, TokenStorageInterface $tokenStorage, AuthorizationCheckerInterface $authenticationChecker, AuthenticationTrustResolverInterface $trustResolver, RoleHierarchyInterface $roleHierarchy = null, ValidatorInterface $validator = null)
     {
         $this->configuration = $configuration;
         $this->expressionLanguage = $expressionLanguage;

MarkingStore/MultipleStateMarkingStore.php

@@ -29,11 +29,7 @@ class MultipleStateMarkingStore implements MarkingStoreInterface
     private $property;
     private $propertyAccessor;
 
-    /**
-     * @param string                         $property
-     * @param PropertyAccessorInterface|null $propertyAccessor
-     */
-    public function __construct($property = 'marking', PropertyAccessorInterface $propertyAccessor = null)
+    public function __construct(string $property = 'marking', PropertyAccessorInterface $propertyAccessor = null)
     {
         $this->property = $property;
         $this->propertyAccessor = $propertyAccessor ?: PropertyAccess::createPropertyAccessor();

MarkingStore/SingleStateMarkingStore.php

@@ -28,11 +28,7 @@ class SingleStateMarkingStore implements MarkingStoreInterface
     private $property;
     private $propertyAccessor;
 
-    /**
-     * @param string                         $property
-     * @param PropertyAccessorInterface|null $propertyAccessor
-     */
-    public function __construct($property = 'marking', PropertyAccessorInterface $propertyAccessor = null)
+    public function __construct(string $property = 'marking', PropertyAccessorInterface $propertyAccessor = null)
     {
         $this->property = $property;
         $this->propertyAccessor = $propertyAccessor ?: PropertyAccess::createPropertyAccessor();

StateMachine.php

@@ -11,7 +11,7 @@
  */
 class StateMachine extends Workflow
 {
-    public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, $name = 'unnamed')
+    public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, string $name = 'unnamed')
     {
         parent::__construct($definition, $markingStore ?: new SingleStateMarkingStore(), $dispatcher, $name);
     }

Transition.php

@@ -28,7 +28,7 @@ class Transition
      * @param string|string[] $froms
      * @param string|string[] $tos
      */
-    public function __construct($name, $froms, $tos)
+    public function __construct(string $name, $froms, $tos)
     {
         if (!preg_match('{^[\w\d_-]+$}', $name)) {
             throw new InvalidArgumentException(sprintf('The transition "%s" contains invalid characters.', $name));

Validator/WorkflowValidator.php

@@ -21,10 +21,7 @@ class WorkflowValidator implements DefinitionValidatorInterface
 {
     private $singlePlace;
 
-    /**
-     * @param bool $singlePlace
-     */
-    public function __construct($singlePlace = false)
+    public function __construct(bool $singlePlace = false)
     {
         $this->singlePlace = $singlePlace;
     }

Workflow.php

@@ -30,7 +30,7 @@ class Workflow
     private $dispatcher;
     private $name;
 
-    public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, $name = 'unnamed')
+    public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, string $name = 'unnamed')
     {
         $this->definition = $definition;
         $this->markingStore = $markingStore ?: new MultipleStateMarkingStore();