nextcloud / fulltextsearch

lib/Model/PlatformWrapper.php

Doc Comments -1 removed lines

@@ -56,7 +56,6 @@
 	 * Provider constructor.
 	 *
 	 * @param string $appId
-	 * @param IFullTextSearchPlatform $platform
 	 */
 	public function __construct($appId, $class) {
 		$this->appId = $appId;

lib/Service/ConfigService.php

Doc Comments +1 added lines, -1 removed lines

@@ -105,8 +105,8 @@
 	/**
 	 * Get a version of an app
 	 *
-	 * @param string $key
 	 *
+	 * @param string $appId
 	 * @return string
 	 */
 	public function getAppVersion($appId) {

lib/Command/Check.php

Spacing +3 added lines, -3 removed lines

@@ -100,7 +100,7 @@ 
 		}
 
 		$output->writeln(
-			'Full text search ' . $this->configService->getAppValue('installed_version')
+			'Full text search '.$this->configService->getAppValue('installed_version')
 		);
 		$output->writeln(' ');
 
@@ -173,7 +173,7 @@ 
 
 		$output->writeln('- Search Platform:');
 
-		$output->writeln($platform->getName() . ' ' . $wrapper->getVersion());
+		$output->writeln($platform->getName().' '.$wrapper->getVersion());
 		echo json_encode($platform->getConfiguration(), JSON_PRETTY_PRINT);
 
 		$output->writeln(' ');
@@ -199,7 +199,7 @@ 
 
 		foreach ($providers as $providerWrapper) {
 			$provider = $providerWrapper->getProvider();
-			$output->writeln($provider->getName() . ' ' . $providerWrapper->getVersion());
+			$output->writeln($provider->getName().' '.$providerWrapper->getVersion());
 			echo json_encode($provider->getConfiguration(), JSON_PRETTY_PRINT);
 			$output->writeln('');
 		}

lib/Command/Test.php

Spacing +12 added lines, -12 removed lines

@@ -207,7 +207,7 @@ 
 			$output->write(' ', true);
 		}
 
-		$output->write($line . ' ', false);
+		$output->write($line.' ', false);
 	}
 
 
@@ -257,7 +257,7 @@ 
 		$this->output($output, 'Testing mocked provider: get indexable documents.');
 		$indexableDocuments =
 			$testProvider->generateIndexableDocuments(TestService::DOCUMENT_USER1);
-		$this->output($output, '(' . sizeof($indexableDocuments) . ' items)', false);
+		$this->output($output, '('.sizeof($indexableDocuments).' items)', false);
 		$this->output($output, true);
 	}
 
@@ -273,12 +273,12 @@ 
 		$wrapper = $this->platformService->getPlatform();
 		$testPlatform = $wrapper->getPlatform();
 
-		$this->output($output, '(' . $testPlatform->getName() . ')', false);
+		$this->output($output, '('.$testPlatform->getName().')', false);
 		$this->output($output, true);
 
 		$this->output($output, 'Testing search platform.');
 		if (!$testPlatform->testPlatform()) {
-			throw new Exception ('Search platform (' . $testPlatform->getName() . ') down ?');
+			throw new Exception('Search platform ('.$testPlatform->getName().') down ?');
 		}
 		$this->output($output, true);
 
@@ -374,13 +374,13 @@ 
 			);
 
 			$this->output(
-				$output, '(size: ' . $indexDocument->getContentSize() . ')', false
+				$output, '(size: '.$indexDocument->getContentSize().')', false
 			);
 			$this->output($output, true);
 		} catch (Exception $e) {
 			throw new Exception(
-				"Issue while getting test document '" . TestService::DOCUMENT_TYPE_LICENSE
-				. "' from search platform: " . $e->getMessage()
+				"Issue while getting test document '".TestService::DOCUMENT_TYPE_LICENSE
+				. "' from search platform: ".$e->getMessage()
 			);
 		}
 
@@ -554,7 +554,7 @@ 
 	) {
 		$this->output(
 			$output,
-			" - '" . $search . "'" . (($moreOutput === '') ? '' : ' - ' . $moreOutput . ' - ')
+			" - '".$search."'".(($moreOutput === '') ? '' : ' - '.$moreOutput.' - ')
 		);
 		$request = new SearchRequest();
 
@@ -568,7 +568,7 @@ 
 
 		$this->output(
 			$output,
-			'(result: ' . $searchResult->getCount() . ', expected: ' . json_encode($expected) . ')',
+			'(result: '.$searchResult->getCount().', expected: '.json_encode($expected).')',
 			false
 		);
 		$this->compareSearchResult($searchResult, $expected);
@@ -632,12 +632,12 @@ 
 	private function compareSearchResult(SearchResult $searchResult, $entries) {
 		$documents = $searchResult->getDocuments();
 		if (sizeof($documents) !== sizeof($entries)) {
-			throw new \Exception('Unexpected SearchResult: ' . json_encode($searchResult));
+			throw new \Exception('Unexpected SearchResult: '.json_encode($searchResult));
 		}
 
 		foreach ($documents as $document) {
 			if (!in_array($document->getId(), $entries)) {
-				throw new \Exception('Unexpected Document: ' . json_encode($document));
+				throw new \Exception('Unexpected Document: '.json_encode($document));
 			}
 		}
 	}
@@ -650,7 +650,7 @@ 
 	 * @throws InterruptException
 	 */
 	private function pause(OutputInterface $output, $s) {
-		$this->output($output, 'Pausing ' . $s . ' seconds');
+		$this->output($output, 'Pausing '.$s.' seconds');
 
 		for ($i = 1; $i <= $s; $i++) {
 			if (time_nanosleep(1, 0) !== true) {

lib/Service/PlatformService.php

Spacing +2 added lines, -2 removed lines

@@ -161,7 +161,7 @@ 
 		$platform = \OC::$server->query((string)$selected->getClass());
 		if (!($platform instanceof IFullTextSearchPlatform)) {
 			throw new PlatformIsNotCompatibleException(
-				$selected . ' is not a compatible FullTextSearchPlatform'
+				$selected.' is not a compatible FullTextSearchPlatform'
 			);
 		}
 
@@ -193,7 +193,7 @@ 
 		}
 
 		throw new PlatformDoesNotExistException(
-			'FullTextSearchPlatform ' . $selected . ' is not available'
+			'FullTextSearchPlatform '.$selected.' is not available'
 		);
 	}
 

lib/Service/ProviderService.php

Spacing +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
 		$provider = \OC::$server->query((string)$providerId);
 		if (!($provider instanceof IFullTextSearchProvider)) {
 			throw new ProviderIsNotCompatibleException(
-				$providerId . ' is not a compatible IFullTextSearchProvider'
+				$providerId.' is not a compatible IFullTextSearchProvider'
 			);
 		}
 
@@ -196,7 +196,7 @@ 
 			}
 		}
 
-		throw new ProviderDoesNotExistException('Provider \'' . $providerId . '\' does not exist');
+		throw new ProviderDoesNotExistException('Provider \''.$providerId.'\' does not exist');
 	}
 
 
@@ -284,7 +284,7 @@ 
 			$knownProvider = $providerWrapper->getProvider();
 			if ($knownProvider->getId() === $provider->getId()) {
 				throw new ProviderIsNotUniqueException(
-					'FullTextSearchProvider ' . $provider->getId() . ' already exist'
+					'FullTextSearchProvider '.$provider->getId().' already exist'
 				);
 			}
 		}