Merge pull request #18955 from kamil-tekiela/Typo-in-accountLocking

MauricioFauth · web-flow · commit 559226d09d4b · 2024-01-31T16:17:34.000-03:00
Fix wrong param name in Account locking controllers Fixes #18954
diff --git a/src/Controllers/Server/Privileges/AccountLockController.php b/src/Controllers/Server/Privileges/AccountLockController.php
@@ -17,7 +17,7 @@
 
 final class AccountLockController extends AbstractController
 {
-    public function __construct(ResponseRenderer $response, Template $template, private AccountLocking $model)
+    public function __construct(ResponseRenderer $response, Template $template, private AccountLocking $accountLocking)
     {
         parent::__construct($response, $template);
     }
@@ -34,7 +34,7 @@ public function __invoke(ServerRequest $request): void
         $hostName = $request->getParsedBodyParam('hostname');
 
         try {
-            $this->model->lock($userName, $hostName);
+            $this->accountLocking->lock($userName, $hostName);
         } catch (Throwable $exception) {
             $this->response->setStatusCode(StatusCodeInterface::STATUS_BAD_REQUEST);
             $this->response->setRequestStatus(false);
diff --git a/src/Controllers/Server/Privileges/AccountUnlockController.php b/src/Controllers/Server/Privileges/AccountUnlockController.php
@@ -17,7 +17,7 @@
 
 final class AccountUnlockController extends AbstractController
 {
-    public function __construct(ResponseRenderer $response, Template $template, private AccountLocking $model)
+    public function __construct(ResponseRenderer $response, Template $template, private AccountLocking $accountLocking)
     {
         parent::__construct($response, $template);
     }
@@ -34,7 +34,7 @@ public function __invoke(ServerRequest $request): void
         $hostName = $request->getParsedBodyParam('hostname');
 
         try {
-            $this->model->unlock($userName, $hostName);
+            $this->accountLocking->unlock($userName, $hostName);
         } catch (Throwable $exception) {
             $this->response->setStatusCode(StatusCodeInterface::STATUS_BAD_REQUEST);
             $this->response->setRequestStatus(false);

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@`
`17`	`17`
`18`	`18`	`final class AccountLockController extends AbstractController`
`19`	`19`	`{`
`20`		`- public function __construct(ResponseRenderer $response, Template $template, private AccountLocking $model)`
	`20`	`+ public function __construct(ResponseRenderer $response, Template $template, private AccountLocking $accountLocking)`
`21`	`21`	`{`
`22`	`22`	`parent::__construct($response, $template);`
`23`	`23`	`}`
`@@ -34,7 +34,7 @@ public function __invoke(ServerRequest $request): void`
`34`	`34`	`$hostName = $request->getParsedBodyParam('hostname');`
`35`	`35`
`36`	`36`	`try {`
`37`		`- $this->model->lock($userName, $hostName);`
	`37`	`+ $this->accountLocking->lock($userName, $hostName);`
`38`	`38`	`} catch (Throwable $exception) {`
`39`	`39`	`$this->response->setStatusCode(StatusCodeInterface::STATUS_BAD_REQUEST);`
`40`	`40`	`$this->response->setRequestStatus(false);`