Ecodev / artisans

server/Application/Traits/HasSubscriptionLastReview.php

<?php

declare(strict_types=1);

namespace Application\Traits;

use Application\Model\Product;
use Application\Repository\ProductRepository;
use Doctrine\ORM\Mapping as ORM;
use GraphQL\Doctrine\Attribute as API;
use InvalidArgumentException;

trait HasSubscriptionLastReview
{
    #[ORM\JoinColumn(onDelete: 'SET NULL')]
    #[ORM\ManyToOne(targetEntity: Product::class)]
    private ?Product $subscriptionLastReview = null;

    /**
     * Get last review number available through a subscription, bypassing all ACL so it also work even if review is not active yet.
     */
    public function getSubscriptionLastReviewNumber(): ?int
    {
        /** @var ProductRepository $productRepository */
        $productRepository = _em()->getRepository(Product::class);

        return $productRepository->getSubscriptionLastReviewNumber($this);
    }

    /**
     * Set last review available through a subscription.
     */
    #[API\Exclude]
    public function setSubscriptionLastReview(?Product $subscriptionLastReview): void
    {
        if ($subscriptionLastReview && !$subscriptionLastReview->getReviewNumber()) {

The expression $subscriptionLastReview->getReviewNumber() of type integer|null is loosely compared to false; this is ambiguous if the integer can be 0. You might want to explicitly use === null instead.

            throw new InvalidArgumentException('The last review of a subscription must be a review, not an article');
        }

        $this->subscriptionLastReview = $subscriptionLastReview;
    }
}