EvilFreelancer / resova-api-php

src/Endpoints/Baskets/Promotions.php

<?php

namespace Resova\Endpoints\Baskets;

use Resova\Client;
use Resova\Models\PromotionRequest;

class Promotions extends Client
{
    /**
     * Create a basket promotion
     * Creates a new basket promotion object.
     *
     * @param PromotionRequest $promotion
     *
     * @return $this
     */
    public function create(PromotionRequest $promotion): self
    {
        // Set HTTP params
        $this->type     = 'post';
        $this->endpoint = '/baskets/' . $this->basket_id . '/promotions';

Are you sure $this->basket_id of type boolean|object can be used in concatenation?

The property basket_id does not exist on Resova\Endpoints\Baskets\Promotions. Since you implemented __get, consider adding a @property annotation.

        $this->params   = $promotion;

        return $this;
    }

    /**
     * Retrieve a basket promotion
     * Retrieves the details of a basket promotion. Provide the unique id for the basket promotion.
     *
     * @param int $promotion_id The basket promotion id
     *
     * @return $this
     */
    public function __invoke(int $promotion_id): self
    {
        $this->promotion_id = $promotion_id;

The property promotion_id does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        // Set HTTP params
        $this->type     = 'get';
        $this->endpoint = '/baskets/' . $this->basket_id . '/promotions/' . $promotion_id;

Are you sure $this->basket_id of type boolean|object can be used in concatenation?

The property basket_id does not exist on Resova\Endpoints\Baskets\Promotions. Since you implemented __get, consider adding a @property annotation.

        return $this;
    }

    /**
     * Delete a basket promotion
     * Permanently deletes a basket promotion. It cannot be undone.
     *
     * @return $this
     */
    public function delete(): self
    {
        // Set HTTP params
        $this->type     = 'delete';
        $this->endpoint = '/baskets/' . $this->basket_id . '/promotions/' . $this->promotion_id;

The property basket_id does not exist on Resova\Endpoints\Baskets\Promotions. Since you implemented __get, consider adding a @property annotation.

Are you sure $this->basket_id of type boolean|object can be used in concatenation?

        return $this;
    }

    /**
     * List all basket promotions
     * Returns a list of your basket promotions.
     * The basket promotions are returned sorted by creation date, with the most recent basket purchase appearing first.
     *
     * @return $this
     */
    public function all(): self
    {
        // Set HTTP params
        $this->type     = 'get';
        $this->endpoint = '/baskets/' . $this->basket_id . '/promotions';

Are you sure $this->basket_id of type boolean|object can be used in concatenation?

The property basket_id does not exist on Resova\Endpoints\Baskets\Promotions. Since you implemented __get, consider adding a @property annotation.

        return $this;
    }
}