Skip to content

Error with Feature Usage validationΒ #20

Open
@stanwarri

Description

@stanwarri

The canUseFeature method is returning the first record in the database instead of filtering based on the slug. This is because the query is being applied to a fresh instance of the model that hasn't been properly utilized to build the query before first() is called in the function below in the SubscriptionUsage.php class

public function scopeByFeatureSlug(Builder $builder, string $featureSlug): Builder
    {
        $model = config('laravel-subscriptions.models.feature', Feature::class);
        $feature = tap(new $model())->where('slug', $featureSlug)->first();

        return $builder->where('feature_id', $feature ? $feature->getKey() : null);
    }

The tap(new $model()) simply returns the model instance itself. The where('slug', $featureSlug) method is then called, but since first() is immediately called on this query, it ends up executing the query before any further conditions are applied.

Instead of using tap(), you should directly chain the query methods on the model instance:

$feature = $model::where('slug', $featureSlug)->first();

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingenhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions