psiphp / object-agent

bridge/doctrine-orm/lib/ExpressionVisitor.php

Doc Comments +1 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
     private $parameters = [];
 
     /**
-     * @param QueryBuilder $expressionFactory
+     * @param Expr $expressionFactory
      */
     public function __construct(Expr $expressionFactory, string $sourceAlias)
     {
@@ -32,7 +32,6 @@ 
      * Walk the given expression to build up the PHPCR-ODM query builder.
      *
      * @param Expression $expr
-     * @param AbstractNode|null $parentNode
      */
     public function dispatch(Expression $expr)
     {

Indentation +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 
         $ormExpressions = [];
         foreach ($expressions as $index => $childExpression) {
-           $ormExpressions[] = $this->dispatch($childExpression);
+            $ormExpressions[] = $this->dispatch($childExpression);
         }
 
         $method = $expression->getType() == Composite::AND ? 'andX' : 'orX';

Spacing +6 added lines, -6 removed lines

@@ -45,7 +45,7 @@ 
                 return $this->walkComposite($expr);
         }
 
-        throw new \RuntimeException('Unknown Expression: ' . get_class($expr));
+        throw new \RuntimeException('Unknown Expression: '.get_class($expr));
     }
 
     public function getParameters()
@@ -99,7 +99,7 @@ 
                 return $this->expressionFactory->isNotNull($field, $this->registerParameter($field, $value));
         }
 
-        throw new \RuntimeException('Unknown comparator: ' . $comparison->getComparator());
+        throw new \RuntimeException('Unknown comparator: '.$comparison->getComparator());
     }
 
     /**
@@ -114,9 +114,9 @@ 
            $ormExpressions[] = $this->dispatch($childExpression);
         }
 
-        $method = $expression->getType() == Composite::AND ? 'andX' : 'orX';
+        $method = $expression->getType() == Composite::AND? 'andX' : 'orX';
 
-        return call_user_func_array([ $this->expressionFactory, $method ], $ormExpressions);
+        return call_user_func_array([$this->expressionFactory, $method], $ormExpressions);
     }
 
     /**
@@ -126,7 +126,7 @@ 
      */
     private function getField($field): string
     {
-        return $this->sourceAlias . '.' . $field;
+        return $this->sourceAlias.'.'.$field;
     }
 
     /**
@@ -148,6 +148,6 @@ 
     private function registerParameter(string $name, $value)
     {
         $this->parameters[$name] = $value;
-        return ':' . $name;
+        return ':'.$name;
     }
 }

bridge/doctrine-orm/tests/Functional/ExpressionVisitorTest.php

Unused Use Statements -6 removed lines

@@ -3,12 +3,6 @@
 
 namespace Psi\Bridge\ObjectAgent\Doctrine\Orm\Tests\Functional;
 
-use Doctrine\ODM\PHPCR\Query\Builder\AbstractNode;
-use Doctrine\ODM\PHPCR\Query\Builder\ConstraintAndx;
-use Doctrine\ODM\PHPCR\Query\Builder\ConstraintComparison;
-use Doctrine\ODM\PHPCR\Query\Builder\ConstraintFieldIsset;
-use Doctrine\ODM\PHPCR\Query\Builder\ConstraintNot;
-use Doctrine\ODM\PHPCR\Query\Builder\ConstraintOrx;
 use Psi\Bridge\ObjectAgent\Doctrine\Orm\ExpressionVisitor;
 use Psi\Component\ObjectAgent\Tests\Functional\Model\Page;
 use Psi\Component\ObjectAgent\Query\Query;

Spacing +6 added lines, -6 removed lines

@@ -101,29 +101,29 @@
             [
                 'null',
                 null,
-                function ($expr) {
+                function($expr) {
                     $this->assertEquals('title IS NULL', $expr);
                 },
             ],
             [
                 'not_null',
                 null,
-                function ($expr) {
+                function($expr) {
                     $this->assertEquals('title IS NOT NULL', $expr);
                 },
             ],
             [
                 'in',
-                [ 1, 2, 3 ],
-                function ($expr) {
+                [1, 2, 3],
+                function($expr) {
                     $this->assertInstanceOf(Func::class, $expr);
                     $this->assertEquals('title IN', $expr->getName());
                 },
             ],
             [
                 'nin',
-                [ 1, 2, 3 ],
-                function ($expr) {
+                [1, 2, 3],
+                function($expr) {
                     $this->assertInstanceOf(Func::class, $expr);
                     $this->assertEquals('title NOT IN', $expr->getName());
                 },

bridge/doctrine-orm/lib/OrmAgent.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-orm/tests/Functional/OrmExtension.php

Spacing +4 added lines, -4 removed lines

@@ -24,19 +24,19 @@ 
     public function getDefaultConfig()
     {
         return [
-            'db_path' => __DIR__ . '/../../../../cache/test.sqlite',
+            'db_path' => __DIR__.'/../../../../cache/test.sqlite',
         ];
     }
 
     public function load(Container $container)
     {
-        $container->register('entity_manager', function (Container $container) {
+        $container->register('entity_manager', function(Container $container) {
             $dbParams = [
                 'driver'    => 'pdo_sqlite',
                 'path' => $container->getParameter('db_path'),
             ];
             $paths = [
-                __DIR__ . '/mappings',
+                __DIR__.'/mappings',
             ];
             $config = Setup::createXMLMetadataConfiguration($paths, true);
             $manager = EntityManager::create($dbParams, $config);
@@ -44,7 +44,7 @@ 
             return $manager;
         });
 
-        $container->register('psi_object_agent.doctrine_orm', function (Container $container) {
+        $container->register('psi_object_agent.doctrine_orm', function(Container $container) {
             return new OrmAgent($container->get('entity_manager'));
         });
     }