EvilFreelancer / resova-api-php

src/Endpoints/Baskets/Bookings.php

<?php

namespace Resova\Endpoints\Baskets;

use Resova\Client;
use Resova\Models\BookingRequest;

class Bookings extends Client
{
    /**
     * Create a basket
     * Creates a new basket object.
     *
     * @param BookingRequest $booking
     *
     * @return $this
     */
    public function create(BookingRequest $booking): self
    {
        $booking->setRequired([
            'instance_id',
            'quantities'
        ]);

        // Set HTTP params
        $this->type     = 'post';
        $this->endpoint = '/baskets/' . $this->basket_id . '/bookings';

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\Bookings. Since you implemented __get, consider adding a @property annotation.

        $this->params   = $booking;

        return $this;
    }

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

The property booking_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 . '/bookings/' . $booking_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\Bookings. Since you implemented __get, consider adding a @property annotation.

        return $this;
    }

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

The property basket_id does not exist on Resova\Endpoints\Baskets\Bookings. 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;
    }

    /**
     * Update a basket booking
     * Updates the specified basket booking by setting the values of the parameters passed.
     * Any parameters not provided will be left unchanged.
     * This request accepts mostly the same arguments as the basket booking creation call.
     *
     * @param null|BookingRequest $booking
     *
     * @return $this
     */
    public function update(BookingRequest $booking = null): self
    {
        // Set HTTP params
        $this->type     = 'put';
        $this->endpoint = '/baskets/' . $this->basket_id . '/bookings';

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\Bookings. Since you implemented __get, consider adding a @property annotation.

        $this->params   = $booking;

        return $this;
    }

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

The property basket_id does not exist on Resova\Endpoints\Baskets\Bookings. 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;
    }
}