Srmklive\Chargify\Traits\ChargifyAPI\ProductsFamilies

Complexity

Total Complexity

4

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	product_families_list()	0	7	1
A	product_family_details()	0	7	1
A	product_family_products()	0	7	1
A	product_family_create()	0	11	1

<?php

namespace Srmklive\Chargify\Traits\ChargifyAPI;

trait ProductsFamilies
{
    /**
     * Create a product family.
     *
     * @param array $data
     *
     * @return array
     */
    public function product_family_create(array $data): array
    {
        $this->apiEndPoint = '/product_families.json';

        $this->verb = 'post';

        $this->options['json'] = [

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

            'product_family' => $data,
        ];

        return $this->doChargifyRequest();

It seems like doChargifyRequest() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Get details for a product family.
     *
     * @param int $family_id
     *
     * @return array
     */
    public function product_family_details(int $family_id): array
    {
        $this->apiEndPoint = "/product_families/{$family_id}.json";

        $this->verb = 'get';

        return $this->doChargifyRequest();
    }

    /**
     * Create a product family.
     *
     * @return array
     */
    public function product_families_list(): array
    {
        $this->apiEndPoint = '/product_families.json';

        $this->verb = 'get';

        return $this->doChargifyRequest();
    }

    /**
     * Get products list for a product family.
     *
     * @param int $family_id
     *
     * @return array
     */
    public function product_family_products(int $family_id): array
    {
        $this->apiEndPoint = "/product_families/{$family_id}/products.json";

        $this->verb = 'get';

        return $this->doChargifyRequest();
    }
}