Skip to content

Commit

Permalink
Merge pull request #47 from EscolaLMS/bugfix/buyable-by-user-should-w…
Browse files Browse the repository at this point in the history
…ork-for-coreuser-model

Fix wrong User type in buyableByUser method
  • Loading branch information
pa-cholek authored Mar 9, 2022
2 parents f29c903 + c251422 commit 57f2c2f
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/Models/Product.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,10 @@
use EscolaLms\Cart\Database\Factories\ProductFactory;
use EscolaLms\Cart\Models\Contracts\ProductInterface;
use EscolaLms\Cart\Models\Contracts\ProductTrait;
use EscolaLms\Cart\Models\User;
use EscolaLms\Cart\QueryBuilders\ProductModelQueryBuilder;
use EscolaLms\Cart\Services\Contracts\ProductServiceContract;
use EscolaLms\Core\Models\User as CoreUser;
use EscolaLms\Tags\Models\Tag;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
Expand Down Expand Up @@ -148,12 +150,12 @@ public function getTaxRate(): int
return $this->tax_rate ?? 0;
}

public function getBuyableByUserAttribute(?User $user = null): bool
public function getBuyableByUserAttribute(?CoreUser $user = null): bool
{
return app(ProductServiceContract::class)->productIsBuyableByUser($this, $user ?? Auth::user());
}

public function getOwnedByUserAttribute(?User $user = null): bool
public function getOwnedByUserAttribute(?CoreUser $user = null): bool
{
return app(ProductServiceContract::class)->productIsOwnedByUser($this, $user ?? Auth::user());
}
Expand Down

0 comments on commit 57f2c2f

Please sign in to comment.