Fixed failing webhook test cases

[bookstack] / app / Http / Controllers / UserController.php

diff --git a/app/Http/Controllers/UserController.php b/app/Http/Controllers/UserController.php
index 414bfefeb3e3db5fab5ad825511f97b600ac8a4d..a78f921f21e2ea187b906be6e3b4783d77d70e23 100644 (file)
--- a/app/Http/Controllers/UserController.php
+++ b/app/Http/Controllers/UserController.php
@@ -13,6 +13,7 @@ use BookStack\Uploads\ImageRepo;
 use Exception;
 use Illuminate\Http\Request;
 use Illuminate\Support\Str;
 use Exception;
 use Illuminate\Http\Request;
 use Illuminate\Support\Str;

+use Illuminate\Validation\Rules\Password;

 use Illuminate\Validation\ValidationException;
 
 class UserController extends Controller
 use Illuminate\Validation\ValidationException;
 
 class UserController extends Controller


@@ -82,7 +83,7 @@ class UserController extends Controller
         $sendInvite = ($request->get('send_invite', 'false') === 'true');
 
         if ($authMethod === 'standard' && !$sendInvite) {
         $sendInvite = ($request->get('send_invite', 'false') === 'true');
 
         if ($authMethod === 'standard' && !$sendInvite) {

-            $validationRules['password'] = ['required', 'min:6'];
+            $validationRules['password'] = ['required', Password::default()];

             $validationRules['password-confirm'] = ['required', 'same:password'];
         } elseif ($authMethod === 'ldap' || $authMethod === 'saml2' || $authMethod === 'openid') {
             $validationRules['external_auth_id'] = ['required'];
             $validationRules['password-confirm'] = ['required', 'same:password'];
         } elseif ($authMethod === 'ldap' || $authMethod === 'saml2' || $authMethod === 'openid') {
             $validationRules['external_auth_id'] = ['required'];


@@ -155,11 +156,11 @@ class UserController extends Controller
         $this->checkPermissionOrCurrentUser('users-manage', $id);
 
         $this->validate($request, [
         $this->checkPermissionOrCurrentUser('users-manage', $id);
 
         $this->validate($request, [

-            'name'             => 'min:2',
+            'name'             => ['min:2'],

             'email'            => ['min:2', 'email', 'unique:users,email,' . $id],
             'email'            => ['min:2', 'email', 'unique:users,email,' . $id],

-            'password'         => ['min:6', 'required_with:password_confirm'],
+            'password'         => ['required_with:password_confirm', Password::default()],

             'password-confirm' => ['same:password', 'required_with:password'],
             'password-confirm' => ['same:password', 'required_with:password'],

-            'setting'          => 'array',
+            'setting'          => ['array'],

             'profile_image'    => array_merge(['nullable'], $this->getImageValidationRules()),
         ]);
 
             'profile_image'    => array_merge(['nullable'], $this->getImageValidationRules()),
         ]);