Stinger-Soft / EntitySearchBundle

Services/Mapping/EntityToDocumentMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
 	/**
 	 * Fetches the mapping for the given object including the mapping of superclasses
 	 *
-	 * @param object $object        	
+	 * @param string $clazz
 	 * @return \StingerSoft\EntitySearchBundle\Services\string[string]
 	 */
 	protected function getMapping($clazz) {

Spacing +20 added lines, -20 removed lines

@@ -47,18 +47,18 @@ 
 	 */
 	public function __construct(SearchService $searchService, array $mapping = array()) {
 		$this->searchService = $searchService;
-		foreach($mapping as $key => $config) {
-			if(!isset($config['mappings'])) {
-				throw new \InvalidArgumentException($key . ' has no mapping defined!');
+		foreach ($mapping as $key => $config) {
+			if (!isset($config['mappings'])) {
+				throw new \InvalidArgumentException($key.' has no mapping defined!');
 			}
-			if(!isset($config['persistence'])) {
-				throw new \InvalidArgumentException($key . ' has no persistence defined!');
+			if (!isset($config['persistence'])) {
+				throw new \InvalidArgumentException($key.' has no persistence defined!');
 			}
-			if(!isset($config['persistence']['model'])) {
-				throw new \InvalidArgumentException($key . ' has no model defined!');
+			if (!isset($config['persistence']['model'])) {
+				throw new \InvalidArgumentException($key.' has no model defined!');
 			}
 			$map = array();
-			foreach($config['mappings'] as $fieldKey => $fieldConfig) {
+			foreach ($config['mappings'] as $fieldKey => $fieldConfig) {
 				$map[$fieldKey] = isset($fieldConfig['propertyPath']) && $fieldConfig['propertyPath'] ? $fieldConfig['propertyPath'] : $fieldKey;
 			}
 			
@@ -73,10 +73,10 @@ 
 	 * @see \StingerSoft\EntitySearchBundle\Services\Mapping\EntityToDocumentMapperInterface::isIndexable()
 	 */
 	public function isIndexable($object) {
-		if($object instanceof SearchableEntity) {
+		if ($object instanceof SearchableEntity) {
 			return true;
 		}
-		if(count($this->getMapping(get_class($object))) > 0) {
+		if (count($this->getMapping(get_class($object))) > 0) {
 			return true;
 		}
 		return false;
@@ -90,10 +90,10 @@ 
 	 */
 	public function isClassIndexable($clazz) {
 		$reflectionClass = new \ReflectionClass($clazz);
-		if(array_key_exists(SearchableEntity::class, $reflectionClass->getInterfaces())) {
+		if (array_key_exists(SearchableEntity::class, $reflectionClass->getInterfaces())) {
 			return true;
 		}
-		if(count($this->getMapping($clazz)) > 0) {
+		if (count($this->getMapping($clazz)) > 0) {
 			return true;
 		}
 		return false;
@@ -105,11 +105,11 @@ 
 	 * @see \StingerSoft\EntitySearchBundle\Services\Mapping\EntityToDocumentMapperInterface::createDocument()
 	 */
 	public function createDocument(ObjectManager $manager, $object) {
-		if(!$this->isIndexable($object))
+		if (!$this->isIndexable($object))
 			return false;
 		$document = $this->getSearchService($manager)->createEmptyDocumentFromEntity($object);
 		$index = $this->fillDocument($document, $object);
-		if($index == false)
+		if ($index == false)
 			return false;
 		
 		return $document;
@@ -122,13 +122,13 @@ 
 	 * @param object $object        	
 	 * @return boolean
 	 */
-	protected function fillDocument(Document &$document, $object) {
-		if($object instanceof SearchableEntity) {
+	protected function fillDocument(Document&$document, $object) {
+		if ($object instanceof SearchableEntity) {
 			return $object->indexEntity($document);
 		}
 		$mapping = $this->getMapping(get_class($object));
 		$accessor = PropertyAccess::createPropertyAccessor();
-		foreach($mapping as $fieldName => $propertyPath) {
+		foreach ($mapping as $fieldName => $propertyPath) {
 			$document->addField($fieldName, $accessor->getValue($object, $propertyPath));
 		}
 		return true;
@@ -141,15 +141,15 @@ 
 	 * @return \StingerSoft\EntitySearchBundle\Services\string[string]
 	 */
 	protected function getMapping($clazz) {
-		if(isset($this->cachedMapping[$clazz])) {
+		if (isset($this->cachedMapping[$clazz])) {
 			return $this->cachedMapping[$clazz];
 		}
 		$ref = new \ReflectionClass($clazz);
 		
 		$mapping = array();
 		
-		foreach($this->mapping as $className => $config) {
-			if($clazz == $className || $ref->isSubclassOf($className)) {
+		foreach ($this->mapping as $className => $config) {
+			if ($clazz == $className || $ref->isSubclassOf($className)) {
 				$mapping = array_merge($mapping, $config);
 			}
 		}

Braces +6 added lines, -4 removed lines

@@ -105,12 +105,14 @@
 	 * @see \StingerSoft\EntitySearchBundle\Services\Mapping\EntityToDocumentMapperInterface::createDocument()
 	 */
 	public function createDocument(ObjectManager $manager, $object) {
-		if(!$this->isIndexable($object))
-			return false;
+		if(!$this->isIndexable($object)) {
+					return false;
+		}
 		$document = $this->getSearchService($manager)->createEmptyDocumentFromEntity($object);
 		$index = $this->fillDocument($document, $object);
-		if($index == false)
-			return false;
+		if($index == false) {
+					return false;
+		}
 		
 		return $document;
 	}

Controller/SearchController.php

Spacing +8 added lines, -8 removed lines

@@ -55,7 +55,7 @@ 
 	private $searchService;
 
 	public function searchAction(Request $request) {
-		if($request->query->get('term', false) !== false) {
+		if ($request->query->get('term', false) !== false) {
 			$this->setSearchTerm($request->getSession(), $request->query->get('term'));
 			return $this->redirectToRoute('stinger_soft_entity_search_search');
 		}
@@ -70,8 +70,8 @@ 
 		));
 		
 		$facetForm->handleRequest($request);
-		if($facetForm->isSubmitted()) {
-			if($facetForm->get('clear')->isClicked()) {
+		if ($facetForm->isSubmitted()) {
+			if ($facetForm->get('clear')->isClicked()) {
 				$query->setFacets($this->getDefaultFacets());
 			}
 			$this->setSearchTerm($request->getSession(), $query->getSearchTerm());
@@ -87,7 +87,7 @@ 
 		
 		$page = $request->query->get('page', 1);
 		$results = array();
-		if($result instanceof PaginatableResultSet) {
+		if ($result instanceof PaginatableResultSet) {
 			$results = $result->paginate($page, self::RESULTS_PER_PAGE);
 		} else {
 			$results = $result->getResults(($page - 1) * self::RESULTS_PER_PAGE, self::RESULTS_PER_PAGE);
@@ -136,7 +136,7 @@ 
 	 * @return string[string][]
 	 */
 	protected function getSearchFacets(SessionInterface $session) {
-		$facets = $session()->get(self::SESSION_PREFIX . '_facets', false);
+		$facets = $session()->get(self::SESSION_PREFIX.'_facets', false);
 		return $facets ? json_decode($facets, true) : $this->getDefaultFacets();
 	}
 
@@ -147,7 +147,7 @@ 
 	 * @param string[string][] $facets
 	 */
 	protected function setSearchFacets(SessionInterface $session, $facets) {
-		$session->set(self::SESSION_PREFIX . '_facets', json_encode($facets));
+		$session->set(self::SESSION_PREFIX.'_facets', json_encode($facets));
 	}
 
 	/**
@@ -157,7 +157,7 @@ 
 	 * @return mixed
 	 */
 	protected function getSearchTerm(SessionInterface $session) {
-		return $session->get(self::SESSION_PREFIX . '_term', false);
+		return $session->get(self::SESSION_PREFIX.'_term', false);
 	}
 
 	/**
@@ -167,7 +167,7 @@ 
 	 * @param string $term        	
 	 */
 	protected function setSearchTerm(SessionInterface $session, $term) {
-		$session->set(self::SESSION_PREFIX . '_term', $term);
+		$session->set(self::SESSION_PREFIX.'_term', $term);
 	}
 
 	/**

Command/SyncCommand.php

Spacing +13 added lines, -13 removed lines

@@ -67,15 +67,15 @@ 
 	 */
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		// Detect upload path
-		if(!self::$defaultUploadPath) {
+		if (!self::$defaultUploadPath) {
 			$root = $this->getContainer()->get('kernel')->getRootDir();
-			self::$defaultUploadPath = $root . '/../web/uploads';
+			self::$defaultUploadPath = $root.'/../web/uploads';
 		}
 		
 		// Get the entity argument
 		$entity = $input->getArgument('entity');
 		
-		if($entity == 'all') {
+		if ($entity == 'all') {
 			/**
 			 * @var EntityManager $entityManager
 			 */
@@ -91,11 +91,11 @@ 
 			/**
 			 * @var ClassMetadata $m
 			 */
-			foreach($meta as $m) {
-				if($m->getReflectionClass()->isAbstract() || $m->getReflectionClass()->isInterface()) {
+			foreach ($meta as $m) {
+				if ($m->getReflectionClass()->isAbstract() || $m->getReflectionClass()->isInterface()) {
 					continue;
 				}
-				if(!$mapper->isClassIndexable($m->getReflectionClass()->getName())) {
+				if (!$mapper->isClassIndexable($m->getReflectionClass()->getName())) {
 					continue;
 				}
 				$this->indexEntity($input, $output, $m->getReflectionClass()->getName());
@@ -115,14 +115,14 @@ 
 		try {
 			// Get repository for the given entity type
 			$repository = $entityManager->getRepository($entity);
-		} catch(\Exception $e) {
+		} 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) {
+		if (count($entities) == 0) {
 			$output->writeln('<comment>No entities found for indexing</comment>');
 			return;
 		}
@@ -130,14 +130,14 @@ 
 		$entitiesIndexed = 0;
 		
 		// Index each entity seperate
-		foreach($entities as $entity) {
-			if($this->getEntityToDocumentMapper()->isIndexable($entity)) {
+		foreach ($entities as $entity) {
+			if ($this->getEntityToDocumentMapper()->isIndexable($entity)) {
 				$document = $this->getEntityToDocumentMapper()->createDocument($entityManager, $entity);
 				$this->getSearchService($entityManager)->saveDocument($document);
 				$entitiesIndexed++;
 			}
 		}
-		$output->writeln('<comment>Indexed ' . $entitiesIndexed . ' entities</comment>');
+		$output->writeln('<comment>Indexed '.$entitiesIndexed.' entities</comment>');
 	}
 
 	/**
@@ -145,7 +145,7 @@ 
 	 * @return EntityToDocumentMapperInterface
 	 */
 	protected function getEntityToDocumentMapper() {
-		if(!$this->entityToDocumentMapper) {
+		if (!$this->entityToDocumentMapper) {
 			$this->entityToDocumentMapper = $this->getContainer()->get(EntityToDocumentMapperInterface::SERVICE_ID);
 		}
 		return $this->entityToDocumentMapper;
@@ -156,7 +156,7 @@ 
 	 * @return SearchService
 	 */
 	protected function getSearchService(ObjectManager $manager) {
-		if(!$this->searchService) {
+		if (!$this->searchService) {
 			$this->searchService = $this->getContainer()->get(SearchService::SERVICE_ID);
 		}
 		$this->searchService->setObjectManager($manager);