Log more info about invalid pm exception (#1582)

laravel · Oct 16, 2023 · f437600 · f437600
1 parent 44ae9dd
commit f437600
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/Exceptions/InvalidPaymentMethod.php b/src/Exceptions/InvalidPaymentMethod.php
@@ -17,7 +17,7 @@ class InvalidPaymentMethod extends Exception
     public static function invalidOwner(StripePaymentMethod $paymentMethod, $owner)
     {
         return new static(
-            "The payment method `{$paymentMethod->id}` does not belong to this customer `$owner->stripe_id`."
+            "The payment method `{$paymentMethod->id}`'s customer `{$paymentMethod->customer}` does not belong to this customer `$owner->stripe_id`."
         );
     }
 }
diff --git a/src/PaymentMethod.php b/src/PaymentMethod.php
@@ -6,6 +6,7 @@
 use Illuminate\Contracts\Support\Jsonable;
 use JsonSerializable;
 use Laravel\Cashier\Exceptions\InvalidPaymentMethod;
+use LogicException;
 use Stripe\PaymentMethod as StripePaymentMethod;
 
 class PaymentMethod implements Arrayable, Jsonable, JsonSerializable
@@ -35,6 +36,10 @@ class PaymentMethod implements Arrayable, Jsonable, JsonSerializable
      */
     public function __construct($owner, StripePaymentMethod $paymentMethod)
     {
+        if (is_null($paymentMethod->customer)) {
+            throw new LogicException('The payment method is not attached to a customer.');
+        }
+
         if ($owner->stripe_id !== $paymentMethod->customer) {
             throw InvalidPaymentMethod::invalidOwner($paymentMethod, $owner);
         }