Allow reusing an existing customer through `PerformsCharges::subscribe()` about cashier-paddle HOT 8 CLOSED

However, when the user already exists, the ManagesCustomer::createAsCustomer() method fails to create a customer with the same email address, throwing a PaddleException.

That's not possible. The method already checks if the billable has a customer attached here:

So I don't see how it's possible Cashier attempts to create a customer twice for the same billable for you?

from cashier-paddle.

Hello @driesvints,

I'm sorry I haven't explained properly. I meant to say this instead:

However, when the user already exists on Paddle's end but not in the database, the ManagesCustomer::createAsCustomer() method fails to create a customer with the same email address, throwing a PaddleException.

In other words, this is how it can happen:

1. I have an existing Customer entity, linked to a billable User entity, with the email of [email protected].
2. Due to whatever reason, I run php artisan migrate:fresh. This recreates all the tables.
3. I try to create the user with the same email address of [email protected]. Since there is no Customer entity attached to the user, $customer = $this->customer fails, and ManagesCustomer::createAsCustomer() proceeds with user creation.
4. Since a user already exists, the Paddle API returns an error stating that said user already exists.

This can also happen if two separate applications using laravel/cashier-paddle are using the same Paddle account. One of them might not have a Customer entity linked, and therefore encounter the aforementioned error when it tries to create what it thinks is a new user, but it already exists on Paddle's end.

from cashier-paddle.

I'm afraid this is impossible. It's not possible to use Cashier Paddle with existing Paddle accounts and customers, only new ones. This is because there's no way from Paddle's side to retrieve a customer by email address.

This can also happen if two separate applications using laravel/cashier-paddle are using the same Paddle account.

This also is impossible to do. You should never try to use two Laravel apps with the same Paddle account.

If there was a way from Paddle to retrieve customers by email address we could attempt to retrieve the customer before creating them and then sync them in through the createAsCustomer method.

from cashier-paddle.

I've tried explaining it here: laravel/docs#9189

from cashier-paddle.

Wait, apparently it is possible to search for a customer by email: laravel/docs#9189. I'll try to see what I can do here.

from cashier-paddle.

Wait, apparently it is possible to search for a customer by email

There is no need to search the customer email by ID in this case, since the error message from Paddle itself returns the conflicting customer ID, and that can be passed on to the GetCustomer API, as I mentioned above.

This also is impossible to do. You should never try to use two Laravel apps with the same Paddle account.

The only issue that I see is the lack of separation of transactions would lead to notifications being sent through the webhook that are not relevant for one application. That discussion is for another issue though, and I could work around that by filtering the product IDs myself, in my own webhook handlers.

from cashier-paddle.

Thanks. Continuing the discussion in #219

from cashier-paddle.

I've sent in a fix for this: #220

from cashier-paddle.