smartboxgroup / core-bundle

Command/Fixtures/GenerateFixtureCommand.php

Doc Comments +7 added lines, -1 removed lines

@@ -101,7 +101,7 @@ 
     }
 
     /**
-     * @param $class
+     * @param null|string $class
      *
      * @return EntityInterface
      */
@@ -192,6 +192,9 @@ 
         return $entity;
     }
 
+    /**
+     * @param string $default
+     */
     protected function ask($text, $default = null)
     {
         $helper = $this->getQuestionHelper();
@@ -202,6 +205,9 @@ 
         return $res;
     }
 
+    /**
+     * @param string $field
+     */
     protected function askForField($field, $tName)
     {
         $question = "$field ($tName): ";

Tests/Fixtures/Entity/TestComplexEntity.php

Doc Comments +4 added lines, -4 removed lines

@@ -192,7 +192,7 @@ 
     }
 
     /**
-     * @return \Smartbox\CoreBundle\Type\IntegerType[]
+     * @return integer[]
      */
     public function getArrayOfIntegers()
     {
@@ -208,7 +208,7 @@ 
     }
 
     /**
-     * @return \Smartbox\CoreBundle\Type\StringType[]
+     * @return string[]
      */
     public function getArrayOfStrings()
     {
@@ -224,7 +224,7 @@ 
     }
 
     /**
-     * @return \Smartbox\CoreBundle\Type\DoubleType[]
+     * @return double[]
      */
     public function getArrayOfDoubles()
     {
@@ -240,7 +240,7 @@ 
     }
 
     /**
-     * @return \Smartbox\CoreBundle\Type\Date[]
+     * @return \DateTime[]
      */
     public function getArrayOfDates()
     {

Tests/Fixtures/Entity/TestNestedEntity.php

Doc Comments +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
     }
 
     /**
-     * @return \Smartbox\CoreBundle\Tests\Fixtures\Entity\TestEntity[]
+     * @return TestEntity
      */
     public function getAssocItems()
     {
@@ -113,7 +113,7 @@ 
     }
 
     /**
-     * @return Entity[]
+     * @return Entity
      */
     public function getGenericItems()
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Smartbox\CoreBundle\Tests\Fixtures\Entity;
 
 use JMS\Serializer\Annotation as JMS;
-use Symfony\Component\Validator\Constraints as Assert;
 use Smartbox\CoreBundle\Type\Entity;
 
 class TestNestedEntity extends Entity

Tests/Serializer/Handler/CachedObjectHandlerTest.php

Doc Comments +3 added lines

@@ -29,6 +29,9 @@
         return $kernel;
     }
 
+    /**
+     * @param string $title
+     */
     protected function createCacheableEntity($title)
     {
         $testEntity = new CacheableEntity();

Tests/Symfony/Validator/Abstract2Dot5ApiTest.php

Doc Comments +3 added lines

@@ -55,6 +55,9 @@
         $this->validator = $this->createValidator($this->metadataFactory);
     }
 
+    /**
+     * @param string $groups
+     */
     protected function validate($value, $constraints = null, $groups = null)
     {
         return $this->validator->validate($value, $constraints, $groups);

Spacing +23 added lines, -23 removed lines

@@ -87,15 +87,15 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message 1');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message 2');
         };
 
         $this->metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $this->metadata->addConstraint(new Callback([
@@ -120,10 +120,10 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Reference violation 1');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Reference violation 2');
         };
 
@@ -151,7 +151,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $violations = $context
                 ->getValidator()
                 // Since the validator is not context aware, the group must
@@ -175,7 +175,7 @@ 
             $context->addViolation('Separate violation');
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback2 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('', $context->getPropertyPath());
@@ -210,7 +210,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test) {
             $previousValue = $context->getValue();
             $previousObject = $context->getObject();
             $previousMetadata = $context->getMetadata();
@@ -232,7 +232,7 @@ 
             $test->assertSame($previousGroup, $context->getGroup());
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback2 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('subpath', $context->getPropertyPath());
@@ -274,7 +274,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test) {
             $previousValue = $context->getValue();
             $previousObject = $context->getObject();
             $previousMetadata = $context->getMetadata();
@@ -296,7 +296,7 @@ 
             $test->assertSame($previousGroup, $context->getGroup());
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback2 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('subpath[key]', $context->getPropertyPath());
@@ -338,7 +338,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[key]', $context->getPropertyPath());
@@ -376,7 +376,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -401,7 +401,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -438,7 +438,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -464,7 +464,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -492,7 +492,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -518,7 +518,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->buildViolation('Message %param%')
                 ->setParameter('%param%', 'value')
                 ->setInvalidValue('Invalid value')
@@ -604,7 +604,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -623,7 +623,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -653,7 +653,7 @@ 
         $entity = new Entity();
         $entity->firstName = 'Bernhard';
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, &$called) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, &$called) {
             $called = true;
             $test->assertSame($entity, $context->getObject());
         };
@@ -679,7 +679,7 @@ 
         $initializer1->expects($this->once())
             ->method('initialize')
             ->with($entity)
-            ->will($this->returnCallback(function ($object) {
+            ->will($this->returnCallback(function($object) {
                 $object->initialized = true;
             }));
 
@@ -695,7 +695,7 @@ 
         // prepare constraint which
         // * checks that "initialized" is set to true
         // * validates the object again
-        $callback = function ($object, ExecutionContextInterface $context) use ($test) {
+        $callback = function($object, ExecutionContextInterface $context) use ($test) {
             $test->assertTrue($object->initialized);
 
             // validate again in same group

Tests/Symfony/Validator/AbstractValidatorTest.php

Doc Comments +12 added lines

@@ -65,10 +65,22 @@
         $this->referenceMetadata = null;
     }
 
+    /**
+     * @param Callback $constraints
+     */
     abstract protected function validate($value, $constraints = null, $groups = null);
 
+    /**
+     * @param string $propertyName
+     * @param string $groups
+     */
     abstract protected function validateProperty($object, $propertyName, $groups = null);
 
+    /**
+     * @param string $propertyName
+     * @param string $value
+     * @param string $groups
+     */
     abstract protected function validatePropertyValue($object, $propertyName, $value, $groups = null);
 
     public function testValidate()

Spacing +44 added lines, -44 removed lines

@@ -75,7 +75,7 @@ 
     {
         $test = $this;
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->assertNull($context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('', $context->getPropertyPath());
@@ -111,7 +111,7 @@ 
         $test = $this;
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('', $context->getPropertyPath());
@@ -149,7 +149,7 @@ 
         $entity = new Entity();
         $entity->firstName = 'Bernhard';
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -189,7 +189,7 @@ 
         $entity = new Entity();
         $entity->setLastName('Schussek');
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('lastName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -229,7 +229,7 @@ 
         $entity = new Entity();
         $array = ['key' => $entity];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[key]', $context->getPropertyPath());
@@ -267,7 +267,7 @@ 
         $entity = new Entity();
         $array = [2 => ['key' => $entity]];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[2][key]', $context->getPropertyPath());
@@ -305,7 +305,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[key]', $context->getPropertyPath());
@@ -345,7 +345,7 @@ 
             2 => new \ArrayIterator(['key' => $entity]),
         ]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[2][key]', $context->getPropertyPath());
@@ -383,7 +383,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference', $context->getPropertyPath());
@@ -423,7 +423,7 @@ 
         $entity->reference = new Reference();
         $entity->reference->value = 'Foobar';
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->referenceMetadata->getPropertyMetadata('value');
 
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
@@ -465,7 +465,7 @@ 
         $entity->reference = new Reference();
         $entity->reference->setPrivateValue('Bamboo');
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->referenceMetadata->getPropertyMetadata('privateValue');
 
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
@@ -532,7 +532,7 @@ 
         $entity = new Entity();
         $entity->reference = ['key' => new Reference()];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[key]', $context->getPropertyPath());
@@ -572,7 +572,7 @@ 
         $entity = new Entity();
         $entity->reference = [2 => ['key' => new Reference()]];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[2][key]', $context->getPropertyPath());
@@ -610,7 +610,7 @@ 
         $entity = new Entity();
         $entity->reference = ['key' => new Reference()];
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
@@ -630,7 +630,7 @@ 
         $entity = new Entity();
         $entity->reference = [2 => ['key' => new Reference()]];
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
@@ -677,7 +677,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[key]', $context->getPropertyPath());
@@ -715,7 +715,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
@@ -754,7 +754,7 @@ 
             2 => new \ArrayIterator(['key' => new Reference()]),
         ]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[2][key]', $context->getPropertyPath());
@@ -796,7 +796,7 @@ 
         $entity->firstName = 'Bernhard';
         $entity->setLastName('Schussek');
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -811,7 +811,7 @@ 
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Other violation');
         };
 
@@ -871,7 +871,7 @@ 
         $entity = new Entity();
         $entity->setLastName('Schussek');
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -886,7 +886,7 @@ 
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Other violation');
         };
 
@@ -922,7 +922,7 @@ 
     {
         $test = $this;
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -937,7 +937,7 @@ 
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Other violation');
         };
 
@@ -1002,7 +1002,7 @@ 
         $entity->reference = new Reference();
         $entity->reference2 = $entity->reference;
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1022,7 +1022,7 @@ 
         $entity->reference = new Reference();
         $entity->reference2 = new Reference();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1040,7 +1040,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1063,7 +1063,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1086,15 +1086,15 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $this->metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $this->metadata->addConstraint(new Callback([
@@ -1120,15 +1120,15 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $this->metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $this->metadata->addConstraint(new Callback([
@@ -1155,10 +1155,10 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Default group');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in group sequence');
         };
 
@@ -1186,10 +1186,10 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in other group');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in group sequence');
         };
 
@@ -1217,16 +1217,16 @@ 
         $sequence = new GroupSequence(['Group 1', 'Group 2', 'Group 3', 'Entity']);
         $entity = new GroupSequenceProviderEntity($sequence);
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $metadata = new ClassMetadata(get_class($entity));
         $metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $metadata->addConstraint(new Callback([
@@ -1253,16 +1253,16 @@ 
         $sequence = ['Group 1', 'Group 2', 'Group 3', 'Entity'];
         $entity = new GroupSequenceProviderEntity($sequence);
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $metadata = new ClassMetadata(get_class($entity));
         $metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $metadata->addConstraint(new Callback([

Tests/Utils/Cache/FakeCacheService.php

Doc Comments +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     }
 
     /**
-     * @param $method
+     * @param string $method
      * @param array $arguments
      * @param null  $result
      */

Utils/Generator/RandomFixtureGenerator.php

Doc Comments +2 added lines, -2 removed lines

@@ -93,8 +93,8 @@
     /**
      * @param $typeName
      * @param array|null $typeParams
-     * @param null       $group
-     * @param null       $version
+     * @param string       $group
+     * @param string       $version
      *
      * @return float|int|string|array|\DateTime|EntityInterface
      */

Serializer/Handler/DateHandler.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 
         $types = ['DateTime', 'DateInterval'];
 
-        $newFormats = ['array'];   // Add here any new format you wish to support
+        $newFormats = ['array']; // Add here any new format you wish to support
 
         foreach ($newFormats as $format) {
             $methods[] = [