FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente

Complexity

Total Complexity

15

Size/Duplication

Total Lines	110
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	cleanRefundData()	0	8	1
A	descripcionTipoComprobante()	0	6	1
B	saveBefore()	0	45	11
A	all()	0	12	2

<?php
/**
 * Copyright (C) 2020 joenilson.
 *
 * This library is free software; you can redistribute it and/or
 * modify it under the terms of the GNU Lesser General Public
 * License as published by the Free Software Foundation; either
 * version 3 of the License, or (at your option) any later version.
 *
 * This library is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 * Lesser General Public License for more details.
 *
 * You should have received a copy of the GNU Lesser General Public
 * License along with this library; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
 * MA 02110-1301  USA
 */

namespace FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model;

use Closure;
use FacturaScripts\Core\Base\DataBase\DataBaseWhere;

The type FacturaScripts\Core\Base\DataBase\DataBaseWhere was not found. Maybe you did not declare it correctly or list all dependencies?

use FacturaScripts\Dinamic\Model\NCFRango;

The type FacturaScripts\Dinamic\Model\NCFRango was not found. Maybe you did not declare it correctly or list all dependencies?

use FacturaScripts\Dinamic\Model\NCFTipo;

The type FacturaScripts\Dinamic\Model\NCFTipo was not found. Maybe you did not declare it correctly or list all dependencies?

use FacturaScripts\Dinamic\Model\NCFTipoMovimiento;

The type FacturaScripts\Dinamic\Model\NCFTipoMovimiento was not found. Maybe you did not declare it correctly or list all dependencies?

use FacturaScripts\Dinamic\Model\Cliente;

This use statement conflicts with another class in this namespace, FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\Cliente. Consider defining an alias.

The type FacturaScripts\Dinamic\Model\Cliente was not found. Maybe you did not declare it correctly or list all dependencies?

use FacturaScripts\Core\App\AppSettings;

The type FacturaScripts\Core\App\AppSettings was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 * Description of FacturaCliente
 *
 * @author joenilson
 */
class FacturaCliente
{
    /**
     * @var date

The type FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\date was not found. Maybe you did not declare it correctly or list all dependencies?

     */
    public $ncffechavencimiento;
    /**
     * @var string
     */
    public $tipocomprobante;
    /**
     *
     * @var string
     */
    public $ncftipopago;
    /**
     *
     * @var string
     */
    public $ncftipomovimiento;
    /**
     *
     * @var string
     */
    public $ncftipoanulacion;

    /**
     * @var string
     */
    public $facturarectnumero2;

    public function saveBefore(): Closure
    {
        return function () {
            if (null !== $this->codigorect && $this->idfactura === null) {

The property codigorect does not exist on FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente. Did you maybe forget to declare it?

The property idfactura does not exist on FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente. Did you maybe forget to declare it?

                $this->cleanRefundData();
            }

            $cliente = new Cliente();
            $appSettins = new AppSettings;
            $actualCliente = $cliente->get($this->codcliente);

The property codcliente does not exist on FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente. Did you maybe forget to declare it?

            $actualCliente->idempresa = $appSettins::get('default', 'idempresa');
            $this->tipocomprobante = $this->tipocomprobante ?? $actualCliente->tipocomprobante;
            $this->tipocomprobante = $_REQUEST['tipocomprobanter'] ?? $this->tipocomprobante;
            if ($this->tipocomprobante !== '' && \in_array($this->numeroncf, ['', null], true)) {
                $tipocomprobante = "02";
                if (($this->tipocomprobante !== null) === true) {
                    $tipocomprobante = $this->tipocomprobante;
                } elseif (($this->tipocomprobante === null) === true) {
                    $tipocomprobante = $actualCliente->tipocomprobante;
                }

                if ('' === $this->numeroncf) {
                    $ncfrango = new NCFRango();
                    $ncfRangoToUse = $ncfrango->getByTipoComprobante($actualCliente->idempresa, $tipocomprobante);
                    if (!$ncfRangoToUse) {
                        $this->toolBox()->i18nLog()->error("no-ncf-range-for-$tipocomprobante");

The method toolBox() does not exist on FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente.

                        return false;
                    }
                    $ncf = $ncfRangoToUse->generateNCF();
                    $this->numeroncf = $ncf;

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

                    $this->ncffechavencimiento = $ncfRangoToUse->fechavencimiento;
                    $this->tipocomprobante = $ncfRangoToUse->tipocomprobante;
                    $ncfRangoToUse->correlativo++;
                    $ncfRangoToUse->save();
                }

                //Verificamos si $this->tipoComprobante es una nota de crédito o debito
                $arrayNCFTypes = ['03','04'];
                if (in_array($this->tipocomprobante, $arrayNCFTypes) === true) {
                    $this->ncftipoanulacion = $_REQUEST['ncftipoanulacionr'] ?? $this->ncftipoanulacion;
                    $this->ncffechavencimiento = $_REQUEST['ncffechavencimientor'] ?? $this->ncffechavencimiento;
                }
            }
            $this->ncffechavencimiento = ($this->ncffechavencimiento === '') ? null : $this->ncffechavencimiento;

The condition $this->ncffechavencimiento === '' is always false.

            return $this;
        };
    }

    public function all(): Closure
    {
        return function () {
            $this->facturarectnumero2 = '';
            if ($this->idfacturarect !== '') {

The property idfacturarect does not exist on FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente. Did you maybe forget to declare it?

                $facturaRectificativa = $this->get($this->idfacturarect);

The method get() does not exist on FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente.

                $this->loadFromData(['facturarectnumero2' => 'SI' ]);

The method loadFromData() does not exist on FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model\FacturaCliente.

                $this->facturarectnumero2 = $facturaRectificativa->numero2;
            } else {
                $this->loadFromData(['facturarectnumero2' => 'NO HAY']);
            }
            return $this;
        };
    }

    public function descripcionTipoComprobante(): Closure
    {
        return function () {
            $ncftipocomprobante = new NCFTipo();
            $ncftipocomprobante->loadFromCode($this->tipocomprobante);
            return $ncftipocomprobante->descripcion;
        };
    }
    protected function cleanRefundData()
    {
        return function () {
            $this->numeroncf = '';

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

            $this->tipocomprobante = null;
            $this->ncffechavencimiento = null;
            $this->ncftipomovimiento = null;
            $this->ncftipopago = null;
        };
    }
}