fix rule unique issue and logout user on email change (#19)

* fix rule unique issue and logout user on email change * some changes * refactor Signed-off-by: Sam Poyigi <[email protected]> --------- Signed-off-by: Sam Poyigi <[email protected]> Co-authored-by: Sam Poyigi <[email protected]>
tastyigniter · Feb 5, 2024 · 06ff32e · 06ff32e
1 parent b6e9645
commit 06ff32e
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/src/Http/Controllers/Users.php b/src/Http/Controllers/Users.php
@@ -50,7 +50,7 @@ class Users extends \Igniter\Admin\Classes\AdminController
         'configFile' => 'user',
     ];
 
-    public $locationConfig = [
+    public array $locationConfig = [
         'addAbsenceConstraint' => false,
     ];
 
@@ -90,7 +90,13 @@ public function account_onSave()
         $usernameChanged = $this->currentUser->username != post('User[username]');
         $passwordChanged = strlen(post('User[password]'));
         $languageChanged = $this->currentUser->language != post('User[language_id]');
-        if ($usernameChanged || $passwordChanged || $languageChanged) {
+        $emailChanged = $this->currentUser->email != post('User[email]');
+        if ($emailChanged || $passwordChanged) {
+            AdminAuth::logout();
+            return redirect('/logout');
+        }
+
+        if ($usernameChanged || $languageChanged) {
             $this->currentUser->reload()->reloadRelations();
             AdminAuth::login($this->currentUser, true);
         }

diff --git a/src/Requests/UserRequest.php b/src/Requests/UserRequest.php
@@ -3,6 +3,7 @@
 namespace Igniter\User\Requests;
 
 use Igniter\System\Classes\FormRequest;
+use Igniter\User\Facades\AdminAuth;
 use Illuminate\Validation\Rule;
 
 class UserRequest extends FormRequest
@@ -46,4 +47,12 @@ public function rules()
             'locations.*' => ['integer'],
         ];
     }
+
+    protected function getRecordId(): string
+    {
+        $slugName = ($slug = $this->route('slug'))
+            ? str_after($slug, '/') : null;
+
+        return $slugName == 'account' ? AdminAuth::id() : $slugName;
+    }
 }