libredte / libredte-lib-core

lib/Sii/Aduana.php

Braces +1 added lines, -2 removed lines

@@ -852,8 +852,7 @@
             $valor = $codigo['CantBultos'].' '.$valor;
             if (!empty($codigo['IdContainer'])) {
                 $valor .= ' ('.$codigo['IdContainer'].' / '.$codigo['Sello'].' / '.$codigo['EmisorSello'].')';
-            }
-            else if (!empty($codigo['Marcas'])) {
+            } else if (!empty($codigo['Marcas'])) {
                 $valor .= ' ('.$codigo['Marcas'].')';
             }
         } else {

lib/Sii/ConsumoFolio.php

Spacing +4 added lines, -4 removed lines

@@ -126,7 +126,7 @@ 
     {
         $fecha = '9999-12-31';
         foreach ($this->detalles as &$d) {
-            if ($d['FchDoc'] < $fecha) {
+            if ($d['FchDoc']<$fecha) {
                 $fecha = $d['FchDoc'];
             }
         }
@@ -143,7 +143,7 @@ 
     {
         $fecha = '0000-01-01';
         foreach ($this->detalles as &$d) {
-            if ($d['FchDoc'] > $fecha) {
+            if ($d['FchDoc']>$fecha) {
                 $fecha = $d['FchDoc'];
             }
         }
@@ -202,7 +202,7 @@ 
         // ajustes post agregar detalles
         foreach ($Resumen as &$r) {
             // obtener folios utilizados = emitidos + anulados
-            $r['FoliosUtilizados'] = $r['FoliosEmitidos'] + $r['FoliosAnulados'];
+            $r['FoliosUtilizados'] = $r['FoliosEmitidos']+$r['FoliosAnulados'];
             $r['RangoUtilizados'] = $this->getRangos($RangoUtilizados[$r['TipoDocumento']]);
         }
         // completar con los resumenes que no se colocaron
@@ -234,7 +234,7 @@ 
         $aux = [];
         $inicial = $folios[0];
         $i = $inicial;
-        foreach($folios as $f) {
+        foreach ($folios as $f) {
             if ($i!=$f) {
                 $inicial = $f;
                 $i = $inicial;

lib/Sii/Dte/PDF/Dte.php

Braces +18 added lines, -12 removed lines

@@ -770,10 +770,12 @@ 
             $this->MultiTexto($receptor['Extranjero']['NumId'], $x+$offset+2);
         }
         $contacto = [];
-        if (!empty($receptor['Contacto']))
-            $contacto[] = $receptor['Contacto'];
-        if (!empty($receptor['CorreoRecep']))
-            $contacto[] = $receptor['CorreoRecep'];
+        if (!empty($receptor['Contacto'])) {
+                    $contacto[] = $receptor['Contacto'];
+        }
+        if (!empty($receptor['CorreoRecep'])) {
+                    $contacto[] = $receptor['CorreoRecep'];
+        }
         if (!empty($contacto)) {
             $this->setFont('', 'B', null);
             $this->Texto('Contacto', $x);
@@ -823,10 +825,12 @@ 
             if (!empty($Transporte['DirDest']) and !empty($Transporte['CmnaDest'])) {
                 $transporte .= 'a '.$Transporte['DirDest'].', '.$Transporte['CmnaDest'];
             }
-            if (!empty($Transporte['RUTTrans']))
-                $transporte .= ' por '.$Transporte['RUTTrans'];
-            if (!empty($Transporte['Patente']))
-                $transporte .= ' en vehículo '.$Transporte['Patente'];
+            if (!empty($Transporte['RUTTrans'])) {
+                            $transporte .= ' por '.$Transporte['RUTTrans'];
+            }
+            if (!empty($Transporte['Patente'])) {
+                            $transporte .= ' en vehículo '.$Transporte['Patente'];
+            }
             if (isset($Transporte['Chofer']) and is_array($Transporte['Chofer'])) {
                 if (!empty($Transporte['Chofer']['NombreChofer'])) {
                     $transporte .= ' con chofer '.$Transporte['Chofer']['NombreChofer'];
@@ -889,8 +893,9 @@ 
      */
     protected function agregarReferencia($referencias, $x = 10, $offset = 22)
     {
-        if (!isset($referencias[0]))
-            $referencias = [$referencias];
+        if (!isset($referencias[0])) {
+                    $referencias = [$referencias];
+        }
         foreach($referencias as $r) {
             $texto = $r['NroLinRef'].' - ';
             if (!empty($r['TpoDocRef'])) {
@@ -1124,8 +1129,9 @@ 
      */
     protected function agregarPagos(array $pagos, $x = 10)
     {
-        if (!isset($pagos[0]))
-            $pagos = [$pagos];
+        if (!isset($pagos[0])) {
+                    $pagos = [$pagos];
+        }
         $this->Texto('Pago(s) programado(s):', $x);
         $this->Ln();
         foreach($pagos as $p) {

Doc Comments +32 added lines, -28 removed lines

@@ -80,7 +80,7 @@ 
     /**
      * Método que asigna la ubicación del logo de la empresa
      * @param logo URI del logo (puede ser local o en una URL)
-     * @param posicion Posición respecto a datos del emisor (=0 izq, =1 arriba). Nota: parámetro válido sólo para formato hoja carta
+     * @param posicion integer respecto a datos del emisor (=0 izq, =1 arriba). Nota: parámetro válido sólo para formato hoja carta
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-04
      */
@@ -333,7 +333,7 @@ 
      * contínuo de 80mm
      * @param dte Arreglo con los datos del XML (tag Documento)
      * @param timbre String XML con el tag TED del DTE
-     * @param width Ancho del papel contínuo en mm (es parámetro porque se usa el mismo método para 75mm)
+     * @param width integer del papel contínuo en mm (es parámetro porque se usa el mismo método para 75mm)
      * @author Pablo Reyes (https://github.com/pabloxp)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2020-03-11
@@ -491,10 +491,11 @@ 
      *  - Dirección casa central del emisor
      *  - Dirección sucursales
      * @param emisor Arreglo con los datos del emisor (tag Emisor del XML)
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho de la información del emisor
-     * @param w_img Ancho máximo de la imagen
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer de la información del emisor
+     * @param w_img integer máximo de la imagen
+     * @param integer $font_size
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-11-22
      */
@@ -603,11 +604,13 @@ 
      * @param rut RUT del emisor
      * @param tipo Código o glosa del tipo de documento
      * @param sucursal_sii Código o glosa de la sucursal del SII del Emisor
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho de la información del emisor
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer de la información del emisor
+     * @param integer $font_size
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-08-05
+     * @return integer
      */
     protected function agregarFolio($rut, $tipo, $folio, $sucursal_sii = null, $x = 130, $y = 15, $w = 70, $font_size = null, array $color = null)
     {
@@ -639,7 +642,7 @@ 
      * Método que agrega los datos de la emisión del DTE que no son los dato del
      * receptor
      * @param IdDoc Información general del documento
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2020-08-11
      */
@@ -727,7 +730,7 @@ 
     /**
      * Método que agrega los datos del receptor
      * @param receptor Arreglo con los datos del receptor (tag Receptor del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-10-06
      */
@@ -814,7 +817,7 @@ 
      * Método que agrega los datos del traslado
      * @param IndTraslado
      * @param Transporte
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-03
      */
@@ -894,7 +897,7 @@ 
     /**
      * Método que agrega las referencias del documento
      * @param referencias Arreglo con las referencias del documento (tag Referencia del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-09-25
      */
@@ -931,7 +934,7 @@ 
     /**
      * Método que agrega el detalle del documento
      * @param detalle Arreglo con el detalle del documento (tag Detalle del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @param y Posición vertical de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2020-08-28
@@ -1041,7 +1044,7 @@ 
     /**
      * Método que agrega el detalle del documento
      * @param detalle Arreglo con el detalle del documento (tag Detalle del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @param y Posición vertical de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @author Pablo Reyes (https://github.com/pabloxp)
@@ -1097,7 +1100,7 @@ 
      * Método que agrega el subtotal del DTE
      * @param detalle Arreglo con los detalles del documentos para poder
      * calcular subtotal
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-17
      */
@@ -1123,7 +1126,7 @@ 
     /**
      * Método que agrega los descuentos y/o recargos globales del documento
      * @param descuentosRecargos Arreglo con los descuentos y/o recargos del documento (tag DscRcgGlobal del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2018-05-29
      */
@@ -1151,7 +1154,7 @@ 
     /**
      * Método que agrega los pagos del documento
      * @param pagos Arreglo con los pagos del documento
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-07-24
      */
@@ -1309,9 +1312,10 @@ 
      *  - Se imprime en el tamaño mínimo: 2x5 cms
      *  - En el lado de abajo con margen izquierdo mínimo de 2 cms
      * @param timbre String con los datos del timbre
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho del timbre
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer del timbre
+     * @param string $position
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-10-06
      */
@@ -1349,10 +1353,10 @@ 
 
     /**
      * Método que agrega el acuse de rebido
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho del acuse de recibo
-     * @param h Alto del acuse de recibo
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer del acuse de recibo
+     * @param h integer del acuse de recibo
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-10-06
      */
@@ -1380,10 +1384,10 @@ 
 
     /**
      * Método que agrega el acuse de rebido
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho del acuse de recibo
-     * @param h Alto del acuse de recibo
+     * @param w integer del acuse de recibo
+     * @param h integer del acuse de recibo
      * @author Pablo Reyes (https://github.com/pabloxp)
      * @version 2015-11-17
      */

Spacing +57 added lines, -57 removed lines

@@ -74,7 +74,7 @@ 
     {
         parent::__construct();
         $this->SetTitle('Documento Tributario Electrónico (DTE) de Chile by LibreDTE');
-        $this->papelContinuo = $papelContinuo === true ? 80 : $papelContinuo;
+        $this->papelContinuo = $papelContinuo===true ? 80 : $papelContinuo;
     }
 
     /**
@@ -193,7 +193,7 @@ 
         $this->Ln();
         // datos del documento
         $y = [];
-        $y[] = $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor'])?$dte['Encabezado']['Emisor']['CdgVendedor']:null);
+        $y[] = $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor']) ? $dte['Encabezado']['Emisor']['CdgVendedor'] : null);
         $y[] = $this->agregarReceptor($dte['Encabezado']);
         $this->setY(max($y));
         $this->agregarTraslado(
@@ -242,7 +242,7 @@ 
         $this->AddPage('P', [$height ? $height : $this->papel_continuo_alto, $width]);
         $x = 1;
         $y = 5;
-        $this->SetXY($x,$y);
+        $this->SetXY($x, $y);
         // agregar datos del documento
         $this->setFont('', '', 8);
         $this->MultiTexto(!empty($dte['Encabezado']['Emisor']['RznSoc']) ? $dte['Encabezado']['Emisor']['RznSoc'] : $dte['Encabezado']['Emisor']['RznSocEmisor'], $x, null, '', $width-2);
@@ -354,15 +354,15 @@ 
             $dte['Encabezado']['IdDoc']['TipoDTE'],
             $dte['Encabezado']['IdDoc']['Folio'],
             $dte['Encabezado']['Emisor']['CmnaOrigen'],
-            $x_start, $y_start, $width-($x_start*4), 10,
-            [0,0,0]
+            $x_start, $y_start, $width-($x_start * 4), 10,
+            [0, 0, 0]
         );
-        $y = $this->agregarEmisor($dte['Encabezado']['Emisor'], $x_start, $y+2, $width-($x_start*45), 8, 9, [0,0,0]);
+        $y = $this->agregarEmisor($dte['Encabezado']['Emisor'], $x_start, $y+2, $width-($x_start * 45), 8, 9, [0, 0, 0]);
         // datos del documento
         $this->SetY($y);
         $this->Ln();
         $this->setFont('', '', 8);
-        $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor'])?$dte['Encabezado']['Emisor']['CdgVendedor']:null, $x_start, $offset, false);
+        $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor']) ? $dte['Encabezado']['Emisor']['CdgVendedor'] : null, $x_start, $offset, false);
         $this->agregarReceptor($dte['Encabezado'], $x_start, $offset);
         $this->agregarTraslado(
             !empty($dte['Encabezado']['IdDoc']['IndTraslado']) ? $dte['Encabezado']['IdDoc']['IndTraslado'] : null,
@@ -432,14 +432,14 @@ 
             2,
             45,
             9,
-            [0,0,0]
+            [0, 0, 0]
         );
-        $y[] = $this->agregarEmisor($dte['Encabezado']['Emisor'], 1, 2, 20, 30, 9, [0,0,0], $y[0]);
+        $y[] = $this->agregarEmisor($dte['Encabezado']['Emisor'], 1, 2, 20, 30, 9, [0, 0, 0], $y[0]);
         $this->SetY(max($y));
         $this->Ln();
         // datos del documento
         $this->setFont('', '', 8);
-        $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor'])?$dte['Encabezado']['Emisor']['CdgVendedor']:null, $x_start, $offset, false);
+        $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor']) ? $dte['Encabezado']['Emisor']['CdgVendedor'] : null, $x_start, $offset, false);
         $this->agregarReceptor($dte['Encabezado'], $x_start, $offset);
         $this->agregarTraslado(
             !empty($dte['Encabezado']['IdDoc']['IndTraslado']) ? $dte['Encabezado']['IdDoc']['IndTraslado'] : null,
@@ -471,7 +471,7 @@ 
         if ($this->cedible and !in_array($dte['Encabezado']['IdDoc']['TipoDTE'], $this->sinAcuseRecibo)) {
             $this->agregarAcuseRecibo(63, $y+6, 45, 40, 15);
             $this->setFont('', 'B', 8);
-            $this->Texto('CEDIBLE'.($dte['Encabezado']['IdDoc']['TipoDTE']==52?' CON SU FACTURA':''), $x_start, $this->y+1, 'L');
+            $this->Texto('CEDIBLE'.($dte['Encabezado']['IdDoc']['TipoDTE']==52 ? ' CON SU FACTURA' : ''), $x_start, $this->y+1, 'L');
         }
         // si el alto no se pasó, entonces es con autocálculo, se elimina esta página y se pasa el alto
         // que se logró determinar para crear la página con el alto correcto
@@ -502,16 +502,16 @@ 
         // logo del documento
         if (isset($this->logo)) {
             // logo centrado (papel continuo)
-            if (!empty($this->logo['posicion']) and $this->logo['posicion'] == 'C') {
+            if (!empty($this->logo['posicion']) and $this->logo['posicion']=='C') {
                 $logo_w = null;
                 $logo_y = null;
                 $logo_position = 'C';
                 $logo_next_pointer = 'N';
             }
             // logo a la derecha (posicion=0) o arriba (posicion=1)
-            else if (empty($this->logo['posicion']) or $this->logo['posicion'] == 1) {
+            else if (empty($this->logo['posicion']) or $this->logo['posicion']==1) {
                 $logo_w = !$this->logo['posicion'] ? $w_img : null;
-                $logo_y = $this->logo['posicion'] ? $w_img/2 : null;
+                $logo_y = $this->logo['posicion'] ? $w_img / 2 : null;
                 $logo_position = '';
                 $logo_next_pointer = 'T';
             }
@@ -530,17 +530,17 @@ 
                 $logo_w,
                 $logo_y,
                 'PNG',
-                (isset($emisor['url'])?$emisor['url']:''),
+                (isset($emisor['url']) ? $emisor['url'] : ''),
                 $logo_next_pointer,
                 2,
                 300,
                 $logo_position
             );
-            if (!empty($this->logo['posicion']) and $this->logo['posicion'] == 'C') {
+            if (!empty($this->logo['posicion']) and $this->logo['posicion']=='C') {
                 $w += 40;
             } else {
                 if ($this->logo['posicion']) {
-                    $this->SetY($this->y + ($w_img/2));
+                    $this->SetY($this->y+($w_img / 2));
                     $w += 40;
                 } else {
                     $x = $this->x+3;
@@ -553,19 +553,19 @@ 
         // agregar datos del emisor
         if ($agregarDatosEmisor) {
             $this->setFont('', 'B', $font_size ? $font_size : 14);
-            $this->SetTextColorArray($color===null?[32, 92, 144]:$color);
-            $this->MultiTexto(!empty($emisor['RznSoc']) ? $emisor['RznSoc'] : $emisor['RznSocEmisor'], $x, $this->y+2, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+            $this->SetTextColorArray($color===null ? [32, 92, 144] : $color);
+            $this->MultiTexto(!empty($emisor['RznSoc']) ? $emisor['RznSoc'] : $emisor['RznSocEmisor'], $x, $this->y+2, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             $this->setFont('', 'B', $font_size ? $font_size : 9);
-            $this->SetTextColorArray([0,0,0]);
-            $this->MultiTexto(!empty($emisor['GiroEmis']) ? $emisor['GiroEmis'] : $emisor['GiroEmisor'], $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+            $this->SetTextColorArray([0, 0, 0]);
+            $this->MultiTexto(!empty($emisor['GiroEmis']) ? $emisor['GiroEmis'] : $emisor['GiroEmisor'], $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             $comuna = !empty($emisor['CmnaOrigen']) ? $emisor['CmnaOrigen'] : null;
             $ciudad = !empty($emisor['CiudadOrigen']) ? $emisor['CiudadOrigen'] : \sasco\LibreDTE\Chile::getCiudad($comuna);
-            $this->MultiTexto($emisor['DirOrigen'].($comuna?(', '.$comuna):'').($ciudad?(', '.$ciudad):''), $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+            $this->MultiTexto($emisor['DirOrigen'].($comuna ? (', '.$comuna) : '').($ciudad ? (', '.$ciudad) : ''), $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             if (!empty($emisor['Sucursal'])) {
-                $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+                $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             }
             if (!empty($this->casa_matriz)) {
-                $this->MultiTexto('Casa matriz: '.$this->casa_matriz, $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+                $this->MultiTexto('Casa matriz: '.$this->casa_matriz, $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             }
             $contacto = [];
             if (!empty($emisor['Telefono'])) {
@@ -580,7 +580,7 @@ 
                 $contacto[] = $emisor['CorreoEmisor'];
             }
             if ($contacto) {
-                $this->MultiTexto(implode(' / ', $contacto), $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+                $this->MultiTexto(implode(' / ', $contacto), $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             }
         }
         return $this->y;
@@ -610,12 +610,12 @@ 
     protected function agregarFolio($rut, $tipo, $folio, $sucursal_sii = null, $x = 130, $y = 15, $w = 70, $font_size = null, array $color = null)
     {
         if ($color===null) {
-            $color = $tipo ? ($tipo==52 ? [0,172,140] : [255,0,0]) : [0,0,0];
+            $color = $tipo ? ($tipo==52 ? [0, 172, 140] : [255, 0, 0]) : [0, 0, 0];
         }
         $this->SetTextColorArray($color);
         // colocar rut emisor, glosa documento y folio
         list($rut, $dv) = explode('-', $rut);
-        $this->setFont ('', 'B', $font_size ? $font_size : 15);
+        $this->setFont('', 'B', $font_size ? $font_size : 15);
         $this->MultiTexto('R.U.T.: '.$this->num($rut).'-'.$dv, $x, $y+4, 'C', $w);
         $this->setFont('', 'B', $font_size ? $font_size : 12);
         $this->MultiTexto($this->getTipo($tipo, $folio), $x, null, 'C', $w);
@@ -628,7 +628,7 @@ 
         if ($tipo) {
             $this->Texto('S.I.I. - '.\sasco\LibreDTE\Sii::getDireccionRegional($sucursal_sii), $x, $this->getY()+4, 'C', $w);
         }
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->Ln();
         return $this->y;
     }
@@ -742,10 +742,10 @@ 
         }
         if (!empty($receptor['RznSocRecep'])) {
             $this->setFont('', 'B', null);
-            $this->Texto(in_array($this->dte, [39, 41]) ? 'Nombre' : ($x==10?'Razón social':'Razón soc.'), $x);
+            $this->Texto(in_array($this->dte, [39, 41]) ? 'Nombre' : ($x==10 ? 'Razón social' : 'Razón soc.'), $x);
             $this->Texto(':', $x+$offset);
             $this->setFont('', '', null);
-            $this->MultiTexto($receptor['RznSocRecep'], $x+$offset+2, null, '', $x==10?105:0);
+            $this->MultiTexto($receptor['RznSocRecep'], $x+$offset+2, null, '', $x==10 ? 105 : 0);
         }
         if (!empty($receptor['GiroRecep'])) {
             $this->setFont('', 'B', null);
@@ -762,7 +762,7 @@ 
             $ciudad = !empty($receptor['CiudadRecep']) ? $receptor['CiudadRecep'] : (
                 !empty($receptor['CmnaRecep']) ? \sasco\LibreDTE\Chile::getCiudad($receptor['CmnaRecep']) : ''
             );
-            $this->MultiTexto($receptor['DirRecep'].(!empty($receptor['CmnaRecep'])?(', '.$receptor['CmnaRecep']):'').($ciudad?(', '.$ciudad):''), $x+$offset+2);
+            $this->MultiTexto($receptor['DirRecep'].(!empty($receptor['CmnaRecep']) ? (', '.$receptor['CmnaRecep']) : '').($ciudad ? (', '.$ciudad) : ''), $x+$offset+2);
         }
         if (!empty($receptor['Extranjero']['Nacionalidad'])) {
             $this->setFont('', 'B', null);
@@ -803,7 +803,7 @@ 
             $this->Texto('Cód. recep.', $x);
             $this->Texto(':', $x+$offset);
             $this->setFont('', '', null);
-            $this->MultiTexto($receptor['CdgIntRecep'], $x+$offset+2, null, '', $x==10?105:0);
+            $this->MultiTexto($receptor['CdgIntRecep'], $x+$offset+2, null, '', $x==10 ? 105 : 0);
         }
         return $this->GetY();
     }
@@ -900,7 +900,7 @@ 
     {
         if (!isset($referencias[0]))
             $referencias = [$referencias];
-        foreach($referencias as $r) {
+        foreach ($referencias as $r) {
             $texto = $r['NroLinRef'].' - ';
             if (!empty($r['TpoDocRef'])) {
                 $texto .= $this->getTipo($r['TpoDocRef']).' ';
@@ -961,7 +961,7 @@ 
             // quitar columnas
             foreach ($item as $col => $valor) {
                 if ($col=='DscItem' and !empty($item['DscItem'])) {
-                    $item['NmbItem'] .= !$this->item_detalle_posicion ? ($html?'<br/>':"\n") : ': ';
+                    $item['NmbItem'] .= !$this->item_detalle_posicion ? ($html ? '<br/>' : "\n") : ': ';
                     $item['DscItem'] = $html ? str_replace("\n", '<br/>', $item['DscItem']) : str_replace(['<br/>', '<br>'], "\n", $item['DscItem']);
                     if ($html) {
                         $item['NmbItem'] .= '<span style="font-size:0.7em">'.$item['DscItem'].'</span>';
@@ -1010,7 +1010,7 @@ 
             }
             $item = array_merge($item_default, $item);
             // si hay código de item se extrae su valor
-            if (!empty($item['CdgItem']['VlrCodigo'])){
+            if (!empty($item['CdgItem']['VlrCodigo'])) {
                 $item['CdgItem'] = $item['CdgItem']['VlrCodigo'];
             }
             // dar formato a números
@@ -1053,9 +1053,9 @@ 
         $this->SetY($this->getY()+1);
         $p1x = $x;
         $p1y = $this->y;
-        $p2x = $this->getPageWidth() - 2;
-        $p2y = $p1y;  // Use same y for a straight line
-        $style = array('width' => 0.2,'color' => array(0, 0, 0));
+        $p2x = $this->getPageWidth()-2;
+        $p2y = $p1y; // Use same y for a straight line
+        $style = array('width' => 0.2, 'color' => array(0, 0, 0));
         $this->Line($p1x, $p1y, $p2x, $p2y, $style);
         $this->Texto($this->detalle_cols['NmbItem']['title'], $x+$offsets[0], $this->y, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
         $this->Texto($this->detalle_cols['PrcItem']['title'], $x+$offsets[1], $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
@@ -1067,7 +1067,7 @@ 
         }
         // mostrar items
         $this->SetY($this->getY()+2);
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             // nombre y descripción del item
             $item = $d['NmbItem'];
             if ($this->papel_continuo_item_detalle and !empty($d['DscItem'])) {
@@ -1077,14 +1077,14 @@ 
             // descuento
             if (!empty($d['DescuentoPct']) or !empty($d['DescuentoMonto'])) {
                 if (!empty($d['DescuentoPct'])) {
-                    $descuento = number_format($d['DescuentoPct'],0,',','.').'%';
+                    $descuento = number_format($d['DescuentoPct'], 0, ',', '.').'%';
                 } else {
-                    $descuento = number_format($d['DescuentoMonto'],0,',','.');
+                    $descuento = number_format($d['DescuentoMonto'], 0, ',', '.');
                 }
                 $this->Texto('Desc.: '.$descuento, $x+$offsets[0], $this->y, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
             }
             // precio y cantidad
-            $this->Texto(number_format($d['PrcItem'],0,',','.'), $x+$offsets[1], $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
+            $this->Texto(number_format($d['PrcItem'], 0, ',', '.'), $x+$offsets[1], $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
             $this->Texto($this->num($d['QtyItem']), $x+$offsets[2], $this->y, ucfirst($this->detalle_cols['QtyItem']['align'][0]), $this->detalle_cols['QtyItem']['width']);
             $this->Texto($this->num($d['MontoItem']), $x+$offsets[3], $this->y, ucfirst($this->detalle_cols['MontoItem']['align'][0]), $this->detalle_cols['MontoItem']['width']);
         }
@@ -1104,7 +1104,7 @@ 
         if (!isset($detalle[0])) {
             $detalle = [$detalle];
         }
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             if (!empty($d['MontoItem'])) {
                 $subtotal += $d['MontoItem'];
             }
@@ -1130,16 +1130,16 @@ 
         if (!isset($descuentosRecargos[0])) {
             $descuentosRecargos = [$descuentosRecargos];
         }
-        foreach($descuentosRecargos as $dr) {
+        foreach ($descuentosRecargos as $dr) {
             $tipo = $dr['TpoMov']=='D' ? 'Descuento' : 'Recargo';
             if (!empty($dr['IndExeDR'])) {
                 $tipo .= ' EX';
             }
             $valor = $dr['TpoValor']=='%' ? $dr['ValorDR'].'%' : $this->num($dr['ValorDR']);
             if ($this->papelContinuo) {
-                $this->Texto($tipo.' global: '.$valor.(!empty($dr['GlosaDR'])?(' ('.$dr['GlosaDR'].')'):''), $x);
+                $this->Texto($tipo.' global: '.$valor.(!empty($dr['GlosaDR']) ? (' ('.$dr['GlosaDR'].')') : ''), $x);
             } else {
-                $this->Texto($tipo.(!empty($dr['GlosaDR'])?(' ('.$dr['GlosaDR'].')'):'').':', 77, null, 'R', 100);
+                $this->Texto($tipo.(!empty($dr['GlosaDR']) ? (' ('.$dr['GlosaDR'].')') : '').':', 77, null, 'R', 100);
                 $this->Texto($valor, 177, null, 'R', 22);
             }
             $this->Ln();
@@ -1159,8 +1159,8 @@ 
             $pagos = [$pagos];
         $this->Texto('Pago(s) programado(s):', $x);
         $this->Ln();
-        foreach($pagos as $p) {
-            $this->Texto('  - '.$this->date($p['FchPago'], false).': $'.$this->num($p['MntPago']).'.-'.(!empty($p['GlosaPagos'])?(' ('.$p['GlosaPagos'].')'):''), $x);
+        foreach ($pagos as $p) {
+            $this->Texto('  - '.$this->date($p['FchPago'], false).': $'.$this->num($p['MntPago']).'.-'.(!empty($p['GlosaPagos']) ? (' ('.$p['GlosaPagos'].')') : ''), $x);
             $this->Ln();
         }
     }
@@ -1194,7 +1194,7 @@ 
             'TpoMoneda' => 'Moneda',
             'MntNeto' => 'Neto $',
             'MntExe' => 'Exento $',
-            'IVA' => 'IVA '.(!empty($totales['TasaIVA'])?(' ('.$totales['TasaIVA'].'%) '):'').'$',
+            'IVA' => 'IVA '.(!empty($totales['TasaIVA']) ? (' ('.$totales['TasaIVA'].'%) ') : '').'$',
             'IVANoRet' => 'IVA no retenido $',
             'CredEC' => 'Desc. 65% IVA $',
             'MntTotal' => 'Total $',
@@ -1211,7 +1211,7 @@ 
             if (!isset($ImptoReten[0])) {
                 $ImptoReten = [$ImptoReten];
             }
-            foreach($ImptoReten as $i) {
+            foreach ($ImptoReten as $i) {
                 $totales['ImptoReten_'.$i['TipoImp']] = $i['MontoImp'];
                 if (!empty($i['TasaImp'])) {
                     $glosas['ImptoReten_'.$i['TipoImp']] = \sasco\LibreDTE\Sii\ImpuestosAdicionales::getGlosa($i['TipoImp']).' ('.$i['TasaImp'].'%) $';
@@ -1324,9 +1324,9 @@ 
                 'vpadding' => 0,
                 'module_width' => 1, // width of a single module in points
                 'module_height' => 1, // height of a single module in points
-                'fgcolor' => [0,0,0],
+                'fgcolor' => [0, 0, 0],
                 'bgcolor' => false, // [255,255,255]
-                'position' => $position === null ? ($this->papelContinuo ? 'C' : 'S') : $position,
+                'position' => $position===null ? ($this->papelContinuo ? 'C' : 'S') : $position,
             ];
             $ecl = version_compare(phpversion(), '7.0.0', '<') ? -1 : $this->ecl;
             $this->write2DBarcode($timbre, 'PDF417,,'.$ecl, $x_timbre, $y, $w, 0, $style, 'B');
@@ -1357,7 +1357,7 @@ 
     protected function agregarAcuseRecibo($x = 83, $y = 190, $w = 60, $h = 40, $line = 25)
     {
         $y = (!$this->papelContinuo and !$this->timbre_pie) ? $this->x_fin_datos : $y;
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->Rect($x, $y, $w, $h, 'D', ['all' => ['width' => 0.1, 'color' => [0, 0, 0]]]);
         $this->setFont('', 'B', 10);
         $this->Texto('Acuse de recibo', $x, $y+1, 'C', $w);
@@ -1387,9 +1387,9 @@ 
      */
     protected function agregarAcuseReciboContinuo($x = 3, $y = null, $w = 68, $h = 40)
     {
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->Rect($x, $y, $w, $h, 'D', ['all' => ['width' => 0.1, 'color' => [0, 0, 0]]]);
-        $style = array('width' => 0.2,'color' => array(0, 0, 0));
+        $style = array('width' => 0.2, 'color' => array(0, 0, 0));
         $this->Line($x, $y+22, $w+3, $y+22, $style);
         //$this->setFont('', 'B', 10);
         //$this->Texto('Acuse de recibo', $x, $y+1, 'C', $w);
@@ -1419,7 +1419,7 @@ 
         $y = (!$this->papelContinuo and !$this->timbre_pie and $this->x_fin_datos<=$y) ? $this->x_fin_datos : $y;
         $y += 48;
         $this->setFont('', 'B', $font_size);
-        $this->Texto('CEDIBLE'.($tipo==52?' CON SU FACTURA':''), null, $y, 'R');
+        $this->Texto('CEDIBLE'.($tipo==52 ? ' CON SU FACTURA' : ''), null, $y, 'R');
     }
 
     /**
@@ -1430,7 +1430,7 @@ 
     protected function agregarLeyendaDestinoContinuo($tipo)
     {
         $this->setFont('', 'B', 8);
-        $this->Texto('CEDIBLE'.($tipo==52?' CON SU FACTURA':''), null, $this->y+6, 'R');
+        $this->Texto('CEDIBLE'.($tipo==52 ? ' CON SU FACTURA' : ''), null, $this->y+6, 'R');
     }
 
 }

lib/Sii/Base/Envio.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
             return false;
         }
         // si no se debe enviar no continuar
-        if ($retry === 0) {
+        if ($retry===0) {
             return false;
         }
         // solicitar token

lib/Sii/RegistroCompraVenta.php

Doc Comments +6 added lines, -4 removed lines

@@ -77,7 +77,7 @@ 
 
     /**
      * Constructor, obtiene el token de la sesión y lo guarda
-     * @param Firma Objeto con la firma electrónica
+     * @param Firma \sasco\LibreDTE\FirmaElectronica con la firma electrónica
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-08-28
      */
@@ -119,6 +119,8 @@ 
      * Método que entrega los eventos asociados a un DTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-08-28
+     * @param string $dte
+     * @param string $folio
      */
     public function listarEventosHistDoc($rut, $dv, $dte, $folio)
     {
@@ -206,10 +208,10 @@ 
 
     /**
      * Método para realizar una solicitud al servicio web del SII
-     * @param request Nombre de la función que se ejecutará en el servicio web
+     * @param request string de la función que se ejecutará en el servicio web
      * @param args Argumentos que se pasarán al servicio web
-     * @param retry Intentos que se realizarán como máximo para obtener respuesta
-     * @return Objeto o String con la respuesta (depende servicio web)
+     * @param retry integer que se realizarán como máximo para obtener respuesta
+     * @return string o String con la respuesta (depende servicio web)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2020-02-12
      */

Spacing +3 added lines, -3 removed lines

@@ -71,8 +71,8 @@ 
     ]; ///< Tipos de transacciones o caracterizaciones/clasificaciones de las compras
 
     public static $estados_ok = [
-        7,  // Evento registrado previamente
-        8,  // Pasados 8 días después de la recepción no es posible registrar reclamos o eventos
+        7, // Evento registrado previamente
+        8, // Pasados 8 días después de la recepción no es posible registrar reclamos o eventos
         27, // No se puede registrar un evento (acuse de recibo, reclamo o aceptación de contenido) de un DTE pagado al contado o gratuito
     ]; ///< Código de estado de respuesta de la asignación de estado que son considerados como OK
 
@@ -251,7 +251,7 @@ 
             return false;
         }
         // hacer consultas al SII
-        for ($i=0; $i<$retry; $i++) {
+        for ($i = 0; $i<$retry; $i++) {
             try {
                 $body = call_user_func_array([$soap, $request], $args);
                 break;

lib/Sii/Dte.php

Doc Comments +11 added lines, -11 removed lines

@@ -237,7 +237,7 @@ 
 
     /**
      * Método que entrega el tipo de DTE
-     * @return Tipo de dte, ej: 33 (factura electrónica)
+     * @return string de dte, ej: 33 (factura electrónica)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-02
      */
@@ -248,7 +248,7 @@ 
 
     /**
      * Método que entrega el folio del DTE
-     * @return Folio del DTE
+     * @return string del DTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-02
      */
@@ -259,7 +259,7 @@ 
 
     /**
      * Método que entrega rut del emisor del DTE
-     * @return RUT del emiro
+     * @return string del emiro
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -275,7 +275,7 @@ 
 
     /**
      * Método que entrega rut del receptor del DTE
-     * @return RUT del emiro
+     * @return string del emiro
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -291,7 +291,7 @@ 
 
     /**
      * Método que entrega fecha de emisión del DTE
-     * @return Fecha de emisión en formato AAAA-MM-DD
+     * @return string de emisión en formato AAAA-MM-DD
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -400,7 +400,7 @@ 
 
     /**
      * Método que realiza el timbrado del DTE
-     * @param Folios Objeto de los Folios con los que se desea timbrar
+     * @param Folios Folios de los Folios con los que se desea timbrar
      * @return =true si se pudo timbrar o =false en caso de error
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-09-01
@@ -486,7 +486,7 @@ 
 
     /**
      * Método que realiza la firma del DTE
-     * @param Firma objeto que representa la Firma Electrónca
+     * @param Firma \sasco\LibreDTE\FirmaElectronica que representa la Firma Electrónca
      * @return =true si el DTE pudo ser fimado o =false si no se pudo firmar
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-10-22
@@ -509,7 +509,7 @@ 
 
     /**
      * Método que entrega el DTE en XML
-     * @return XML con el DTE (podría: con o sin timbre y con o sin firma)
+     * @return string con el DTE (podría: con o sin timbre y con o sin firma)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-08-20
      */
@@ -1764,7 +1764,7 @@ 
      *  - Firma del DTE
      *  - RUT del emisor (si se pasó uno para comparar)
      *  - RUT del receptor (si se pasó uno para comparar)
-     * @return Código del estado de la validación
+     * @return integer del estado de la validación
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-07-03
      */
@@ -1877,7 +1877,7 @@ 
 
     /**
      * Método que obtiene el estado del DTE
-     * @param Firma objeto que representa la Firma Electrónca
+     * @param Firma \sasco\LibreDTE\FirmaElectronica que representa la Firma Electrónca
      * @return Arreglo con el estado del DTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-24
@@ -1921,7 +1921,7 @@ 
 
     /**
      * Método que obtiene el estado avanzado del DTE
-     * @param Firma objeto que representa la Firma Electrónca
+     * @param Firma \sasco\LibreDTE\FirmaElectronica que representa la Firma Electrónca
      * @return Arreglo con el estado del DTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-05

Spacing +17 added lines, -17 removed lines

@@ -139,7 +139,7 @@ 
                 ]
             ]);
             $parent = $this->xml->getElementsByTagName($this->tipo_general)->item(0);
-            $this->xml->generate($datos + ['TED' => null], null, $parent);
+            $this->xml->generate($datos+['TED' => null], null, $parent);
             $this->datos = $datos;
             if ($normalizar and !$this->verificarDatos()) {
                 return false;
@@ -382,7 +382,7 @@ 
         $xml->documentElement->removeAttributeNS('http://www.w3.org/2001/XMLSchema-instance', 'xsi');
         $xml->documentElement->removeAttributeNS('http://www.sii.cl/SiiDte', '');
         $TED = $xml->getFlattened('/');
-        return mb_detect_encoding($TED, ['UTF-8', 'ISO-8859-1']) != 'ISO-8859-1' ? utf8_decode($TED) : $TED;
+        return mb_detect_encoding($TED, ['UTF-8', 'ISO-8859-1'])!='ISO-8859-1' ? utf8_decode($TED) : $TED;
     }
 
     /**
@@ -395,7 +395,7 @@ 
     {
         $datos = $this->getDatos();
         $idk = !empty($datos['TED']['DD']['CAF']['DA']['IDK']) ? (int)$datos['TED']['DD']['CAF']['DA']['IDK'] : null;
-        return $idk ? $idk === 100 : null;
+        return $idk ? $idk===100 : null;
     }
 
     /**
@@ -532,7 +532,7 @@ 
     {
         $this->getDatos();
         // generar resumen
-        $resumen =  [
+        $resumen = [
             'TpoDoc' => (int)$this->datos['Encabezado']['IdDoc']['TipoDTE'],
             'NroDoc' => (int)$this->datos['Encabezado']['IdDoc']['Folio'],
             'TasaImp' => 0,
@@ -577,10 +577,10 @@ 
      */
     private function calcularNetoIVA($total, $tasa = null)
     {
-        if ($tasa === 0 or $tasa === false) {
+        if ($tasa===0 or $tasa===false) {
             return [0, 0];
         }
-        if ($tasa === null) {
+        if ($tasa===null) {
             $tasa = \sasco\LibreDTE\Sii::getIVA();
         }
         // WARNING: el IVA obtenido puede no ser el NETO*(TASA/100)
@@ -588,8 +588,8 @@ 
         // entre el total y el neto, ya que hay casos de borde como:
         //  - BRUTO:   680 => NETO:   571 e IVA:   108 => TOTAL:   679
         //  - BRUTO: 86710 => NETO: 72866 e IVA: 13845 => TOTAL: 86711
-        $neto = round($total / (1+($tasa/100)));
-        $iva = $total - $neto;
+        $neto = round($total / (1+($tasa / 100)));
+        $iva = $total-$neto;
         return [$neto, $iva];
     }
 
@@ -1436,12 +1436,12 @@ 
                     );
                     // aplicar descuento
                     if ($d['DescuentoPct']) {
-                        $d['DescuentoMonto'] = round($d['MontoItem'] * (float)$d['DescuentoPct']/100);
+                        $d['DescuentoMonto'] = round($d['MontoItem'] * (float)$d['DescuentoPct'] / 100);
                     }
                     $d['MontoItem'] -= $d['DescuentoMonto'];
                     // aplicar recargo
                     if ($d['RecargoPct']) {
-                        $d['RecargoMonto'] = round($d['MontoItem'] * (float)$d['RecargoPct']/100);
+                        $d['RecargoMonto'] = round($d['MontoItem'] * (float)$d['RecargoPct'] / 100);
                     }
                     $d['MontoItem'] += $d['RecargoMonto'];
                     // aproximar monto del item
@@ -1527,7 +1527,7 @@ 
                 }
                 $valor =
                     $dr['TpoValor']=='%'
-                    ? $this->round(($dr['ValorDR']/100)*$datos['Encabezado']['Totales'][$monto], $datos['Encabezado']['Totales']['TpoMoneda'])
+                    ? $this->round(($dr['ValorDR'] / 100) * $datos['Encabezado']['Totales'][$monto], $datos['Encabezado']['Totales']['TpoMoneda'])
                     : $dr['ValorDR']
                 ;
                 // aplicar descuento
@@ -1600,7 +1600,7 @@ 
             // si el monto no existe se asigna
             if ($datos['Encabezado']['Totales']['ImptoReten'][$i]['MontoImp']===null) {
                 $datos['Encabezado']['Totales']['ImptoReten'][$i]['MontoImp'] = round(
-                    $neto * $datos['Encabezado']['Totales']['ImptoReten'][$i]['TasaImp']/100
+                    $neto * $datos['Encabezado']['Totales']['ImptoReten'][$i]['TasaImp'] / 100
                 );
             }
         }
@@ -1608,7 +1608,7 @@ 
         if (isset($datos['Encabezado']['Totales']['ImptoReten']) and is_array($datos['Encabezado']['Totales']['ImptoReten'])) {
             $codigos = array_keys($montos);
             $n_impuestos = count($datos['Encabezado']['Totales']['ImptoReten']);
-            for ($i=0; $i<$n_impuestos; $i++) {
+            for ($i = 0; $i<$n_impuestos; $i++) {
                 if (!in_array($datos['Encabezado']['Totales']['ImptoReten'][$i]['TipoImp'], $codigos)) {
                     unset($datos['Encabezado']['Totales']['ImptoReten'][$i]);
                 }
@@ -1633,7 +1633,7 @@ 
         // valor IndMntNeto = 2 indica que los montosde las líneas on netos en cuyo caso no aplica el cálculo
         // de neto e iva a partir del total y deberá venir informado de otra forma (aun no definido)
         if ($this->esBoleta() and (empty($datos['Encabezado']['IdDoc']['IndMntNeto']) or $datos['Encabezado']['IdDoc']['IndMntNeto']!=2)) {
-            $total = (int)$datos['Encabezado']['Totales']['MntTotal'] - (int)$datos['Encabezado']['Totales']['MntExe'];
+            $total = (int)$datos['Encabezado']['Totales']['MntTotal']-(int)$datos['Encabezado']['Totales']['MntExe'];
             if ($total and (empty($datos['Encabezado']['Totales']['MntNeto']) or empty($datos['Encabezado']['Totales']['IVA']))) {
                 list($datos['Encabezado']['Totales']['MntNeto'], $datos['Encabezado']['Totales']['IVA']) = $this->calcularNetoIVA($total);
             }
@@ -1648,7 +1648,7 @@ 
             } else {
                 if (empty($datos['Encabezado']['Totales']['IVA']) and !empty($datos['Encabezado']['Totales']['TasaIVA'])) {
                     $datos['Encabezado']['Totales']['IVA'] = round(
-                        $datos['Encabezado']['Totales']['MntNeto']*($datos['Encabezado']['Totales']['TasaIVA']/100)
+                        $datos['Encabezado']['Totales']['MntNeto'] * ($datos['Encabezado']['Totales']['TasaIVA'] / 100)
                     );
                 }
             }
@@ -1674,7 +1674,7 @@ 
                 if (ImpuestosAdicionales::getTipo($ImptoReten['TipoImp'])=='R') {
                     $datos['Encabezado']['Totales']['MntTotal'] -= $ImptoReten['MontoImp'];
                     if ($ImptoReten['MontoImp']!=$datos['Encabezado']['Totales']['IVA']) {
-                        $datos['Encabezado']['Totales']['IVANoRet'] = $datos['Encabezado']['Totales']['IVA'] - $ImptoReten['MontoImp'];
+                        $datos['Encabezado']['Totales']['IVANoRet'] = $datos['Encabezado']['Totales']['IVA']-$ImptoReten['MontoImp'];
                     }
                 }
                 // si es adicional se suma al total
@@ -1832,7 +1832,7 @@ 
         $SignatureValue = trim(str_replace(["\n", ' ', "\t"], '', $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue));
         $X509Certificate = trim(str_replace(["\n", ' ', "\t"], '', $Signature->getElementsByTagName('X509Certificate')->item(0)->nodeValue));
         $X509Certificate = '-----BEGIN CERTIFICATE-----'."\n".wordwrap($X509Certificate, 64, "\n", true)."\n".'-----END CERTIFICATE----- ';
-        $valid = openssl_verify($SignedInfo->C14N(), base64_decode($SignatureValue), $X509Certificate) === 1 ? true : false;
+        $valid = openssl_verify($SignedInfo->C14N(), base64_decode($SignatureValue), $X509Certificate)===1 ? true : false;
         return $valid;
         //return $valid and $DigestValue===base64_encode(sha1($Documento->C14N(), true));
     }

Braces +62 added lines, -42 removed lines

@@ -57,10 +57,11 @@ 
      */
     public function __construct($datos, $normalizar = true)
     {
-        if (is_array($datos))
-            $this->setDatos($datos, $normalizar);
-        else if (is_string($datos))
-            $this->loadXML($datos);
+        if (is_array($datos)) {
+                    $this->setDatos($datos, $normalizar);
+        } else if (is_string($datos)) {
+                    $this->loadXML($datos);
+        }
         $this->timestamp = date('Y-m-d\TH:i:s');
     }
 
@@ -121,8 +122,9 @@ 
             if ($normalizar) {
                 $this->normalizar($datos);
                 $method = 'normalizar_'.$this->tipo;
-                if (method_exists($this, $method))
-                    $this->$method($datos);
+                if (method_exists($this, $method)) {
+                                    $this->$method($datos);
+                }
                 $this->normalizar_final($datos);
             }
             $this->tipo_general = $this->getTipoGeneral($this->tipo);
@@ -199,8 +201,9 @@ 
      */
     public function getJSON()
     {
-        if (!$this->getDatos())
-            return false;
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return json_encode($this->datos, JSON_PRETTY_PRINT);
     }
 
@@ -225,9 +228,10 @@ 
      */
     private function getTipoGeneral($dte)
     {
-        foreach ($this->tipos as $tipo => $codigos)
-            if (in_array($dte, $codigos))
+        foreach ($this->tipos as $tipo => $codigos) {
+                    if (in_array($dte, $codigos))
                 return $tipo;
+        }
         \sasco\LibreDTE\Log::write(
             \sasco\LibreDTE\Estado::DTE_ERROR_TIPO,
             \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::DTE_ERROR_TIPO, $dte)
@@ -266,10 +270,12 @@ 
     public function getEmisor()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Emisor/RUTEmisor')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Emisor']['RUTEmisor'];
     }
 
@@ -282,10 +288,12 @@ 
     public function getReceptor()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Receptor/RUTRecep')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Receptor']['RUTRecep'];
     }
 
@@ -298,10 +306,12 @@ 
     public function getFechaEmision()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/IdDoc/FchEmis')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['IdDoc']['FchEmis'];
     }
 
@@ -314,10 +324,12 @@ 
     public function getMontoTotal()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Totales/MntTotal')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Totales']['MntTotal'];
     }
 
@@ -330,10 +342,12 @@ 
     public function getMoneda()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Totales/TpoMoneda')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Totales']['TpoMoneda'];
     }
 
@@ -376,8 +390,9 @@ 
             return false;*/
         $xml = new \sasco\LibreDTE\XML();
         $TED = $this->xml->getElementsByTagName('TED')->item(0);
-        if (!$TED)
-            return '<TED/>';
+        if (!$TED) {
+                    return '<TED/>';
+        }
         $xml->loadXML($TED->C14N());
         $xml->documentElement->removeAttributeNS('http://www.w3.org/2001/XMLSchema-instance', 'xsi');
         $xml->documentElement->removeAttributeNS('http://www.sii.cl/SiiDte', '');
@@ -698,8 +713,9 @@ 
         }
         // si existe descuento o recargo global se normalizan
         if (!empty($datos['DscRcgGlobal'])) {
-            if (!isset($datos['DscRcgGlobal'][0]))
-                $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            if (!isset($datos['DscRcgGlobal'][0])) {
+                            $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            }
             $NroLinDR = 1;
             foreach ($datos['DscRcgGlobal'] as &$dr) {
                 $dr = array_merge([
@@ -1364,8 +1380,9 @@ 
      */
     private function normalizar_detalle(array &$datos)
     {
-        if (!isset($datos['Detalle'][0]))
-            $datos['Detalle'] = [$datos['Detalle']];
+        if (!isset($datos['Detalle'][0])) {
+                    $datos['Detalle'] = [$datos['Detalle']];
+        }
         $item = 1;
         foreach ($datos['Detalle'] as &$d) {
             $d = array_merge([
@@ -1427,8 +1444,9 @@ 
                 ];
             }
             if ($d['PrcItem']) {
-                if (!$d['QtyItem'])
-                    $d['QtyItem'] = 1;
+                if (!$d['QtyItem']) {
+                                    $d['QtyItem'] = 1;
+                }
                 if (empty($d['MontoItem'])) {
                     $d['MontoItem'] = $this->round(
                         (float)$d['QtyItem'] * (float)$d['PrcItem'],
@@ -1494,8 +1512,9 @@ 
     private function normalizar_aplicar_descuentos_recargos(array &$datos)
     {
         if (!empty($datos['DscRcgGlobal'])) {
-            if (!isset($datos['DscRcgGlobal'][0]))
-                $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            if (!isset($datos['DscRcgGlobal'][0])) {
+                            $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            }
             foreach ($datos['DscRcgGlobal'] as &$dr) {
                 $dr = array_merge([
                     'NroLinDR' => false,
@@ -1562,8 +1581,9 @@ 
         $montos = [];
         foreach ($datos['Detalle'] as &$d) {
             if (!empty($d['CodImpAdic'])) {
-                if (!isset($montos[$d['CodImpAdic']]))
-                    $montos[$d['CodImpAdic']] = 0;
+                if (!isset($montos[$d['CodImpAdic']])) {
+                                    $montos[$d['CodImpAdic']] = 0;
+                }
                 $montos[$d['CodImpAdic']] += $d['MontoItem'];
             }
         }