Hand in the route and the parameters of the request

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>

Author: nickvergessen

apps/comments/lib/Search/CommentsSearchProvider.php

@@ -77,7 +77,11 @@ public function getName(): string {
 	/**
 	 * @inheritDoc
 	 */
-	public function getOrder(string $from): int {
+	public function getOrder(string $route, array $routeParameters): int {
+		if ($route === 'files.View.index') {
+			// Files first
+			return 0;
+		}
 		return 10;
 	}
 

apps/dav/lib/Search/ContactsSearchProvider.php

@@ -96,8 +96,11 @@ public function getName(): string {
 	/**
 	 * @inheritDoc
 	 */
-	public function getOrder(string $from): int {
-		return 7;
+	public function getOrder(string $route, array $routeParameters): int {
+		if ($route === 'contacts.Page.index') {
+			return -1;
+		}
+		return 20;
 	}
 
 	/**

apps/dav/lib/Search/EventsSearchProvider.php

@@ -82,7 +82,10 @@ public function getName(): string {
 	/**
 	 * @inheritDoc
 	 */
-	public function getOrder(string $from): int {
+	public function getOrder(string $route, array $routeParameters): int {
+		if ($route === 'calendar.View.index') {
+			return -1;
+		}
 		return 10;
 	}
 

apps/dav/lib/Search/TasksSearchProvider.php

@@ -74,7 +74,10 @@ public function getName(): string {
 	/**
 	 * @inheritDoc
 	 */
-	public function getOrder(string $from): int {
+	public function getOrder(string $route, array $routeParameters): int {
+		if ($route === 'tasks.Page.index') {
+			return -1;
+		}
 		return 10;
 	}
 

apps/files/lib/Search/FilesSearchProvider.php

@@ -77,7 +77,11 @@ public function getName(): string {
 	/**
 	 * @inheritDoc
 	 */
-	public function getOrder(string $from): int {
+	public function getOrder(string $route, array $routeParameters): int {
+		if ($route === 'files.View.index') {
+			// Before comments
+			return -5;
+		}
 		return 5;
 	}
 

apps/settings/lib/Search/SectionSearch.php

@@ -75,9 +75,8 @@ public function getName(): string {
 	/**
 	 * @inheritDoc
 	 */
-	public function getOrder(string $from): int {
-		if (strpos($from, $this->urlGenerator->linkToRoute('settings.PersonalSettings.index') === 0)
-		|| strpos($from, $this->urlGenerator->linkToRoute('settings.AdminSettings.index')) === 0) {
+	public function getOrder(string $route, array $routeParameters): int {
+		if ($route === 'settings.PersonalSettings.index' || $route === 'settings.AdminSettings.index') {
 			return -1;
 		}
 		return 20;

core/Controller/UnifiedSearchController.php

@@ -32,7 +32,9 @@
 use OCP\AppFramework\Http\JSONResponse;
 use OCP\IRequest;
 use OCP\IUserSession;
+use OCP\Route\IRouter;
 use OCP\Search\ISearchQuery;
+use Symfony\Component\Routing\Exception\ResourceNotFoundException;
 
 class UnifiedSearchController extends Controller {
 
@@ -42,26 +44,33 @@ class UnifiedSearchController extends Controller {
 	/** @var IUserSession */
 	private $userSession;
 
+	/** @var IRouter */
+	private $router;
+
 	public function __construct(IRequest $request,
 								IUserSession $userSession,
-								SearchComposer $composer) {
+								SearchComposer $composer,
+								IRouter $router) {
 		parent::__construct('core', $request);
 
 		$this->composer = $composer;
 		$this->userSession = $userSession;
+		$this->router = $router;
 	}
 
 	/**
 	 * @NoAdminRequired
 	 * @NoCSRFRequired
-	 * 
+	 *
 	 * @param string $from the url the user is currently at
-	 * 
+	 *
 	 * @return JSONResponse
 	 */
-	public function getProviders(string $from): JSONResponse {
+	public function getProviders(string $from = ''): JSONResponse {
+		[$route, $parameters] = $this->getRouteInformation($from);
+
 		return new JSONResponse(
-			$this->composer->getProviders($from)
+			$this->composer->getProviders($route, $parameters)
 		);
 	}
 
@@ -74,17 +83,20 @@ public function getProviders(string $from): JSONResponse {
 	 * @param int|null $sortOrder
 	 * @param int|null $limit
 	 * @param int|string|null $cursor
+	 * @param string $from
 	 *
 	 * @return JSONResponse
 	 */
 	public function search(string $providerId,
 						   string $term = '',
 						   ?int $sortOrder = null,
 						   ?int $limit = null,
-						   $cursor = null): JSONResponse {
+						   $cursor = null,
+						   string $from = ''): JSONResponse {
 		if (empty(trim($term))) {
 			return new JSONResponse(null, Http::STATUS_BAD_REQUEST);
 		}
+		[$route, $routeParameters] = $this->getRouteInformation($from);
 
 		return new JSONResponse(
 			$this->composer->search(
@@ -94,9 +106,45 @@ public function search(string $providerId,
 					$term,
 					$sortOrder ?? ISearchQuery::SORT_DATE_DESC,
 					$limit ?? SearchQuery::LIMIT_DEFAULT,
-					$cursor
+					$cursor,
+					$route,
+					$routeParameters
 				)
 			)
 		);
 	}
+
+	protected function getRouteInformation(string $url): array {
+		$routeStr = '';
+		$parameters = [];
+
+		if ($url !== '') {
+			$urlParts = parse_url($url);
+
+			try {
+				$parameters = $this->router->findMatchingRoute($urlParts['path']);
+
+				// contacts.PageController.index => contacts.Page.index
+				$route = $parameters['caller'];
+				if (substr($route[1], -10) === 'Controller') {
+					$route[1] = substr($route[1], 0, -10);
+				}
+				$routeStr = implode('.', $route);
+
+				// cleanup
+				unset($parameters['_route'], $parameters['action'], $parameters['caller']);
+			} catch (ResourceNotFoundException $exception) {
+			}
+
+			if (isset($urlParts['query'])) {
+				parse_str($urlParts['query'], $queryParameters);
+				$parameters = array_merge($parameters, $queryParameters);
+			}
+		}
+
+		return [
+			$routeStr,
+			$parameters,
+		];
+	}
 }