libredte / libredte-lib-core

lib/FirmaElectronica.php

Doc Comments +9 added lines, -9 removed lines

@@ -100,7 +100,7 @@ 
     /**
      * Método para generar un error usando una excepción de SowerPHP o terminar
      * el script si no se está usando el framework
-     * @param msg Mensaje del error
+     * @param msg string del error
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-08-04
      */
@@ -188,7 +188,7 @@ 
 
     /**
      * Método que entrega desde cuando es válida la firma
-     * @return validFrom_time_t
+     * @return string
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-22
      */
@@ -199,7 +199,7 @@ 
 
     /**
      * Método que entrega hasta cuando es válida la firma
-     * @return validTo_time_t
+     * @return string
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-22
      */
@@ -232,7 +232,7 @@ 
 
     /**
      * Método que obtiene el módulo de la clave privada
-     * @return Módulo en base64
+     * @return string en base64
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2014-12-07
      */
@@ -244,7 +244,7 @@ 
 
     /**
      * Método que obtiene el exponente público de la clave privada
-     * @return Exponente público en base64
+     * @return string público en base64
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2014-12-06
      */
@@ -294,9 +294,9 @@ 
 
     /**
      * Método para realizar la firma de datos
-     * @param data Datos que se desean firmar
+     * @param data string que se desean firmar
      * @param signature_alg Algoritmo que se utilizará para firmar (por defect SHA1)
-     * @return Firma digital de los datos en base64 o =false si no se pudo firmar
+     * @return boolean|string digital de los datos en base64 o =false si no se pudo firmar
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2014-12-08
      */
@@ -311,7 +311,7 @@ 
 
     /**
      * Método que verifica la firma digital de datos
-     * @param data Datos que se desean verificar
+     * @param data string que se desean verificar
      * @param signature Firma digital de los datos en base64
      * @param pub_key Certificado digital, clave pública, de la firma
      * @param signature_alg Algoritmo que se usó para firmar (por defect SHA1)
@@ -334,7 +334,7 @@ 
      *
      * @param xml Datos XML que se desean firmar
      * @param reference Referencia a la que hace la firma
-     * @return XML firmado o =false si no se pudo fimar
+     * @return boolean|string firmado o =false si no se pudo fimar
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-07-24
      */

lib/Sii/Folios.php

Doc Comments +6 added lines, -6 removed lines

@@ -114,7 +114,7 @@ 
 
     /**
      * Método que entrega el RUT de a quién se está autorizando el CAF
-     * @return Rut del emisor del CAF
+     * @return false|string del emisor del CAF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-30
      */
@@ -156,7 +156,7 @@ 
 
     /**
      * Método que entrega la firma del SII sobre el nodo DA
-     * @return Firma en base64
+     * @return false|string en base64
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-30
      */
@@ -171,7 +171,7 @@ 
     /**
      * Método que entrega el IDK (serial number) de la clave pública del SII
      * utilizada para firmar el CAF
-     * @return Serial number
+     * @return \sasco\LibreDTE\IDK|null number
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-30
      */
@@ -185,7 +185,7 @@ 
 
     /**
      * Método que entrega la clave privada proporcionada por el SII para el CAF
-     * @return Clave privada en base64
+     * @return false|string privada en base64
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-30
      */
@@ -199,7 +199,7 @@ 
 
     /**
      * Método que entrega la clave pública proporcionada por el SII para el CAF
-     * @return Clave pública en base64
+     * @return false|string pública en base64
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-30
      */
@@ -227,7 +227,7 @@ 
 
     /**
      * Método que entrega la fecha de autorización con la que se emitió el CAF
-     * @return Fecha de autorización del CAF
+     * @return false|string de autorización del CAF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-07-19
      */

Braces +39 added lines, -26 removed lines

@@ -63,8 +63,9 @@ 
         // validar firma del SII sobre los folios
         $firma = $this->getFirma();
         $idk = $this->getIDK();
-        if (!$firma or !$idk)
-            return false;
+        if (!$firma or !$idk) {
+                    return false;
+        }
         $pub_key = \sasco\LibreDTE\Sii::cert($idk);
         if (!$pub_key or openssl_verify($this->xml->getFlattened('/AUTORIZACION/CAF/DA'), base64_decode($firma), $pub_key)!==1) {
             \sasco\LibreDTE\Log::write(
@@ -75,8 +76,9 @@ 
         }
         // validar clave privada y pública proporcionada por el SII
         $private_key = $this->getPrivateKey();
-        if (!$private_key)
-            return false;
+        if (!$private_key) {
+                    return false;
+        }
         $plain = md5(date('U'));
         if (!openssl_private_encrypt($plain, $crypt, $private_key)) {
             \sasco\LibreDTE\Log::write(
@@ -86,8 +88,9 @@ 
             return false;
         }
         $public_key = $this->getPublicKey();
-        if (!$public_key)
-            return false;
+        if (!$public_key) {
+                    return false;
+        }
         if (!openssl_public_decrypt($crypt, $plain_firmado, $public_key)) {
             \sasco\LibreDTE\Log::write(
                 \sasco\LibreDTE\Estado::FOLIOS_ERROR_DESENCRIPTAR,
@@ -106,8 +109,9 @@ 
      */
     public function getCaf()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $CAF = $this->xml->getElementsByTagName('CAF')->item(0);
         return $CAF ? $CAF : false;
     }
@@ -120,8 +124,9 @@ 
      */
     public function getEmisor()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $RE = $this->xml->getElementsByTagName('RE')->item(0);
         return $RE ? $RE->nodeValue : false;
     }
@@ -134,8 +139,9 @@ 
      */
     public function getDesde()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $D = $this->xml->getElementsByTagName('D')->item(0);
         return $D ? (int)$D->nodeValue : false;
     }
@@ -148,8 +154,9 @@ 
      */
     public function getHasta()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $H = $this->xml->getElementsByTagName('H')->item(0);
         return $H ? (int)$H->nodeValue : false;
     }
@@ -162,8 +169,9 @@ 
      */
     private function getFirma()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $FRMA = $this->xml->getElementsByTagName('FRMA')->item(0);
         return $FRMA ? $FRMA->nodeValue : false;
     }
@@ -177,8 +185,9 @@ 
      */
     private function getIDK()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $IDK = $this->xml->getElementsByTagName('IDK')->item(0);
         return $IDK ? (int)$IDK->nodeValue : false;
     }
@@ -191,8 +200,9 @@ 
      */
     public function getPrivateKey()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $RSASK = $this->xml->getElementsByTagName('RSASK')->item(0);
         return $RSASK ? $RSASK->nodeValue : false;
     }
@@ -205,8 +215,9 @@ 
      */
     public function getPublicKey()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $RSAPUBK = $this->xml->getElementsByTagName('RSAPUBK')->item(0);
         return $RSAPUBK ? $RSAPUBK->nodeValue : false;
     }
@@ -219,8 +230,9 @@ 
      */
     public function getTipo()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $TD = $this->xml->getElementsByTagName('TD')->item(0);
         return $TD ? (int)$TD->nodeValue : false;
     }
@@ -233,8 +245,9 @@ 
      */
     public function getFechaAutorizacion()
     {
-        if (!$this->xml)
-            return false;
+        if (!$this->xml) {
+                    return false;
+        }
         $FA = $this->xml->getElementsByTagName('FA')->item(0);
         return $FA ? $FA->nodeValue : false;
     }

lib/Sii/PDF/Dte.php

Doc Comments +31 added lines, -28 removed lines

@@ -174,7 +174,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)
+     * @param posicion integer respecto a datos del emisor (=0 izq, =1 arriba)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-04
      */
@@ -336,7 +336,7 @@ 
      * contínuo
      * @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
+     * @param width boolean del papel contínuo en mm
      * @author Pablo Reyes (https://github.com/pabloxp)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-06-15
@@ -482,10 +482,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 2016-12-02
      */
@@ -555,11 +556,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 2016-12-02
+     * @return integer
      */
     private function agregarFolio($rut, $tipo, $folio, $sucursal_sii = null, $x = 130, $y = 15, $w = 70, $font_size = null, array $color = null)
     {
@@ -605,7 +608,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 2017-06-15
      */
@@ -693,7 +696,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 2017-06-15
      */
@@ -780,7 +783,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
      */
@@ -851,7 +854,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 2016-08-03
      */
@@ -874,7 +877,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 2016-08-05
@@ -941,7 +944,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 Pablo Reyes (https://github.com/pabloxp)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
@@ -977,7 +980,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
      */
@@ -1003,7 +1006,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 2016-08-17
      */
@@ -1027,7 +1030,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
      */
@@ -1137,9 +1140,9 @@ 
      *  - 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
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-07-13
      */
@@ -1176,10 +1179,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 2015-09-08
      */
@@ -1206,10 +1209,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 +39 added lines, -39 removed lines

@@ -168,7 +168,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;
     }
 
     /**
@@ -303,7 +303,7 @@ 
         $this->setY(max($y));
         $this->Ln();
         $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(
@@ -350,7 +350,7 @@ 
         // determinar alto de la página y agregarla
         $height = 145;
         $n_detalle = count($dte['Detalle']);
-        if ($n_detalle>1) $height += $n_detalle*20;
+        if ($n_detalle>1) $height += $n_detalle * 20;
         if ($this->cedible) $height += 50;
         $this->AddPage('P', [$height, $width]);
         // agregar cabecera del documento
@@ -359,15 +359,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, 40, 8, 9, [0,0,0]);
+        $y = $this->agregarEmisor($dte['Encabezado']['Emisor'], $x_start, $y+2, 40, 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,
@@ -417,7 +417,7 @@ 
         $this->AddPage('P', [$height, $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);
@@ -497,13 +497,13 @@ 
                 $this->logo['uri'],
                 $x,
                 $y,
-                !$this->logo['posicion']?$w_img:null, $this->logo['posicion']?($w_img/2):null,
+                !$this->logo['posicion'] ? $w_img : null, $this->logo['posicion'] ? ($w_img / 2) : null,
                 'PNG',
-                (isset($emisor['url'])?$emisor['url']:''),
+                (isset($emisor['url']) ? $emisor['url'] : ''),
                 'T'
             );
             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;
@@ -514,13 +514,13 @@ 
         }
         // agregar datos del emisor
         $this->setFont('', 'B', $font_size ? $font_size : 14);
-        $this->SetTextColorArray($color===null?[32, 92, 144]:$color);
+        $this->SetTextColorArray($color===null ? [32, 92, 144] : $color);
         $this->MultiTexto(!empty($emisor['RznSoc']) ? $emisor['RznSoc'] : $emisor['RznSocEmisor'], $x, $this->y+2, 'L', $w);
         $this->setFont('', 'B', $font_size ? $font_size : 9);
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->MultiTexto(!empty($emisor['GiroEmis']) ? $emisor['GiroEmis'] : $emisor['GiroEmisor'], $x, $this->y, 'L', $w);
         $ciudad = !empty($emisor['CiudadOrigen']) ? $emisor['CiudadOrigen'] : \sasco\LibreDTE\Chile::getCiudad($emisor['CmnaOrigen']);
-        $this->MultiTexto($emisor['DirOrigen'].', '.$emisor['CmnaOrigen'].($ciudad?(', '.$ciudad):''), $x, $this->y, 'L', $w);
+        $this->MultiTexto($emisor['DirOrigen'].', '.$emisor['CmnaOrigen'].($ciudad ? (', '.$ciudad) : ''), $x, $this->y, 'L', $w);
         if (!empty($emisor['Sucursal'])) {
             $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', $w);
         }
@@ -564,12 +564,12 @@ 
     private 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), $x, null, 'C', $w);
@@ -582,7 +582,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;
     }
@@ -713,7 +713,7 @@ 
             $this->Texto('Señor(es)', $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);
@@ -730,7 +730,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);
@@ -771,7 +771,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();
     }
@@ -859,7 +859,7 @@ 
     {
         if (!isset($referencias[0]))
             $referencias = [$referencias];
-        foreach($referencias as $r) {
+        foreach ($referencias as $r) {
             $texto = $r['NroLinRef'].' - '.$this->getTipo($r['TpoDocRef']).' N° '.$r['FolioRef'].' del '.$r['FchRef'];
             if (isset($r['RazonRef']) and $r['RazonRef']!==false)
                 $texto = $texto.': '.$r['RazonRef'];
@@ -952,9 +952,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+1, $this->y, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
         $this->Texto($this->detalle_cols['PrcItem']['title'], $x+15, $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
@@ -964,9 +964,9 @@ 
         if (!isset($detalle[0]))
             $detalle = [$detalle];
         $this->SetY($this->getY()+2);
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             $this->MultiTexto($d['NmbItem'], $x+1, $this->y+4, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
-            $this->Texto(number_format($d['PrcItem'],0,',','.'), $x+15, $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
+            $this->Texto(number_format($d['PrcItem'], 0, ',', '.'), $x+15, $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
             $this->Texto($this->num($d['QtyItem']), $x+35, $this->y, ucfirst($this->detalle_cols['QtyItem']['align'][0]), $this->detalle_cols['QtyItem']['width']);
             $this->Texto($this->num($d['MontoItem']), $x+45, $this->y, ucfirst($this->detalle_cols['MontoItem']['align'][0]), $this->detalle_cols['MontoItem']['width']);
         }
@@ -986,7 +986,7 @@ 
         if (!isset($detalle[0])) {
             $detalle = [$detalle];
         }
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             if (!empty($d['MontoItem'])) {
                 $subtotal += $d['MontoItem'];
             }
@@ -1011,13 +1011,13 @@ 
     {
         if (!isset($descuentosRecargos[0]))
             $descuentosRecargos = [$descuentosRecargos];
-        foreach($descuentosRecargos as $dr) {
+        foreach ($descuentosRecargos as $dr) {
             $tipo = $dr['TpoMov']=='D' ? 'Descuento' : 'Recargo';
             $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();
@@ -1037,8 +1037,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();
         }
     }
@@ -1088,7 +1088,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'].'%) $';
@@ -1153,7 +1153,7 @@ 
                 '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' => $this->papelContinuo ? 'C' : 'S',
             ];
@@ -1185,7 +1185,7 @@ 
      */
     private function agregarAcuseRecibo($x = 93, $y = 190, $w = 50, $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]]]);
         $this->setFont('', 'B', 10);
         $this->Texto('Acuse de recibo', $x, $y+1, 'C', $w);
@@ -1215,9 +1215,9 @@ 
      */
     private 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);
@@ -1245,7 +1245,7 @@ 
     private function agregarLeyendaDestino($tipo, $y = 254, $font_size = 10)
     {
         $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');
     }
 
     /**
@@ -1276,7 +1276,7 @@ 
         $dias = ['Domingo', 'Lunes', 'Martes', 'Miércoles', 'Jueves', 'Viernes', 'Sábado'];
         $meses = ['enero', 'febrero', 'marzo', 'abril', 'mayo', 'junio', 'julio', 'agosto', 'septiembre', 'octubre', 'noviembre', 'diciembre'];
         $unixtime = strtotime($date);
-        $fecha = date(($mostrar_dia?'\D\I\A ':'').'j \d\e \M\E\S \d\e\l Y', $unixtime);
+        $fecha = date(($mostrar_dia ? '\D\I\A ' : '').'j \d\e \M\E\S \d\e\l Y', $unixtime);
         $dia = $dias[date('w', $unixtime)];
         $mes = $meses[date('n', $unixtime)-1];
         return str_replace(array('DIA', 'MES'), array($dia, $mes), $fecha);

Braces +101 added lines, -64 removed lines

@@ -310,15 +310,17 @@ 
             !empty($dte['Encabezado']['IdDoc']['IndTraslado']) ? $dte['Encabezado']['IdDoc']['IndTraslado'] : null,
             !empty($dte['Encabezado']['Transporte']) ? $dte['Encabezado']['Transporte'] : null
         );
-        if (!empty($dte['Referencia']))
-            $this->agregarReferencia($dte['Referencia']);
+        if (!empty($dte['Referencia'])) {
+                    $this->agregarReferencia($dte['Referencia']);
+        }
         $this->agregarDetalle($dte['Detalle']);
         if (!empty($dte['DscRcgGlobal'])) {
             $this->agregarSubTotal($dte['Detalle']);
             $this->agregarDescuentosRecargos($dte['DscRcgGlobal']);
         }
-        if (!empty($dte['Encabezado']['IdDoc']['MntPagos']))
-            $this->agregarPagos($dte['Encabezado']['IdDoc']['MntPagos']);
+        if (!empty($dte['Encabezado']['IdDoc']['MntPagos'])) {
+                    $this->agregarPagos($dte['Encabezado']['IdDoc']['MntPagos']);
+        }
         $this->agregarTotales($dte['Encabezado']['Totales']);
         // agregar observaciones
         $this->agregarObservacion($dte['Encabezado']['IdDoc']);
@@ -350,8 +352,12 @@ 
         // determinar alto de la página y agregarla
         $height = 145;
         $n_detalle = count($dte['Detalle']);
-        if ($n_detalle>1) $height += $n_detalle*20;
-        if ($this->cedible) $height += 50;
+        if ($n_detalle>1) {
+            $height += $n_detalle*20;
+        }
+        if ($this->cedible) {
+            $height += 50;
+        }
         $this->AddPage('P', [$height, $width]);
         // agregar cabecera del documento
         $y = $this->agregarFolio(
@@ -412,8 +418,12 @@ 
     {
         // determinar alto de la página y agregarla
         $height = 100;
-        if ($this->cedible) $height += 50;
-        if ($dte['Encabezado']['Receptor']['RUTRecep']!='66666666-6') $height += 40;
+        if ($this->cedible) {
+            $height += 50;
+        }
+        if ($dte['Encabezado']['Receptor']['RUTRecep']!='66666666-6') {
+            $height += 40;
+        }
         $this->AddPage('P', [$height, $width]);
         $x = 1;
         $y = 5;
@@ -526,10 +536,12 @@ 
         }
         $contacto = [];
         if (!empty($emisor['Telefono'])) {
-            if (!is_array($emisor['Telefono']))
-                $emisor['Telefono'] = [$emisor['Telefono']];
-            foreach ($emisor['Telefono'] as $t)
-                $contacto[] = $t;
+            if (!is_array($emisor['Telefono'])) {
+                            $emisor['Telefono'] = [$emisor['Telefono']];
+            }
+            foreach ($emisor['Telefono'] as $t) {
+                            $contacto[] = $t;
+            }
         }
         if (!empty($emisor['CorreoEmisor'])) {
             $contacto[] = $emisor['CorreoEmisor'];
@@ -596,8 +608,9 @@ 
      */
     private function getTipo($tipo)
     {
-        if (!is_numeric($tipo) and !isset($this->tipos[$tipo]))
-            return $tipo;
+        if (!is_numeric($tipo) and !isset($this->tipos[$tipo])) {
+                    return $tipo;
+        }
         return isset($this->tipos[$tipo]) ? strtoupper($this->tipos[$tipo]) : 'Documento '.$tipo;
     }
 
@@ -747,10 +760,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);
@@ -800,15 +815,19 @@ 
             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'];
-                if (!empty($Transporte['Chofer']['RUTChofer']))
-                    $transporte .= ' ('.$Transporte['Chofer']['RUTChofer'].')';
+                if (!empty($Transporte['Chofer']['NombreChofer'])) {
+                                    $transporte .= ' con chofer '.$Transporte['Chofer']['NombreChofer'];
+                }
+                if (!empty($Transporte['Chofer']['RUTChofer'])) {
+                                    $transporte .= ' ('.$Transporte['Chofer']['RUTChofer'].')';
+                }
             }
             if ($transporte) {
                 $this->setFont('', 'B', null);
@@ -822,8 +841,9 @@ 
         if (!empty($Transporte['Aduana']) and is_array($Transporte['Aduana'])) {
             $col = 0;
             foreach ($Transporte['Aduana'] as $tag => $codigo) {
-                if ($codigo===false)
-                    continue;
+                if ($codigo===false) {
+                                    continue;
+                }
                 $glosa = \sasco\LibreDTE\Sii\Aduana::getGlosa($tag);
                 $valor = \sasco\LibreDTE\Sii\Aduana::getValor($tag, $codigo);
                 if ($glosa!==false and $valor!==false) {
@@ -836,15 +856,18 @@ 
                     $this->Texto(':', $x+$offset+$col);
                     $this->setFont('', '', null);
                     $this->Texto($valor, $x+$offset+2+$col);
-                    if ($tag=='TipoBultos')
-                        $col = abs($col-110);
-                    if ($col)
-                        $this->Ln();
+                    if ($tag=='TipoBultos') {
+                                            $col = abs($col-110);
+                    }
+                    if ($col) {
+                                            $this->Ln();
+                    }
                     $col = abs($col-110);
                 }
             }
-            if ($col)
-                $this->Ln();
+            if ($col) {
+                            $this->Ln();
+            }
         }
     }
 
@@ -857,12 +880,14 @@ 
      */
     private 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'].' - '.$this->getTipo($r['TpoDocRef']).' N° '.$r['FolioRef'].' del '.$r['FchRef'];
-            if (isset($r['RazonRef']) and $r['RazonRef']!==false)
-                $texto = $texto.': '.$r['RazonRef'];
+            if (isset($r['RazonRef']) and $r['RazonRef']!==false) {
+                            $texto = $texto.': '.$r['RazonRef'];
+            }
             $this->setFont('', 'B', null);
             $this->Texto('Referencia', $x);
             $this->Texto(':', $x+$offset);
@@ -881,8 +906,9 @@ 
      */
     private function agregarDetalle($detalle, $x = 10)
     {
-        if (!isset($detalle[0]))
-            $detalle = [$detalle];
+        if (!isset($detalle[0])) {
+                    $detalle = [$detalle];
+        }
         $this->setFont('', '', $this->detalle_fuente);
         // titulos
         $titulos = [];
@@ -899,36 +925,42 @@ 
                     $item['NmbItem'] .= !$this->item_detalle_posicion ? '<br/>' : ': ';
                     $item['NmbItem'] .= '<span style="font-size:0.7em">'.$item['DscItem'].'</span>';
                 }
-                if (!in_array($col, $titulos_keys) or ($dte_exento and $col=='IndExe'))
-                    unset($item[$col]);
+                if (!in_array($col, $titulos_keys) or ($dte_exento and $col=='IndExe')) {
+                                    unset($item[$col]);
+                }
             }
             // ajustes a IndExe
             if (isset($item['IndExe'])) {
-                if ($item['IndExe']==1)
-                    $item['IndExe'] = 'EX';
-                else if ($item['IndExe']==2)
-                    $item['IndExe'] = 'NF';
+                if ($item['IndExe']==1) {
+                                    $item['IndExe'] = 'EX';
+                } else if ($item['IndExe']==2) {
+                                    $item['IndExe'] = 'NF';
+                }
             }
             // agregar todas las columnas que se podrían imprimir en la tabla
             $item_default = [];
-            foreach ($this->detalle_cols as $key => $info)
-                $item_default[$key] = false;
+            foreach ($this->detalle_cols as $key => $info) {
+                            $item_default[$key] = false;
+            }
             $item = array_merge($item_default, $item);
             // si hay código de item se extrae su valor
-            if ($item['CdgItem'])
-                $item['CdgItem'] = $item['CdgItem']['VlrCodigo'];
+            if ($item['CdgItem']) {
+                            $item['CdgItem'] = $item['CdgItem']['VlrCodigo'];
+            }
             // dar formato a números
             foreach (['QtyItem', 'PrcItem', 'DescuentoMonto', 'RecargoMonto', 'MontoItem'] as $col) {
-                if ($item[$col])
-                    $item[$col] = $this->num($item[$col]);
+                if ($item[$col]) {
+                                    $item[$col] = $this->num($item[$col]);
+                }
             }
         }
         // opciones
         $options = ['align'=>[]];
         $i = 0;
         foreach ($this->detalle_cols as $info) {
-            if (isset($info['width']))
-                $options['width'][$i] = $info['width'];
+            if (isset($info['width'])) {
+                            $options['width'][$i] = $info['width'];
+            }
             $options['align'][$i] = $info['align'];
             $i++;
         }
@@ -961,8 +993,9 @@ 
         $this->Texto($this->detalle_cols['QtyItem']['title'], $x+35, $this->y, ucfirst($this->detalle_cols['QtyItem']['align'][0]), $this->detalle_cols['QtyItem']['width']);
         $this->Texto($this->detalle_cols['MontoItem']['title'], $x+45, $this->y, ucfirst($this->detalle_cols['MontoItem']['align'][0]), $this->detalle_cols['MontoItem']['width']);
         $this->Line($p1x, $p1y+4, $p2x, $p2y+4, $style);
-        if (!isset($detalle[0]))
-            $detalle = [$detalle];
+        if (!isset($detalle[0])) {
+                    $detalle = [$detalle];
+        }
         $this->SetY($this->getY()+2);
         foreach($detalle as  &$d) {
             $this->MultiTexto($d['NmbItem'], $x+1, $this->y+4, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
@@ -1009,8 +1042,9 @@ 
      */
     private function agregarDescuentosRecargos(array $descuentosRecargos, $x = 10)
     {
-        if (!isset($descuentosRecargos[0]))
-            $descuentosRecargos = [$descuentosRecargos];
+        if (!isset($descuentosRecargos[0])) {
+                    $descuentosRecargos = [$descuentosRecargos];
+        }
         foreach($descuentosRecargos as $dr) {
             $tipo = $dr['TpoMov']=='D' ? 'Descuento' : 'Recargo';
             $valor = $dr['TpoValor']=='%' ? $dr['ValorDR'].'%' : $this->num($dr['ValorDR']);
@@ -1033,8 +1067,9 @@ 
      */
     private 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) {
@@ -1258,11 +1293,13 @@ 
      */
     private function num($n)
     {
-        if (!is_numeric($n))
-            return $n;
+        if (!is_numeric($n)) {
+                    return $n;
+        }
         $broken_number = explode('.', (string)$n);
-        if (isset($broken_number[1]))
-            return number_format($broken_number[0], 0, ',', '.').','.$broken_number[1];
+        if (isset($broken_number[1])) {
+                    return number_format($broken_number[0], 0, ',', '.').','.$broken_number[1];
+        }
         return number_format($broken_number[0], 0, ',', '.');
     }
 

lib/Sii/LibroGuia.php

Spacing +7 added lines, -7 removed lines

@@ -79,11 +79,11 @@ 
         }
         // calcular valores que no se hayan entregado
         if (!$detalle['IVA'] and $detalle['TasaImp'] and $detalle['MntNeto']) {
-            $detalle['IVA'] = round($detalle['MntNeto'] * ($detalle['TasaImp']/100));
+            $detalle['IVA'] = round($detalle['MntNeto'] * ($detalle['TasaImp'] / 100));
         }
         // calcular monto total si no se especificó
         if ($detalle['MntTotal']===false) {
-            $detalle['MntTotal'] = $detalle['MntNeto'] + $detalle['IVA'];
+            $detalle['MntTotal'] = $detalle['MntNeto']+$detalle['IVA'];
         }
     }
 
@@ -100,7 +100,7 @@ 
         $data = \sasco\LibreDTE\CSV::read($archivo);
         $n_data = count($data);
         $detalles = [];
-        for ($i=1; $i<$n_data; $i++) {
+        for ($i = 1; $i<$n_data; $i++) {
             // detalle genérico
             $detalle = [
                 'Folio' => $data[$i][0],
@@ -203,17 +203,17 @@ 
             // se contabiliza si la guía está anulada
             if ($d['Anulado']==1 or $d['Anulado']==2) {
                 if ($d['Anulado']==1) {
-                    $ResumenPeriodo['TotFolAnulado'] = (int)$ResumenPeriodo['TotFolAnulado'] + 1;
+                    $ResumenPeriodo['TotFolAnulado'] = (int)$ResumenPeriodo['TotFolAnulado']+1;
                 } else {
-                    $ResumenPeriodo['TotGuiaAnulada'] = (int)$ResumenPeriodo['TotGuiaAnulada'] + 1;
+                    $ResumenPeriodo['TotGuiaAnulada'] = (int)$ResumenPeriodo['TotGuiaAnulada']+1;
                 }
             }
             // si no está anulado
             else {
                 // si es de venta
                 if ($d['TpoOper']==1) {
-                    $ResumenPeriodo['TotGuiaVenta'] = (int)$ResumenPeriodo['TotGuiaVenta'] + 1;
-                    $ResumenPeriodo['TotMntGuiaVta'] = (int)$ResumenPeriodo['TotMntGuiaVta'] + $d['MntTotal'];
+                    $ResumenPeriodo['TotGuiaVenta'] = (int)$ResumenPeriodo['TotGuiaVenta']+1;
+                    $ResumenPeriodo['TotMntGuiaVta'] = (int)$ResumenPeriodo['TotMntGuiaVta']+$d['MntTotal'];
                 }
                 // si no es de venta
                 else {

lib/Sii/PDF/LibroCompraVenta.php

Braces +10 added lines, -7 removed lines

@@ -57,8 +57,9 @@ 
     {
         $this->startPageGroup();
         $this->AddPage();
-        if (isset($libro['LibroCompraVenta']))
-            $libro = $libro['LibroCompraVenta'];
+        if (isset($libro['LibroCompraVenta'])) {
+                    $libro = $libro['LibroCompraVenta'];
+        }
         // título del libro
         $this->SetFont('helvetica', 'B', 16);
         $this->Texto('Libro de '.ucfirst(strtolower($libro['EnvioLibro']['Caratula']['TipoOperacion'])), null, null, 'C');
@@ -109,8 +110,7 @@ 
                     $total_resumen['TotIVARetTotal'] += !empty($total['TotIVARetTotal']) ? $total['TotIVARetTotal'] : 0;
                     $total_resumen['TotIVANoRetenido'] += !empty($total['TotIVANoRetenido']) ? $total['TotIVANoRetenido'] : 0;
                     $total_resumen['TotMntTotal'] += $total['TotMntTotal'];
-                }
-                else if (in_array($total['TpoDoc'], $this->dte_tipo_operacion['resta'])) {
+                } else if (in_array($total['TpoDoc'], $this->dte_tipo_operacion['resta'])) {
                     $total_resumen['TotMntExe'] -= !empty($total['TotMntExe']) ? $total['TotMntExe'] : 0;
                     $total_resumen['TotMntNeto'] -= !empty($total['TotMntNeto']) ? $total['TotMntNeto'] : 0;
                     $total_resumen['TotMntIVA'] -= !empty($total['TotMntIVA']) ? $total['TotMntIVA'] : 0;
@@ -171,10 +171,13 @@ 
             foreach ($libro['EnvioLibro']['Detalle'] as $d) {
                 // impuesto adicional
                 if (!empty($d['OtrosImp'])) {
-                    if (!isset($d['OtrosImp'][0]))
-                        $d['OtrosImp'] = [$d['OtrosImp']];
+                    if (!isset($d['OtrosImp'][0])) {
+                                            $d['OtrosImp'] = [$d['OtrosImp']];
+                    }
                     $n_OtrosImp = count($d['OtrosImp']);
-                } else $n_OtrosImp = 0;
+                } else {
+                    $n_OtrosImp = 0;
+                }
                 // se agrega detalle
                 $detalle[] = [
                     $d['TpoDoc'],

lib/Sii/LibroCompraVenta.php

Braces +54 added lines, -36 removed lines

@@ -96,10 +96,12 @@ 
      */
     public function agregar(array $detalle, $normalizar = true)
     {
-        if ($normalizar)
-            $this->normalizarDetalle($detalle);
-        if (!$detalle['TpoDoc'])
-            return false;
+        if ($normalizar) {
+                    $this->normalizarDetalle($detalle);
+        }
+        if (!$detalle['TpoDoc']) {
+                    return false;
+        }
         $this->detalles[] = $detalle;
         return true;
     }
@@ -166,8 +168,9 @@ 
         $detalle['RznSoc'] = mb_substr($detalle['RznSoc'], 0, 50);
         // calcular valores que no se hayan entregado
         if (isset($detalle['FctProp'])) {
-            if ($detalle['IVAUsoComun']===false)
-                $detalle['IVAUsoComun'] = round($detalle['MntNeto'] * ($detalle['TasaImp']/100));
+            if ($detalle['IVAUsoComun']===false) {
+                            $detalle['IVAUsoComun'] = round($detalle['MntNeto'] * ($detalle['TasaImp']/100));
+            }
         } else if (!$detalle['MntIVA'] and !is_array($detalle['IVANoRec']) and $detalle['TasaImp'] and $detalle['MntNeto']) {
             $detalle['MntIVA'] = round($detalle['MntNeto'] * ($detalle['TasaImp']/100));
         }
@@ -177,8 +180,9 @@ 
         }
         // normalizar IVA no recuperable
         if (!empty($detalle['IVANoRec'])) {
-            if (!isset($detalle['IVANoRec'][0]))
-                $detalle['IVANoRec'] = [$detalle['IVANoRec']];
+            if (!isset($detalle['IVANoRec'][0])) {
+                            $detalle['IVANoRec'] = [$detalle['IVANoRec']];
+            }
             // si son múltiples iva no recuperable se arma arreglo real
             if (strpos($detalle['IVANoRec'][0]['CodIVANoRec'], ',')) {
                 $CodIVANoRec = explode(',', $detalle['IVANoRec'][0]['CodIVANoRec']);
@@ -195,8 +199,9 @@ 
         }
         // normalizar otros impuestos
         if (!empty($detalle['OtrosImp'])) {
-            if (!isset($detalle['OtrosImp'][0]))
-                $detalle['OtrosImp'] = [$detalle['OtrosImp']];
+            if (!isset($detalle['OtrosImp'][0])) {
+                            $detalle['OtrosImp'] = [$detalle['OtrosImp']];
+            }
             // si son múltiples impuestos se arma arreglo real
             if (strpos($detalle['OtrosImp'][0]['CodImp'], ',')) {
                 $CodImp = explode(',', $detalle['OtrosImp'][0]['CodImp']);
@@ -257,8 +262,9 @@ 
         }
         // si el código de sucursal no existe se pone a falso, esto básicamente
         // porque algunos sistemas podrían usar 0 cuando no hay CdgSIISucur
-        if (!$detalle['CdgSIISucur'])
-            $detalle['CdgSIISucur'] = false;
+        if (!$detalle['CdgSIISucur']) {
+                    $detalle['CdgSIISucur'] = false;
+        }
     }
 
     /**
@@ -513,8 +519,9 @@ 
             'TipoEnvio' => 'TOTAL',
             'FolioNotificacion' => false,
         ], $caratula);
-        if ($this->caratula['TipoEnvio']=='ESPECIAL')
-            $this->caratula['FolioNotificacion'] = null;
+        if ($this->caratula['TipoEnvio']=='ESPECIAL') {
+                    $this->caratula['FolioNotificacion'] = null;
+        }
         $this->id = 'LibreDTE_LIBRO_'.$this->caratula['TipoOperacion'].'_'.str_replace('-', '', $this->caratula['RutEmisorLibro']).'_'.str_replace('-', '', $this->caratula['PeriodoTributario']).'_'.date('U');
     }
 
@@ -528,8 +535,9 @@ 
     public function generar($incluirDetalle = true)
     {
         // si ya se había generado se entrega directamente
-        if ($this->xml_data)
-            return $this->xml_data;
+        if ($this->xml_data) {
+                    return $this->xml_data;
+        }
         // generar totales de DTE y sus montos
         $TotalesPeriodo = $this->getResumen();
         $ResumenPeriodo = $TotalesPeriodo ? ['TotalesPeriodo'=>$TotalesPeriodo] : false;
@@ -604,13 +612,16 @@ 
             }
             $totales[$d['TpoDoc']]['TotMntTotal'] += $d['MntTotal'];
             // contabilizar documentos anulados
-            if (!empty($d['Anulado']) and $d['Anulado']=='A')
-                $totales[$d['TpoDoc']]['TotAnulado']++;
+            if (!empty($d['Anulado']) and $d['Anulado']=='A') {
+                            $totales[$d['TpoDoc']]['TotAnulado']++;
+            }
             // si hay activo fijo se contabiliza
-            if (!empty($d['MntActivoFijo']))
-                $totales[$d['TpoDoc']]['TotMntActivoFijo'] += $d['MntActivoFijo'];
-            if (!empty($d['MntIVAActivoFijo']))
-                $totales[$d['TpoDoc']]['TotMntIVAActivoFijo'] += $d['MntIVAActivoFijo'];
+            if (!empty($d['MntActivoFijo'])) {
+                            $totales[$d['TpoDoc']]['TotMntActivoFijo'] += $d['MntActivoFijo'];
+            }
+            if (!empty($d['MntIVAActivoFijo'])) {
+                            $totales[$d['TpoDoc']]['TotMntIVAActivoFijo'] += $d['MntIVAActivoFijo'];
+            }
             // si hay iva no recuperable se contabiliza
             if (!empty($d['IVANoRec'])) {
                 foreach ($d['IVANoRec'] as $IVANoRec) {
@@ -633,8 +644,9 @@ 
                 unset($d['FctProp']); // se quita el factor de proporcionalidad del detalle ya que no es parte del XML
             }
             // contabilizar IVA fuera de plazo
-            if (!empty($d['IVAFueraPlazo']))
-                $totales[$d['TpoDoc']]['TotIVAFueraPlazo'] += $d['IVAFueraPlazo'];
+            if (!empty($d['IVAFueraPlazo'])) {
+                            $totales[$d['TpoDoc']]['TotIVAFueraPlazo'] += $d['IVAFueraPlazo'];
+            }
             // si hay otro tipo de impuesto se contabiliza
             if (!empty($d['OtrosImp'])) {
                 foreach ($d['OtrosImp'] as $OtrosImp) {
@@ -648,17 +660,21 @@ 
                 }
             }
             // contabilizar impuesto sin derecho a crédito
-            if (!empty($d['MntSinCred']))
-                $totales[$d['TpoDoc']]['TotImpSinCredito'] += $d['MntSinCred'];
+            if (!empty($d['MntSinCred'])) {
+                            $totales[$d['TpoDoc']]['TotImpSinCredito'] += $d['MntSinCred'];
+            }
             // contabilidad IVA retenido total
-            if (!empty($d['IVARetTotal']))
-                $totales[$d['TpoDoc']]['TotIVARetTotal'] += $d['IVARetTotal'];
+            if (!empty($d['IVARetTotal'])) {
+                            $totales[$d['TpoDoc']]['TotIVARetTotal'] += $d['IVARetTotal'];
+            }
             // contabilizar IVA retenido parcial
-            if (!empty($d['IVARetParcial']))
-                $totales[$d['TpoDoc']]['TotIVARetParcial'] += $d['IVARetParcial'];
+            if (!empty($d['IVARetParcial'])) {
+                            $totales[$d['TpoDoc']]['TotIVARetParcial'] += $d['IVARetParcial'];
+            }
             // contabilizar IVA no retenido
-            if (!empty($d['IVANoRetenido']))
-                $totales[$d['TpoDoc']]['TotIVANoRetenido'] += $d['IVANoRetenido'];
+            if (!empty($d['IVANoRetenido'])) {
+                            $totales[$d['TpoDoc']]['TotIVANoRetenido'] += $d['IVANoRetenido'];
+            }
         }
         // agregar resumenes pasados que no se hayan generado por los detalles
         foreach ($this->resumen as $tipo => $resumen) {
@@ -682,8 +698,9 @@ 
         $manual = [];
         if (isset($this->toArray()['LibroCompraVenta']['EnvioLibro']['ResumenPeriodo']['TotalesPeriodo'])) {
             $totales = $this->toArray()['LibroCompraVenta']['EnvioLibro']['ResumenPeriodo']['TotalesPeriodo'];
-            if (!isset($totales[0]))
-                $totales = [$totales];
+            if (!isset($totales[0])) {
+                            $totales = [$totales];
+            }
             foreach ($totales as $total) {
                 if (isset($total['TpoDoc']) and in_array($total['TpoDoc'], [35, 38, 48])) {
                     $manual[$total['TpoDoc']] = array_merge($this->total_default, $total);
@@ -703,8 +720,9 @@ 
         $manual = [];
         if (isset($this->toArray()['LibroCompraVenta']['EnvioLibro']['ResumenPeriodo']['TotalesPeriodo'])) {
             $totales = $this->toArray()['LibroCompraVenta']['EnvioLibro']['ResumenPeriodo']['TotalesPeriodo'];
-            if (!isset($totales[0]))
-                $totales = [$totales];
+            if (!isset($totales[0])) {
+                            $totales = [$totales];
+            }
             foreach ($totales as $total) {
                 if (in_array($total['TpoDoc'], [39, 41])) {
                     $manual[$total['TpoDoc']] = array_merge($this->total_default, $total);