Merge pull request 'use app href for redirection' (#245) from fix/issue244 into develop
Some checks failed
ci/woodpecker/push/woodpecker Pipeline failed

Reviewed-on: #245
This commit is contained in:
Simon Vieille 2023-04-14 21:11:29 +02:00
commit 88c14b6aec

View file

@ -78,29 +78,33 @@ class AppController extends Controller
continue;
}
return $this->redirectToApp($app['id']);
return $this->redirectToApp($app, true);
}
return $this->redirectToApp('files');
}
protected function redirectToApp($appId): RedirectResponse
protected function redirectToApp($app, bool $isHref = false): RedirectResponse
{
$isIgnoreFrontController = true === OC::$server->getConfig()->getSystemValue(
'htaccess.IgnoreFrontController',
false
);
if (!$isHref) {
$isIgnoreFrontController = true === OC::$server->getConfig()->getSystemValue(
'htaccess.IgnoreFrontController',
false
);
$isFrontControllerActive = 'true' === getenv('front_controller_active');
$isFrontControllerActive = 'true' === getenv('front_controller_active');
if ($isIgnoreFrontController || $isFrontControllerActive) {
$path = '/apps/%s/';
if ($isIgnoreFrontController || $isFrontControllerActive) {
$path = '/apps/%s/';
} else {
$path = '/index.php/apps/%s/';
}
$url = $this->urlGenerator->getAbsoluteURL(sprintf($path, $app));
} else {
$path = '/index.php/apps/%s/';
$url = $app['href'];
}
$url = $this->urlGenerator->getAbsoluteURL(sprintf($path, $appId));
return new RedirectResponse($url);
}
}