From d3a977303ae7a530dc942b4f6b5e80a8ed13ff85 Mon Sep 17 00:00:00 2001
From: Marijus Kilmanas <marijus.kilmanas@visiongroup.io>
Date: Mon, 22 Jan 2024 13:09:36 +0200
Subject: [PATCH] Remove $container parameter from controller constructors

---
 psalm.xml.dist                                  | 5 +++++
 src/Controllers/AuthenticationController.php    | 4 ----
 src/Controllers/BackchannelLogoutController.php | 5 +----
 3 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/psalm.xml.dist b/psalm.xml.dist
index 6a528d4..0cf0f11 100644
--- a/psalm.xml.dist
+++ b/psalm.xml.dist
@@ -24,5 +24,10 @@
         <PossiblyNullArgument errorLevel="suppress" />
         <PossiblyNullPropertyAssignmentValue errorLevel="suppress" />
         <PossiblyNullReference errorLevel="suppress" />
+        <PropertyNotSetInConstructor>
+            <errorLevel type="suppress">
+                <referencedProperty name="Symfony\Bundle\FrameworkBundle\Controller\AbstractController::$container" />    <!-- Injected by Symfony DI via setContainer() because marked as #[Required] -->
+            </errorLevel>
+        </PropertyNotSetInConstructor>
     </issueHandlers>
 </psalm>
diff --git a/src/Controllers/AuthenticationController.php b/src/Controllers/AuthenticationController.php
index 5d27454..09f21ca 100644
--- a/src/Controllers/AuthenticationController.php
+++ b/src/Controllers/AuthenticationController.php
@@ -7,9 +7,7 @@
 use Auth0\SDK\Auth0;
 use Auth0\Symfony\Contracts\Controllers\AuthenticationControllerInterface;
 use Auth0\Symfony\Security\Authenticator;
-use Psr\Container\ContainerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
-use Symfony\Component\HttpFoundation\Exception\{BadRequestException, ConflictingHeadersException, SuspiciousOperationException};
 use Symfony\Component\HttpFoundation\{RedirectResponse, Request, Response};
 use Symfony\Component\Routing\RouterInterface;
 use Throwable;
@@ -22,9 +20,7 @@ final class AuthenticationController extends AbstractController implements Authe
     public function __construct(
         private Authenticator $authenticator,
         private RouterInterface $router,
-        ContainerInterface $container,
     ) {
-        $this->container = $container;
     }
 
     /**
diff --git a/src/Controllers/BackchannelLogoutController.php b/src/Controllers/BackchannelLogoutController.php
index 4e06f5c..eb56a7c 100644
--- a/src/Controllers/BackchannelLogoutController.php
+++ b/src/Controllers/BackchannelLogoutController.php
@@ -7,9 +7,8 @@
 use Auth0\SDK\Auth0;
 use Auth0\Symfony\Contracts\Controllers\AuthenticationControllerInterface;
 use Auth0\Symfony\Security\Authenticator;
-use Psr\Container\ContainerInterface;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
-use Symfony\Component\HttpFoundation\{RedirectResponse, Request, Response};
+use Symfony\Component\HttpFoundation\{Request, Response};
 use Throwable;
 
 use function is_string;
@@ -18,9 +17,7 @@ final class BackchannelLogoutController extends AbstractController implements Au
 {
     public function __construct(
         private Authenticator $authenticator,
-        ContainerInterface $container,
     ) {
-        $this->container = $container;
     }
 
     /**