improve admin creation logic

wangdeer · Sep 29, 2021 · 70b167c · 70b167c
1 parent 603656a
commit 70b167c
Showing 1 changed file with 15 additions and 3 deletions.
diff --git a/app/Containers/AppSection/User/Actions/CreateAdminAction.php b/app/Containers/AppSection/User/Actions/CreateAdminAction.php
@@ -8,11 +8,15 @@
 use App\Containers\AppSection\User\UI\API\Requests\CreateAdminRequest;
 use App\Ship\Exceptions\CreateResourceFailedException;
 use App\Ship\Parents\Actions\Action;
+use App\Ship\Parents\Exceptions\Exception;
+use Illuminate\Support\Facades\DB;
+use Throwable;
 
 class CreateAdminAction extends Action
 {
     /**
      * @throws CreateResourceFailedException
+     * @throws Throwable
      */
     public function run(CreateAdminRequest $request): User
     {
@@ -24,10 +28,18 @@ public function run(CreateAdminRequest $request): User
             'birth',
         ]);
 
-        $admin = app(CreateUserByCredentialsTask::class)->run($sanitizedData);
+        DB::beginTransaction();
 
-        app(AssignRolesToUserTask::class)->run($admin, [config('appSection-authorization.admin_role')]);
+        try {
+            $admin = app(CreateUserByCredentialsTask::class)->run($sanitizedData);
+            app(AssignRolesToUserTask::class)->run($admin, [config('appSection-authorization.admin_role')]);
 
-        return $admin;
+            DB::commit();
+
+            return $admin;
+        } catch (Exception $exception) {
+            DB::rollBack();
+            throw $exception;
+        }
     }
 }