add comment about improving when registerRedirect is called

julien-nc · julien-nc · commit 8856ee3f5dbe · 2024-01-19T15:00:29.000+01:00
Signed-off-by: Julien Veyssier &lt;julien-nc@posteo.net&gt;
diff --git a/lib/AppInfo/Application.php b/lib/AppInfo/Application.php
@@ -86,6 +86,10 @@ public function boot(IBootContext $context): void {
 	}
 
 	private function registerRedirect(IRequest $request, IURLGenerator $urlGenerator, SettingsService $settings, ProviderMapper $providerMapper): void {
+		// TODO when min supported version is >=28 :
+		// run this in a listener of OCP\AppFramework\Http\Events\BeforeLoginTemplateRenderedEvent
+		// to avoid doing useless stuff on data requests and template requests that are not the login page
+
 		$providers = $this->getCachedProviders($providerMapper);
 		$redirectUrl = $request->getParam('redirect_url');
 

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,10 @@ public function boot(IBootContext $context): void {`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`private function registerRedirect(IRequest $request, IURLGenerator $urlGenerator, SettingsService $settings, ProviderMapper $providerMapper): void {`
	`89`	`+ // TODO when min supported version is >=28 :`
	`90`	`+ // run this in a listener of OCP\AppFramework\Http\Events\BeforeLoginTemplateRenderedEvent`
	`91`	`+ // to avoid doing useless stuff on data requests and template requests that are not the login page`
	`92`	`+`
`89`	`93`	`$providers = $this->getCachedProviders($providerMapper);`
`90`	`94`	`$redirectUrl = $request->getParam('redirect_url');`
`91`	`95`