Completed
Push — master ( 77cc29...840912 )
by Gerrit
10:19
created
Mapping/ChoiceMapping.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,6 @@
 block discarded – undo
14 14
 use Addiks\RDMBundle\Mapping\MappingInterface;
15 15
 use Doctrine\DBAL\Schema\Column;
16 16
 use Doctrine\DBAL\Types\Type;
17
-use Doctrine\DBAL\Types\TextType;
18 17
 use Addiks\RDMBundle\Exception\InvalidMappingException;
19 18
 use Addiks\RDMBundle\Hydration\HydrationContextInterface;
20 19
 use Symfony\Component\DependencyInjection\ContainerInterface;
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -165,7 +165,7 @@
 block discarded – undo
165 165
             $choiceValue = $choiceMapping->resolveValue(
166 166
                 $context,
167 167
                 [] # <= I'm not sure how this parameter should be handled correctly in the future,
168
-                   #    but with the current supported features it *should* be irrelevant.
168
+                    #    but with the current supported features it *should* be irrelevant.
169 169
             );
170 170
 
171 171
             if ($choiceValue === $valueFromEntityField) {
Please login to merge, or discard this patch.
Mapping/Drivers/MappingXmlDriver.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,6 @@
 block discarded – undo
40 40
 use Symfony\Component\HttpKernel\KernelInterface;
41 41
 use Addiks\RDMBundle\Mapping\NullableMapping;
42 42
 use Addiks\RDMBundle\Mapping\NullableMappingInterface;
43
-use Symfony\Component\DependencyInjection\ContainerInterface;
44 43
 
45 44
 final class MappingXmlDriver implements MappingDriverInterface
46 45
 {
Please login to merge, or discard this patch.
Tests/DataLoader/SimpleSelectDataLoaderTest.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -12,7 +12,6 @@  discard block
 block discarded – undo
12 12
 
13 13
 use Addiks\RDMBundle\DataLoader\SimpleSelectDataLoader;
14 14
 use Addiks\RDMBundle\Mapping\Drivers\MappingDriverInterface;
15
-use Addiks\RDMBundle\Mapping\EntityMapping;
16 15
 use Addiks\RDMBundle\Tests\Hydration\EntityExample;
17 16
 use Addiks\RDMBundle\Mapping\ChoiceMapping;
18 17
 use Addiks\RDMBundle\ValueResolver\ValueResolverInterface;
@@ -23,7 +22,6 @@  discard block
 block discarded – undo
23 22
 use Doctrine\ORM\Query\Expr;
24 23
 use Doctrine\DBAL\Driver\Statement;
25 24
 use PHPUnit\Framework\TestCase;
26
-use Addiks\RDMBundle\Mapping\ServiceMapping;
27 25
 use Doctrine\ORM\UnitOfWork;
28 26
 use Addiks\RDMBundle\Tests\Hydration\ServiceExample;
29 27
 use Addiks\RDMBundle\Mapping\MappingInterface;
Please login to merge, or discard this patch.
Tests/Hydration/EntityHydratorTest.php 1 patch
Unused Use Statements   -6 removed lines patch added patch discarded remove patch
@@ -13,17 +13,11 @@
 block discarded – undo
13 13
 use PHPUnit\Framework\TestCase;
14 14
 use Addiks\RDMBundle\Hydration\EntityHydrator;
15 15
 use Addiks\RDMBundle\Mapping\Drivers\MappingDriverInterface;
16
-use Addiks\RDMBundle\Mapping\Annotation\Service;
17 16
 use Addiks\RDMBundle\Tests\Hydration\ServiceExample;
18 17
 use Addiks\RDMBundle\Tests\Hydration\EntityExample;
19
-use ErrorException;
20
-use Addiks\RDMBundle\Mapping\ServiceMapping;
21 18
 use Addiks\RDMBundle\Mapping\EntityMapping;
22
-use Addiks\RDMBundle\ValueResolver\ValueResolverInterface;
23
-use Addiks\RDMBundle\Exception\FailedRDMAssertionExceptionInterface;
24 19
 use Addiks\RDMBundle\DataLoader\DataLoaderInterface;
25 20
 use Doctrine\ORM\EntityManagerInterface;
26
-use Addiks\RDMBundle\Mapping\EntityMappingInterface;
27 21
 use Addiks\RDMBundle\Mapping\MappingInterface;
28 22
 use Addiks\RDMBundle\Hydration\HydrationContextInterface;
29 23
 use Symfony\Component\DependencyInjection\ContainerInterface;
Please login to merge, or discard this patch.
Tests/Mapping/ArrayMappingTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,6 @@
 block discarded – undo
14 14
 use Addiks\RDMBundle\Mapping\ArrayMapping;
15 15
 use Addiks\RDMBundle\Mapping\MappingInterface;
16 16
 use Doctrine\DBAL\Schema\Column;
17
-use InvalidArgumentException;
18 17
 use Addiks\RDMBundle\Hydration\HydrationContextInterface;
19 18
 use Addiks\RDMBundle\Tests\Hydration\EntityExample;
20 19
 use Addiks\RDMBundle\Exception\FailedRDMAssertionExceptionInterface;
Please login to merge, or discard this patch.
Tests/Mapping/ChoiceMappingTest.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -37,6 +37,9 @@
 block discarded – undo
37 37
      */
38 38
     private $optionMappingB;
39 39
 
40
+    /**
41
+     * @param string $column
42
+     */
40 43
     public function setUp(
41 44
         $column = null
42 45
     ) {
Please login to merge, or discard this patch.
Tests/Mapping/Drivers/MappingStaticPHPDriverTest.php 1 patch
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -12,9 +12,6 @@
 block discarded – undo
12 12
 
13 13
 use PHPUnit\Framework\TestCase;
14 14
 use Addiks\RDMBundle\Tests\Hydration\EntityExample;
15
-use Addiks\RDMBundle\Mapping\Annotation\Service;
16
-use ReflectionProperty;
17
-use Addiks\RDMBundle\Mapping\Drivers\MappingDriverInterface;
18 15
 use Addiks\RDMBundle\Mapping\Drivers\MappingStaticPHPDriver;
19 16
 use Addiks\RDMBundle\Mapping\EntityMapping;
20 17
 use Addiks\RDMBundle\Mapping\ServiceMapping;
Please login to merge, or discard this patch.
Tests/Mapping/Drivers/MappingXmlDriverTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@
 block discarded – undo
88 88
             'bar' => new ServiceMapping($this->container, 'other_service', false, "in file '{$mappingFilePath}'"),
89 89
             'baz' => new ChoiceMapping('baz_column', [
90 90
                 'lorem' => new ServiceMapping($this->container, "lorem_service", false, "in file '{$mappingFilePath}'"),
91
-                'ipsum' => new ServiceMapping($this->container, "ipsum_service", true,  "in file '{$mappingFilePath}'"),
91
+                'ipsum' => new ServiceMapping($this->container, "ipsum_service", true, "in file '{$mappingFilePath}'"),
92 92
             ], "in file '{$mappingFilePath}'"),
93 93
             'faz' => new ChoiceMapping(new Column("faz_column", Type::getType('string'), ['notnull' => true]), [
94 94
                 'lorem' => new ServiceMapping($this->container, "lorem_service", false, "in file '{$mappingFilePath}'"),
Please login to merge, or discard this patch.