From c923aaceb01d2ac7f73a7de4b925b093970bf543 Mon Sep 17 00:00:00 2001 From: Pinga <121483313+getpinga@users.noreply.github.com> Date: Wed, 19 Feb 2025 14:11:44 +0200 Subject: [PATCH] Hotfix for password update --- cp/app/Controllers/RegistrarsController.php | 8 +++++--- cp/app/Controllers/UsersController.php | 8 +++++--- 2 files changed, 10 insertions(+), 6 deletions(-) diff --git a/cp/app/Controllers/RegistrarsController.php b/cp/app/Controllers/RegistrarsController.php index 8492101..6cf7496 100644 --- a/cp/app/Controllers/RegistrarsController.php +++ b/cp/app/Controllers/RegistrarsController.php @@ -657,9 +657,11 @@ class RegistrarsController extends Controller return $response->withHeader('Location', '/registrar/update/'.$registrar)->withStatus(302); } - if (!checkPasswordComplexity($data['panelPassword'])) { - $this->container->get('flash')->addMessage('error', 'Password too weak. Use a stronger password'); - return $response->withHeader('Location', '/registrar/update/'.$registrar)->withStatus(302); + if (isset($data['panelPassword']) && $data['panelPassword']) { + if (!checkPasswordComplexity($data['panelPassword'])) { + $this->container->get('flash')->addMessage('error', 'Password too weak. Use a stronger password'); + return $response->withHeader('Location', '/registrar/update/'.$registrar)->withStatus(302); + } } if (!empty($_SESSION['registrars_user_email'])) { diff --git a/cp/app/Controllers/UsersController.php b/cp/app/Controllers/UsersController.php index a01bbf9..f938607 100644 --- a/cp/app/Controllers/UsersController.php +++ b/cp/app/Controllers/UsersController.php @@ -312,9 +312,11 @@ class UsersController extends Controller return $response->withHeader('Location', '/user/update/'.$old_username)->withStatus(302); } - if (!checkPasswordComplexity($password)) { - $this->container->get('flash')->addMessage('error', 'Password too weak. Use a stronger password'); - return $response->withHeader('Location', '/user/update/'.$old_username)->withStatus(302); + if (!empty($password)) { + if (!checkPasswordComplexity($password)) { + $this->container->get('flash')->addMessage('error', 'Password too weak. Use a stronger password'); + return $response->withHeader('Location', '/user/update/'.$old_username)->withStatus(302); + } } // Check if username already exists (excluding the current user)