psiphp / object-agent

bridge/doctrine-collections/lib/CollectionsVisitor.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Psi\Bridge\ObjectAgent\Doctrine\Collections;
 
@@ -45,7 +45,7 @@ 
                 return $this->walkComposite($expr);
         }
 
-        throw new \RuntimeException('Unknown Expression: ' . get_class($expr));
+        throw new \RuntimeException('Unknown Expression: '.get_class($expr));
     }
 
     /**
@@ -95,7 +95,7 @@ 
 
         }
 
-        throw new \RuntimeException('Unknown comparator: ' . $comparison->getComparator());
+        throw new \RuntimeException('Unknown comparator: '.$comparison->getComparator());
     }
 
     /**
@@ -110,7 +110,7 @@ 
             $ormExpressions[] = $this->dispatch($childExpression);
         }
 
-        $method = $expression->getType() == Composite::AND ? 'andX' : 'orX';
+        $method = $expression->getType() == Composite::AND? 'andX' : 'orX';
 
         return call_user_func_array([$this->expressionBuilder, $method], $ormExpressions);
     }

lib/Exception/BadMethodCallException.php

Indentation +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     {
         throw new self(sprintf(
             'Comparison "%s" is not supported.',
-             $comparison
+                $comparison
         ));
     }
 }

bridge/doctrine-collections/tests/Functional/CollectionsVisitorTest.php

Spacing +3 added lines, -3 removed lines

@@ -106,7 +106,7 @@ 
             [
                 'null',
                 null,
-                function ($expr) {
+                function($expr) {
                     $this->assertEquals('=', $expr->getOperator());
                     $this->assertNull($expr->getValue()->getValue());
                 },
@@ -114,7 +114,7 @@ 
             [
                 'in',
                 [1, 2, 3],
-                function ($expr) {
+                function($expr) {
                     $this->assertEquals('IN', $expr->getOperator());
                     $this->assertEquals([1, 2, 3], $expr->getValue()->getValue());
                 },
@@ -122,7 +122,7 @@ 
             [
                 'nin',
                 [1, 2, 3],
-                function ($expr) {
+                function($expr) {
                     $this->assertEquals('NIN', $expr->getOperator());
                     $this->assertEquals([1, 2, 3], $expr->getValue()->getValue());
                 },

bridge/doctrine-collections/lib/Store.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Psi\Bridge\ObjectAgent\Doctrine\Collections;
 
@@ -19,7 +19,7 @@ 
 
     public function addCollection($classFqn, array $collection)
     {
-        array_map(function ($object) use ($classFqn) {
+        array_map(function($object) use ($classFqn) {
             if (get_class($object) !== $classFqn) {
                 throw new \InvalidArgumentException(sprintf(
                     'All objects in collection must be of class "%s", got "%s"',

bridge/doctrine-collections/lib/CollectionsAgent.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Psi\Bridge\ObjectAgent\Doctrine\PhpcrOdm;
 

bridge/doctrine-phpcr-odm/lib/PhpcrOdmAgent.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Psi\Bridge\ObjectAgent\Doctrine\PhpcrOdm;
 
@@ -133,7 +133,7 @@ 
         $orderBy = $queryBuilder->orderBy();
         foreach ($query->getOrderings() as $field => $order) {
             $order = strtolower($order);
-            $orderBy->{$order}()->field($sourceAlias . '.' .  $field);
+            $orderBy->{$order}()->field($sourceAlias.'.'.$field);
         }
 
         if (null !== $query->getFirstResult()) {

bridge/doctrine-phpcr-odm/tests/Functional/PhpcrOdmAgentTest.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
         $page = new Page();
         $page->title = $title;
-        $page->path = '/test/page-' . $id++;
+        $page->path = '/test/page-'.$id++;
         $this->documentManager->persist($page);
         $this->documentManager->flush();
 

bridge/doctrine-orm/lib/OrmAgent.php

Unused Use Statements -3 removed lines

@@ -9,8 +9,6 @@ 
 use Doctrine\Common\Util\ClassUtils;
 use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\ORM\QueryBuilder;
-use Doctrine\ORM\Query as OrmQuery;
-use Doctrine\ORM\Tools\Pagination\Paginator;
 use Psi\Component\ObjectAgent\AgentInterface;
 use Psi\Component\ObjectAgent\Capabilities;
 use Psi\Component\ObjectAgent\Exception\BadMethodCallException;
@@ -18,7 +16,6 @@ 
 use Psi\Component\ObjectAgent\Query\Comparison;
 use Psi\Component\ObjectAgent\Query\Query;
 use Psi\Component\ObjectAgent\Query\Join;
-use Doctrine\ORM\Query\Expr\Select;
 
 class OrmAgent implements AgentInterface
 {

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Psi\Bridge\ObjectAgent\Doctrine\Orm;
 
@@ -103,7 +103,7 @@ 
         $idField = reset($idFields);
 
         $queryBuilder = $this->entityManager->getRepository($class)->createQueryBuilder('a');
-        $queryBuilder->where($queryBuilder->expr()->in('a.' . $idField, ':identifiers'));
+        $queryBuilder->where($queryBuilder->expr()->in('a.'.$idField, ':identifiers'));
         $queryBuilder->setParameter('identifiers', $identifiers);
 
         return $queryBuilder->getQuery()->execute();
@@ -205,7 +205,7 @@ 
         // TODO: Make a clone method on the Query, which allows passing an
         //       array of parts to replace in the clone.
         $query = Query::create($query->getClassFqn(), [
-            'selects' => [ 'count(' . self::SOURCE_ALIAS . '.' . $idField . ')' ],
+            'selects' => ['count('.self::SOURCE_ALIAS.'.'.$idField.')'],
             'joins' => $query->getJoins(),
             'criteria' => $query->hasExpression() ? $query->getExpression() : null,
         ]);
@@ -245,7 +245,7 @@ 
 
         $selects = [];
         foreach ($query->getSelects() as $selectName => $selectAlias) {
-            $select = $selectName . ' ' . $selectAlias;
+            $select = $selectName.' '.$selectAlias;
             if (is_int($selectName)) {
                 $select = $selectAlias;
             }
@@ -268,7 +268,7 @@ 
         }
 
         foreach ($query->getOrderings() as $field => $order) {
-            $queryBuilder->addOrderBy(self::SOURCE_ALIAS . '.' . $field, $order);
+            $queryBuilder->addOrderBy(self::SOURCE_ALIAS.'.'.$field, $order);
         }
 
         if (null !== $query->getFirstResult()) {

lib/Query/Query.php

Doc Comments +3 added lines

@@ -60,6 +60,9 @@
         return new self($classFqn, $query['selects'], $query['joins'], $query['criteria'], $query['orderings'], $query['firstResult'], $query['maxResults']);
     }
 
+    /**
+     * @param string $value1
+     */
     public static function comparison(string $comparator, $value1, $value2): Comparison
     {
         return new Comparison($comparator, $value1, $value2);

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         $this->joins = $joins;
         $this->selects = $selects;
 
-        array_walk($joins, function (Join $join) {
+        array_walk($joins, function(Join $join) {
         });
     }