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-10-22
      */

lib/Sii.php

Doc Comments +7 added lines, -7 removed lines

@@ -154,8 +154,8 @@ 
 
     /**
      * Método para realizar una solicitud al servicio web del SII
-     * @param wsdl Nombre del WSDL que se usará
-     * @param request Nombre de la función que se ejecutará en el servicio web
+     * @param wsdl string del WSDL que se usará
+     * @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 SimpleXMLElement con la espuesta del servicio web consultado
@@ -249,7 +249,7 @@ 
      * @param usuario RUN del usuario que envía el DTE
      * @param empresa RUT de la empresa emisora del DTE
      * @param dte Documento XML con el DTE que se desea enviar a SII
-     * @param token Token de autenticación automática ante el SII
+     * @param token string de autenticación automática ante el SII
      * @param gzip Permite enviar el archivo XML comprimido al servidor
      * @param retry Intentos que se realizarán como máximo para obtener respuesta
      * @return Respuesta XML desde SII o bien null si no se pudo obtener respuesta
@@ -357,7 +357,7 @@ 
      * \endcode
      *
      * @param idk IDK de la clave pública del SII. Si no se indica se tratará de determinar con el ambiente que se esté usando
-     * @return Contenido del certificado
+     * @return string|false del certificado
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-16
      */
@@ -382,7 +382,7 @@ 
     /**
      * Método que asigna el ambiente que se usará por defecto (si no está
      * asignado con la constante _LibreDTE_CERTIFICACION_)
-     * @param ambiente Ambiente a usar: Sii::PRODUCCION o Sii::CERTIFICACION
+     * @param ambiente integer a usar: Sii::PRODUCCION o Sii::CERTIFICACION
      * @warning No se está verificando SSL en ambiente de certificación
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-28
@@ -417,7 +417,7 @@ 
 
     /**
      * Método que entrega la tasa de IVA vigente
-     * @return Tasa de IVA vigente
+     * @return integer de IVA vigente
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-03
      */
@@ -493,7 +493,7 @@ 
     /**
      * Método que entrega la dirección regional según la comuna que se esté
      * consultando
-     * @param comuna de la sucursal del emior o bien código de la sucursal del SII
+     * @param comuna Sii\PDF\Código|null la sucursal del emior o bien código de la sucursal del SII
      * @return Dirección regional del SII
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-11-07

Spacing +6 added lines, -6 removed lines

@@ -197,7 +197,7 @@ 
             \sasco\LibreDTE\Log::write(Estado::REQUEST_ERROR_SOAP, Estado::get(Estado::REQUEST_ERROR_SOAP, $msg));
             return false;
         }
-        for ($i=0; $i<$retry; $i++) {
+        for ($i = 0; $i<$retry; $i++) {
             try {
                 if ($args) {
                     $body = call_user_func_array([$soap, $request], $args);
@@ -265,7 +265,7 @@ 
             $dte = '<?xml version="1.0" encoding="ISO-8859-1"?>'."\n".$dte;
         }
         do {
-            $file = sys_get_temp_dir().'/dte_'.md5(microtime().$token.$dte).'.'.($gzip?'gz':'xml');
+            $file = sys_get_temp_dir().'/dte_'.md5(microtime().$token.$dte).'.'.($gzip ? 'gz' : 'xml');
         } while (file_exists($file));
         if ($gzip) {
             $dte = gzencode($dte);
@@ -314,7 +314,7 @@ 
             curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);
         }
         // enviar XML al SII
-        for ($i=0; $i<$retry; $i++) {
+        for ($i = 0; $i<$retry; $i++) {
             $response = curl_exec($curl);
             if ($response and $response!='Error 500') {
                 break;
@@ -343,7 +343,7 @@ 
         if ($xml->STATUS!=0) {
             \sasco\LibreDTE\Log::write(
                 $xml->STATUS,
-                Estado::get($xml->STATUS).(isset($xml->DETAIL)?'. '.implode("\n", (array)$xml->DETAIL->ERROR):'')
+                Estado::get($xml->STATUS).(isset($xml->DETAIL) ? '. '.implode("\n", (array)$xml->DETAIL->ERROR) : '')
             );
         }
         return $xml;
@@ -475,12 +475,12 @@ 
         $lines = explode("\n", $response);
         $n_lines = count($lines);
         $data = [];
-        for ($i=1; $i<$n_lines; $i++) {
+        for ($i = 1; $i<$n_lines; $i++) {
             $row = str_getcsv($lines[$i], ';', '');
             unset($lines[$i]);
             if (!isset($row[5]))
                 continue;
-            for ($j=0; $j<6; $j++)
+            for ($j = 0; $j<6; $j++)
                 $row[$j] = trim($row[$j]);
             $row[1] = utf8_decode($row[1]);
             $row[4] = strtolower($row[4]);

Braces +29 added lines, -20 removed lines

@@ -138,8 +138,9 @@ 
         // entregar WSDL local (modificados para ambiente de certificación)
         if ($ambiente==self::CERTIFICACION) {
             $wsdl = dirname(dirname(__FILE__)).'/wsdl/'.self::$config['servidor'][$ambiente].'/'.$servicio.'.jws';
-            if (is_readable($wsdl))
-                return $wsdl;
+            if (is_readable($wsdl)) {
+                            return $wsdl;
+            }
         }
         // entregar WSDL oficial desde SII
         $location = isset(self::$config['wsdl'][$servicio]) ? self::$config['wsdl'][$servicio] : self::$config['wsdl']['*'];
@@ -168,10 +169,12 @@ 
             $retry = (int)$args;
             $args = null;
         }
-        if (!$retry)
-            $retry = self::$retry;
-        if ($args and !is_array($args))
-            $args = [$args];
+        if (!$retry) {
+                    $retry = self::$retry;
+        }
+        if ($args and !is_array($args)) {
+                    $args = [$args];
+        }
         if (!self::$verificar_ssl) {
             if (self::getAmbiente()==self::PRODUCCION) {
                 $msg = Estado::get(Estado::ENVIO_SSL_SIN_VERIFICAR);
@@ -366,8 +369,9 @@ 
         // si se pasó un idk y existe el archivo asociado se entrega
         if ($idk) {
             $cert = dirname(dirname(__FILE__)).'/certs/'.$idk.'.cer';
-            if (is_readable($cert))
-                return file_get_contents($cert);
+            if (is_readable($cert)) {
+                            return file_get_contents($cert);
+            }
         }
         // buscar certificado y entregar si existe o =false si no
         $ambiente = self::getAmbiente();
@@ -407,10 +411,11 @@ 
     public static function getAmbiente($ambiente = null)
     {
         if ($ambiente===null) {
-            if (defined('_LibreDTE_CERTIFICACION_'))
-                $ambiente = (int)_LibreDTE_CERTIFICACION_;
-            else
-                $ambiente = self::$ambiente;
+            if (defined('_LibreDTE_CERTIFICACION_')) {
+                            $ambiente = (int)_LibreDTE_CERTIFICACION_;
+            } else {
+                            $ambiente = self::$ambiente;
+            }
         }
         return $ambiente;
     }
@@ -436,8 +441,9 @@ 
     {
         // solicitar token
         $token = \sasco\LibreDTE\Sii\Autenticacion::getToken($Firma);
-        if (!$token)
-            return false;
+        if (!$token) {
+                    return false;
+        }
         // definir ambiente y servidor
         $ambiente = self::getAmbiente($ambiente);
         $servidor = self::$config['servidor'][$ambiente];
@@ -466,8 +472,9 @@ 
         }
         // realizar consulta curl
         $response = curl_exec($curl);
-        if (!$response)
-            return false;
+        if (!$response) {
+                    return false;
+        }
         // cerrar sesión curl
         curl_close($curl);
         // entregar datos del archivo CSV
@@ -478,10 +485,12 @@ 
         for ($i=1; $i<$n_lines; $i++) {
             $row = str_getcsv($lines[$i], ';', '');
             unset($lines[$i]);
-            if (!isset($row[5]))
-                continue;
-            for ($j=0; $j<6; $j++)
-                $row[$j] = trim($row[$j]);
+            if (!isset($row[5])) {
+                            continue;
+            }
+            for ($j=0; $j<6; $j++) {
+                            $row[$j] = trim($row[$j]);
+            }
             $row[1] = utf8_decode($row[1]);
             $row[4] = strtolower($row[4]);
             $row[5] = strtolower($row[5]);

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
      */
@@ -1750,7 +1750,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
      * @warning No se está validando la firma
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-08
@@ -1861,7 +1861,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
@@ -1902,7 +1902,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
      * @todo Corregir warning y también definir que se retornará (sobre todo en caso de error)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)

Spacing +16 added lines, -16 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;
     }
 
     /**
@@ -529,7 +529,7 @@ 
     {
         $this->getDatos();
         // generar resumen
-        $resumen =  [
+        $resumen = [
             'TpoDoc' => (int)$this->datos['Encabezado']['IdDoc']['TipoDTE'],
             'NroDoc' => (int)$this->datos['Encabezado']['IdDoc']['Folio'],
             'TasaImp' => 0,
@@ -574,17 +574,17 @@ 
      */
     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)
         // se calcula el monto neto y luego se obtiene el IVA haciendo la resta
         // 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];
     }
 
@@ -1418,12 +1418,12 @@ 
                     );
                     // aplicar descuento
                     if ($d['DescuentoPct']) {
-                        $d['DescuentoMonto'] = round($d['MontoItem'] * (int)$d['DescuentoPct']/100);
+                        $d['DescuentoMonto'] = round($d['MontoItem'] * (int)$d['DescuentoPct'] / 100);
                     }
                     $d['MontoItem'] -= $d['DescuentoMonto'];
                     // aplicar recargo
                     if ($d['RecargoPct']) {
-                        $d['RecargoMonto'] = round($d['MontoItem'] * (int)$d['RecargoPct']/100);
+                        $d['RecargoMonto'] = round($d['MontoItem'] * (int)$d['RecargoPct'] / 100);
                     }
                     $d['MontoItem'] += $d['RecargoMonto'];
                     // aproximar monto del item
@@ -1509,7 +1509,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
@@ -1582,7 +1582,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
                 );
             }
         }
@@ -1590,7 +1590,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]);
                 }
@@ -1618,7 +1618,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)
                     );
                 }
             }
@@ -1642,7 +1642,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
@@ -1794,7 +1794,7 @@ 
         $SignatureValue = $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue;
         $X509Certificate = $Signature->getElementsByTagName('X509Certificate')->item(0)->nodeValue;
         $X509Certificate = '-----BEGIN CERTIFICATE-----'."\n".wordwrap(trim($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));
     }

lib/Sii/PDF/Dte.php

Doc Comments +31 added lines, -28 removed lines

@@ -177,7 +177,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
      */
@@ -356,7 +356,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-10-24
@@ -512,10 +512,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
      */
@@ -586,11 +587,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)
     {
@@ -636,7 +639,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
      */
@@ -724,7 +727,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
      */
@@ -811,7 +814,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
      */
@@ -882,7 +885,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
      */
@@ -919,7 +922,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
@@ -986,7 +989,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)
@@ -1022,7 +1025,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
      */
@@ -1048,7 +1051,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
      */
@@ -1072,7 +1075,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
      */
@@ -1209,9 +1212,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-10-05
      */
@@ -1249,10 +1252,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 2017-10-05
      */
@@ -1280,10 +1283,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

@@ -171,7 +171,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;
     }
 
     /**
@@ -316,7 +316,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(
@@ -376,15 +376,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,
@@ -437,7 +437,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);
@@ -527,13 +527,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;
@@ -544,14 +544,14 @@ 
         }
         // 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);
         $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', $w);
+        $this->MultiTexto($emisor['DirOrigen'].($comuna ? (', '.$comuna) : '').($ciudad ? (', '.$ciudad) : ''), $x, $this->y, 'L', $w);
         if (!empty($emisor['Sucursal'])) {
             $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', $w);
         }
@@ -595,12 +595,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);
@@ -613,7 +613,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;
     }
@@ -744,7 +744,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);
@@ -761,7 +761,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);
@@ -802,7 +802,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();
     }
@@ -890,7 +890,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']).' ';
@@ -997,9 +997,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']);
@@ -1009,9 +1009,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']);
         }
@@ -1031,7 +1031,7 @@ 
         if (!isset($detalle[0])) {
             $detalle = [$detalle];
         }
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             if (!empty($d['MontoItem'])) {
                 $subtotal += $d['MontoItem'];
             }
@@ -1056,13 +1056,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();
@@ -1082,8 +1082,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();
         }
     }
@@ -1134,7 +1134,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'].'%) $';
@@ -1226,7 +1226,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',
             ];
@@ -1259,7 +1259,7 @@ 
     private function agregarAcuseRecibo($x = 93, $y = 190, $w = 50, $h = 40)
     {
         $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);
@@ -1289,9 +1289,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);
@@ -1321,7 +1321,7 @@ 
         $y = (!$this->papelContinuo and !$this->timbre_pie) ? $this->x_fin_datos : $y;
         $y += 64;
         $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');
     }
 
     /**
@@ -1332,7 +1332,7 @@ 
     private 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');
     }
 
     /**
@@ -1363,7 +1363,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);

lib/XML.php

Doc Comments +6 added lines, -5 removed lines

@@ -52,8 +52,8 @@ 
      * Método que genera nodos XML a partir de un arreglo
      * @param data Arreglo con los datos que se usarán para generar XML
      * @param namespace Arreglo con el espacio de nombres para el XML que se generará (URI y prefijo)
-     * @param parent DOMElement padre para los elementos, o =null para que sea la raíz
-     * @return Objeto \sasco\LibreDTE\XML
+     * @param parent \DOMElement padre para los elementos, o =null para que sea la raíz
+     * @return XML \sasco\LibreDTE\XML
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-10-22
      */
@@ -153,7 +153,7 @@ 
     /**
      * Método para realizar consultas XPATH al documento XML
      * @param expression Expresión XPath a ejecutar
-     * @return DOMNodeList
+     * @return \DOMNodeList
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-08-05
      */
@@ -274,7 +274,7 @@ 
      * Método que cuenta los nodos con el mismo nombre hijos deun DOMElement
      * No sirve usar: $dom->getElementsByTagName($tagName)->length ya que esto
      * entrega todos los nodos con el nombre, sean hijos, nietos, etc.
-     * @return Cantidad de nodos hijos con el mismo nombre en el DOMElement
+     * @return integer de nodos hijos con el mismo nombre en el DOMElement
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -314,7 +314,7 @@ 
 
     /**
      * Método que entrega el nombre del archivo del schema del XML
-     * @return Nombre del schema o bien =false si no se encontró
+     * @return string del schema o bien =false si no se encontró
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-12-14
      */
@@ -356,6 +356,7 @@ 
      * respeta el estándar y las requiere convertidas
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-01-20
+     * @param string $xml
      */
     private function fixEntities($xml)
     {

Spacing +4 added lines, -4 removed lines

@@ -57,7 +57,7 @@ 
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-10-22
      */
-    public function generate(array $data, array $namespace = null, \DOMElement &$parent = null)
+    public function generate(array $data, array $namespace = null, \DOMElement & $parent = null)
     {
         if ($parent===null) {
             $parent = &$this;
@@ -197,7 +197,7 @@ 
      */
     private function utf2iso($string)
     {
-        return mb_detect_encoding($string, ['UTF-8', 'ISO-8859-1']) != 'ISO-8859-1' ? utf8_decode($string) : $string;
+        return mb_detect_encoding($string, ['UTF-8', 'ISO-8859-1'])!='ISO-8859-1' ? utf8_decode($string) : $string;
     }
 
     /**
@@ -237,7 +237,7 @@ 
         }
         // agregar nodos hijos
         if ($dom->hasChildNodes()) {
-            foreach($dom->childNodes as $child) {
+            foreach ($dom->childNodes as $child) {
                 if ($child instanceof \DOMText) {
                     $textContent = trim($child->textContent);
                     if ($textContent!="") {
@@ -362,7 +362,7 @@ 
         $newXML = '';
         $n_letras = strlen($xml);
         $convertir = false;
-        for ($i=0; $i<$n_letras; ++$i) {
+        for ($i = 0; $i<$n_letras; ++$i) {
             if ($xml[$i]=='>')
                 $convertir = true;
             if ($xml[$i]=='<')

Braces +42 added lines, -30 removed lines

@@ -120,8 +120,9 @@ 
     private function sanitize($txt)
     {
         // si no se paso un texto o bien es un número no se hace nada
-        if (!$txt or is_numeric($txt))
-            return $txt;
+        if (!$txt or is_numeric($txt)) {
+                    return $txt;
+        }
         // convertir "predefined entities" de XML
         $txt = str_replace(
             ['&', '&', '<', '<', '>', '&#62', '"', '"', ''', '''],
@@ -174,8 +175,9 @@ 
     {
         if ($xpath) {
             $node = $this->xpath($xpath)->item(0);
-            if (!$node)
-                return false;
+            if (!$node) {
+                            return false;
+            }
             $xml = $this->utf2iso($node->C14N());
             $xml = $this->fixEntities($xml);
         } else {
@@ -222,12 +224,15 @@ 
     public function toArray(\DOMElement $dom = null, array &$array = null, $arregloNodos = false)
     {
         // determinar valores de parámetros
-        if (!$dom)
-            $dom = $this->documentElement;
-        if (!$dom)
-            return false;
-        if ($array===null)
-            $array = [$dom->tagName => null];
+        if (!$dom) {
+                    $dom = $this->documentElement;
+        }
+        if (!$dom) {
+                    return false;
+        }
+        if ($array===null) {
+                    $array = [$dom->tagName => null];
+        }
         // agregar atributos del nodo
         if ($dom->hasAttributes()) {
             $array[$dom->tagName]['@attributes'] = [];
@@ -243,22 +248,24 @@ 
                     if ($textContent!="") {
                         if ($dom->childNodes->length==1 and empty($array[$dom->tagName])) {
                             $array[$dom->tagName] = $textContent;
-                        } else
-                            $array[$dom->tagName]['@value'] = $textContent;
+                        } else {
+                                                    $array[$dom->tagName]['@value'] = $textContent;
+                        }
                     }
-                }
-                else if ($child instanceof \DOMElement) {
+                } else if ($child instanceof \DOMElement) {
                     $nodos_gemelos = $this->countTwins($dom, $child->tagName);
                     if ($nodos_gemelos==1) {
-                        if ($arregloNodos)
-                            $this->toArray($child, $array);
-                        else
-                            $this->toArray($child, $array[$dom->tagName]);
+                        if ($arregloNodos) {
+                                                    $this->toArray($child, $array);
+                        } else {
+                                                    $this->toArray($child, $array[$dom->tagName]);
+                        }
                     }
                     // crear arreglo con nodos hijos que tienen el mismo nombre de tag
                     else {
-                        if (!isset($array[$dom->tagName][$child->tagName]))
-                            $array[$dom->tagName][$child->tagName] = [];
+                        if (!isset($array[$dom->tagName][$child->tagName])) {
+                                                    $array[$dom->tagName][$child->tagName] = [];
+                        }
                         $siguiente = count($array[$dom->tagName][$child->tagName]);
                         $array[$dom->tagName][$child->tagName][$siguiente] = [];
                         $this->toArray($child, $array[$dom->tagName][$child->tagName][$siguiente], true);
@@ -282,8 +289,9 @@ 
     {
         $twins = 0;
         foreach ($dom->childNodes as $child) {
-            if ($child instanceof \DOMElement and $child->tagName==$tagName)
-                $twins++;
+            if ($child instanceof \DOMElement and $child->tagName==$tagName) {
+                            $twins++;
+            }
         }
         return $twins;
     }
@@ -297,8 +305,9 @@ 
     public function getErrors()
     {
         $errors = [];
-        foreach (libxml_get_errors() as $e)
-            $errors[] = $e->message;
+        foreach (libxml_get_errors() as $e) {
+                    $errors[] = $e->message;
+        }
         return $errors;
     }
 
@@ -321,8 +330,9 @@ 
     public function getSchema()
     {
         $schemaLocation = $this->documentElement->getAttribute('xsi:schemaLocation');
-        if (!$schemaLocation or strpos($schemaLocation, ' ')===false)
-            return false;
+        if (!$schemaLocation or strpos($schemaLocation, ' ')===false) {
+                    return false;
+        }
         list($uri, $xsd) = explode(' ', $schemaLocation);
         return $xsd;
     }
@@ -363,10 +373,12 @@ 
         $n_letras = strlen($xml);
         $convertir = false;
         for ($i=0; $i<$n_letras; ++$i) {
-            if ($xml[$i]=='>')
-                $convertir = true;
-            if ($xml[$i]=='<')
-                $convertir = false;
+            if ($xml[$i]=='>') {
+                            $convertir = true;
+            }
+            if ($xml[$i]=='<') {
+                            $convertir = false;
+            }
             if ($convertir) {
                 $l = $xml[$i]=='\'' ? '&apos;' : ($xml[$i]=='"' ? '&quot;' : $xml[$i]);
             } else {

lib/Sii/LibroCompraVenta.php

Braces +30 added lines, -20 removed lines

@@ -102,10 +102,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;
     }
@@ -174,8 +176,9 @@ 
         }
         // 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));
         }
@@ -185,8 +188,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']);
@@ -203,8 +207,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']);
@@ -265,8 +270,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;
+        }
     }
 
     /**
@@ -521,8 +527,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');
     }
 
@@ -536,8 +543,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;
@@ -702,8 +710,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);
@@ -723,8 +732,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);

lib/Sii/Certificacion/SetPruebas.php

Braces +46 added lines, -32 removed lines

@@ -154,8 +154,9 @@ 
         foreach ($casos as $caso) {
             // determinar tipo documento y folio
             $TipoDTE = self::$tipos[$caso['documento']];
-            if (!isset($folios[$TipoDTE]))
-                $folios[$TipoDTE] = 1;
+            if (!isset($folios[$TipoDTE])) {
+                            $folios[$TipoDTE] = 1;
+            }
             $Folio = $folios[$TipoDTE];
             // crear encabezado del documento
             $documento = [
@@ -257,14 +258,17 @@ 
                     foreach ($item as $col => $val) {
                         $col = self::$item_cols[$col];
                         // procesar cada valor de acuerdo al nombre de la columna
-                        if (in_array($col, ['DescuentoPct', 'RecargoPct']))
-                            $detalle[$col] = substr($val, 0, -1);
-                        else
-                            $detalle[$col] = utf8_encode($val); // se convierte de ISO-8859-1 a UTF-8
+                        if (in_array($col, ['DescuentoPct', 'RecargoPct'])) {
+                                                    $detalle[$col] = substr($val, 0, -1);
+                        } else {
+                                                    $detalle[$col] = utf8_encode($val);
+                        }
+                        // se convierte de ISO-8859-1 a UTF-8
                     }
                     // si el item es EXENTO se agrega campo que lo indica
-                    if (strpos($detalle['NmbItem'], 'EXENTO'))
-                        $detalle['IndExe'] = 1;
+                    if (strpos($detalle['NmbItem'], 'EXENTO')) {
+                                            $detalle['IndExe'] = 1;
+                    }
                     // si hay una referencia se completa con los campos del
                     // detalle de la referencia que no estén en este detalle
                     if (!empty($caso['referencia'])) {
@@ -274,8 +278,9 @@ 
                         for ($i=0; $i<$n_detalle_r; $i++) {
                             if ($detalle_r[$i]['NmbItem']==$detalle['NmbItem']) {
                                 foreach ($detalle_r[$i] as $attr => $val) {
-                                    if (!isset($detalle[$attr]))
-                                        $detalle[$attr] = $val;
+                                    if (!isset($detalle[$attr])) {
+                                                                            $detalle[$attr] = $val;
+                                    }
                                 }
                             }
                         }
@@ -359,8 +364,9 @@ 
                     'ValorDR' => substr($caso['recargo'], 0, -1),
                 ];
             }
-            if (empty($documento['DscRcgGlobal']))
-                unset($documento['DscRcgGlobal']);
+            if (empty($documento['DscRcgGlobal'])) {
+                            unset($documento['DscRcgGlobal']);
+            }
             // agregar descuento del documento de la referencia
             else if (!empty($caso['referencia'])) {
                 $referencia = self::getReferencia($caso['referencia']['razon']);
@@ -392,14 +398,16 @@ 
                 // agregar totales
                 $documento['Encabezado']['Totales'] = $referencia['Totales'];
                 // agregar tasa de IVA si corresponde
-                if (isset($documento['Encabezado']['Totales']['TasaIVA']))
-                    $documento['Encabezado']['Totales']['TasaIVA'] = \sasco\LibreDTE\Sii::getIVA();
+                if (isset($documento['Encabezado']['Totales']['TasaIVA'])) {
+                                    $documento['Encabezado']['Totales']['TasaIVA'] = \sasco\LibreDTE\Sii::getIVA();
+                }
                 // si el documento referenciado es factura exenta y hay MntExe
                 if (isset($documento['Encabezado']['Totales']['MntExe'])) {
-                    if ($documentos[$caso['referencia']['caso']]['Encabezado']['IdDoc']['TipoDTE']==34)
-                        $documento['Encabezado']['Totales']['MntExe'] = 0;
-                    else
-                        unset($documento['Encabezado']['Totales']['MntExe']);
+                    if ($documentos[$caso['referencia']['caso']]['Encabezado']['IdDoc']['TipoDTE']==34) {
+                                            $documento['Encabezado']['Totales']['MntExe'] = 0;
+                    } else {
+                                            unset($documento['Encabezado']['Totales']['MntExe']);
+                    }
                 }
                 // si es documento de exportación se resetean los totales y se copia el tipo de moneda si no existe
                 if (in_array($documento['Encabezado']['IdDoc']['TipoDTE'], [111, 112])) {
@@ -412,8 +420,9 @@ 
             }
             // agregar referencia de exportación si existe
             if (!empty($caso['exportacion']['REFERENCIA'])) {
-                if (!is_array($caso['exportacion']['REFERENCIA']))
-                    $caso['exportacion']['REFERENCIA'] = [$caso['exportacion']['REFERENCIA']];
+                if (!is_array($caso['exportacion']['REFERENCIA'])) {
+                                    $caso['exportacion']['REFERENCIA'] = [$caso['exportacion']['REFERENCIA']];
+                }
                 foreach ($caso['exportacion']['REFERENCIA'] as $ref) {
                     $documento['Referencia'][] = [
                         'TpoDocRef' => self::$referencias_exportacion[$ref],
@@ -433,14 +442,16 @@ 
             if (isset($documento['Encabezado']['Totales'])) {
                 $hayValor = false;
                 foreach ($documento['Detalle'] as $d) {
-                    if (!empty($d['PrcItem']))
-                        $hayValor = true;
+                    if (!empty($d['PrcItem'])) {
+                                            $hayValor = true;
+                    }
                 }
                 if (!$hayValor) {
-                    if (isset($documento['Encabezado']['Totales']['MntExe']))
-                        $documento['Encabezado']['Totales'] = ['MntExe'=>0];
-                    else
-                        $documento['Encabezado']['Totales'] = [];
+                    if (isset($documento['Encabezado']['Totales']['MntExe'])) {
+                                            $documento['Encabezado']['Totales'] = ['MntExe'=>0];
+                    } else {
+                                            $documento['Encabezado']['Totales'] = [];
+                    }
                     $documento['Encabezado']['Totales']['MntTotal'] = 0;
                 }
             }
@@ -548,8 +559,9 @@ 
                     $n_lineas = count($lineas);
                     for ($i=$i; $i<$n_lineas; $i++) {
                         // si la línea está vacía se omite
-                        if (!$lineas[$i])
-                            continue;
+                        if (!$lineas[$i]) {
+                                                    continue;
+                        }
                         // si hay descuento global se guarda
                         if (strpos($lineas[$i], 'DESCUENTO GLOBAL ITEMES AFECTOS')===0) {
                             $aux = explode("\t", $lineas[$i]);
@@ -585,13 +597,15 @@ 
                                 }
                                 // agregar a los datos de aduanas
                                 else {
-                                    if (!isset($datos['exportacion']))
-                                        $datos['exportacion'] = [];
+                                    if (!isset($datos['exportacion'])) {
+                                                                            $datos['exportacion'] = [];
+                                    }
                                     if (!isset($datos['exportacion'][$var])) {
                                         $datos['exportacion'][$var] = $val;
                                     } else {
-                                        if (!is_array($datos['exportacion'][$var]))
-                                            $datos['exportacion'][$var] = [$datos['exportacion'][$var]];
+                                        if (!is_array($datos['exportacion'][$var])) {
+                                                                                    $datos['exportacion'][$var] = [$datos['exportacion'][$var]];
+                                        }
                                         $datos['exportacion'][$var][] = $val;
                                     }
                                 }