bankiru / doctrine-api-client

EntityManager.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Bankiru\Api\Doctrine\Proxy\ApiCollection;
 use Bankiru\Api\Doctrine\Proxy\ProxyFactory;
 use Bankiru\Api\Doctrine\Utility\IdentifierFixer;
-use Doctrine\Common\Collections\AbstractLazyCollection;
 use Doctrine\Common\Persistence\ObjectRepository;
 use Doctrine\Common\Proxy\Proxy;
 

EntityRepository.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace Bankiru\Api\Doctrine;
 
 use Bankiru\Api\Doctrine\Mapping\ApiMetadata;
-use Bankiru\Api\Doctrine\Proxy\ApiCollection;
-use Doctrine\Common\Collections\Collection;
 use Doctrine\Common\Persistence\ObjectRepository;
 use ScayTrase\Api\Rpc\RpcClientInterface;
 

Cache/KeyStrategyInterface.php

Spacing +1 added lines, -1 removed lines

@@ -24,5 +24,5 @@
      *
      * @return string
      */
-    public function getEntityKey(ApiMetadata $metadata,array $identifier);
+    public function getEntityKey(ApiMetadata $metadata, array $identifier);
 }

Mapping/Driver/YmlMetadataDriver.php

Spacing +1 added lines, -2 removed lines

@@ -87,8 +87,7 @@
             if (array_key_exists('entityPath', $element['client'])) {
                 $pathSeparator  =
                     array_key_exists('entityPathSeparator', $element['client']) ?
-                        $element['client']['entityPathSeparator'] :
-                        null;
+                        $element['client']['entityPathSeparator'] : null;
                 $methodProvider = new EntityMethodProvider($element['client']['entityPath'], $pathSeparator);
             }
 

Mapping/EntityMetadata.php

Spacing +1 added lines, -1 removed lines

@@ -491,7 +491,7 @@
         }
 
         // Fetch mode. Default fetch mode to LAZY, if not set.
-        if ( ! isset($mapping['fetch'])) {
+        if (!isset($mapping['fetch'])) {
             $mapping['fetch'] = self::FETCH_LAZY;
         }
 

Proxy/ApiCollection.php

Spacing +2 added lines, -2 removed lines

@@ -204,7 +204,7 @@ 
         return array_udiff_assoc(
             $this->snapshot,
             $this->collection->toArray(),
-            function ($a, $b) {
+            function($a, $b) {
                 return $a === $b ? 0 : 1;
             }
         );
@@ -221,7 +221,7 @@ 
         return array_udiff_assoc(
             $this->collection->toArray(),
             $this->snapshot,
-            function ($a, $b) {
+            function($a, $b) {
                 return $a === $b ? 0 : 1;
             }
         );

Rpc/SearchArgumentsTransformer.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
                 /** @var EntityMetadata $target */
                 $target = $this->manager->getClassMetadata($mapping['target']);
 
-                $converter = function ($value) use ($target) {
+                $converter = function($value) use ($target) {
                     if (!is_object($value)) {
                         return $value;
                     }
@@ -70,7 +70,7 @@ 
                     $values = $converter($values);
                 }
             } else {
-                $caster = function ($value) use ($field) {
+                $caster = function($value) use ($field) {
                     $type = $this->manager
                         ->getConfiguration()
                         ->getTypeRegistry()->get($this->metadata->getTypeOfField($field));

UnitOfWork.php

Spacing +5 added lines, -5 removed lines

@@ -1683,7 +1683,7 @@ 
         $class               = $this->em->getClassMetadata(get_class($entity));
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) {
+            function($assoc) {
                 return $assoc['isCascadeRefresh'];
             }
         );
@@ -1722,7 +1722,7 @@ 
         $class               = $this->em->getClassMetadata(get_class($entity));
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) {
+            function($assoc) {
                 return $assoc['isCascadeDetach'];
             }
         );
@@ -1762,7 +1762,7 @@ 
         $class               = $this->manager->getClassMetadata(get_class($entity));
         $associationMappings = array_filter(
             $class->getAssociationMappings(),
-            function ($assoc) {
+            function($assoc) {
                 return $assoc['isCascadeMerge'];
             }
         );
@@ -1800,11 +1800,11 @@ 
         $class               = $this->em->getClassMetadata(get_class($entity));
         $associationMappings = array_filter(
             $class->associationMappings,
-            function ($assoc) {
+            function($assoc) {
                 return $assoc['isCascadeRemove'];
             }
         );
-        $entitiesToCascade   = [];
+        $entitiesToCascade = [];
         foreach ($associationMappings as $assoc) {
             if ($entity instanceof Proxy && !$entity->__isInitialized__) {
                 $entity->__load();

Tests/CollectionLoadingTest.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         foreach ($entities as $entity) {
             self::assertInternalType('int', $entity->getId());
             self::assertInstanceOf(SubEntity::class, $entity);
-            self::assertEquals('test-payload-'.$entity->getId(), $entity->getPayload());
+            self::assertEquals('test-payload-' . $entity->getId(), $entity->getPayload());
             self::assertEquals('sub-payload', $entity->getSubPayload());
 
             if (null !== $entity->getStringPayload()) {