minor symfony#19187 [Workflow] CS tweaks (ro0NL)

This PR was squashed before being merged into the 3.2-dev branch (closes symfony#19187).

Discussion
----------

[Workflow] CS tweaks

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | ~
| License       | MIT
| Doc PR        | ~

Just a few minor tweaks/fixes after playing around with it. Some phpdoc should still be added imo, like missing `@throws`, right?

Commits
-------

5428a92 [Workflow] CS tweaks

Author: fabpot

src/Symfony/Component/Workflow/Definition.php

@@ -21,9 +21,7 @@
 class Definition
 {
     private $places = array();
-
     private $transitions = array();
-
     private $initialPlace;
 
     /**

src/Symfony/Component/Workflow/Event/Event.php

@@ -22,15 +22,13 @@
 class Event extends BaseEvent
 {
     private $subject;
-
     private $marking;
-
     private $transition;
 
     /**
      * Event constructor.
      *
-     * @param mixed      $subject
+     * @param object     $subject
      * @param Marking    $marking
      * @param Transition $transition
      */

src/Symfony/Component/Workflow/EventListener/AuditTrailListener.php

@@ -30,19 +30,19 @@ public function __construct(LoggerInterface $logger)
     public function onLeave(Event $event)
     {
         foreach ($event->getTransition()->getFroms() as $place) {
-            $this->logger->info(sprintf('leaving "%s" for subject of class "%s"', $place, get_class($event->getSubject())));
+            $this->logger->info(sprintf('Leaving "%s" for subject of class "%s".', $place, get_class($event->getSubject())));
         }
     }
 
     public function onTransition(Event $event)
     {
-        $this->logger->info(sprintf('transition "%s" for subject of class "%s"', $event->getTransition()->getName(), get_class($event->getSubject())));
+        $this->logger->info(sprintf('Transition "%s" for subject of class "%s".', $event->getTransition()->getName(), get_class($event->getSubject())));
     }
 
     public function onEnter(Event $event)
     {
         foreach ($event->getTransition()->getTos() as $place) {
-            $this->logger->info(sprintf('entering "%s" for subject of class "%s"', $place, get_class($event->getSubject())));
+            $this->logger->info(sprintf('Entering "%s" for subject of class "%s".', $place, get_class($event->getSubject())));
         }
     }
 

src/Symfony/Component/Workflow/MarkingStore/PropertyAccessorMarkingStore.php

@@ -23,7 +23,6 @@
 class PropertyAccessorMarkingStore implements MarkingStoreInterface
 {
     private $property;
-
     private $propertyAccessor;
 
     /**

src/Symfony/Component/Workflow/MarkingStore/ScalarMarkingStore.php

@@ -23,7 +23,6 @@
 class ScalarMarkingStore implements MarkingStoreInterface, UniqueTransitionOutputInterface
 {
     private $property;
-
     private $propertyAccessor;
 
     /**

src/Symfony/Component/Workflow/Registry.php

@@ -23,19 +23,19 @@ class Registry
 
     /**
      * @param Workflow $workflow
-     * @param string   $classname
+     * @param string   $className
      */
-    public function add(Workflow $workflow, $classname)
+    public function add(Workflow $workflow, $className)
     {
-        $this->workflows[] = array($workflow, $classname);
+        $this->workflows[] = array($workflow, $className);
     }
 
     public function get($subject, $workflowName = null)
     {
         $matched = null;
 
-        foreach ($this->workflows as list($workflow, $classname)) {
-            if ($this->supports($workflow, $classname, $subject, $workflowName)) {
+        foreach ($this->workflows as list($workflow, $className)) {
+            if ($this->supports($workflow, $className, $subject, $workflowName)) {
                 if ($matched) {
                     throw new InvalidArgumentException('At least two workflows match this subject. Set a different name on each and use the second (name) argument of this method.');
                 }
@@ -50,9 +50,9 @@ public function get($subject, $workflowName = null)
         return $matched;
     }
 
-    private function supports(Workflow $workflow, $classname, $subject, $name)
+    private function supports(Workflow $workflow, $className, $subject, $name)
     {
-        if (!$subject instanceof $classname) {
+        if (!$subject instanceof $className) {
             return false;
         }
 

src/Symfony/Component/Workflow/Tests/DefinitionTest.php

@@ -28,7 +28,7 @@ public function testSetInitialPlace()
     }
 
     /**
-     * @expectedException Symfony\Component\Workflow\Exception\LogicException
+     * @expectedException \Symfony\Component\Workflow\Exception\LogicException
      * @expectedExceptionMessage Place "d" cannot be the initial place as it does not exist.
      */
     public function testSetInitialPlaceAndPlaceIsNotDefined()
@@ -50,7 +50,7 @@ public function testAddTransition()
     }
 
     /**
-     * @expectedException Symfony\Component\Workflow\Exception\LogicException
+     * @expectedException \Symfony\Component\Workflow\Exception\LogicException
      * @expectedExceptionMessage Place "c" referenced in transition "name" does not exist.
      */
     public function testAddTransitionAndFromPlaceIsNotDefined()
@@ -61,7 +61,7 @@ public function testAddTransitionAndFromPlaceIsNotDefined()
     }
 
     /**
-     * @expectedException Symfony\Component\Workflow\Exception\LogicException
+     * @expectedException \Symfony\Component\Workflow\Exception\LogicException
      * @expectedExceptionMessage Place "c" referenced in transition "name" does not exist.
      */
     public function testAddTransitionAndToPlaceIsNotDefined()

src/Symfony/Component/Workflow/Tests/EventListener/AuditTrailListenerTest.php

@@ -34,9 +34,9 @@ public function testItWorks()
         $workflow->apply($object, 't1');
 
         $expected = array(
-            'leaving "a" for subject of class "stdClass"',
-            'transition "t1" for subject of class "stdClass"',
-            'entering "b" for subject of class "stdClass"',
+            'Leaving "a" for subject of class "stdClass".',
+            'Transition "t1" for subject of class "stdClass".',
+            'Entering "b" for subject of class "stdClass".',
         );
 
         $this->assertSame($expected, $logger->logs);

src/Symfony/Component/Workflow/Tests/RegistryTest.php

@@ -44,7 +44,7 @@ public function testGetWithSuccess()
     }
 
     /**
-     * @expectedException Symfony\Component\Workflow\Exception\InvalidArgumentException
+     * @expectedException \Symfony\Component\Workflow\Exception\InvalidArgumentException
      * @expectedExceptionMessage At least two workflows match this subject. Set a different name on each and use the second (name) argument of this method.
      */
     public function testGetWithMultipleMatch()

src/Symfony/Component/Workflow/Tests/TransitionTest.php

@@ -7,7 +7,7 @@
 class TransitionTest extends \PHPUnit_Framework_TestCase
 {
     /**
-     * @expectedException Symfony\Component\Workflow\Exception\InvalidArgumentException
+     * @expectedException \Symfony\Component\Workflow\Exception\InvalidArgumentException
      * @expectedExceptionMessage The transition "foo.bar" contains invalid characters.
      */
     public function testValidateName()