fix(psalm): ignore missing IAlternativeLoginProvider (@SInCE 34) and rename getAlternativeLogin to getAlternativeLogins

julien-nc · julien-nc · commit a53483295549 · 2026-04-15T12:22:31.000+02:00
Signed-off-by: Julien Veyssier &lt;julien-nc@posteo.net&gt;
diff --git a/lib/AlternativeLogin/AlternativeLoginProvider.php b/lib/AlternativeLogin/AlternativeLoginProvider.php
@@ -32,7 +32,7 @@ public function __construct(
 	) {
 	}
 
-	public function getAlternativeLogin(): array {
+	public function getAlternativeLogins(): array {
 		$alternativeLogins = [];
 		$redirectUrl = $this->request->getParam('redirect_url');
 		$absoluteRedirectUrl = !empty($redirectUrl) ? $this->urlGenerator->getAbsoluteURL($redirectUrl) : $redirectUrl;
diff --git a/psalm.xml b/psalm.xml
@@ -48,6 +48,7 @@
 				<referencedClass name="OCA\OIDCIdentityProvider\AppInfo\Application" />
 				<referencedClass name="OCP\Files\ISetupManager" />
 				<referencedClass name="OCP\User\Events\UserFirstTimeLoggedInEvent" />
+				<referencedClass name="OCP\Authentication\IAlternativeLoginProvider" />
 			</errorLevel>
 		</UndefinedClass>
 		<UndefinedDocblockClass>

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public function __construct(`
`32`	`32`	`) {`
`33`	`33`	`}`
`34`	`34`
`35`		`- public function getAlternativeLogin(): array {`
	`35`	`+ public function getAlternativeLogins(): array {`
`36`	`36`	`$alternativeLogins = [];`
`37`	`37`	`$redirectUrl = $this->request->getParam('redirect_url');`
`38`	`38`	`$absoluteRedirectUrl = !empty($redirectUrl) ? $this->urlGenerator->getAbsoluteURL($redirectUrl) : $redirectUrl;`