nextcloud / fulltextsearch_elasticsearch

lib/Service/IndexMappingService.php

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
 			'index' =>
 				[
 					'index' => $this->configService->getElasticIndex(),
-					'id'    => $document->getProviderId() . ':' . $document->getId(),
+					'id'    => $document->getProviderId().':'.$document->getId(),
 					'type'  => 'standard',
 					'body'  => $this->generateIndexBody($document)
 				]
@@ -105,7 +105,7 @@ 
 			'index' =>
 				[
 					'index' => $this->configService->getElasticIndex(),
-					'id'    => $document->getProviderId() . ':' . $document->getId(),
+					'id'    => $document->getProviderId().':'.$document->getId(),
 					'type'  => 'standard',
 					'body'  => ['doc' => $this->generateIndexBody($document)]
 				]
@@ -133,7 +133,7 @@ 
 			'index' =>
 				[
 					'index' => $this->configService->getElasticIndex(),
-					'id'    => $providerId . ':' . $documentId,
+					'id'    => $providerId.':'.$documentId,
 					'type'  => 'standard'
 				]
 		];

lib/Service/SearchService.php

Doc Comments +1 added lines, -1 removed lines

@@ -189,7 +189,7 @@
 
 	/**
 	 * @param ISearchResult $searchResult
-	 * @param array $result
+	 * @param callable $result
 	 */
 	private function updateSearchResult(ISearchResult $searchResult, array $result) {
 		$searchResult->setRawResult(json_encode($result));

Spacing +5 added lines, -5 removed lines

@@ -85,7 +85,7 @@ 
 		Client $client, ISearchResult $searchResult, IDocumentAccess $access
 	) {
 		try {
-			$this->miscService->log('New Search Request; SearchResult Model: ' . json_encode($searchResult), 0);
+			$this->miscService->log('New Search Request; SearchResult Model: '.json_encode($searchResult), 0);
 			$query = $this->searchMappingService->generateSearchQuery(
 				$searchResult->getRequest(), $access, $searchResult->getProvider()
 																   ->getId()
@@ -95,25 +95,25 @@ 
 		}
 
 		try {
-			$this->miscService->log('Searching ES: ' . json_encode($query['params']), 0);
+			$this->miscService->log('Searching ES: '.json_encode($query['params']), 0);
 
 			$result = $client->search($query['params']);
 		} catch (Exception $e) {
 			$this->miscService->log(
-				'debug - request: ' . json_encode($searchResult->getRequest()) . '   - query: '
+				'debug - request: '.json_encode($searchResult->getRequest()).'   - query: '
 				. json_encode($query)
 			);
 			throw $e;
 		}
 
-		$this->miscService->log('Result from ES: ' . json_encode($result), 0);
+		$this->miscService->log('Result from ES: '.json_encode($result), 0);
 		$this->updateSearchResult($searchResult, $result);
 
 		foreach ($result['hits']['hits'] as $entry) {
 			$searchResult->addDocument($this->parseSearchEntry($entry, $access->getViewerId()));
 		}
 
-		$this->miscService->log('Filled SearchResult Model: ' . json_encode($searchResult), 0);
+		$this->miscService->log('Filled SearchResult Model: '.json_encode($searchResult), 0);
 	}
 
 

lib/Service/SearchMappingService.php

Spacing +3 added lines, -3 removed lines

@@ -279,7 +279,7 @@ 
 
 		foreach ($request->getWildcardFields() as $field) {
 			if (!$this->fieldIsOutLimit($request, $field)) {
-				$queryFields[] = ['wildcard' => [$field => '*' . $content->getWord() . '*']];
+				$queryFields[] = ['wildcard' => [$field => '*'.$content->getWord().'*']];
 			}
 		}
 
@@ -380,7 +380,7 @@ 
 		return [
 			'index' => $this->configService->getElasticIndex(),
 			'type'  => 'standard',
-			'id'    => $providerId . ':' . $documentId
+			'id'    => $providerId.':'.$documentId
 		];
 	}
 
@@ -393,7 +393,7 @@ 
 	private function getPartsFields(ISearchRequest $request) {
 		return array_map(
 			function($value) {
-				return 'parts.' . $value;
+				return 'parts.'.$value;
 			}, $request->getParts()
 		);
 	}

lib/Platform/ElasticSearchPlatform.php

Spacing +3 added lines, -3 removed lines

@@ -126,12 +126,12 @@
 		$hosts = $this->configService->getElasticHost();
 		foreach ($hosts as $host) {
 			$parsedHost = parse_url($host);
-			$safeHost = $parsedHost['scheme'] . '://';
+			$safeHost = $parsedHost['scheme'].'://';
 			if (array_key_exists('user', $parsedHost)) {
-				$safeHost .= $parsedHost['user'] . ':' . '********' . '@';
+				$safeHost .= $parsedHost['user'].':'.'********'.'@';
 			}
 			$safeHost .= $parsedHost['host'];
-			$safeHost .= ':' . $parsedHost['port'];
+			$safeHost .= ':'.$parsedHost['port'];
 
 			$sanitizedHosts[] = $safeHost;
 		}

lib/Service/IUserStoragesService.php

Indentation +5 added lines, -5 removed lines

@@ -30,9 +30,9 @@
 namespace OCA\FullTextSearch_ElasticSearch\Service;
 
 interface IUserStoragesService {
-    /**
-     * Returns an array of strings with all external mountpoints of the current user
-     * @return array
-     */
-    function getAllStoragesForUser();
+	/**
+	 * Returns an array of strings with all external mountpoints of the current user
+	 * @return array
+	 */
+	function getAllStoragesForUser();
 }
\ No newline at end of file

lib/Service/UserStoragesService.php

Indentation +11 added lines, -11 removed lines

@@ -36,19 +36,19 @@ 
  */
 class UserStoragesService implements IUserStoragesService {
     
-    /** @var UserGlobalStoragesService */
-    private $userGlobalStoragesService;
+	/** @var UserGlobalStoragesService */
+	private $userGlobalStoragesService;
 
-    public function __construct(UserGlobalStoragesService $userGlobalStoragesService) {
-        $this->userGlobalStoragesService = $userGlobalStoragesService;
-    }
+	public function __construct(UserGlobalStoragesService $userGlobalStoragesService) {
+		$this->userGlobalStoragesService = $userGlobalStoragesService;
+	}
 
    /**
-     * Returns an array of strings with all external mountpoints of the current user
-     * @return array
-     */
-    public function getAllStoragesForUser() {
-        $userStorages = $this->userGlobalStoragesService->getAllStoragesForUser();
+    * Returns an array of strings with all external mountpoints of the current user
+    * @return array
+    */
+	public function getAllStoragesForUser() {
+		$userStorages = $this->userGlobalStoragesService->getAllStoragesForUser();
 		$mountPoints = [];
 		foreach($userStorages as $userStorage){
 			/** @var $userStorage OCA\Files_External\Lib\StorageConfig */
@@ -62,5 +62,5 @@ 
 			$mountPoints[] = $mountPoint;
 		}
 		return $mountPoints;
-    }
+	}
 }
\ No newline at end of file

Spacing +3 added lines, -3 removed lines

@@ -50,13 +50,13 @@
     public function getAllStoragesForUser() {
         $userStorages = $this->userGlobalStoragesService->getAllStoragesForUser();
 		$mountPoints = [];
-		foreach($userStorages as $userStorage){
+		foreach ($userStorages as $userStorage) {
 			/** @var $userStorage OCA\Files_External\Lib\StorageConfig */
 			$mountPoint = $userStorage->getMountPoint();
-			if (substr($mountPoint, 0, 1) === '/'){
+			if (substr($mountPoint, 0, 1) === '/') {
 				$mountPoint = substr($mountPoint, 1, strlen($mountPoint) - 1);
 			}
-			if (substr($mountPoint, strlen($mountPoint) - 1, 1) !== '/'){
+			if (substr($mountPoint, strlen($mountPoint) - 1, 1) !== '/') {
 				$mountPoint .= '/';
 			}
 			$mountPoints[] = $mountPoint;

lib/AppInfo/Application.php

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
 use OCP\AppFramework\Bootstrap\IRegistrationContext;
 use Psr\Container\ContainerExceptionInterface;
 
-require_once __DIR__ . '/../../vendor/autoload.php';
+require_once __DIR__.'/../../vendor/autoload.php';
 
 
 /**
@@ -71,7 +71,7 @@ 
 		// Make SearchMappingService also work without external storage
 		// if app is inactive or not installed.
 		$context->registerService(SearchMappingService::class, function($c) {
-			try{
+			try {
 				$userStoragesService = $c->query(UserGlobalStoragesService::class);
 				return new SearchMappingService(
 					$c->query(ConfigService::class),

Braces +1 added lines, -2 removed lines

@@ -78,8 +78,7 @@
 					$c->query(MiscService::class),
 					new UserStoragesService($userStoragesService)
 				);
-			}
-			catch (ContainerExceptionInterface $e) {
+			} catch (ContainerExceptionInterface $e) {
 				return new SearchMappingService(
 					$c->query(ConfigService::class),
 					$c->query(MiscService::class)