Fixed: If password field is blank when saving a user

bymayo · Jul 11, 2024 · fc581cc · fc581cc
1 parent c52a1fd
commit fc581cc
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Porter Changelog
 
+## 5.0.2 - 2024-07-11
+### Fixed
+- If password field is blank when saving a user in the CP, skip password validation
+
 ## 5.0.1 - 2024-05-30
 ### Changed
 - Icon to a new shiny (literally) icon

diff --git a/composer.json b/composer.json
@@ -2,7 +2,7 @@
     "name": "bymayo/porter",
     "description": "A toolkit with lots of helpers for users and accounts",
     "type": "craft-plugin",
-    "version": "5.0.1",
+    "version": "5.0.2",
     "keywords": [
         "craft",
         "cms",

diff --git a/src/Porter.php b/src/Porter.php
@@ -191,7 +191,7 @@ function (ModelEvent $event) {
 
                 }
 
-                if ($this->settings->passwordForcePolicy && ($user->newPassword || strlen($user->newPassword) >= 0))
+                if ($this->settings->passwordForcePolicy && ($user->newPassword || strlen($user->newPassword) > 0))
                 {
 
                     $errors = $this->emailPassword->checkPasswordPolicy($user->newPassword);
-Original file line number
+Diff line change
@@ Expand Up / @@ -191,7 +191,7 @@ function (ModelEvent $event) { @@
                     }
-                    if ($this->settings->passwordForcePolicy && ($user->newPassword || strlen($user->newPassword) >= 0))
+                    if ($this->settings->passwordForcePolicy && ($user->newPassword || strlen($user->newPassword) > 0))
                     {
                         $errors = $this->emailPassword->checkPasswordPolicy($user->newPassword);
@@ Expand Down @@