F72X\Sunat\Document\NoteMixin

Complexity

Total Complexity

3

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	addDiscrepancyResponse()	0	17	1
A	addRequestedMonetaryTotal()	0	16	1
A	addBillingReference()	0	17	1

<?php

/**
 * MÓDULO DE EMISIÓN ELECTRÓNICA F72X
 * UBL 2.1
 * Version 1.1
 * 
 * Copyright 2018, Jaime Cruz
 */

namespace F72X\Sunat\Document;

use F72X\UblComponent\DiscrepancyResponse;
use F72X\UblComponent\BillingReference;
use F72X\UblComponent\InvoiceDocumentReference;
use F72X\UblComponent\RequestedMonetaryTotal;

trait NoteMixin {

    public function addDiscrepancyResponse() {

        // Data
        $dataMap = $this->dataMap;
        $affectedDocId = $dataMap->getNoteAffectedDocId();
        $type = $dataMap->getNoteType();
        $description = $dataMap->getNoteDescription();

        // XML Nodes
        $DiscrepancyResponse = new DiscrepancyResponse();
        $DiscrepancyResponse
                ->setReferenceID($affectedDocId)
                ->setResponseCode($type)
                ->setDescription($description);

        // Add Node
        $this->setDiscrepancyResponse($DiscrepancyResponse);

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

    }

    public function addBillingReference() {

        // Data
        $dataMap = $this->dataMap;
        $affDocId = $dataMap->getNoteAffectedDocId();
        $affDocType = $dataMap->getNoteAffectedDocType();

        // XML Nodes
        $BillingReference = new BillingReference();
        $InvoiceDocumentReference = new InvoiceDocumentReference();
        $BillingReference
                ->setInvoiceDocumentReference($InvoiceDocumentReference
                        ->setID($affDocId)
                        ->setDocumentTypeCode($affDocType));

        // Add Node
        $this->setBillingReference($BillingReference);

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

    }
    private function addRequestedMonetaryTotal() {
        $dataMap            = $this->dataMap;
        $currencyID         = $this->getDocumentCurrencyCode(); // Tipo de moneda

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

        $totalAllowances    = $dataMap->getTotalAllowances();   // Total descuentos
        $payableAmount      = $dataMap->getPayableAmount();     // Total a pagar
        $billableAmount     = $dataMap->getBillableValue();
        // RequestedMonetaryTotal
        $RequestedMonetaryTotal = new RequestedMonetaryTotal();
        $RequestedMonetaryTotal
                ->setCurrencyID($currencyID)
                ->setLineExtensionAmount($billableAmount)
                ->setTaxInclusiveAmount($payableAmount)
                ->setAllowanceTotalAmount($totalAllowances)
                ->setPayableAmount($payableAmount);

        $this->setRequestedMonetaryTotal($RequestedMonetaryTotal);

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

    }
}