Stinger-Soft / EntitySearchBundle

Tests/Fixtures/ORM/Car.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 	 *
 	 * @see \StingerSoft\EntitySearchBundle\Model\SearchableEntity::indexEntity()
 	 */
-	public function indexEntity(Document &$document) {
+	public function indexEntity(Document&$document) {
 		$document->addField(Document::FIELD_TITLE, $this->getTitle());
 		return self::$index;
 	}

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

Command/SyncCommand.php

Spacing +14 added lines, -14 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,12 +91,12 @@ 
 			/**
 			 * @var ClassMetadata $m
 			 */
-			foreach($meta as $m) {
+			foreach ($meta as $m) {
 				
-				if($m->getReflectionClass()->isAbstract() || $m->getReflectionClass()->isInterface()) {
+				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());
@@ -117,7 +117,7 @@ 
 		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;
 		}
@@ -128,7 +128,7 @@ 
 		$entityCount = $countQueryBuilder->getQuery()->getSingleScalarResult();
 		
 		$iterableResult = $queryBuilder->getQuery()->iterate();
-		if($entityCount == 0) {
+		if ($entityCount == 0) {
 			$output->writeln('<comment>No entities found for indexing</comment>');
 			return;
 		}
@@ -138,12 +138,12 @@ 
 		// Index each entity seperate
 		foreach ($iterableResult as $row) {
 			$entity = $row[0];
-			if($this->getEntityToDocumentMapper()->isIndexable($entity)) {
+			if ($this->getEntityToDocumentMapper()->isIndexable($entity)) {
 				$document = $this->getEntityToDocumentMapper()->createDocument($entityManager, $entity);
-				if($document === false) continue;
+				if ($document === false) continue;
 				$this->getSearchService($entityManager)->saveDocument($document);
 				$entitiesIndexed++;
-				if($entitiesIndexed % 50 == 0) {
+				if ($entitiesIndexed % 50 == 0) {
 					$entityManager->flush();
 				}
 			}
@@ -151,7 +151,7 @@ 
 		}
 		$entityManager->flush();
 		$entityManager->clear();
-		$output->writeln('<comment>Indexed ' . $entitiesIndexed . ' entities</comment>');
+		$output->writeln('<comment>Indexed '.$entitiesIndexed.' entities</comment>');
 	}
 
 	/**
@@ -159,7 +159,7 @@ 
 	 * @return EntityToDocumentMapperInterface
 	 */
 	protected function getEntityToDocumentMapper() {
-		if(!$this->entityToDocumentMapper) {
+		if (!$this->entityToDocumentMapper) {
 			$this->entityToDocumentMapper = $this->getContainer()->get(EntityToDocumentMapperInterface::SERVICE_ID);
 		}
 		return $this->entityToDocumentMapper;
@@ -170,7 +170,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);

Braces +3 added lines, -1 removed lines

@@ -140,7 +140,9 @@
 			$entity = $row[0];
 			if($this->getEntityToDocumentMapper()->isIndexable($entity)) {
 				$document = $this->getEntityToDocumentMapper()->createDocument($entityManager, $entity);
-				if($document === false) continue;
+				if($document === false) {
+					continue;
+				}
 				$this->getSearchService($entityManager)->saveDocument($document);
 				$entitiesIndexed++;
 				if($entitiesIndexed % 50 == 0) {

Tests/application.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 // application.php
-require __DIR__ . '/../vendor/autoload.php';
+require __DIR__.'/../vendor/autoload.php';
 
 use StingerSoft\EntitySearchBundle\Command\ClearIndexCommand;
 use StingerSoft\EntitySearchBundle\Command\SyncCommand;

Tests/Command/SyncCommandTest.php

Spacing +2 added lines, -2 removed lines

@@ -43,8 +43,8 @@
 			'get' 
 		))->disableOriginalConstructor()->getMock();
 		
-		$container->method('get')->willReturnCallback(function ($serviceId) use ($that) {
-			switch($serviceId) {
+		$container->method('get')->willReturnCallback(function($serviceId) use ($that) {
+			switch ($serviceId) {
 				case 'kernel':
 					return $that;
 				case 'doctrine.orm.entity_manager':