controller refactoring part 9

src/Domain/Payments/Controllers/PaymentMethodsController.php

@@ -3,10 +3,10 @@ namespace Domain\Payments\Controllers;
 
 use Auth;
 use Illuminate\Http\Request;
+use Illuminate\Http\Response;
 use Laravel\Cashier\PaymentMethod;
 use App\Http\Controllers\Controller;
 use Illuminate\Support\Facades\Cache;
-use Support\Demo\Actions\DemoService;
 use Domain\Subscriptions\Services\StripeService;
 use Domain\Payments\Resources\PaymentCardResource;
 use Domain\Payments\Resources\PaymentCardCollection;
@@ -17,16 +17,12 @@ class PaymentMethodsController extends Controller
 {
     public function __construct(
         private StripeService $stripe,
-        private DemoService $demo,
-    ) {
-    }
+    ) {}
 
     /**
      * Get user payment methods grouped by default and others
-     *
-     * @return array
      */
-    public function index()
+    public function index(): array
     {
         $user = Auth::user();
 
@@ -80,12 +76,8 @@ class PaymentMethodsController extends Controller
 
     /**
      * Update default payment method
-     *
-     * @param Request $request
-     * @param $id
-     * @return \Illuminate\Contracts\Routing\ResponseFactory|\Illuminate\Http\Response
      */
-    public function update($id)
+    public function update(string $id): Response
     {
         $user = Auth::user();
 
@@ -100,8 +92,8 @@ class PaymentMethodsController extends Controller
 
         // Clear cached payment methods
         cache_forget_many([
-            'payment-methods-user-' . $user->id,
-            'default-payment-methods-user-' . $user->id,
+            "payment-methods-user-{$user->id}",
+            "default-payment-methods-user-{$user->id}",
         ]);
 
         return response('Done', 204);
@@ -109,17 +101,14 @@ class PaymentMethodsController extends Controller
 
     /**
      * Register new payment method for user
-     *
-     * @param Request $request
-     * @return \Illuminate\Contracts\Routing\ResponseFactory|\Illuminate\Http\Response
      */
-    public function store(RegisterNewPaymentMethodRequest $request)
+    public function store(RegisterNewPaymentMethodRequest $request): Response
     {
         // Get user
         $user = Auth::user();
 
         // Check if is demo
-        if (is_demo($user->id)) {
+        if (is_demo_account($user->email)) {
             return response('Done', 201);
         }
 
@@ -131,14 +120,13 @@ class PaymentMethodsController extends Controller
 
     /**
      * Delete user payment method
-     *
      */
-    public function delete($id)
+    public function delete($id): Response
     {
         $user = Auth::user();
 
         // Check if is demo
-        abort_if(is_demo_account('howdy@hi5ve.digital'), 204, 'Done.');
+        abort_if(is_demo_account($user->email), 204, 'Done.');
 
         // Get payment method
         $paymentMethod = $user->findPaymentMethod($id);
@@ -151,8 +139,8 @@ class PaymentMethodsController extends Controller
 
         // Clear cached payment methods
         cache_forget_many([
-            'payment-methods-user-' . $user->id,
-            'default-payment-methods-user-' . $user->id,
+            "payment-methods-user-{$user->id}",
+            "default-payment-methods-user-{$user->id}",
         ]);
 
         return response('Done!', 204);

src/Domain/Subscriptions/Controllers/GetSetupIntentController.php

@@ -0,0 +1,27 @@
+<?php
+
+
+namespace Domain\Subscriptions\Controllers;
+
+
+use App\Http\Controllers\Controller;
+use Auth;
+use Domain\Subscriptions\Services\StripeService;
+use Illuminate\Http\Response;
+
+/**
+ * Generate setup intent
+ */
+class GetSetupIntentController extends Controller
+{
+    public function __construct(
+        public StripeService $stripe,
+    ) {}
+
+    public function setup_intent(): Response
+    {
+        return response(
+            $this->stripe->getSetupIntent(Auth::user()), 201
+        );
+    }
+}

src/Domain/Subscriptions/Controllers/SubscriptionCancelController.php

@@ -0,0 +1,34 @@
+<?php
+
+
+namespace Domain\Subscriptions\Controllers;
+
+
+use App\Http\Controllers\Controller;
+use Auth;
+use Illuminate\Http\Response;
+use Illuminate\Support\Facades\Cache;
+
+class SubscriptionCancelController extends Controller
+{
+    /**
+     * Cancel Subscription
+     */
+    public function __invoke(): Response
+    {
+        $user = Auth::user();
+
+        // Check if is demo
+        if (is_demo_account($user->email)) {
+            return response('Done!', 204);
+        }
+
+        // Cancel subscription
+        $user->subscription('main')->cancel();
+
+        // Forget user subscription
+        Cache::forget("subscription-user-{$user->id}");
+
+        return response('Done!', 204);
+    }
+}

src/Domain/Subscriptions/Controllers/SubscriptionController.php

@@ -22,19 +22,6 @@ class SubscriptionController extends Controller
     ) {
     }
 
-    /**
-     * Generate setup intent
-     *
-     * @return Application|ResponseFactory|Response|SetupIntent
-     */
-    public function setup_intent()
-    {
-        return response(
-            $this->stripe->getSetupIntent(Auth::user()),
-            201
-        );
-    }
-
     /**
      * Get user subscription detail
      *
@@ -60,89 +47,4 @@ class SubscriptionController extends Controller
             );
         });
     }
-
-    /**
-     * Upgrade account to subscription
-     *
-     * @param StoreUpgradeAccountRequest $request
-     * @return ResponseFactory|Response
-     */
-    public function upgrade(StoreUpgradeAccountRequest $request)
-    {
-        // Get user
-        $user = Auth::user();
-
-        // Check if is demo
-        if (is_demo($user->id)) {
-            return $this->demo->response_204();
-        }
-
-        // Forget user subscription
-        Cache::forget('subscription-user-' . $user->id);
-
-        // Get requested plan
-        $plan = $this->stripe->getPlan($request->input('plan.data.id'));
-
-        // Set user billing
-        $user->setBilling($request->input('billing'));
-
-        // Update stripe customer billing info
-        $this->stripe->updateCustomerDetails($user);
-
-        // Make subscription
-        $this->stripe->createOrReplaceSubscription($request, $user);
-
-        // Update user storage limit
-        $user->settings()->update([
-            'storage_capacity' => $plan['product']['metadata']['capacity'],
-        ]);
-
-        return response('Done!', 204);
-    }
-
-    /**
-     * Cancel Subscription
-     *
-     * @return ResponseFactory|Response
-     */
-    public function cancel()
-    {
-        $user = User::find(Auth::id());
-
-        // Check if is demo
-        if (is_demo($user->id)) {
-            return $this->demo->response_204();
-        }
-
-        // Cancel subscription
-        $user->subscription('main')->cancel();
-
-        // Forget user subscription
-        Cache::forget('subscription-user-' . $user->id);
-
-        return response('Done!', 204);
-    }
-
-    /**
-     * Resume Subscription
-     *
-     * @return ResponseFactory|Response
-     */
-    public function resume()
-    {
-        $user = User::find(Auth::id());
-
-        // Check if is demo
-        if (is_demo($user->id)) {
-            return $this->demo->response_204();
-        }
-
-        // Resume subscription
-        $user->subscription('main')->resume();
-
-        // Forget user subscription
-        Cache::forget('subscription-user-' . $user->id);
-
-        return response('Done!', 204);
-    }
 }

src/Domain/Subscriptions/Controllers/SubscriptionResumeController.php

@@ -0,0 +1,33 @@
+<?php
+
+
+namespace Domain\Subscriptions\Controllers;
+
+use App\Http\Controllers\Controller;
+use Auth;
+use Illuminate\Http\Response;
+use Illuminate\Support\Facades\Cache;
+
+class SubscriptionResumeController extends Controller
+{
+    /**
+     * Resume Subscription
+     */
+    public function __invoke(): Response
+    {
+        $user = Auth::user();
+
+        // Check if is demo
+        if (is_demo_account($user->email)) {
+            return response('Done!', 204);
+        }
+
+        // Resume subscription
+        $user->subscription('main')->resume();
+
+        // Forget user subscription
+        Cache::forget("subscription-user-{$user->id}");
+
+        return response('Done!', 204);
+    }
+}

src/Domain/Subscriptions/Controllers/SubscriptionUpgradeController.php

@@ -0,0 +1,60 @@
+<?php
+
+
+namespace Domain\Subscriptions\Controllers;
+
+
+use App\Http\Controllers\Controller;
+use Auth;
+use Domain\Subscriptions\Requests\StoreUpgradeAccountRequest;
+use Domain\Subscriptions\Services\StripeService;
+use Illuminate\Http\Response;
+use Illuminate\Support\Facades\Cache;
+
+/**
+ * Upgrade account to subscription
+ */
+class SubscriptionUpgradeController extends Controller
+{
+    public function __construct(
+        public StripeService $stripe,
+    ) {}
+
+    public function __invoke(StoreUpgradeAccountRequest $request): Response
+    {
+        $user = Auth::user();
+
+        // Check if is demo
+        if (is_demo_account($user->email)) {
+            return response('Done!', 204);
+        }
+
+        // Forget user subscription
+        Cache::forget("subscription-user-{$user->id}");
+
+        // Get requested plan
+        $plan = $this->stripe->getPlan(
+            $request->input('plan.data.id')
+        );
+
+        // Set user billing
+        $user->setBilling(
+            $request->input('billing')
+        );
+
+        // Update stripe customer billing info
+        $this->stripe->updateCustomerDetails($user);
+
+        // Make subscription
+        $this->stripe->createOrReplaceSubscription($request, $user);
+
+        // Update user storage limit
+        $user
+            ->settings()
+            ->update([
+                'storage_capacity' => $plan['product']['metadata']['capacity'],
+            ]);
+
+        return response('Done!', 204);
+    }
+}