Stinger-Soft / EntitySearchBundle

DependencyInjection/Configuration.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
-use Symfony\Component\HttpKernel\Kernel;
 
 
 /**

Services/Mapping/DocumentToEntityMapperInterface.php

Doc Comments -1 removed lines

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

Controller/SearchControllerTrait.php

Spacing +14 added lines, -14 removed lines

@@ -32,7 +32,7 @@ 
 	private $facetFormatter;
 
 	public function searchAction(Request $request, DocumentToEntityMapperInterface $mapper) {
-		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');
 		}
@@ -48,8 +48,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());
@@ -69,9 +69,9 @@ 
 
 			$page = (int)$request->query->get('page', 1);
 			$results = null;
-			if($result instanceof PaginatableResultSet) {
+			if ($result instanceof PaginatableResultSet) {
 				$results = $result->paginate($page, $this->getResultsPerPage());
-			} elseif($result !== null) {
+			} elseif ($result !== null) {
 				$results = $result->getResults(($page - 1) * $this->getResultsPerPage(), $this->getResultsPerPage());
 			}
 
@@ -82,7 +82,7 @@ 
 				'mapper'    => $mapper,
 				'facetForm' => $facetForm->createView()
 			));
-		} catch(\Exception $exception) {
+		} catch (\Exception $exception) {
 			$response = $this->render($this->getErrorTemplate(), array(
 				'error' => $exception->getMessage(),
 				'term'  => $query->getSearchTerm()
@@ -126,7 +126,7 @@ 
 	protected function getConfiguredUsedFacets(array $queryUsedFacets) {
 		$availableFacets = $this->getAvailableFacets();
 		$usedFacets = array();
-		foreach($queryUsedFacets as $queryUsedFacet) {
+		foreach ($queryUsedFacets as $queryUsedFacet) {
 			$usedFacets[$queryUsedFacet] = $availableFacets[$queryUsedFacet];
 		}
 		return $usedFacets;
@@ -193,15 +193,15 @@ 
 	}
 
 	protected function initFacets() {
-		if(!$this->availableFacets) {
+		if (!$this->availableFacets) {
 			$this->availableFacets = array();
 			$this->facetFormatter = array();
 
 			$facetServices = $this->getParameter('stinger_soft.entity_search.available_facets');
-			foreach($facetServices as $facetServiceId) {
+			foreach ($facetServices as $facetServiceId) {
 				$facetService = $this->searchService->getFacet($facetServiceId);
 				$this->availableFacets[$facetService->getField()] = $facetService->getFormOptions();
-				if($facetService->getFacetFormatter()) {
+				if ($facetService->getFacetFormatter()) {
 					$this->facetFormatter[$facetService->getField()] = $facetService->getFacetFormatter();
 				}
 			}
@@ -215,7 +215,7 @@ 
 	 * @return string[string][]
 	 */
 	protected function getSearchFacets(SessionInterface $session) {
-		$facets = $session->get($this->getSessionPrefix() . '_facets', false);
+		$facets = $session->get($this->getSessionPrefix().'_facets', false);
 		return $facets ? \json_decode($facets, true) : $this->getDefaultFacets();
 	}
 
@@ -226,7 +226,7 @@ 
 	 * @param string[string][] $facets
 	 */
 	protected function setSearchFacets(SessionInterface $session, $facets) {
-		$session->set($this->getSessionPrefix() . '_facets', \json_encode($facets));
+		$session->set($this->getSessionPrefix().'_facets', \json_encode($facets));
 	}
 
 	/**
@@ -236,7 +236,7 @@ 
 	 * @return mixed
 	 */
 	protected function getSearchTerm(SessionInterface $session) {
-		return $session->get($this->getSessionPrefix() . '_term', false);
+		return $session->get($this->getSessionPrefix().'_term', false);
 	}
 
 	/**
@@ -246,7 +246,7 @@ 
 	 * @param string $term
 	 */
 	protected function setSearchTerm(SessionInterface $session, $term) {
-		$session->set($this->getSessionPrefix() . '_term', $term);
+		$session->set($this->getSessionPrefix().'_term', $term);
 	}
 
 }
\ No newline at end of file

Services/DoctrineListener.php

Spacing +9 added lines, -9 removed lines

@@ -118,7 +118,7 @@ 
 	 * @param LifecycleEventArgs $args
 	 */
 	public function postPersist(LifecycleEventArgs $args): void {
-		if(!$this->enableIndexing) {
+		if (!$this->enableIndexing) {
 			return;
 		}
 		$this->updateEntity($args->getObject(), $args->getObjectManager());
@@ -130,7 +130,7 @@ 
 	 * @throws \Doctrine\ORM\OptimisticLockException
 	 */
 	public function postFlush(PostFlushEventArgs $eventArgs): void {
-		if($this->needsFlush) {
+		if ($this->needsFlush) {
 			$this->needsFlush = false;
 			$eventArgs->getEntityManager()->flush();
 		}
@@ -142,7 +142,7 @@ 
 	 * @param LifecycleEventArgs $args
 	 */
 	public function preRemove(LifecycleEventArgs $args): void {
-		if(!$this->enableIndexing) {
+		if (!$this->enableIndexing) {
 			return;
 		}
 		$this->removeEntity($args->getObject(), $args->getObjectManager());
@@ -154,7 +154,7 @@ 
 	 * @param LifecycleEventArgs $args
 	 */
 	public function postUpdate(LifecycleEventArgs $args): void {
-		if(!$this->enableIndexing) return;
+		if (!$this->enableIndexing) return;
 		$this->updateEntity($args->getObject(), $args->getObjectManager());
 	}
 
@@ -179,12 +179,12 @@ 
 	 * @param object $entity
 	 */
 	protected function updateEntity(object $entity, ObjectManager $manager): void {
-		if($entity instanceof SearchableAlias) {
+		if ($entity instanceof SearchableAlias) {
 			$entity = $entity->getEntityToIndex();
 		}
 		$document = $this->getEntityToDocumentMapper()->createDocument($manager, $entity);
-		if($document !== null) {
-			if($this->getEventDispatcher()) {
+		if ($document !== null) {
+			if ($this->getEventDispatcher()) {
 				$event = new DocumentPreSaveEvent($document);
 				$this->getEventDispatcher()->dispatch($event, DocumentPreSaveEvent::NAME);
 			}
@@ -198,11 +198,11 @@ 
 	 * @param object $entity
 	 */
 	protected function removeEntity(object $entity, ObjectManager $manager): void {
-		if($entity instanceof SearchableAlias) {
+		if ($entity instanceof SearchableAlias) {
 			$entity = $entity->getEntityToIndex();
 		}
 		$document = $this->getEntityToDocumentMapper()->createDocument($manager, $entity);
-		if($document !== null) {
+		if ($document !== null) {
 			$this->getSearchService()->removeDocument($document);
 			$this->needsFlush = true;
 		}

Services/Mapping/DocumentToEntityMapper.php

Spacing +3 added lines, -3 removed lines

@@ -36,14 +36,14 @@
 	public function getEntity(Document $document): ?object {
 		$clazz = $document->getEntityClass();
 
-		if($clazz === null) {
+		if ($clazz === null) {
 			return null;
 		}
-		if(!$document->getEntityId()) {
+		if (!$document->getEntityId()) {
 			return null;
 		}
 		$manager = $this->managerRegistry->getManagerForClass($clazz);
-		if($manager === null) {
+		if ($manager === null) {
 			return null;
 		}
 		return $manager->getRepository($clazz)->find($document->getEntityId());

Services/AbstractSearchService.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 	 * @required
 	 */
 	public function setObjectManager(EntityManagerInterface $om): void {
-		if($this->objectManager)
+		if ($this->objectManager)
 			return;
 		$this->objectManager = $om;
 	}

Braces +3 added lines, -2 removed lines

@@ -53,8 +53,9 @@
 	 * @required
 	 */
 	public function setObjectManager(EntityManagerInterface $om): void {
-		if($this->objectManager)
-			return;
+		if($this->objectManager) {
+					return;
+		}
 		$this->objectManager = $om;
 	}
 

StingerSoftEntitySearchBundle.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
 	public static function getRequiredBundles(string $env, array &$requiredBundles = []): array {
 
-		if(isset($requiredBundles['StingerSoftEntitySearchBundle'])) {
+		if (isset($requiredBundles['StingerSoftEntitySearchBundle'])) {
 			return $requiredBundles;
 		}
 

Form/QueryType.php

Spacing +15 added lines, -15 removed lines

@@ -55,25 +55,25 @@ 
 		$preferredFilterChoices = $options['preferred_filter_choices'];
 		$maxChoiceGroupCount = (int)$options['max_choice_group_count'];
 
-		if($usedFacets && !$result) {
+		if ($usedFacets && !$result) {
 			$data = [];
-			foreach($usedFacets as $facetType => $facetTypeOptions) {
+			foreach ($usedFacets as $facetType => $facetTypeOptions) {
 				$facetTypeOptions = is_array($facetTypeOptions) ? $facetTypeOptions : [];
 				$preferredChoices = $preferredFilterChoices[$facetType] ?? [];
 				$i = 0;
-				$builder->add('facet_' . $facetType, FacetType::class, array_merge(array(
-					'label'             => 'stinger_soft_entity_search.forms.query.' . $facetType . '.label',
+				$builder->add('facet_'.$facetType, FacetType::class, array_merge(array(
+					'label'             => 'stinger_soft_entity_search.forms.query.'.$facetType.'.label',
 					'multiple'          => true,
 					'expanded'          => true,
 					'preferred_choices' => function($val) use ($preferredChoices, $data, $facetType, $maxChoiceGroupCount, &$i) {
-						$facetKey = 'facet_' . $facetType;
+						$facetKey = 'facet_'.$facetType;
 						return $i++ < $maxChoiceGroupCount || $maxChoiceGroupCount === 0 || \in_array($val, $preferredChoices) || (isset($data[$facetKey]) && \in_array($val, $data[$facetKey]));
 					}
 				), $facetTypeOptions));
 				unset($i);
 			}
 		}
-		if($result) {
+		if ($result) {
 			$builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) use ($options, $result) {
 				$this->createFacets($event->getForm(), $result->getFacets(), $options, $event->getData());
 			});
@@ -137,14 +137,14 @@ 
 		$selectedFacets = $data->getFacets();
 		$usedFacets = $options['used_facets'];
 
-		foreach($facets->getFacets() as $facetType => $facetValues) {
+		foreach ($facets->getFacets() as $facetType => $facetValues) {
 			$preferredChoices = $preferredFilterChoices[$facetType] ?? [];
 
 			$i = 0;
 			$facetTypeOptions = $usedFacets[$facetType] ?? [];
 			$formatter = $options['facet_formatter'][$facetType] ?? null;
-			$builder->add('facet_' . $facetType, FacetType::class, array_merge(array(
-				'label'             => 'stinger_soft_entity_search.forms.query.' . $facetType . '.label',
+			$builder->add('facet_'.$facetType, FacetType::class, array_merge(array(
+				'label'             => 'stinger_soft_entity_search.forms.query.'.$facetType.'.label',
 				'multiple'          => true,
 				'expanded'          => true,
 				'choices'           => $this->generateFacetChoices($facetType, $facetValues, $selectedFacets[$facetType] ?? [], $formatter),
@@ -167,17 +167,17 @@ 
 	protected function generateFacetChoices($facetType, array $facets, array $selectedFacets = [], callable $formatter = null): array {
 		$choices = [];
 		$handledFacets = [];
-		foreach($facets as $facet => $data) {
+		foreach ($facets as $facet => $data) {
 			$value = $data['value'];
 			$count = $data['count'];
-			if($count === 0 && !\in_array($facet, $selectedFacets)) {
+			if ($count === 0 && !\in_array($facet, $selectedFacets)) {
 				continue;
 			}
 			$handledFacets[$facet] = true;
 			$choices[$this->formatFacet($formatter, $facetType, $facet, $value, $count)] = $facet;
 		}
-		foreach($selectedFacets as $facet) {
-			if(!isset($facets[$facet]) || isset($handledFacets[$facet])) {
+		foreach ($selectedFacets as $facet) {
+			if (!isset($facets[$facet]) || isset($handledFacets[$facet])) {
 				continue;
 			}
 			$value = $facets[$facet]['value'];
@@ -196,8 +196,8 @@ 
 	 * @return string
 	 */
 	protected function formatFacet($formatter, $facetType, $facet, $value, $count):string {
-		$default = $facet . ' (' . $count . ')';
-		if(!$formatter) {
+		$default = $facet.' ('.$count.')';
+		if (!$formatter) {
 			return $default;
 		}
 		return $formatter($facetType, $value, $count, $default);