Stinger-Soft / EntitySearchBundle

Model/ResultSetAdapter.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 	/**
 	 * @param Document[] $results
 	 */
-	public function setResults(array $results){
+	public function setResults(array $results) {
 		$this->results = $results;
 	}
 }
\ No newline at end of file

Model/Result/FacetSetAdapter.php

Spacing +3 added lines, -3 removed lines

@@ -30,10 +30,10 @@ 
 	}
 
 	public function addFacetValue($key, $value, $increaseCounterBy = 1) {
-		if(!isset($this->facets[$key])){
+		if (!isset($this->facets[$key])) {
 			$this->facets[$key] = array();
 		}
-		if(!isset($this->facets[$key][$value])){
+		if (!isset($this->facets[$key][$value])) {
 			$this->facets[$key][$value] = 0;
 		}
 		$this->facets[$key][$value] = $this->facets[$key][$value] + $increaseCounterBy;
@@ -46,7 +46,7 @@ 
 	 * @see \StingerSoft\EntitySearchBundle\Model\Result\FacetSet::getFacet()
 	 */
 	public function getFacet($key) {
-		if(isset($this->facets[$key])) {
+		if (isset($this->facets[$key])) {
 			return $this->facets[$key];
 		} else {
 			return;

Services/DoctrineListener.php

Spacing +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
 	 */
 	protected function updateEntity($entity, ObjectManager $manager) {
 		$document = $this->getEntityHandler()->createDocument($manager, $entity);
-		if($document !== false) {
+		if ($document !== false) {
 			$this->getSearchService($manager)->saveDocument($document);
 		}
 	}
@@ -108,7 +108,7 @@ 
 	 */
 	protected function removeEntity($entity, ObjectManager $manager) {
 		$document = $this->getEntityHandler()->createDocument($manager, $entity);
-		if($document !== false) {
+		if ($document !== false) {
 			$this->getSearchService($manager)->removeDocument($document);
 		}
 	}

Services/Mapping/EntityToDocumentMapper.php

Spacing +18 added lines, -18 removed lines

@@ -46,18 +46,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;
 			}
 			
@@ -72,10 +72,10 @@ 
 	 * @return boolean
 	 */
 	public function isIndexable($object) {
-		if($object instanceof SearchableEntity) {
+		if ($object instanceof SearchableEntity) {
 			return true;
 		}
-		if(count($this->getMapping($object)) > 0) {
+		if (count($this->getMapping($object)) > 0) {
 			return true;
 		}
 		return false;
@@ -89,11 +89,11 @@ 
 	 * @return boolean|Document Returns false if no document could be created
 	 */
 	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;
@@ -106,13 +106,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($object);
 		$accessor = PropertyAccess::createPropertyAccessor();
-		foreach($mapping as $fieldName => $propertyPath) {
+		foreach ($mapping as $fieldName => $propertyPath) {
 			$document->addField($fieldName, $accessor->getValue($object, $propertyPath));
 		}
 		return true;
@@ -126,15 +126,15 @@ 
 	 */
 	protected function getMapping($object) {
 		$clazz = get_class($object);
-		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

@@ -89,12 +89,14 @@
 	 * @return boolean|Document Returns false if no document could be created
 	 */
 	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;
 	}

Command/SyncCommand.php

Spacing +10 added lines, -10 removed lines

@@ -65,15 +65,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') {
 			// $indexHandler = $this->getIndexHandler();
 			// $entities = $indexHandler->getSearchableEntities();
 			// foreach($entities as $bundle => $searchableEntities) {
@@ -98,14 +98,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;
 		}
@@ -113,14 +113,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>');
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 	 * @return EntityToDocumentMapperInterface
 	 */
 	protected function getEntityToDocumentMapper() {
-		if(!$this->entityToDocumentMapper) {
+		if (!$this->entityToDocumentMapper) {
 			$this->entityToDocumentMapper = $this->getContainer()->get(EntityToDocumentMapperInterface::SERVICE_ID);
 		}
 		return $this->entityToDocumentMapper;
@@ -139,7 +139,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);

Services/Mapping/DocumentToEntityMapperInterface.php

Doc Comments -1 removed lines

@@ -25,7 +25,6 @@
 	 * Tries to create a document from the given object
 	 *
 	 * @param ObjectManager $manager        	
-	 * @param object $object        	
 	 * @return boolean|Document Returns false if no document could be created
 	 */
 	public function getEntity(ObjectManager $manager, Document $document);

Form/QueryType.php

Braces +3 added lines, -2 removed lines

@@ -66,8 +66,9 @@
 		$choices = array();
 		
 		foreach($facets as $facet => $count) {
-			if($count == 0)
-				break;
+			if($count == 0) {
+							break;
+			}
 			$choices[$facet] = $facet . ' (' . $count . ')';
 		}
 		return $choices;

Spacing +7 added lines, -7 removed lines

@@ -59,13 +59,13 @@ 
 	 * @param FacetSet $facets        	
 	 */
 	protected function createFacets(FormBuilderInterface $builder, FacetSet $facets) {
-		foreach($facets->getFacets() as $facetType => $facetValues) {
-			$builder->add('facet_' . $facetType, ChoiceType::class, array(
-				'label' => 'stinger_soft_entity_search.forms.query.' . $facetType . '.label',
+		foreach ($facets->getFacets() as $facetType => $facetValues) {
+			$builder->add('facet_'.$facetType, ChoiceType::class, array(
+				'label' => 'stinger_soft_entity_search.forms.query.'.$facetType.'.label',
 				'multiple' => true,
 				'expanded' => true,
 				'choices_as_values' => true,
-				'property_path' => 'facets[' . $facetType . ']',
+				'property_path' => 'facets['.$facetType.']',
 				'choices' => $this->generateFacetChoices($facetType, $facetValues) 
 			));
 		}
@@ -79,10 +79,10 @@ 
 	protected function generateFacetChoices($facetType, array $facets) {
 		$choices = array();
 		
-		foreach($facets as $facet => $count) {
-			if($count == 0)
+		foreach ($facets as $facet => $count) {
+			if ($count == 0)
 				break;
-			$choices[$facet . ' (' . $count . ')'] = $facet;
+			$choices[$facet.' ('.$count.')'] = $facet;
 		}
 		return $choices;
 	}