Completed
Push — master ( 3e8b02...37f80c )
by Florian
16:58
created
Model/Result/FacetSetAdapter.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -30,10 +30,10 @@  discard block
 block discarded – undo
30 30
 	}
31 31
 
32 32
 	public function addFacetValue($key, $value, $increaseCounterBy = 1) {
33
-		if(!isset($this->facets[$key])){
33
+		if (!isset($this->facets[$key])) {
34 34
 			$this->facets[$key] = array();
35 35
 		}
36
-		if(!isset($this->facets[$key][$value])){
36
+		if (!isset($this->facets[$key][$value])) {
37 37
 			$this->facets[$key][$value] = 0;
38 38
 		}
39 39
 		$this->facets[$key][$value] = $this->facets[$key][$value] + $increaseCounterBy;
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 	 * @see \StingerSoft\EntitySearchBundle\Model\Result\FacetSet::getFacet()
47 47
 	 */
48 48
 	public function getFacet($key) {
49
-		if(isset($this->facets[$key])) {
49
+		if (isset($this->facets[$key])) {
50 50
 			return $this->facets[$key];
51 51
 		} else {
52 52
 			return;
Please login to merge, or discard this patch.
Services/DummySearchService.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
 		$id = $document->getEntityClass();
30 30
 		$id .= '#';
31 31
 		$entityId = $document->getEntityId();
32
-		if(is_scalar($entityId)) {
32
+		if (is_scalar($entityId)) {
33 33
 			$id .= $entityId;
34 34
 		} else {
35 35
 			$id .= md5(serialize($entityId));
@@ -75,15 +75,15 @@  discard block
 block discarded – undo
75 75
 	 */
76 76
 	public function autocomplete($search, $maxResults = 10) {
77 77
 		$words = array();
78
-		foreach($this->index as $doc) {
79
-			foreach($doc->getFields() as $content) {
80
-				if(is_string($content)) {
78
+		foreach ($this->index as $doc) {
79
+			foreach ($doc->getFields() as $content) {
80
+				if (is_string($content)) {
81 81
 					$words = array_merge($words, explode(' ', $content));
82 82
 				}
83 83
 			}
84 84
 		}
85 85
 		
86
-		return array_filter($words, function ($word) use ($search) {
86
+		return array_filter($words, function($word) use ($search) {
87 87
 			return stripos($word, $search) === 0;
88 88
 		});
89 89
 	}
@@ -101,10 +101,10 @@  discard block
 block discarded – undo
101 101
 		$facets = new FacetSetAdapter();
102 102
 		
103 103
 		$hits = array();
104
-		foreach($this->index as $key => $doc) {
105
-			foreach($doc->getFields() as $content) {
106
-				if(is_string($content) && stripos($content, $term) !== false) {
107
-					if(!isset($hits[$key])) {
104
+		foreach ($this->index as $key => $doc) {
105
+			foreach ($doc->getFields() as $content) {
106
+				if (is_string($content) && stripos($content, $term) !== false) {
107
+					if (!isset($hits[$key])) {
108 108
 						$hits[$key] = 0;
109 109
 					}
110 110
 					$hits[$key] = $hits[$key] + 1;
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
 		}
114 114
 		arsort($hits);
115 115
 		$results = array();
116
-		foreach(array_keys($hits) as $docId) {
116
+		foreach (array_keys($hits) as $docId) {
117 117
 			$doc = $this->index[$docId];
118 118
 			$facets->addFacetValue(FacetSet::FACET_ENTITY_TYPE, $doc->getEntityClass());
119 119
 			$facets->addFacetValue(FacetSet::FACET_AUTHOR, $doc->getFieldValue(Document::FIELD_AUTHOR));
Please login to merge, or discard this patch.
Form/FacetChoiceLoader.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -39,9 +39,9 @@  discard block
 block discarded – undo
39 39
 	public function loadValuesForChoices(array $choices, $value = null) {
40 40
 		// is called on form creat with $choices containing the preset of the bound entity
41 41
 		$values = array();
42
-		foreach($choices as $key => $choice) {
42
+		foreach ($choices as $key => $choice) {
43 43
 			// we use a DataTransformer, thus only plain values arrive as choices which can be used directly as value
44
-			if(is_callable($value)) {
44
+			if (is_callable($value)) {
45 45
 				$values[$key] = (string)call_user_func($value, $choice, $key);
46 46
 			} else {
47 47
 				$values[$key] = $choice;
@@ -85,9 +85,9 @@  discard block
 block discarded – undo
85 85
 	public function loadChoicesForValues(array $values, $value = null) {
86 86
 		// is called on form submit after loadValuesForChoices of form create and loadChoiceList of form view create
87 87
 		$choices = array();
88
-		foreach($values as $key => $val) {
88
+		foreach ($values as $key => $val) {
89 89
 			// we use a DataTransformer, thus only plain values arrive as choices which can be used directly as value
90
-			if(is_callable($value)) {
90
+			if (is_callable($value)) {
91 91
 				$choices[$key] = (string)call_user_func($value, $val, $key);
92 92
 			} else {
93 93
 				$choices[$key] = $val;
Please login to merge, or discard this patch.
Tests/Form/QueryTypeTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -116,7 +116,7 @@
 block discarded – undo
116 116
 		$view = $form->createView();
117 117
 		$children = $view->children;
118 118
 		
119
-		foreach(array_keys($formData) as $key) {
119
+		foreach (array_keys($formData) as $key) {
120 120
 			$this->assertArrayHasKey($key, $children);
121 121
 		}
122 122
 		
Please login to merge, or discard this patch.
Tests/Model/Result/FacetSetAdapterTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
 
17 17
 class FacetSetAdapterTest extends \PHPUnit_Framework_TestCase {
18 18
 
19
-	public function testAddFacetValue(){
19
+	public function testAddFacetValue() {
20 20
 		$facets = new FacetSetAdapter(array());
21 21
 		$this->assertCount(0, $facets);
22 22
 		$facets->addFacetValue(Document::FIELD_AUTHOR, 'Oliver Kotte');
@@ -26,13 +26,13 @@  discard block
 block discarded – undo
26 26
 		$this->assertCount(1, $facets);
27 27
 		$this->assertCount(2, $facets->getFacet(Document::FIELD_AUTHOR));
28 28
 		
29
-		foreach($facets as $facetKey => $facetValues){
29
+		foreach ($facets as $facetKey => $facetValues) {
30 30
 			$this->assertEquals(Document::FIELD_AUTHOR, $facetKey);
31 31
 			$this->assertCount(2, $facetValues);
32 32
 		}
33 33
 	}
34 34
 	
35
-	public function testGetFacet(){
35
+	public function testGetFacet() {
36 36
 		$facets = new FacetSetAdapter(array());
37 37
 		$this->assertCount(0, $facets);
38 38
 		$this->assertNull($facets->getFacet('NotExisting'));
Please login to merge, or discard this patch.
Services/DoctrineListener.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -73,12 +73,12 @@  discard block
 block discarded – undo
73 73
 	 * @param LifecycleEventArgs $args        	
74 74
 	 */
75 75
 	public function postPersist(LifecycleEventArgs $args) {
76
-		if(!$this->enableIndexing) return;
76
+		if (!$this->enableIndexing) return;
77 77
 		$this->updateEntity($args->getObject(), $args->getObjectManager());
78 78
 	}
79 79
 
80 80
 	public function postFlush(PostFlushEventArgs $eventArgs) {
81
-		if($this->needsFlush) {
81
+		if ($this->needsFlush) {
82 82
 			$this->needsFlush = false;
83 83
 			$eventArgs->getEntityManager()->flush();
84 84
 		}
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	 * @param LifecycleEventArgs $args        	
91 91
 	 */
92 92
 	public function preRemove(LifecycleEventArgs $args) {
93
-		if(!$this->enableIndexing) return;
93
+		if (!$this->enableIndexing) return;
94 94
 		$this->removeEntity($args->getObject(), $args->getObjectManager());
95 95
 	}
96 96
 
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	 * @param LifecycleEventArgs $args        	
101 101
 	 */
102 102
 	public function postUpdate(LifecycleEventArgs $args) {
103
-		if(!$this->enableIndexing) return;
103
+		if (!$this->enableIndexing) return;
104 104
 		$this->updateEntity($args->getObject(), $args->getObjectManager());
105 105
 	}
106 106
 
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 	 */
128 128
 	protected function updateEntity($entity, ObjectManager $manager) {
129 129
 		$document = $this->getEntityToDocumentMapper()->createDocument($manager, $entity);
130
-		if($document !== false) {
130
+		if ($document !== false) {
131 131
 			$this->getSearchService($manager)->saveDocument($document);
132 132
 		}
133 133
 	}
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 	 */
139 139
 	protected function removeEntity($entity, ObjectManager $manager) {
140 140
 		$document = $this->getEntityToDocumentMapper()->createDocument($manager, $entity);
141
-		if($document !== false) {
141
+		if ($document !== false) {
142 142
 			$this->getSearchService($manager)->removeDocument($document);
143 143
 		}
144 144
 	}
Please login to merge, or discard this patch.
Services/AbstractSearchService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
 	 * @see \StingerSoft\EntitySearchBundle\Services\SearchService::setObjectManager()
32 32
 	 */
33 33
 	public function setObjectManager(ObjectManager $om) {
34
-		if($this->objectManager)
34
+		if ($this->objectManager)
35 35
 			return;
36 36
 		$this->objectManager = $om;
37 37
 	}
Please login to merge, or discard this patch.
Tests/AbstractORMTestCase.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
 		);
45 45
 		$config = null === $config ? $this->getMockAnnotatedConfig() : $config;
46 46
 		$em = EntityManager::create($conn, $config, $evm ?: $this->getEventManager());
47
-		$schema = array_map(function ($class) use ($em) {
47
+		$schema = array_map(function($class) use ($em) {
48 48
 			return $em->getClassMetadata($class);
49 49
 		}, (array)$this->getUsedEntityFixtures());
50 50
 		$schemaTool = new SchemaTool($em);
@@ -69,13 +69,13 @@  discard block
 block discarded – undo
69 69
 		$refl = new \ReflectionClass($configurationClass);
70 70
 		$methods = $refl->getMethods();
71 71
 		$mockMethods = array();
72
-		foreach($methods as $method) {
73
-			if($method->name !== 'addFilter' && $method->name !== 'getFilterClassName') {
72
+		foreach ($methods as $method) {
73
+			if ($method->name !== 'addFilter' && $method->name !== 'getFilterClassName') {
74 74
 				$mockMethods[] = $method->name;
75 75
 			}
76 76
 		}
77 77
 		$config = $this->getMockBuilder($configurationClass)->setMethods($mockMethods)->getMock();
78
-		$config->expects($this->once())->method('getProxyDir')->will($this->returnValue(__DIR__ . '/../../temp'));
78
+		$config->expects($this->once())->method('getProxyDir')->will($this->returnValue(__DIR__.'/../../temp'));
79 79
 		$config->expects($this->once())->method('getProxyNamespace')->will($this->returnValue('Proxy'));
80 80
 		$config->expects($this->any())->method('getDefaultQueryHints')->will($this->returnValue(array()));
81 81
 		$config->expects($this->once())->method('getAutoGenerateProxyClasses')->will($this->returnValue(true));
Please login to merge, or discard this patch.
Form/FacetType.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -31,12 +31,12 @@  discard block
 block discarded – undo
31 31
 		$builder->resetModelTransformers();
32 32
 		$builder->resetViewTransformers();
33 33
 		
34
-		if($options['multiple']) {
35
-			$builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event) {
34
+		if ($options['multiple']) {
35
+			$builder->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) {
36 36
 				$event->stopPropagation();
37 37
 			}, 1);
38 38
 		}
39
-		$builder->addEventListener(FormEvents::SUBMIT, function (FormEvent $event) {
39
+		$builder->addEventListener(FormEvents::SUBMIT, function(FormEvent $event) {
40 40
 			$data = $event->getForm()->getExtraData();
41 41
 			//$event->setData(array_unique(array_merge($data, $event->getData())));
42 42
 			$event->setData($data);
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 	public function configureOptions(OptionsResolver $resolver) {
63 63
 		$resolver->setDefault('translation_domain', 'StingerSoftEntitySearchBundle');
64 64
 		$resolver->setDefault('by_reference', true);
65
-		if(Kernel::VERSION_ID < 30000) {
65
+		if (Kernel::VERSION_ID < 30000) {
66 66
 			$resolver->setDefault('choices_as_values', true);
67 67
 		}
68 68
 	}
Please login to merge, or discard this patch.