gordonbanderson / silverstripe-elastica

tests/ElasticSearchPageControllerTest.php

Spacing +9 added lines, -9 removed lines

@@ -243,7 +243,7 @@ 
 	public function testQueryInSearchBoxForOneFormOnly() {
 		$searchPageObj = $this->ElasticSearchPage2;
 		$url = rtrim($searchPageObj->Link(), '/');
-		$url .= "?q=Auckland&sfid=".$searchPageObj->Identifier;
+		$url .= "?q=Auckland&sfid=" . $searchPageObj->Identifier;
 		$response = $this->get($url);
 		$this->assertEquals(200, $response->getStatusCode());
 		$this->assertAttributeHasExactValue('#ElasticSearchForm_SearchForm_q', 'q',
@@ -378,7 +378,7 @@ 
 		$ctr = 0;
 
 		// results vary slightly due to sharding, hence check for a string instead of absolute results
-		while ($ctr < 18) {
+		while($ctr < 18) {
 			$this->assertSelectorContains('div.searchResult a', $ctr, 'New Orleans, Southern Pacific');
 			$ctr++;
 			$this->assertSelectorStartsWithOrEquals('div.searchResult a', $ctr, 'Similar');
@@ -397,13 +397,13 @@ 
 		$url = rtrim($searchPageObj->Link(), '/');
 		$url .= "?q=New%20Zealind&TestMode=true";
 		$response = $this->get($url);
-		error_log(print_r($response,1));
+		error_log(print_r($response, 1));
 		$this->assertEquals(200, $response->getStatusCode());
 
 		$ctr = 0;
 
 		// results vary slightly due to sharding, hence check for a string instead of absolute results
-		while ($ctr < 18) {
+		while($ctr < 18) {
 			$this->assertSelectorContains('div.searchResult a', $ctr, 'New');
 			$ctr++;
 			$this->assertSelectorStartsWithOrEquals('div.searchResult a', $ctr, 'Similar');
@@ -501,7 +501,7 @@ 
 		//Note pages need to be published, by default fixtures only reside in Stage
 		$searchPageObj = $this->ElasticSearchPage2;
 		$url = rtrim($searchPageObj->Link(), '/');
-		$url = $url.'?q='.$searchTerm;
+		$url = $url . '?q=' . $searchTerm;
 		$response = $this->get($url);
 		$this->assertEquals(200, $response->getStatusCode());
 
@@ -568,7 +568,7 @@ 
 		//Note pages need to be published, by default fixtures only reside in Stage
 		$searchPageObj = $this->ElasticSearchPage;
 		$url = rtrim($searchPageObj->Link(), '/');
-		$url = $url.'?q='.$searchTerm;
+		$url = $url . '?q=' . $searchTerm;
 		$response = $this->get($url);
 		$this->assertEquals(200, $response->getStatusCode());
 
@@ -699,18 +699,18 @@ 
 		//increase the number of results and assert that they are the same as per pages 1,2 joined
 		$searchPageObj->ResultsPerPage = 20;
 		$searchPageObj->write();
-		$searchPageObj->publish('Stage','Live');
+		$searchPageObj->publish('Stage', 'Live');
 		$response3 = $this->get($firstPageURL);
 	}
 
 
 	private function enableHighlights() {
-		foreach (SearchableField::get()->filter('Name', 'Title') as $sf) {
+		foreach(SearchableField::get()->filter('Name', 'Title') as $sf) {
 			$sf->ShowHighlights = true;
 			$sf->write();
 		}
 
-		foreach (SearchableField::get()->filter('Name', 'Content') as $sf) {
+		foreach(SearchableField::get()->filter('Name', 'Content') as $sf) {
 			$sf->ShowHighlights = true;
 			$sf->write();
 		}

tests/ElasticsearchFunctionalTestBase.php

Spacing +20 added lines, -20 removed lines

@@ -9,8 +9,8 @@ 
 	public static $ignoreFixtureFileFor = array();
 
 	protected $extraDataObjects = array(
-		'SearchableTestPage','FlickrPhotoTO','FlickrAuthorTO','FlickrSetTO','FlickrTagTO',
-		'SearchableTestFatherPage','SearchableTestGrandFatherPage'
+		'SearchableTestPage', 'FlickrPhotoTO', 'FlickrAuthorTO', 'FlickrSetTO', 'FlickrTagTO',
+		'SearchableTestFatherPage', 'SearchableTestGrandFatherPage'
 	);
 
 
@@ -27,7 +27,7 @@ 
 
 	public function setUp() {
 
-		error_log("*************** TEST: ".$this->getName());
+		error_log("*************** TEST: " . $this->getName());
 
 		$cache = SS_Cache::factory('elasticsearch');
 		$cache->clean(Zend_Cache::CLEANING_MODE_ALL);
@@ -35,7 +35,7 @@ 
 
 		// this needs to be called in order to create the list of searchable
 		// classes and fields that are available.  Simulates part of a build
-		$classes = array('SearchableTestPage','SiteTree','Page','FlickrPhotoTO','FlickrSetTO',
+		$classes = array('SearchableTestPage', 'SiteTree', 'Page', 'FlickrPhotoTO', 'FlickrSetTO',
 			'FlickrTagTO', 'FlickrAuthorTO');
 		$this->requireDefaultRecordsFrom = $classes;
 
@@ -54,7 +54,7 @@ 
 			$this->service->setTestMode(true);
 
 			// A previous test may have deleted the index and then failed, so check for this
-			if (!$this->service->getIndex()->exists()) {
+			if(!$this->service->getIndex()->exists()) {
 				$this->service->getIndex()->create();
 			}
 			$this->service->reset();
@@ -65,9 +65,9 @@ 
 			// load fixtures
 			$orig_fixture_file = static::$fixture_file;
 
-			foreach (static::$ignoreFixtureFileFor as $testPattern) {
-				$pattern = '/'.$testPattern.'/';
-				if (preg_match($pattern, $this->getName())) {
+			foreach(static::$ignoreFixtureFileFor as $testPattern) {
+				$pattern = '/' . $testPattern . '/';
+				if(preg_match($pattern, $this->getName())) {
 					static::$fixture_file = null;
 				}
 			}
@@ -79,7 +79,7 @@ 
 		// this has to be executed otherwise nesting exceptions occur
 		parent::setUp();
 
-		if ($elasticaException) {
+		if($elasticaException) {
 			$this->fail('T1 Exception with Elasticsearch');
 		}
 
@@ -101,18 +101,18 @@ 
 			$elasticaException = true;
 		}
 
-		if ($elasticaException) {
+		if($elasticaException) {
 			$this->fail('T2 Exception with Elasticsearch');
 		}
 	}
 
 
 	private function publishSiteTree() {
-		foreach (SiteTree::get()->getIterator() as $page) {
+		foreach(SiteTree::get()->getIterator() as $page) {
 			// temporarily disable Elasticsearch indexing, it will be done in a batch
 			$page->IndexingOff = true;
 
-			$page->publish('Stage','Live');
+			$page->publish('Stage', 'Live');
 		}
 	}
 
@@ -133,7 +133,7 @@ 
 		$items = $this->cssParser()->getBySelector($selector);
 
 		$ctr = 0;
-		foreach ($items as $item) {
+		foreach($items as $item) {
 			$text = strip_tags($item);
 			$escaped = str_replace("'", "\'", $text);
 			$ctr++;
@@ -163,7 +163,7 @@ 
 		$items = $this->cssParser()->getBySelector($selector);
 
 		$ctr = 0;
-		foreach ($items as $item) {
+		foreach($items as $item) {
 			$text = strip_tags($item);
 			$escaped = str_replace("'", "\'", $text);
 			$ctr++;
@@ -185,7 +185,7 @@ 
 	 */
 	public function assertAttributeHasExactValue($selector, $attributeName, $expectedValue) {
 		$items = $this->cssParser()->getBySelector($selector);
-		foreach ($items as $item) {
+		foreach($items as $item) {
 			$this->assertEquals($expectedValue, $item['value']);
 		}
 	}
@@ -194,9 +194,9 @@ 
 	public function assertAttributesHaveExactValues($selector, $expectedValues) {
 		$attributeNames = array_keys($expectedValues);
 		$items = $this->cssParser()->getBySelector($selector);
-		foreach ($items as $item) {
+		foreach($items as $item) {
 			$actualValues = array();
-			foreach ($attributeNames as $attributeName) {
+			foreach($attributeNames as $attributeName) {
 				$actualValues[$attributeName] = (string)$item[$attributeName];
 			}
 			$this->assertEquals($expectedValues, $actualValues);
@@ -206,7 +206,7 @@ 
 
 	public function assertNumberOfNodes($selector, $expectedAmount) {
 		$items = $this->cssParser()->getBySelector($selector);
-		foreach ($items as $item) {
+		foreach($items as $item) {
 			$text = strip_tags($item);
 		}
 
@@ -219,9 +219,9 @@ 
 	public function collateSearchResults() {
 		$items = $this->cssParser()->getBySelector('div.searchResults .searchResult');
 		$result = array();
-		foreach ($items as $item) {
+		foreach($items as $item) {
 			$attr = $item->attributes()->id;
-			array_push($result, $attr."");
+			array_push($result, $attr . "");
 		}
 
 		return $result;