FeiMx\Tax\Traits\Taxable

Complexity

Total Complexity

20

Size/Duplication

Total Lines	174
Duplicated Lines	20.69 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	1
Bugs	0	Features	1

10 Methods

Rating	Name	Duplication	Size	Complexity
A	priceColumn()	0	4	1
A	taxGroups()	0	10	1
A	hasTaxGroups()	0	4	1
B	hasTaxGroup()	0	20	6
A	assignTaxGroup()	0	17	2
A	removeTaxGroup()	0	4	1
A	syncTaxGroups()	0	6	1
A	total()	18	18	2
A	getAmounts()	18	18	2
A	getStoredTaxGroup()	0	11	3

<?php

namespace FeiMx\Tax\Traits;

use FeiMx\Tax\Exceptions\TaxErrorException;
use FeiMx\Tax\Models\TaxGroup;
use Illuminate\Database\Eloquent\Relations\MorphToMany;

trait Taxable
{
    /**
     * Column name used for get the price of the model.
     *
     * @return string Column name
     */
    public static function priceColumn()
    {
        return 'price';
    }

    /**
     * [taxGroups description].
     *
     * @return [type] [description]

The doc-type [type] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     */
    public function taxGroups(): MorphToMany
    {
        return $this->morphToMany(

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

            TaxGroup::class,
            'model',
            'model_has_tax_groups',
            'model_id',
            'tax_group_id'
        );
    }

    public function hasTaxGroups()
    {
        return (bool) $this->taxGroups->count();

The property taxGroups does not exist. Did you maybe forget to declare it?

    }

    /**
     * Determine if the model has (one of) the given tax group(s).
     *
     * @param string|array|\FeiMx\Tax\Models\TaxGroup|\Illuminate\Support\Collection $taxGroups
     *
     * @return bool
     */
    public function hasTaxGroup($taxGroups)
    {
        if (is_string($taxGroups)) {
            return $this->taxGroups->contains('name', $taxGroups);
        }
        if ($taxGroups instanceof TaxGroup) {
            return $this->taxGroups->contains('id', $taxGroups->id);
        }
        if (is_array($taxGroups)) {
            foreach ($taxGroups as $taxGroup) {
                if ($this->hasTaxGroup($taxGroup)) {
                    return true;
                }
            }

            return false;
        }

        return $taxGroups->intersect($this->taxGroups)->isNotEmpty();
    }

    /**
     * Assign the given taxGroup to the model.
     *
     * @param array|string|\FeiMx\Tax\Models\TaxGroup ...$taxGroups
     *
     * @return $this
     */
    public function assignTaxGroup(...$taxGroups)
    {
        if (0 == count($taxGroups)) {
            throw new TaxErrorException('You must pass a valid TaxGroup');
        }

        $taxGroups = collect($taxGroups)
            ->flatten()
            ->map(function ($taxGroup) {
                return $this->getStoredTaxGroup($taxGroup);
            })
            ->all();

        $this->taxGroups()->saveMany($taxGroups);

        return $this;
    }

    /**
     * Revoke the given role from the model.
     *
     * @param string|\FeiMx\Tax\Models\TaxGroup $taxGroup
     */
    public function removeTaxGroup($taxGroup)
    {
        $this->taxGroups()->detach($this->getStoredTaxGroup($taxGroup));
    }

    /**
     * Remove all current tax groups and set the given ones.
     *
     * @param array|\FeiMx\Tax\Models\TaxGroup|string ...$taxGroups
     *
     * @return $this
     */
    public function syncTaxGroups(...$taxGroups)
    {
        $this->taxGroups()->detach();

        return $this->assignTaxGroup($taxGroups);
    }

    /**
     * Get total amount for current TaxGroup.
     *
     * @param \FeiMx\Tax\Models\TaxGroup|string $taxGroup
     *
     * @return $total

The doc-type $total could not be parsed: Unknown type name "$total" at position 0. (view supported doc-types)

     */
    public function total($taxGroup = null)

This method seems to be duplicated in your project.

    {
        if (null === $taxGroup) {
            throw new TaxErrorException('You must pass a valid TaxGroup');
        }

        $taxGroup = $this->getStoredTaxGroup($taxGroup);
        $column = self::priceColumn();

        return $total = $taxGroup->taxManager($this->{$column})

$total is not used, you could remove the assignment.

            ->addTaxes(
                $taxGroup->taxes->map(function ($tax) {
                    return $tax->info();
                })
                ->all()
            )
            ->total();
    }

    /**
     * Get a list of taxes with amount calculated for the given TaxGroup.
     *
     * @param \FeiMx\Tax\Models\TaxGroup|string $taxGroup
     *
     * @return $total

The doc-type $total could not be parsed: Unknown type name "$total" at position 0. (view supported doc-types)

     */
    public function getAmounts($taxGroup = null)

This method seems to be duplicated in your project.

    {
        if (null === $taxGroup) {
            throw new TaxErrorException('You must pass a valid TaxGroup');
        }

        $taxGroup = $this->getStoredTaxGroup($taxGroup);
        $column = self::priceColumn();

        return $taxGroup->taxManager($this->{$column})
            ->addTaxes(
                $taxGroup->taxes->map(function ($tax) {
                    return $tax->info();
                })
                ->all()
            )
            ->get();
    }

    protected function getStoredTaxGroup($taxGroup): TaxGroup
    {
        if (is_numeric($taxGroup)) {
            return TaxGroup::find($taxGroup);
        }
        if (is_string($taxGroup)) {
            return TaxGroup::whereName($taxGroup)->first();
        }

        return $taxGroup;
    }
}