cleaning code

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>

lib/Listeners/ListenersCore.php

@@ -33,7 +33,6 @@ public function __construct(
 	) {
 	}
 
-
 	/**
 	 * @return bool
 	 */

lib/Provider/FilesProvider.php

@@ -41,7 +41,7 @@
  */
 class FilesProvider implements IFullTextSearchProvider {
 	public const FILES_PROVIDER_ID = 'files';
-	private IRunner $runner;
+	private ?IRunner $runner = null;
 	private IIndexOptions $indexOptions;
 
 	public function __construct(
@@ -54,23 +54,20 @@ public function __construct(
 	) {
 	}
 
-
 	/**
 	 * return unique id of the provider
 	 */
 	public function getId(): string {
 		return self::FILES_PROVIDER_ID;
 	}
 
-
 	/**
 	 * return name of the provider
 	 */
 	public function getName(): string {
 		return 'Files';
 	}
 
-
 	/**
 	 * @return array
 	 */
@@ -83,7 +80,6 @@ public function getConfiguration(): array {
 		return $config;
 	}
 
-
 	/**
 	 * @param IRunner $runner
 	 */
@@ -92,15 +88,13 @@ public function setRunner(IRunner $runner) {
 		$this->filesService->setRunner($runner);
 	}
 
-
 	/**
 	 * @param IIndexOptions $options
 	 */
 	public function setIndexOptions(IIndexOptions $options) {
 		$this->indexOptions = $options;
 	}
 
-
 	/**
 	 * @return ISearchTemplate
 	 */
@@ -291,8 +285,8 @@ public function unloadProvider() {
 	 *
 	 * @param ISearchRequest $request
 	 */
-	public function improveSearchRequest(ISearchRequest $request) {
-		$this->searchService->improveSearchRequest($request);
+	public function improveSearchRequest(ISearchRequest $searchRequest) {
+		$this->searchService->improveSearchRequest($searchRequest);
 	}
 
 

lib/Service/ExternalFilesService.php

@@ -48,7 +48,6 @@ public function __construct(
 	) {
 	}
 
-
 	/**
 	 * @param string $userId
 	 */
@@ -135,8 +134,6 @@ public function updateDocumentAccess(FilesDocument &$document, Node $file) {
 		$document->getIndex()
 				 ->addOptionInt('external_mount_id', $mount->getId());
 		$document->setAccess($access);
-
-		$document->setAccess($access);
 	}
 
 
@@ -263,8 +260,6 @@ private function getRandomUserFromMountPoint(MountPoint $mount): string {
 			}
 		}
 
-		throw new ExternalMountWithNoViewerException(
-			'cannot get a valid user for external mount'
-		);
+		throw new ExternalMountWithNoViewerException('cannot get a valid user for external mount');
 	}
 }

lib/Service/FilesService.php

@@ -736,7 +736,7 @@ private function updateShareNames(FilesDocument $document, Node $file): array {
 				$shareNames[$this->secureUsername($username)] =
 					(!is_string($path)) ? $path = '' : $path;
 			} catch (Throwable $e) {
-				$this->logger->warning('Issue while getting information on documentId:' . $document->getId(), ['exception' => $e]);
+				$this->logger->debug('Issue while getting information on documentId:' . $document->getId(), ['exception' => $e]);
 			}
 		}
 
@@ -1143,6 +1143,8 @@ private function manageContentErrorException(IIndexDocument $document, Throwable
 			$t->getMessage(),
 			IIndex::ERROR_SEV_3
 		);
+
+		$this->logger->debug('content error', ['exception' => $t]);
 	}