From 62f2e51608a6b59aff69ecbd9c9771aeb3d538c4 Mon Sep 17 00:00:00 2001
From: Mohamed Alsharaf <mohamed.alsharaf@silverstripe.com>
Date: Mon, 18 Mar 2024 16:23:29 +1300
Subject: [PATCH] Ensure not to use singleton for form

When using Injector to create a singleton constructor arguments are
ignored. This causes getForm to fail as SAMLLoginForm requires
constructor arguments.
---
 src/Control/SAMLController.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Control/SAMLController.php b/src/Control/SAMLController.php
index 8222f4f..62f4f39 100644
--- a/src/Control/SAMLController.php
+++ b/src/Control/SAMLController.php
@@ -348,6 +348,6 @@ public function getLogger()
      */
     public function getForm()
     {
-        return Injector::inst()->get(SAMLLoginForm::class, true, [$this, SAMLAuthenticator::class, 'LoginForm']);
+        return Injector::inst()->get(SAMLLoginForm::class, false, [$this, SAMLAuthenticator::class, 'LoginForm']);
     }
 }