joenilson / fsRepublicaDominicana

Controller/EditFacturaCliente.php

Spacing +3 added lines, -3 removed lines

@@ -67,9 +67,9 @@
         $data = $this->getBusinessFormData();
         $cliente = $cliente0->get($data['subject']['codcliente']);
         
-        $data['form']['codsubtipodoc'] =(isset($data['form']['codsubtipodoc']))?$cliente->codsubtipodoc:"02";
-        $data['form']['codoperaciondoc'] = (isset($data['form']['codoperaciondoc']))?"01":"LIMPIO";
-        $data['form']['ncftipopago'] = (!isset($data['form']['ncftipopago']))?$cliente->ncftipopago:"";
+        $data['form']['codsubtipodoc'] = (isset($data['form']['codsubtipodoc'])) ? $cliente->codsubtipodoc : "02";
+        $data['form']['codoperaciondoc'] = (isset($data['form']['codoperaciondoc'])) ? "01" : "LIMPIO";
+        $data['form']['ncftipopago'] = (!isset($data['form']['ncftipopago'])) ? $cliente->ncftipopago : "";
 
         $merged = array_merge($data['custom'], $data['final'], $data['form'], $data['subject']);
         $this->views[$this->active]->loadFromData($merged);

Controller/ListNCFTipoMovimiento.php

Spacing +2 added lines, -2 removed lines

@@ -63,13 +63,13 @@
     {
         
         $this->addView('ListNCFTipoMovimiento-1', 'NCFTipoMovimiento', 'sales', 'fas fa-store');
-        $this->addSearchFields('ListNCFTipoMovimiento-1', ['tipomovimiento','codigo','descripcion']);
+        $this->addSearchFields('ListNCFTipoMovimiento-1', ['tipomovimiento', 'codigo', 'descripcion']);
         $this->addOrderBy('ListNCFTipoMovimiento-1', ['id'], 'code');
         $this->addOrderBy('ListNCFTipoMovimiento-1', ['descripcion'], 'description');
         $this->addRestoreButton('ListNCFTipoMovimiento-1');
         
         $this->addView('ListNCFTipoMovimiento-2', 'NCFTipoMovimiento', 'purchases', 'fas fa-credit-card');
-        $this->addSearchFields('ListNCFTipoMovimiento-2', ['tipomovimiento','codigo','descripcion']);
+        $this->addSearchFields('ListNCFTipoMovimiento-2', ['tipomovimiento', 'codigo', 'descripcion']);
         $this->addOrderBy('ListNCFTipoMovimiento-2', ['id'], 'code');
         $this->addOrderBy('ListNCFTipoMovimiento-2', ['descripcion'], 'description');
         $this->addRestoreButton('ListNCFTipoMovimiento-2');

Extension/Controller/EditFacturaCliente.php

Spacing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
 {
     public function createViews()
     {
-        return function () {
+        return function() {
 //            $ncfTipo = new NCFTipo();
 //            $ncfTipos = $ncfTipo->allFor('ventas', 'suma');
 //            $customValues = [];
@@ -43,11 +43,11 @@ 
             $ncfTiposPago = $ncfTipoPago->findAllByTipopago('01');
             $customValuesNTP = [];
             $customValuesNTP[] = ['value'=>'', 'title'=>'-----------'];
-            foreach($ncfTiposPago as $tipopago) {
+            foreach ($ncfTiposPago as $tipopago) {
                 $customValuesNTP[] = ['value'=>$tipopago->codigo, 'title'=>$tipopago->descripcion];
             }
             $columnToModifyNTP = $this->views['EditFacturaCliente']->columnForName('ncf-payment-type');
-            if($columnToModifyNTP) {
+            if ($columnToModifyNTP) {
                 $columnToModifyNTP->widget->setValuesFromArray($customValuesNTP);
             }
 
@@ -55,11 +55,11 @@ 
             $ncfTiposAnulacion = $ncfTipoAnulacion->all();
             $customValuesNTA = [];
             $customValuesNTA[] = ['value'=>'', 'title'=>'-----------'];
-            foreach($ncfTiposAnulacion as $tipoanulacion) {
+            foreach ($ncfTiposAnulacion as $tipoanulacion) {
                 $customValuesNTA[] = ['value'=>$tipoanulacion->codigo, 'title'=>$tipoanulacion->descripcion];
             }
             $columnToModifyNTA = $this->views['EditFacturaCliente']->columnForName('ncf-cancellation-type');
-            if($columnToModifyNTA) {
+            if ($columnToModifyNTA) {
                 $columnToModifyNTA->widget->setValuesFromArray($customValuesNTA);
             }
 
@@ -67,11 +67,11 @@ 
             $ncfTiposMovimiento = $ncfTipoMovimiento->findAllByTipomovimiento('VEN');
             $customValuesNTM = [];
             $customValuesNTM[] = ['value'=>'', 'title'=>'-----------'];
-            foreach($ncfTiposMovimiento as $tipomovimiento) {
+            foreach ($ncfTiposMovimiento as $tipomovimiento) {
                 $customValuesNTM[] = ['value'=>$tipomovimiento->codigo, 'title'=>$tipomovimiento->descripcion];
             }
             $columnToModifyNTM = $this->views['EditFacturaCliente']->columnForName('ncf-movement-type');
-            if($columnToModifyNTM) {
+            if ($columnToModifyNTM) {
                 $columnToModifyNTM->widget->setValuesFromArray($customValuesNTM);
             }
         };

Extension/Controller/EditCliente.php

Spacing +5 added lines, -5 removed lines

@@ -26,16 +26,16 @@ 
 
     public function createViews()
     {
-        return function () {
+        return function() {
             $ncfTipo = new NCFTipo();
             $ncfTipos = $ncfTipo->allFor('ventas', 'suma');
             $customValues = [];
             $customValues[] = ['value'=>'', 'title'=>'-----------'];
-            foreach($ncfTipos as $tipo) {
+            foreach ($ncfTipos as $tipo) {
                 $customValues[] = ['value'=>$tipo->tipocomprobante, 'title'=>$tipo->descripcion];
             }
             $columnToModify = $this->views['EditCliente']->columnForName('codsubtipodoc');
-            if($columnToModify) {
+            if ($columnToModify) {
                 $columnToModify->widget->setValuesFromArray($customValues);
             }
 
@@ -43,11 +43,11 @@ 
             $ncfTiposPago = $ncfTipoPago->findAllByTipopago('01');
             $customValuesNTP = [];
             $customValuesNTP[] = ['value'=>'', 'title'=>'-----------'];
-            foreach($ncfTiposPago as $tipopago) {
+            foreach ($ncfTiposPago as $tipopago) {
                 $customValuesNTP[] = ['value'=>$tipopago->codigo, 'title'=>$tipopago->descripcion];
             }
             $columnToModifyNTP = $this->views['EditCliente']->columnForName('ncf-payment-types');
-            if($columnToModifyNTP) {
+            if ($columnToModifyNTP) {
                 $columnToModifyNTP->widget->setValuesFromArray($customValuesNTP);
             }
         };

Extension/Model/FacturaCliente.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
     
     public function saveInsert()
     {
-        return function () {
+        return function() {
             $ncfrango = new NCFRango();
             $cliente = new Cliente();
             $appSettins = new AppSettings;

Extension/Model/Cliente.php

Indentation +8 added lines, -8 removed lines

@@ -16,15 +16,15 @@ 
  * along with fsRepublicaDominicana. If not, see <http://www.gnu.org/licenses/>.
  */
 
- namespace FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model;
+    namespace FacturaScripts\Plugins\fsRepublicaDominicana\Extension\Model;
 
- class Cliente 
- {
+    class Cliente 
+    {
     /**
-    * Type of NCF to generate to the customer
-    *
-    * @var string
-    */
+     * Type of NCF to generate to the customer
+     *
+     * @var string
+     */
     public $codsubtipodoc;
     /**
      * Payment type based on DGII table
@@ -32,4 +32,4 @@ 
      */
     public $ncftipopago;
 
- }
+    }

Model/NCFTipoPago.php

Spacing +29 added lines, -29 removed lines

@@ -61,20 +61,20 @@ 
      * @var array
      */
     public $arrayTipos = array(
-        array ('tipopago'=>'01','codigo' => '17', 'descripcion' => 'EFECTIVO','estado'=>true),
-        array ('tipopago'=>'01','codigo' => '18', 'descripcion' => 'CHEQUES/TRANSFERENCIAS/DEPOSITO','estado'=>true),
-        array ('tipopago'=>'01','codigo' => '19', 'descripcion' => 'TARJETA CRÉDITO/DÉBITO','estado'=>true),
-        array ('tipopago'=>'01','codigo' => '20', 'descripcion' => 'VENTA A CREDITO','estado'=>true),
-        array ('tipopago'=>'01','codigo' => '21', 'descripcion' => 'BONOS O CERTIFICADOS DE REGALO','estado'=>true),
-        array ('tipopago'=>'01','codigo' => '22', 'descripcion' => 'PERMUTA','estado'=>true),
-        array ('tipopago'=>'01','codigo' => '23', 'descripcion' => 'OTRAS FORMAS DE VENTAS','estado'=>true),
-        array ('tipopago'=>'02','codigo' => '01', 'descripcion' => 'EFECTIVO','estado'=>true),
-        array ('tipopago'=>'02','codigo' => '02', 'descripcion' => 'CHEQUES/TRANSFERENCIAS/DEPOSITO','estado'=>true),
-        array ('tipopago'=>'02','codigo' => '03', 'descripcion' => 'TARJETA CRÉDITO/DÉBITO','estado'=>true),
-        array ('tipopago'=>'02','codigo' => '04', 'descripcion' => 'COMPRA A CREDITO','estado'=>true),
-        array ('tipopago'=>'02','codigo' => '05', 'descripcion' => 'PERMUTA','estado'=>true),
-        array ('tipopago'=>'02','codigo' => '06', 'descripcion' => 'NOTA DE CREDITO','estado'=>true),
-        array ('tipopago'=>'02','codigo' => '07', 'descripcion' => 'MIXTO','estado'=>true)
+        array('tipopago'=>'01', 'codigo' => '17', 'descripcion' => 'EFECTIVO', 'estado'=>true),
+        array('tipopago'=>'01', 'codigo' => '18', 'descripcion' => 'CHEQUES/TRANSFERENCIAS/DEPOSITO', 'estado'=>true),
+        array('tipopago'=>'01', 'codigo' => '19', 'descripcion' => 'TARJETA CRÉDITO/DÉBITO', 'estado'=>true),
+        array('tipopago'=>'01', 'codigo' => '20', 'descripcion' => 'VENTA A CREDITO', 'estado'=>true),
+        array('tipopago'=>'01', 'codigo' => '21', 'descripcion' => 'BONOS O CERTIFICADOS DE REGALO', 'estado'=>true),
+        array('tipopago'=>'01', 'codigo' => '22', 'descripcion' => 'PERMUTA', 'estado'=>true),
+        array('tipopago'=>'01', 'codigo' => '23', 'descripcion' => 'OTRAS FORMAS DE VENTAS', 'estado'=>true),
+        array('tipopago'=>'02', 'codigo' => '01', 'descripcion' => 'EFECTIVO', 'estado'=>true),
+        array('tipopago'=>'02', 'codigo' => '02', 'descripcion' => 'CHEQUES/TRANSFERENCIAS/DEPOSITO', 'estado'=>true),
+        array('tipopago'=>'02', 'codigo' => '03', 'descripcion' => 'TARJETA CRÉDITO/DÉBITO', 'estado'=>true),
+        array('tipopago'=>'02', 'codigo' => '04', 'descripcion' => 'COMPRA A CREDITO', 'estado'=>true),
+        array('tipopago'=>'02', 'codigo' => '05', 'descripcion' => 'PERMUTA', 'estado'=>true),
+        array('tipopago'=>'02', 'codigo' => '06', 'descripcion' => 'NOTA DE CREDITO', 'estado'=>true),
+        array('tipopago'=>'02', 'codigo' => '07', 'descripcion' => 'MIXTO', 'estado'=>true)
     );
     
     /**
@@ -102,27 +102,27 @@ 
     public function install() 
     {
         parent::install();
-        return "INSERT INTO rd_ncftipopagos (tipopago, codigo, descripcion, estado) VALUES " .
-            "('01','17','EFECTIVO',true), " .
-            "('01','18','CHEQUES/TRANSFERENCIAS/DEPOSITO',true), " .
-            "('01','19','TARJETA CRÉDITO/DÉBITO',true), " .
-            "('01','20','VENTA A CRÉDITO',true), " .
-            "('01','21','BONOS O CERTIFICADOS DE REGALO',true), " .
-            "('01','22','PERMUTA',true), " .
-            "('01','23','OTRAS FORMAS DE VENTAS',true), " .
-            "('02','01','EFECTIVO',true), " .
-            "('02','02','CHEQUES/TRANSFERENCIAS/DEPOSITO',true), " .
-            "('02','03','TARJETA CRÉDITO/DÉBITO',true), " .
-            "('02','04','COMPRA A CREDITO',true), " .
-            "('02','05','PERMUTA',true), " .
-            "('02','06','NOTA DE CREDITO',true), " .
+        return "INSERT INTO rd_ncftipopagos (tipopago, codigo, descripcion, estado) VALUES ".
+            "('01','17','EFECTIVO',true), ".
+            "('01','18','CHEQUES/TRANSFERENCIAS/DEPOSITO',true), ".
+            "('01','19','TARJETA CRÉDITO/DÉBITO',true), ".
+            "('01','20','VENTA A CRÉDITO',true), ".
+            "('01','21','BONOS O CERTIFICADOS DE REGALO',true), ".
+            "('01','22','PERMUTA',true), ".
+            "('01','23','OTRAS FORMAS DE VENTAS',true), ".
+            "('02','01','EFECTIVO',true), ".
+            "('02','02','CHEQUES/TRANSFERENCIAS/DEPOSITO',true), ".
+            "('02','03','TARJETA CRÉDITO/DÉBITO',true), ".
+            "('02','04','COMPRA A CREDITO',true), ".
+            "('02','05','PERMUTA',true), ".
+            "('02','06','NOTA DE CREDITO',true), ".
             "('02','07','MIXTO',true);";
     }
     
     public function restoreData()
     {
         $dataBase = new DataBase();
-        $sqlClean = "DELETE FROM " . $this->tableName() . ";";
+        $sqlClean = "DELETE FROM ".$this->tableName().";";
         $dataBase->exec($sqlClean);
         foreach ($this->arrayTipos as $arrayItem) {
             $initialData = new NCFTipoMovimiento($arrayItem);

Model/NCFTipoMovimiento.php

Spacing +19 added lines, -19 removed lines

@@ -61,23 +61,23 @@ 
      * @var array
      */
     private $arrayTiposMovimiento = array(
-        array ('tipomovimiento'=>'COM','codigo' => '01', 'descripcion' => 'GASTOS DE PERSONAL', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '02', 'descripcion' => 'GASTOS POR TRABAJOS, SUMINISTROS Y SERVICIOS', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '03', 'descripcion' => 'ARRENDAMIENTOS', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '04', 'descripcion' => 'GASTOS DE ACTIVOS FIJOS', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '05', 'descripcion' => 'GASTOS DE REPRESENTACIÓN', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '06', 'descripcion' => 'OTRAS DEDUCCIONES ADMITIDAS', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '07', 'descripcion' => 'GASTOS FINANCIEROS', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '08', 'descripcion' => 'GASTOS EXTRAORDINARIOS', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '09', 'descripcion' => 'COMPRAS Y GASTOS QUE FORMARÁN PARTE DEL COSTO DE VENTA', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '10', 'descripcion' => 'ADQUISICIONES DE ACTIVOS', 'estado' => true),
-        array ('tipomovimiento'=>'COM','codigo' => '11', 'descripcion' => 'GASTOS DE SEGUROS', 'estado' => true),
-        array ('tipomovimiento'=>'VEN','codigo' => '1', 'descripcion' => 'Ingresos por operaciones (No financieros)', 'estado' => true),
-        array ('tipomovimiento'=>'VEN','codigo' => '2', 'descripcion' => 'Ingresos Financieros', 'estado' => true),
-        array ('tipomovimiento'=>'VEN','codigo' => '3', 'descripcion' => 'Ingresos Extraordinarios', 'estado' => true),
-        array ('tipomovimiento'=>'VEN','codigo' => '4', 'descripcion' => 'Ingresos por Arrendamientos', 'estado' => true),
-        array ('tipomovimiento'=>'VEN','codigo' => '5', 'descripcion' => 'Ingresos por Venta de Activo Depreciable', 'estado' => true),
-        array ('tipomovimiento'=>'VEN','codigo' => '6', 'descripcion' => 'Otros Ingresos', 'estado' => true)
+        array('tipomovimiento'=>'COM', 'codigo' => '01', 'descripcion' => 'GASTOS DE PERSONAL', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '02', 'descripcion' => 'GASTOS POR TRABAJOS, SUMINISTROS Y SERVICIOS', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '03', 'descripcion' => 'ARRENDAMIENTOS', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '04', 'descripcion' => 'GASTOS DE ACTIVOS FIJOS', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '05', 'descripcion' => 'GASTOS DE REPRESENTACIÓN', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '06', 'descripcion' => 'OTRAS DEDUCCIONES ADMITIDAS', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '07', 'descripcion' => 'GASTOS FINANCIEROS', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '08', 'descripcion' => 'GASTOS EXTRAORDINARIOS', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '09', 'descripcion' => 'COMPRAS Y GASTOS QUE FORMARÁN PARTE DEL COSTO DE VENTA', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '10', 'descripcion' => 'ADQUISICIONES DE ACTIVOS', 'estado' => true),
+        array('tipomovimiento'=>'COM', 'codigo' => '11', 'descripcion' => 'GASTOS DE SEGUROS', 'estado' => true),
+        array('tipomovimiento'=>'VEN', 'codigo' => '1', 'descripcion' => 'Ingresos por operaciones (No financieros)', 'estado' => true),
+        array('tipomovimiento'=>'VEN', 'codigo' => '2', 'descripcion' => 'Ingresos Financieros', 'estado' => true),
+        array('tipomovimiento'=>'VEN', 'codigo' => '3', 'descripcion' => 'Ingresos Extraordinarios', 'estado' => true),
+        array('tipomovimiento'=>'VEN', 'codigo' => '4', 'descripcion' => 'Ingresos por Arrendamientos', 'estado' => true),
+        array('tipomovimiento'=>'VEN', 'codigo' => '5', 'descripcion' => 'Ingresos por Venta de Activo Depreciable', 'estado' => true),
+        array('tipomovimiento'=>'VEN', 'codigo' => '6', 'descripcion' => 'Otros Ingresos', 'estado' => true)
     );
     
     /**
@@ -105,7 +105,7 @@ 
     public function install() 
     {
         parent::install();
-        return "INSERT INTO rd_ncftipomovimiento (tipomovimiento, codigo, descripcion, estado) VALUES " .
+        return "INSERT INTO rd_ncftipomovimiento (tipomovimiento, codigo, descripcion, estado) VALUES ".
             "('COM','01','GASTOS DE PERSONAL',true), ".
             "('COM','02','GASTOS POR TRABAJOS, SUMINISTROS Y SERVICIOS',true), ".
             "('COM','03','ARRENDAMIENTOS',true), ".
@@ -128,7 +128,7 @@ 
     public function restoreData()
     {
         $dataBase = new DataBase();
-        $sqlClean = "DELETE FROM " . $this->tableName() . ";";
+        $sqlClean = "DELETE FROM ".$this->tableName().";";
         $dataBase->exec($sqlClean);
         foreach ($this->arrayTiposMovimiento as $arrayItem) {
             $initialData = new NCFTipoMovimiento($arrayItem);