Stinger-Soft / EntitySearchBundle

Services/AbstractSearchService.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	 * @see \StingerSoft\EntitySearchBundle\Services\SearchService::setObjectManager()
 	 */
 	public function setObjectManager(ObjectManager $om) {
-		if($this->objectManager)
+		if ($this->objectManager)
 			return;
 		$this->objectManager = $om;
 	}

Braces +3 added lines, -2 removed lines

@@ -31,8 +31,9 @@
 	 * @see \StingerSoft\EntitySearchBundle\Services\SearchService::setObjectManager()
 	 */
 	public function setObjectManager(ObjectManager $om) {
-		if($this->objectManager)
-			return;
+		if($this->objectManager) {
+					return;
+		}
 		$this->objectManager = $om;
 	}
 

Tests/Services/DoctrineListenerTest.php

Doc Comments +4 added lines, -4 removed lines

@@ -25,9 +25,9 @@ 
 
 	/**
 	 *
-	 * @param number $save
+	 * @param integer $save
 	 *        	Number of expected saves
-	 * @param number $delete
+	 * @param integer $delete
 	 *        	Number of expected deletions
 	 */
 	protected function registerDoctrineListener($save = 0, $delete = 0) {
@@ -50,9 +50,9 @@ 
 
 	/**
 	 *
-	 * @param number $save
+	 * @param integer $save
 	 *        	Number of expected saves
-	 * @param number $delete
+	 * @param integer $delete
 	 *        	Number of expected deletions
 	 */
 	protected function registerSearchService($save = 0, $delete = 0) {

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 boolean|Document Returns false if no document could be created
 	 */
 	public function getEntity(Document $document);

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) {

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;
 	}

Spacing +19 added lines, -19 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;
@@ -123,12 +123,12 @@ 
 	 * @return boolean
 	 */
 	protected function fillDocument(Document &$document, $object) {
-		if($object instanceof SearchableEntity) {
+		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);
 			}
 		}

Tests/AbstractORMTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -44,7 +44,7 @@ 
 		);
 		$config = null === $config ? $this->getMockAnnotatedConfig() : $config;
 		$em = EntityManager::create($conn, $config, $evm ?: $this->getEventManager());
-		$schema = array_map(function ($class) use ($em) {
+		$schema = array_map(function($class) use ($em) {
 			return $em->getClassMetadata($class);
 		}, (array)$this->getUsedEntityFixtures());
 		$schemaTool = new SchemaTool($em);
@@ -69,13 +69,13 @@ 
 		$refl = new \ReflectionClass($configurationClass);
 		$methods = $refl->getMethods();
 		$mockMethods = array();
-		foreach($methods as $method) {
-			if($method->name !== 'addFilter' && $method->name !== 'getFilterClassName') {
+		foreach ($methods as $method) {
+			if ($method->name !== 'addFilter' && $method->name !== 'getFilterClassName') {
 				$mockMethods[] = $method->name;
 			}
 		}
 		$config = $this->getMockBuilder($configurationClass)->setMethods($mockMethods)->getMock();
-		$config->expects($this->once())->method('getProxyDir')->will($this->returnValue(__DIR__ . '/../../temp'));
+		$config->expects($this->once())->method('getProxyDir')->will($this->returnValue(__DIR__.'/../../temp'));
 		$config->expects($this->once())->method('getProxyNamespace')->will($this->returnValue('Proxy'));
 		$config->expects($this->any())->method('getDefaultQueryHints')->will($this->returnValue(array()));
 		$config->expects($this->once())->method('getAutoGenerateProxyClasses')->will($this->returnValue(true));

Form/FacetType.php

Spacing +4 added lines, -4 removed lines

@@ -31,12 +31,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())));
 			$event->setData($data);
@@ -62,7 +62,7 @@ 
 	public function configureOptions(OptionsResolver $resolver) {
 		$resolver->setDefault('translation_domain', 'StingerSoftEntitySearchBundle');
 		$resolver->setDefault('by_reference', true);
-		if(Kernel::VERSION_ID < 30000) {
+		if (Kernel::VERSION_ID < 30000) {
 			$resolver->setDefault('choices_as_values', true);
 		}
 	}

Model/DocumentAdapter.php

Spacing +3 added lines, -3 removed lines

@@ -81,11 +81,11 @@ 
 	 * @see \StingerSoft\EntitySearchBundle\Model\Document::addMultiValueField()
 	 */
 	public function addMultiValueField($field, $value) {
-		if(!array_key_exists($field, $this->fields)) {
+		if (!array_key_exists($field, $this->fields)) {
 			$this->fields[$field] = array(
 				$value 
 			);
-		} else if(!in_array($value, $this->fields[$field])) {
+		} else if (!in_array($value, $this->fields[$field])) {
 			$this->fields[$field][] = $value;
 		}
 	}
@@ -98,7 +98,7 @@ 
 	 */
 	public function setEntityClass($clazz) {
 		$this->entityClass = $clazz;
-		if(!$this->entityType) {
+		if (!$this->entityType) {
 			$this->entityType = $clazz;
 		}
 	}

Services/Mapping/DocumentToEntityMapper.php

Spacing +2 added lines, -2 removed lines

@@ -34,10 +34,10 @@
 	public function getEntity(Document $document) {
 		$clazz = $document->getEntityClass();
 		
-		if($clazz == null) {
+		if ($clazz == null) {
 			return null;
 		}
-		if(!$document->getEntityId()) {
+		if (!$document->getEntityId()) {
 			return null;
 		}
 		return $this->managerRegistry->getManagerForClass($clazz)->getRepository($clazz)->find($document->getEntityId());

Model/Document.php

Doc Comments +7 added lines, -1 removed lines

@@ -92,6 +92,7 @@ 
 	 *        	The name of the field
 	 * @param mixed $value
 	 *        	The value of this field
+	 * @return void
 	 */
 	public function addField($fieldname, $value);
 
@@ -120,6 +121,7 @@ 
 	 *        	The name of the field
 	 * @param mixed $value
 	 *        	The additional value of this field
+	 * @return void
 	 */
 	public function addMultiValueField($field, $value);
 
@@ -131,6 +133,7 @@ 
 	 *
 	 * @param string $clazz
 	 *        	The classname
+	 * @return void
 	 */
 	public function setEntityClass($clazz);
 
@@ -150,6 +153,7 @@ 
 	 *
 	 * @param mixed $id
 	 *        	The ID of the corresponding entity
+	 * @return void
 	 */
 	public function setEntityId($id);
 
@@ -166,6 +170,7 @@ 
 	 * hiding some programatically needed complexity from the user
 	 *
 	 * @param string $type        	
+	 * @return void
 	 */
 	public function setEntityType($type);
 
@@ -176,7 +181,7 @@ 
 	 *
 	 * If no entity type is set, the class will be used instead
 	 *
-	 * @param return $type        	
+	 * @return string
 	 */
 	public function getEntityType();
 
@@ -186,6 +191,7 @@ 
 	 * <strong>note:</strong> This may not supported by the underlying implementation
 	 *
 	 * @param string $path        	
+	 * @return void
 	 */
 	public function setFile($path);