SyncCommand::execute() - Code Metrics - Inspection of "Updated sync and clear command" - Stinger-Soft/EntitySearchBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6c2a09...c47409 )

by Florian

created 2017-04-11 13:23 UTC

SyncCommand::execute() C

↳ Parent: SyncCommand

Complexity

Conditions	7
Paths	10

Size

Total Lines	40
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
dl	0
loc	40
rs	6.7272
c	2
b	0
f	0
cc	7
eloc	17
nc	10
nop	2

<?php

/*
 * This file is part of the Stinger Entity Search package.
 *
 * (c) Oliver Kotte <[email protected]>
 * (c) Florian Meyer <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace StingerSoft\EntitySearchBundle\Command;

use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Doctrine\Common\Persistence\ObjectManager;
use StingerSoft\EntitySearchBundle\Services\Mapping\EntityToDocumentMapperInterface;
use StingerSoft\EntitySearchBundle\Services\SearchService;
use Doctrine\ORM\EntityManager;
use Doctrine\ORM\Mapping\ClassMetadata;

class SyncCommand extends ContainerAwareCommand {

	/**
	 *
	 * @var EntityToDocumentMapperInterface
	 */
	protected $entityToDocumentMapper;

	/**
	 *
	 * @var SearchService
	 */
	protected $searchService;

	/**
	 *
	 * Cache for the default upload path of this platform
	 *
	 * @var string
	 */
	protected static $defaultUploadPath = null;

	/**
	 *
	 * {@inheritdoc}
	 *
	 * @see \Symfony\Component\Console\Command\Command::configure()
	 */
	protected function configure() {
		/* @formatter:off */
		$this
			->setName('stinger:search:sync')
			->addArgument('entity', InputArgument::REQUIRED, 'The entity you want to index')
			->addOption('source', null, InputArgument::OPTIONAL, 'specify a source from where to load entities [relational, mongodb] (unsupported!)', 'relational')
			->setDescription('Index all entities');
		/* @formatter:on */
	}

	/**
	 *
	 * {@inheritdoc}
	 *
	 * @see \Symfony\Component\Console\Command\Command::execute()
	 */
	protected function execute(InputInterface $input, OutputInterface $output) {
		// Detect upload path
		if(!self::$defaultUploadPath) {
			$root = $this->getContainer()->get('kernel')->getRootDir();
			self::$defaultUploadPath = $root . '/../web/uploads';
		}
		
		// Get the entity argument
		$entity = $input->getArgument('entity');
		
		if($entity == 'all') {
			/**
			 * @var EntityManager $entityManager
			 */
			$entityManager = $this->getContainer()->get('doctrine.orm.entity_manager');
			
			$meta = $entityManager->getMetadataFactory()->getAllMetadata();
			
			/**
			 * @var EntityToDocumentMapperInterface $mapper
			 */
			$mapper = $this->getContainer()->get(EntityToDocumentMapperInterface::SERVICE_ID);
			
			/**
			 * @var ClassMetadata $m
			 */
			foreach($meta as $m) {
				
				if($m->getReflectionClass()->isAbstract() || $m->getReflectionClass()->isInterface()) {
					continue;
				}
				if(!$mapper->isClassIndexable($m->getReflectionClass()->getName())) {
					continue;
				}
				$this->indexEntity($input, $output, $m->getReflectionClass()->getName());
			}
		} else {
			$this->indexEntity($input, $output, $entity);
		}
	}

	protected function indexEntity(InputInterface $input, OutputInterface $output, $entity) {

		$output->writeln(sprintf('<comment>Indexing entities of type "%s"</comment>', $entity));
		/**
		 *
		 * @var EntityManager $entityManager
		 */
		$entityManager = $this->getContainer()->get('doctrine.orm.entity_manager');
		$repository = null;
		try {
			// Get repository for the given entity type
			$repository = $entityManager->getRepository($entity);
		} catch(\Exception $e) {
			$output->writeln(sprintf('<error>No repository found for "%s", check your input</error>', $entity));
			return;
		}
		
		// Get all entities
		$entities = $repository->findAll();
		if(count($entities) == 0) {
			$output->writeln('<comment>No entities found for indexing</comment>');
			return;
		}
		
		$entitiesIndexed = 0;
		
		// Index each entity seperate
		foreach($entities as $entity) {
			if($this->getEntityToDocumentMapper()->isIndexable($entity)) {
				$document = $this->getEntityToDocumentMapper()->createDocument($entityManager, $entity);
				if($document === false) continue;
				$this->getSearchService($entityManager)->saveDocument($document);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
				$entitiesIndexed++;
				if($entitiesIndexed % 50 == 0) {
					$entityManager->flush();
				}
			}
			
		}
		$entityManager->flush();
		$output->writeln('<comment>Indexed ' . $entitiesIndexed . ' entities</comment>');
	}

	/**
	 *
	 * @return EntityToDocumentMapperInterface
	 */
	protected function getEntityToDocumentMapper() {
		if(!$this->entityToDocumentMapper) {
			$this->entityToDocumentMapper = $this->getContainer()->get(EntityToDocumentMapperInterface::SERVICE_ID);
		}
		return $this->entityToDocumentMapper;
	}

	/**
	 *
	 * @return SearchService
	 */
	protected function getSearchService(ObjectManager $manager) {
		if(!$this->searchService) {
			$this->searchService = $this->getContainer()->get(SearchService::SERVICE_ID);
		}
		$this->searchService->setObjectManager($manager);
		return $this->searchService;
	}
}

1			<?php
2
3			/*
4			* This file is part of the Stinger Entity Search package.
5			*
6			* (c) Oliver Kotte <[email protected]>
7			* (c) Florian Meyer <[email protected]>
8			*
9			* For the full copyright and license information, please view the LICENSE
10			* file that was distributed with this source code.
11			*/
12			namespace StingerSoft\EntitySearchBundle\Command;
13
14			use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
15			use Symfony\Component\Console\Input\InputArgument;
16			use Symfony\Component\Console\Input\InputInterface;
17			use Symfony\Component\Console\Output\OutputInterface;
18			use Doctrine\Common\Persistence\ObjectManager;
19			use StingerSoft\EntitySearchBundle\Services\Mapping\EntityToDocumentMapperInterface;
20			use StingerSoft\EntitySearchBundle\Services\SearchService;
21			use Doctrine\ORM\EntityManager;
22			use Doctrine\ORM\Mapping\ClassMetadata;
23
24			class SyncCommand extends ContainerAwareCommand {
25
26			/**
27			*
28			* @var EntityToDocumentMapperInterface
29			*/
30			protected $entityToDocumentMapper;
31
32			/**
33			*
34			* @var SearchService
35			*/
36			protected $searchService;
37
38			/**
39			*
40			* Cache for the default upload path of this platform
41			*
42			* @var string
43			*/
44			protected static $defaultUploadPath = null;
45
46			/**
47			*
48			* {@inheritdoc}
49			*
50			* @see \Symfony\Component\Console\Command\Command::configure()
51			*/
52			protected function configure() {
53			/* @formatter:off */
54			$this
55			->setName('stinger:search:sync')
56			->addArgument('entity', InputArgument::REQUIRED, 'The entity you want to index')
57			->addOption('source', null, InputArgument::OPTIONAL, 'specify a source from where to load entities [relational, mongodb] (unsupported!)', 'relational')
58			->setDescription('Index all entities');
59			/* @formatter:on */
60			}
61
62			/**
63			*
64			* {@inheritdoc}
65			*
66			* @see \Symfony\Component\Console\Command\Command::execute()
67			*/
68			protected function execute(InputInterface $input, OutputInterface $output) {
69			// Detect upload path
70			if(!self::$defaultUploadPath) {
71			$root = $this->getContainer()->get('kernel')->getRootDir();
72			self::$defaultUploadPath = $root . '/../web/uploads';
73			}
74
75			// Get the entity argument
76			$entity = $input->getArgument('entity');
77
78			if($entity == 'all') {
79			/**
80			* @var EntityManager $entityManager
81			*/
82			$entityManager = $this->getContainer()->get('doctrine.orm.entity_manager');
83
84			$meta = $entityManager->getMetadataFactory()->getAllMetadata();
85
86			/**
87			* @var EntityToDocumentMapperInterface $mapper
88			*/
89			$mapper = $this->getContainer()->get(EntityToDocumentMapperInterface::SERVICE_ID);
90
91			/**
92			* @var ClassMetadata $m
93			*/
94			foreach($meta as $m) {
95
96			if($m->getReflectionClass()->isAbstract() \|\| $m->getReflectionClass()->isInterface()) {
97			continue;
98			}
99			if(!$mapper->isClassIndexable($m->getReflectionClass()->getName())) {
100			continue;
101			}
102			$this->indexEntity($input, $output, $m->getReflectionClass()->getName());
103			}
104			} else {
105			$this->indexEntity($input, $output, $entity);
106			}
107			}
108
109			protected function indexEntity(InputInterface $input, OutputInterface $output, $entity) {
			0 ignored issues – show Unused Code introduced 2016-06-22 11:30 UTC by Report Bug Copy Issue Report The parameter `$input` 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...
110			$output->writeln(sprintf('<comment>Indexing entities of type "%s"</comment>', $entity));
111			/**
112			*
113			* @var EntityManager $entityManager
114			*/
115			$entityManager = $this->getContainer()->get('doctrine.orm.entity_manager');
116			$repository = null;
117			try {
118			// Get repository for the given entity type
119			$repository = $entityManager->getRepository($entity);
120			} catch(\Exception $e) {
121			$output->writeln(sprintf('<error>No repository found for "%s", check your input</error>', $entity));
122			return;
123			}
124
125			// Get all entities
126			$entities = $repository->findAll();
127			if(count($entities) == 0) {
128			$output->writeln('<comment>No entities found for indexing</comment>');
129			return;
130			}
131
132			$entitiesIndexed = 0;
133
134			// Index each entity seperate
135			foreach($entities as $entity) {
136			if($this->getEntityToDocumentMapper()->isIndexable($entity)) {
137			$document = $this->getEntityToDocumentMapper()->createDocument($entityManager, $entity);
138			if($document === false) continue;
139			$this->getSearchService($entityManager)->saveDocument($document);
			0 ignored issues – show Bug introduced 2016-06-22 11:30 UTC by Report Bug Copy Issue Report It seems like `$document` defined by `$this->getEntityToDocume...entityManager, $entity)` on line `137` can also be of type `boolean`; however, `StingerSoft\EntitySearch...Service::saveDocument()` does only seem to accept `object<StingerSoft\Entit...hBundle\Model\Document>`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
140			$entitiesIndexed++;
141			if($entitiesIndexed % 50 == 0) {
142			$entityManager->flush();
143			}
144			}
145
146			}
147			$entityManager->flush();
148			$output->writeln('<comment>Indexed ' . $entitiesIndexed . ' entities</comment>');
149			}
150
151			/**
152			*
153			* @return EntityToDocumentMapperInterface
154			*/
155			protected function getEntityToDocumentMapper() {
156			if(!$this->entityToDocumentMapper) {
157			$this->entityToDocumentMapper = $this->getContainer()->get(EntityToDocumentMapperInterface::SERVICE_ID);
158			}
159			return $this->entityToDocumentMapper;
160			}
161
162			/**
163			*
164			* @return SearchService
165			*/
166			protected function getSearchService(ObjectManager $manager) {
167			if(!$this->searchService) {
168			$this->searchService = $this->getContainer()->get(SearchService::SERVICE_ID);
169			}
170			$this->searchService->setObjectManager($manager);
171			return $this->searchService;
172			}
173			}

Stinger-Soft / EntitySearchBundle

GitHub Access Token became invalid

Push — master ( 6c2a09...c47409 )

SyncCommand::execute() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like