refactor(clean): clean code with js

SalamLang · Jan 29, 2025 · 7644db9 · 7644db9
1 parent ba40941
commit 7644db9
Show file tree

Hide file tree

Showing 7 changed files with 25 additions and 24 deletions.
diff --git a/app/Http/Controllers/Admin/AdminController.php b/app/Http/Controllers/Admin/AdminController.php
@@ -7,7 +7,6 @@
 use App\Models\Code;
 use App\Models\User;
 use App\Models\Visit;
-use Illuminate\Http\Request;
 
 class AdminController extends Controller
 {
@@ -20,15 +19,15 @@ public function index()
         $visit_count = Visit::all()->count();
 
         return $this->success([
-            "codes" => [
-                "count" => $codes_count
+            'codes' => [
+                'count' => $codes_count,
             ],
-            "users" => [
-                "count" => $users_count
+            'users' => [
+                'count' => $users_count,
+            ],
+            'visit' => [
+                'count' => $visit_count,
             ],
-            "visit" => [
-                "count" => $visit_count
-            ]
         ]);
     }
 }
diff --git a/app/Http/Controllers/InfoController.php b/app/Http/Controllers/InfoController.php
@@ -15,13 +15,13 @@ public function info(Request $request)
         $code_count = $user?->codes()?->count();
 
         return $this->success([
-            "user" => [
-                "name" => $user->name,
-                "family" => $user->family,
+            'user' => [
+                'name' => $user->name,
+                'family' => $user->family,
+            ],
+            'code' => [
+                'count' => $code_count,
             ],
-            "code" => [
-                "count" => $code_count,
-            ]
         ]);
     }
 }
diff --git a/app/Http/Middleware/CheckAdminMiddleware.php b/app/Http/Middleware/CheckAdminMiddleware.php
@@ -11,6 +11,7 @@
 class CheckAdminMiddleware
 {
     use ApiResponseTrait;
+
     /**
      * Handle an incoming request.
      *
@@ -23,9 +24,9 @@ public function handle(Request $request, Closure $next): Response
         $personalAccessToken = $token ? PersonalAccessToken::findToken($token) : null;
         $user = $personalAccessToken?->tokenable;
 
-        if ($user and $user?->getRoleNames()->toArray()[0] === "admin") {
+        if ($user and $user?->getRoleNames()->toArray()[0] === 'admin') {
             return $next($request);
-        }else {
+        } else {
             return $this->fail();
         }
     }

diff --git a/app/Http/Middleware/VisitMiddleware.php b/app/Http/Middleware/VisitMiddleware.php
@@ -17,9 +17,10 @@ class VisitMiddleware
     public function handle(Request $request, Closure $next): Response
     {
         Visit::create([
-            "ip" => $request->ip(),
-            "user_agent" => $request->userAgent(),
+            'ip' => $request->ip(),
+            'user_agent' => $request->userAgent(),
         ]);
+
         return $next($request);
     }
 }
diff --git a/app/Models/Visit.php b/app/Models/Visit.php
@@ -10,7 +10,7 @@ class Visit extends Model
     use SoftDeletes;
 
     protected $fillable = [
-        "ip",
-        "user_agent"
+        'ip',
+        'user_agent',
     ];
 }
diff --git a/database/migrations/2025_01_29_182815_create_visits_table.php b/database/migrations/2025_01_29_182815_create_visits_table.php
@@ -13,8 +13,8 @@ public function up(): void
     {
         Schema::create('visits', function (Blueprint $table) {
             $table->id();
-            $table->string("ip");
-            $table->string("user_agent");
+            $table->string('ip');
+            $table->string('user_agent');
             $table->softDeletes();
             $table->timestamps();
         });

diff --git a/routes/api.php b/routes/api.php
@@ -22,8 +22,8 @@
                 Route::delete('code', [CodeController::class, 'destroy'])->name('code');
                 Route::post('info', [InfoController::class, 'info'])->name('info');
 
-                Route::prefix("admin")->name(".admin")->middleware(CheckAdminMiddleware::class)->group(function () {
-                    Route::post("/", [AdminController::class, "index"]);
+                Route::prefix('admin')->name('.admin')->middleware(CheckAdminMiddleware::class)->group(function () {
+                    Route::post('/', [AdminController::class, 'index']);
                 });
             });
         });