Treestoneit\ShoppingCart\BuyableTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getBuyableIdentifier()	0	4	1
A	getBuyableDescription()	0	4	1
A	getBuyablePrice()	0	4	1
A	getExtraFees()	0	4	1
A	getOptions()	0	4	1

<?php

namespace Treestoneit\ShoppingCart;

use Exception;

trait BuyableTrait
{
    /**
     * Get the identifier of the Buyable item.
     *
     * @return int|string
     */
    public function getBuyableIdentifier()
    {
        return $this->getKey();

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

    }

    /**
     * Get the description or title of the Buyable item.
     *
     * @return string
     * @throws \Exception
     */
    public function getBuyableDescription()
    {
        throw new Exception('Buyable description has not been set.');
    }

    /**
     * Get the price of the Buyable item.
     *
     * @return float|null
     * @throws \Exception
     */
    public function getBuyablePrice()
    {
        throw new Exception('Buyable price has not been set.');
    }

    /**
     * Any extra fees not based on product quantity.
     *
     * @return float|int
     */
    public function getExtraFees()
    {
        return 0;
    }

    /**
     * An array of options (color, size, etc.) for this buyable item.
     *
     * @return array
     */
    public function getOptions(): array
    {
        return [];
    }
}