ByTIC\Payments\Models\AbstractModels\HasPurchaseParent

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getPurchaseFk()	0	3	1
A	populateFromPayment()	0	7	1

<?php

namespace ByTIC\Payments\Models\AbstractModels;

use ByTIC\Payments\Models\Purchase\Traits\IsPurchasableModelTrait;

/**
 * Trait HasPurchaseParent
 * @package ByTIC\Payments\Models\AbstractModels
 */
trait HasPurchaseParent
{
    /**
     * @param IsPurchasableModelTrait $payment
     * @return $this
     */
    public function populateFromPayment($payment)
    {
        $this->{$this->getPurchaseFk()} = $payment->id;
        $this->amount = floor($payment->getPurchaseAmount() * 100);

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

        $this->currency = $payment->getPurchaseCurrency();

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

        return $this;
    }

    /**
     * @return string
     */
    protected function getPurchaseFk()
    {
        return 'id_purchase';
    }
}