Fix wrong unified search link to folder

[skjnldsv]

It now opens the folder directly
I think it's safer to use the shortened link as this is where we transform the final url. This is where we did all the changes in the past like

• Open the sidebar straight away or not
• Open the folder directly or not
• Open the file main action directly or not
• etc etc

[skjnldsv]

/backport to stable23

[skjnldsv]

/backport to stable22

[skjnldsv]

/backport to stable21

[ArtificialOwl]

might be better to use this new link for folders only ?

[skjnldsv]

might be better to use this new link for folders only ?

Well, it's the same, if we decide to change the url format, it means we'll also have to update this here too.
While if we keep using the /f/fileid shortcut, we make sure everything is at the same location

server/apps/files/lib/Controller/ViewController.php

Lines 397 to 429 in a325141

	private function redirectToFile($fileId, bool $setOpenfile = false) {
	$uid = $this->userSession->getUser()->getUID();
	$baseFolder = $this->rootFolder->getUserFolder($uid);
	$files = $baseFolder->getById($fileId);
	$params = [];
	if (empty($files) && $this->appManager->isEnabledForUser('files_trashbin')) {
	$baseFolder = $this->rootFolder->get($uid . '/files_trashbin/files/');
	$files = $baseFolder->getById($fileId);
	$params['view'] = 'trashbin';
	}
	if (!empty($files)) {
	$file = current($files);
	if ($file instanceof Folder) {
	// set the full path to enter the folder
	$params['dir'] = $baseFolder->getRelativePath($file->getPath());
	} else {
	// set parent path as dir
	$params['dir'] = $baseFolder->getRelativePath($file->getParent()->getPath());
	// and scroll to the entry
	$params['scrollto'] = $file->getName();
	if ($setOpenfile) {
	// forward the openfile URL parameter.
	$params['openfile'] = $fileId;
	}
	}
	return new RedirectResponse($this->urlGenerator->linkToRoute('files.view.index', $params));
	}
	throw new \OCP\Files\NotFoundException();
	}

[PVince81]

👍 makes sense