ElasticWalker::getExecutor() - Code Metrics - Inspection of "More tests, some refatorations, and child parent f..." - andsalves/doctrine-elastic - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a11bd1...3fc1f5 )

by Anderson

created 2017-04-10 00:23 UTC

ElasticWalker::getExecutor() A

↳ Parent: ElasticWalker::walkSelectStatement()

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	0

<?php

namespace DoctrineElastic\Query;

use Doctrine\ORM\Query\AST\FromClause;
use Doctrine\ORM\Query\AST\GroupByClause;
use Doctrine\ORM\Query\AST\HavingClause;
use Doctrine\ORM\Query\AST\OrderByClause;
use Doctrine\ORM\Query\AST\OrderByItem;
use Doctrine\ORM\Query\AST\SelectClause;
use Doctrine\ORM\Query\AST\SelectStatement;
use Doctrine\ORM\Query\AST\WhereClause;
use DoctrineElastic\Elastic\ElasticQuery;
use DoctrineElastic\Elastic\SearchParams;
use DoctrineElastic\Mapping\Type;
use DoctrineElastic\Query\Walker\Helper\WalkerHelper;
use DoctrineElastic\Query\Walker\WhereWalker;
use DoctrineElastic\Service\ElasticSearchService;

/**
 * Main walker for queries
 *
 * @uahtor Ands
 */
class ElasticWalker {

    /** @var ElasticQuery */
    protected $query;

    /** @var string */
    private $_className;

    /** @var WalkerHelper */
    private $walkerHelper;

    /** @var SelectStatement */
    private $_ast;

    public function __construct(ElasticQuery $query, SelectStatement $AST, $className) {
        $this->query = $query;
        $this->walkerHelper = new WalkerHelper();
        $this->_ast = $AST;
        $this->_className = $className;
    }

    /**
     * @return SearchParams
     */
    public function walkSelectStatement() {
        $searchParams = new SearchParams();
        $size = $this->query->getMaxResults();
        $offset = $this->query->getFirstResult();

        $type = $this->getEntityElasticType($this->_className);

        $searchParams->setType($type->getName());
        $searchParams->setIndex($type->getIndex());

        $this->walkSelectClause($this->_ast->selectClause, $searchParams);
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
        $this->walkFromClause($this->_ast->fromClause, $searchParams);
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}

        if ($this->_ast->whereClause) {
            $this->walkWhereClause($this->_ast->whereClause, $searchParams);
        }

        if ($this->_ast->groupByClause) {
            $this->walkGroupByClause($this->_ast->groupByClause, $searchParams);
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
        }

        if ($this->_ast->havingClause) {
            $this->walkHavingClause($this->_ast->havingClause, $searchParams);
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
        }

        if ($this->_ast->orderByClause) {
            $this->walkOrderByClause($this->_ast->orderByClause, $searchParams);
        }

        $searchParams->setSize($size);
        $searchParams->setFrom($offset);

        return $searchParams;
    }

    private function walkSelectClause(SelectClause $selectClause, SearchParams $searchParams) {


    }

    private function walkFromClause(FromClause $fromClause, SearchParams $searchParams) {


    }

    private function walkWhereClause(WhereClause $whereClause, SearchParams $searchParams) {
        $whereWalker = new WhereWalker($this->query, $this->_className, $this->walkerHelper);
        $whereWalker->walk($whereClause, $searchParams);
    }

    private function walkGroupByClause(GroupByClause $groupByClause, SearchParams $searchParams) {


    }

    private function walkHavingClause(HavingClause $havingClause, SearchParams $searchParams) {


    }

    private function walkOrderByClause(OrderByClause $orderByClause, SearchParams $searchParams) {
        /** @var OrderByItem $item */
        foreach ($orderByClause->orderByItems as $item) {
            $order = $item->type;
            $propertyName = $item->expression->field;
            $ESField = $this->walkerHelper->getEntityElasticField($propertyName, $this->_className, $this->query);

            $searchParams->setSort([$ESField->name => strtolower($order)]);
        }
    }

    /**
     * @param $className
     * @return Type
     */
    private function getEntityElasticType($className) {
        $classMetadata = $this->query->getEntityManager()->getClassMetadata($className);

        $entityPersister = $this->query->getEntityManager()->getUnitOfWork()
            ->getEntityPersister($className);
        /** @var Type $type */
        $type = $entityPersister->getAnnotionReader()
            ->getClassAnnotation($classMetadata->getReflectionClass(), Type::class);

        return $type;
    }
}

1			<?php
2
3			namespace DoctrineElastic\Query;
4
5			use Doctrine\ORM\Query\AST\FromClause;
6			use Doctrine\ORM\Query\AST\GroupByClause;
7			use Doctrine\ORM\Query\AST\HavingClause;
8			use Doctrine\ORM\Query\AST\OrderByClause;
9			use Doctrine\ORM\Query\AST\OrderByItem;
10			use Doctrine\ORM\Query\AST\SelectClause;
11			use Doctrine\ORM\Query\AST\SelectStatement;
12			use Doctrine\ORM\Query\AST\WhereClause;
13			use DoctrineElastic\Elastic\ElasticQuery;
14			use DoctrineElastic\Elastic\SearchParams;
15			use DoctrineElastic\Mapping\Type;
16			use DoctrineElastic\Query\Walker\Helper\WalkerHelper;
17			use DoctrineElastic\Query\Walker\WhereWalker;
18			use DoctrineElastic\Service\ElasticSearchService;
19
20			/**
21			* Main walker for queries
22			*
23			* @uahtor Ands
24			*/
25			class ElasticWalker {
26
27			/** @var ElasticQuery */
28			protected $query;
29
30			/** @var string */
31			private $_className;
32
33			/** @var WalkerHelper */
34			private $walkerHelper;
35
36			/** @var SelectStatement */
37			private $_ast;
38
39			public function __construct(ElasticQuery $query, SelectStatement $AST, $className) {
40			$this->query = $query;
41			$this->walkerHelper = new WalkerHelper();
42			$this->_ast = $AST;
43			$this->_className = $className;
44			}
45
46			/**
47			* @return SearchParams
48			*/
49			public function walkSelectStatement() {
50			$searchParams = new SearchParams();
51			$size = $this->query->getMaxResults();
52			$offset = $this->query->getFirstResult();
53
54			$type = $this->getEntityElasticType($this->_className);
55
56			$searchParams->setType($type->getName());
57			$searchParams->setIndex($type->getIndex());
58
59			$this->walkSelectClause($this->_ast->selectClause, $searchParams);
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The call to the method `DoctrineElastic\Query\El...ker::walkSelectClause()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
60			$this->walkFromClause($this->_ast->fromClause, $searchParams);
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The call to the method `DoctrineElastic\Query\El...alker::walkFromClause()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
61
62			if ($this->_ast->whereClause) {
63			$this->walkWhereClause($this->_ast->whereClause, $searchParams);
64			}
65
66			if ($this->_ast->groupByClause) {
67			$this->walkGroupByClause($this->_ast->groupByClause, $searchParams);
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The call to the method `DoctrineElastic\Query\El...er::walkGroupByClause()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
68			}
69
70			if ($this->_ast->havingClause) {
71			$this->walkHavingClause($this->_ast->havingClause, $searchParams);
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The call to the method `DoctrineElastic\Query\El...ker::walkHavingClause()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
72			}
73
74			if ($this->_ast->orderByClause) {
75			$this->walkOrderByClause($this->_ast->orderByClause, $searchParams);
76			}
77
78			$searchParams->setSize($size);
79			$searchParams->setFrom($offset);
80
81			return $searchParams;
82			}
83
84			private function walkSelectClause(SelectClause $selectClause, SearchParams $searchParams) {
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$selectClause` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$searchParams` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
85
86			}
87
88			private function walkFromClause(FromClause $fromClause, SearchParams $searchParams) {
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$fromClause` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$searchParams` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
89
90			}
91
92			private function walkWhereClause(WhereClause $whereClause, SearchParams $searchParams) {
93			$whereWalker = new WhereWalker($this->query, $this->_className, $this->walkerHelper);
94			$whereWalker->walk($whereClause, $searchParams);
95			}
96
97			private function walkGroupByClause(GroupByClause $groupByClause, SearchParams $searchParams) {
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$groupByClause` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$searchParams` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
98
99			}
100
101			private function walkHavingClause(HavingClause $havingClause, SearchParams $searchParams) {
			0 ignored issues – show Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$havingClause` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2017-03-10 23:06 UTC by Report Bug Copy Issue Report The parameter `$searchParams` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
102
103			}
104
105			private function walkOrderByClause(OrderByClause $orderByClause, SearchParams $searchParams) {
106			/** @var OrderByItem $item */
107			foreach ($orderByClause->orderByItems as $item) {
108			$order = $item->type;
109			$propertyName = $item->expression->field;
110			$ESField = $this->walkerHelper->getEntityElasticField($propertyName, $this->_className, $this->query);
111
112			$searchParams->setSort([$ESField->name => strtolower($order)]);
113			}
114			}
115
116			/**
117			* @param $className
118			* @return Type
119			*/
120			private function getEntityElasticType($className) {
121			$classMetadata = $this->query->getEntityManager()->getClassMetadata($className);
122
123			$entityPersister = $this->query->getEntityManager()->getUnitOfWork()
124			->getEntityPersister($className);
125			/** @var Type $type */
126			$type = $entityPersister->getAnnotionReader()
127			->getClassAnnotation($classMetadata->getReflectionClass(), Type::class);
128
129			return $type;
130			}
131			}

andsalves / doctrine-elastic

GitHub Access Token became invalid

Push — master ( a11bd1...3fc1f5 )

ElasticWalker::getExecutor() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like