nextcloud / fulltextsearch

lib/Service/ProviderService.php

Doc Comments +3 added lines

@@ -214,6 +214,9 @@
 	}
 
 
+	/**
+	 * @param boolean $boolean
+	 */
 	public function setProviderAsIndexed(IFullTextSearchProvider $provider, $boolean) {
 		$this->configService->setProviderOptions(
 			$provider->getId(), ConfigService::PROVIDER_INDEXED, (($boolean) ? '1' : '0')

Spacing +3 added lines, -3 removed lines

@@ -106,7 +106,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'
 			);
 		}
 
@@ -187,7 +187,7 @@ 
 			}
 		}
 
-		throw new ProviderDoesNotExistException('Provider \'' . $providerId . '\' does not exist');
+		throw new ProviderDoesNotExistException('Provider \''.$providerId.'\' does not exist');
 	}
 
 
@@ -272,7 +272,7 @@ 
 		foreach ($this->providers AS $knownProvider) {
 			if ($knownProvider->getId() === $provider->getId()) {
 				throw new ProviderIsNotUniqueException(
-					'FullTextSearchProvider ' . $provider->getId() . ' already exist'
+					'FullTextSearchProvider '.$provider->getId().' already exist'
 				);
 			}
 		}

lib/Command/Search.php

Spacing +3 added lines, -3 removed lines

@@ -105,13 +105,13 @@
 
 	private function displaySearchResult(SearchResult $searchResult) {
 
-		echo '> ' . $searchResult->getProvider()
-								 ->getName() . "\n";
+		echo '> '.$searchResult->getProvider()
+								 ->getName()."\n";
 
 		/** @var IndexDocument[] $result */
 		$result = $searchResult->getDocuments();
 		foreach ($result as $document) {
-			echo ' - ' . $document->getId() . ' score:' . $document->getScore() . "\n";
+			echo ' - '.$document->getId().' score:'.$document->getScore()."\n";
 		}
 	}
 

lib/Service/RunningService.php

Spacing +5 added lines, -5 removed lines

@@ -224,18 +224,18 @@
 		$preAction = $tick->getAction();
 
 		if ($preAction !== '') {
-			$preActionTotal = $tick->getInfo($preAction . 'Total', 0);
-			$preActionStart = $tick->getInfo($preAction . 'Init', 0);
+			$preActionTotal = $tick->getInfo($preAction.'Total', 0);
+			$preActionStart = $tick->getInfo($preAction.'Init', 0);
 
 			if ($preActionStart > 0) {
 
 				$preActionTotal += ($now - $preActionStart);
-				$tick->setInfo($preAction . 'Total', $preActionTotal);
-				$tick->unsetInfo($preAction . 'Init');
+				$tick->setInfo($preAction.'Total', $preActionTotal);
+				$tick->unsetInfo($preAction.'Init');
 			}
 		}
 		$tick->setAction($action)
-			 ->setInfo($action . 'Init', $now);
+			 ->setInfo($action.'Init', $now);
 	}
 
 

lib/Service/SearchService.php

Spacing +1 added lines, -1 removed lines

@@ -184,7 +184,7 @@
 			try {
 				$rights->setCircles(Circles::joinedCircleIds($user->getUID()));
 			} catch (Exception $e) {
-				$this->miscService->log('Circles is set as enabled but: ' . $e->getMessage());
+				$this->miscService->log('Circles is set as enabled but: '.$e->getMessage());
 			}
 		}
 

lib/Service/PlatformService.php

Spacing +2 added lines, -2 removed lines

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

lib/Model/Index.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
 	/**
 	 * @param int $status
 	 *
-	 * @return bool
+	 * @return integer
 	 */
 	public function isStatus($status) {
 		return ((int)$status & $this->getStatus());

lib/Command/Check.php

Spacing +3 added lines, -3 removed lines

@@ -103,7 +103,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() . ' ' . $platform->getVersion());
+		$output->writeln($platform->getName().' '.$platform->getVersion());
 		echo json_encode($platform->getConfiguration(), JSON_PRETTY_PRINT);
 
 		$output->writeln(' ');
@@ -198,7 +198,7 @@ 
 		$output->writeln('- Content Providers:');
 
 		foreach ($providers as $provider) {
-			$output->writeln($provider->getName() . ' ' . $provider->getVersion());
+			$output->writeln($provider->getName().' '.$provider->getVersion());
 			echo json_encode($provider->getConfiguration(), JSON_PRETTY_PRINT);
 			$output->writeln('');
 		}

lib/Model/SearchRequest.php

Doc Comments +2 added lines, -2 removed lines

@@ -212,8 +212,8 @@
 	}
 
 	/**
-	 * @param $key
-	 * @param $value
+	 * @param string $key
+	 * @param string $value
 	 *
 	 * @return $this
 	 */

Spacing +2 added lines, -2 removed lines

@@ -169,7 +169,7 @@ 
 
 		$options = ['is', 'show'];
 		if (in_array($kw, $options)) {
-			$this->addOption($kw . '_' . $value, '1');
+			$this->addOption($kw.'_'.$value, '1');
 
 			return true;
 		}
@@ -422,7 +422,7 @@ 
 		foreach ($ak as $source) {
 			$tags = $this->subTags[$source];
 			foreach ($tags as $tag) {
-				$subTags[] = $source . '_' . $tag;
+				$subTags[] = $source.'_'.$tag;
 			}
 		}
 

lib/Command/Test.php

Doc Comments +1 added lines, -1 removed lines

@@ -210,7 +210,7 @@
 
 	/**
 	 * @param string|bool $line
-	 * @param $isNewLine
+	 * @param boolean $isNewLine
 	 *
 	 * @return string
 	 */

Spacing +12 added lines, -12 removed lines

@@ -204,7 +204,7 @@ 
 			$output->write(' ', true);
 		}
 
-		$output->write($line . ' ', false);
+		$output->write($line.' ', false);
 	}
 
 
@@ -254,7 +254,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);
 	}
 
@@ -268,12 +268,12 @@ 
 	private function testLoadingPlatform($output) {
 		$this->output($output, 'Loading search platform.');
 		$testPlatform = $this->platformService->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);
 
@@ -369,13 +369,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()
 			);
 		}
 
@@ -549,7 +549,7 @@ 
 	) {
 		$this->output(
 			$output,
-			" - '" . $search . "'" . (($moreOutput === '') ? '' : ' - ' . $moreOutput . ' - ')
+			" - '".$search."'".(($moreOutput === '') ? '' : ' - '.$moreOutput.' - ')
 		);
 		$request = new SearchRequest();
 
@@ -563,7 +563,7 @@ 
 
 		$this->output(
 			$output,
-			'(result: ' . $searchResult->getCount() . ', expected: ' . json_encode($expected) . ')',
+			'(result: '.$searchResult->getCount().', expected: '.json_encode($expected).')',
 			false
 		);
 		$this->compareSearchResult($searchResult, $expected);
@@ -627,12 +627,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));
 			}
 		}
 	}
@@ -645,7 +645,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) {