When multiple canceled subscriptions, subscribed() function returns false, even when Billable model has active subscription

Cashier Stripe Version

14.14.0

Laravel Version

10.48.13

PHP Version

8.3

Database Driver & Version

MySQL v8

Description

I have a SaaS running cashier with stripe. I noticed on stripe a customer being active, but on my app, he was inactive.

When analyzing the Billable model's return for $user->subscribed('default');, it came back false. That meant the user was not subscribed, although Stripe was clear on showing me the last bill was paid and the user had an active subscription.

After further investigation, I noticed the user had more than one subscription in the database, all of them up to date with stripe. No webhook problems. 3 where canceled, and one active.

This is probably an edge case, but when a billable model has more than one subcription on the database, and only one is active, the subscribed() function can return false, despite there beign an active subscription.

The issue, as far as i can tell, lies on the following code:

    public function subscription($type = 'default')
    {
        return $this->subscriptions->where('type', $type)->first();
    }

first() is called, not taking in account the stripe_status, so if the user has more than one subscription, the first one is returned, no matter the status.

Let me know if I implemented cashier wrong. Reporting in case this is something important or not previously reported.

Thanks in advance for all the amazing work.

Steps To Reproduce

Manually add subscriptions to a Billable model
Make sure there is only one stripe_status marked as active
Call subscribed on the billable model

laravel / cashier-stripe