Stinger-Soft / EntitySearchBundle

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 +11 added lines, -11 removed lines

@@ -36,10 +36,10 @@ 
 		));
 
 		$usedFacets = $options['used_facets'];
-		if($usedFacets){
-			foreach($usedFacets as $facetType) {
-				$builder->add('facet_' . $facetType, FacetType::class, array(
-					'label' => 'stinger_soft_entity_search.forms.query.' . $facetType . '.label',
+		if ($usedFacets) {
+			foreach ($usedFacets as $facetType) {
+				$builder->add('facet_'.$facetType, FacetType::class, array(
+					'label' => 'stinger_soft_entity_search.forms.query.'.$facetType.'.label',
 					'multiple' => true,
 					'expanded' => true,
 				));
@@ -75,13 +75,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) 
 			));
 		}
@@ -95,10 +95,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;
 	}

Services/DummySearchService.php

Spacing +10 added lines, -10 removed lines

@@ -29,7 +29,7 @@ 
 		$id = $document->getEntityClass();
 		$id .= '#';
 		$entityId = $document->getEntityId();
-		if(is_scalar($entityId)) {
+		if (is_scalar($entityId)) {
 			$id .= $entityId;
 		} else {
 			$id .= md5(serialize($entityId));
@@ -75,15 +75,15 @@ 
 	 */
 	public function autocomplete($search, $maxResults = 10) {
 		$words = array();
-		foreach($this->index as $doc) {
-			foreach($doc->getFields() as $content) {
-				if(is_string($content)) {
+		foreach ($this->index as $doc) {
+			foreach ($doc->getFields() as $content) {
+				if (is_string($content)) {
 					$words = array_merge($words, explode(' ', $content));
 				}
 			}
 		}
 		
-		return array_filter($words, function ($word) use ($search) {
+		return array_filter($words, function($word) use ($search) {
 			return stripos($word, $search) === 0;
 		});
 	}
@@ -101,10 +101,10 @@ 
 		$facets = new FacetSetAdapter();
 		
 		$hits = array();
-		foreach($this->index as $key => $doc) {
-			foreach($doc->getFields() as $content) {
-				if(is_string($content) && stripos($content, $term) !== false) {
-					if(!isset($hits[$key])) {
+		foreach ($this->index as $key => $doc) {
+			foreach ($doc->getFields() as $content) {
+				if (is_string($content) && stripos($content, $term) !== false) {
+					if (!isset($hits[$key])) {
 						$hits[$key] = 0;
 					}
 					$hits[$key] = $hits[$key] + 1;
@@ -113,7 +113,7 @@ 
 		}
 		arsort($hits);
 		$results = array();
-		foreach(array_keys($hits) as $docId) {
+		foreach (array_keys($hits) as $docId) {
 			$doc = $this->index[$docId];
 			$facets->addFacetValue(FacetSet::FACET_ENTITY_TYPE, $doc->getEntityClass());
 			$facets->addFacetValue(FacetSet::FACET_AUTHOR, $doc->getFieldValue(Document::FIELD_AUTHOR));

Form/FacetType.php

Unused Use Statements -2 removed lines

@@ -11,11 +11,9 @@
  */
 namespace StingerSoft\EntitySearchBundle\Form;
 
-use StingerSoft\EntitySearchBundle\Model\Query;
 use StingerSoft\EntitySearchBundle\Model\Result\FacetSet;
 use StingerSoft\EntitySearchBundle\Model\ResultSet;
 use Symfony\Component\Form\AbstractType;
-use Symfony\Component\Form\ChoiceList\LazyChoiceList;
 use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Component\Form\FormEvent;

Spacing +10 added lines, -10 removed lines

@@ -36,12 +36,12 @@ 
 		$builder->resetModelTransformers();
 		$builder->resetViewTransformers();
 		
-		if($options['multiple']) {
-			$builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) {
+		if ($options['multiple']) {
+			$builder->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) {
 				$event->stopPropagation();
 			}, 1);
 		}
-		$builder->addEventListener(FormEvents::SUBMIT, function(FormEvent $event){
+		$builder->addEventListener(FormEvents::SUBMIT, function(FormEvent $event) {
 			$data = $event->getForm()->getExtraData();
 			$event->setData(array_unique(array_merge($data, $event->getData())));
 		});
@@ -69,13 +69,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) 
 			));
 		}
@@ -89,10 +89,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;
 	}

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;

Tests/Form/QueryTypeTest.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 	}
 	
 	
-	public function testWithNotExistingFacets(){
+	public function testWithNotExistingFacets() {
 		
 		$query = new Query('Hemelinger', array(), array(
 			Document::FIELD_TYPE