silvercommerce / tax-admin

tests/model/TestProduct.php

<?php

namespace SilverCommerce\TaxAdmin\Tests\Model;

use SilverStripe\i18n\i18n;
use SilverStripe\Dev\TestOnly;
use SilverStripe\ORM\DataObject;
use SilverCommerce\TaxAdmin\Model\TaxRate;
use SilverCommerce\TaxAdmin\Traits\Taxable;
use SilverCommerce\TaxAdmin\Model\TaxCategory;
use SilverCommerce\TaxAdmin\Interfaces\TaxableProvider;

class TestProduct extends DataObject implements TestOnly, TaxableProvider
{
    use Taxable;

The trait SilverCommerce\TaxAdmin\Traits\Taxable requires the property $Rate which is not provided by SilverCommerce\TaxAdmin\Tests\Model\TestProduct.

    /**
     * Default behaviour for price with tax (if current instance not set)
     *
     * @var boolean
     */
    private static $show_price_with_tax = false;

The private property $show_price_with_tax is not used, and could be removed.

    /**
     * Default behaviour for adding the tax string to the rendered currency.
     *
     * @var boolean
     */
    private static $show_tax_string = false;

The private property $show_tax_string is not used, and could be removed.

    private static $db = [

The private property $db is not used, and could be removed.

        "Title" => "Varchar",
        "BasePrice" => 'Decimal(9,3)',
        "StockID" => "Varchar",
        "StockLevel" => "Int",
        "Weight" => "Decimal"
    ];

    private static $has_one = [

The private property $has_one is not used, and could be removed.

        'TaxRate' => TaxRate::class,
        'TaxCategory' => TaxCategory::class
    ];

    public function getBasePrice()
    {
        return $this->dbObject('BasePrice')->getValue();
    }

    /**
     * Find a tax rate based on the selected ID, or revert to using the valid tax
     * from the current category
     *
     * @return \SilverCommerce\TaxAdmin\Model\TaxRate
     */
    public function getTaxRate()
    {
        $tax = TaxRate::get()->byID($this->getOwner()->TaxRateID);

The method getOwner() does not exist on SilverCommerce\TaxAdmin\Tests\Model\TestProduct. Since you implemented __call, consider adding a @method annotation.

        // If no tax explicity set, try to get from category
        if (empty($tax)) {
            $category = TaxCategory::get()->byID($this->getOwner()->TaxCategoryID);

            $tax = (!empty($category)) ? $category->ValidTax() : null ;
        }

        if (empty($tax)) {
            $tax = TaxRate::create();
            $tax->ID = -1;
        }

        return $tax;
    }

    /**
     * Get should this field automatically show the price including TAX?
     *
     * @return bool
     */
    public function getShowPriceWithTax()
    {
        return (bool)$this->config()->get('show_price_with_tax');
    }

    /**
     * Get if this field should add a "Tax String" (EG Includes VAT) to the rendered
     * currency?
     *
     * @return bool|null
     */
    public function getShowTaxString()
    {
        return (bool)$this->config()->get('show_tax_string');
    }

    /**
     * Return the currently available locale
     *
     * @return string
     */
    public function getLocale()
    {
        return i18n::get_locale();
    }
}