Merge pull request #27 from wirecli/perf-optimize-user-role-save-6580931905620183491

⚡ [performance improvement] Optimize N+1 user role save operations

Author: flydev-fr

src/Helpers/PwUserTools.php

@@ -94,8 +94,6 @@ public function attachRolesToUser($user, $roles, $output, $reset = false) {
         $editedUser->removeRole($role->name);
       }
     }
-    $editedUser->save();
-
     // remove existing roles
     if ($reset === true) {
       foreach ($editedUser->roles as $role) {
@@ -109,9 +107,10 @@ public function attachRolesToUser($user, $roles, $output, $reset = false) {
       $this->checkIfRoleExists($role, $output);
 
       $editedUser->addRole($role);
-      $editedUser->save();
     }
 
+    $editedUser->save();
+
     return $editedUser;
   }