Update

fix #224
Blair2004 · Jul 31, 2021 · c0826f2 · c0826f2
1 parent f9465fa
commit c0826f2
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 3 deletions.
diff --git a/app/Http/Controllers/AuthController.php b/app/Http/Controllers/AuthController.php
@@ -204,6 +204,22 @@ public function postSignUp( SignUpRequest $request )
     {
         Hook::action( 'ns-register-form', $request );
 
+        /**
+         * check user existence
+         */
+        $user   =   User::where( 'email', $request->input( 'email' ) )->first();
+        if ( $user instanceof User ) {
+            throw new NotAllowedException( __( 'Unable to register using this email.' ) );
+        }
+
+        /**
+         * check user existence
+         */
+        $user   =   User::where( 'username', $request->input( 'username' ) )->first();
+        if ( $user instanceof User ) {
+            throw new NotAllowedException( __( 'Unable to register using this username.' ) );
+        }
+
         $options                    =   app()->make( Options::class );
         $role                       =   $options->get( 'ns_registration_role' );
         $registration_validated     =   $options->get( 'ns_registration_validated', 'yes' );

diff --git a/app/Http/Requests/SignUpRequest.php b/app/Http/Requests/SignUpRequest.php
@@ -32,8 +32,8 @@ public function authorize()
     public function rules()
     {
         return [
-            'username'          =>  'required|min:6|unique:nexopos_users',
-            'email'             =>  'email|unique:nexopos_users',
+            'username'          =>  'required|min:6',
+            'email'             =>  'email',
             'password'          =>  'required',
             'password_confirm'  => 'same:password'
         ];

diff --git a/config/nexopos.php b/config/nexopos.php
@@ -1,6 +1,6 @@
 <?php
 return [
-    'version'       =>  '4.4.0',
+    'version'       =>  '4.4.1',
     'pos'           =>  [
         // deprecated
         'payments'  =>  [