libredte / libredte-lib-core

lib/Sii/Certificacion/SetPruebas.php

Spacing +5 added lines, -5 removed lines

@@ -262,7 +262,7 @@ 
                         // buscar el caso y copiar sus columnas que no estén
                         $detalle_r = $documentos[$caso['referencia']['caso']]['Detalle'];
                         $n_detalle_r = count($detalle_r);
-                        for ($i=0; $i<$n_detalle_r; $i++) {
+                        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]))
@@ -325,7 +325,7 @@ 
                 $documento['DscRcgGlobal'][] = [
                     'TpoMov' => 'R',
                     'TpoValor' => '$',
-                    'ValorDR' => round((substr($caso['recargo-total-clausula'], 0, -1)/100) * $documento['Encabezado']['Transporte']['Aduana']['TotClauVenta'], 2),
+                    'ValorDR' => round((substr($caso['recargo-total-clausula'], 0, -1) / 100) * $documento['Encabezado']['Transporte']['Aduana']['TotClauVenta'], 2),
                 ];
             }
             // agregar recargo por flete y/o seguro, se agrega sólo en factura
@@ -466,7 +466,7 @@ 
         $casos = explode($separador, $archivo);
         $separador_len = strlen($separador);
         $n_casos = count($casos);
-        for ($i=1; $i<$n_casos; $i++) {
+        for ($i = 1; $i<$n_casos; $i++) {
             $caso = trim($casos[$i]);
             $caso_anterior = trim($casos[$i-1]);
             $caso_titulo = substr($caso_anterior, -$separador_len);
@@ -523,7 +523,7 @@ 
                 $titulos = array_slice(array_filter(explode("\t", $lineas[$linea_titulos_detalles])), 0);
                 // extraer detalles
                 $datos['detalle'] = [];
-                $i = $linea_titulos_detalles + 1;
+                $i = $linea_titulos_detalles+1;
                 while (!empty($lineas[$i])) {
                     $item = array_slice(array_filter(explode("\t", $lineas[$i])), 0);
                     $n_item = count($item);
@@ -537,7 +537,7 @@ 
                 // sólo continuar si hay más líneas, será, por ej, el descuento global
                 if (isset($lineas[$i])) {
                     $n_lineas = count($lineas);
-                    for ($i=$i; $i<$n_lineas; $i++) {
+                    for ($i = $i; $i<$n_lineas; $i++) {
                         // si la línea está vacía se omite
                         if (!$lineas[$i])
                             continue;

Braces +49 added lines, -34 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 = [
@@ -219,8 +220,9 @@ 
                         $documento['Encabezado']['Transporte']['Aduana']['TipoBultos']['EmisorSello'] = 'Sellos de Chile';
                     }
                 }
-                if (empty($documento['Encabezado']['Transporte']['Aduana']))
-                    unset($documento['Encabezado']['Transporte']);
+                if (empty($documento['Encabezado']['Transporte']['Aduana'])) {
+                                    unset($documento['Encabezado']['Transporte']);
+                }
                 // agregar moneda a los totales
                 if (!empty($caso['exportacion']['MONEDA DE LA OPERACION'])) {
                     $documento['Encabezado']['Totales']['TpoMoneda'] = $caso['exportacion']['MONEDA DE LA OPERACION'];
@@ -248,14 +250,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'])) {
@@ -265,8 +270,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;
+                                    }
                                 }
                             }
                         }
@@ -350,8 +356,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']);
@@ -383,14 +390,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])) {
@@ -403,8 +412,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],
@@ -424,14 +434,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;
                 }
             }
@@ -539,8 +551,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]);
@@ -576,13 +589,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;
                                     }
                                 }

lib/Sii/LibroBoleta.php

Spacing +2 added lines, -2 removed lines

@@ -170,13 +170,13 @@
                 }
                 // determinar neto e iva
                 $tasa = \sasco\LibreDTE\Sii::getIVA();
-                $neto = round(($d['MntTotal'] - $d['MntExe']) / (1 + $tasa/100));
+                $neto = round(($d['MntTotal']-$d['MntExe']) / (1+$tasa / 100));
                 if ($neto) {
                     $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntNeto'] += $neto;
                     $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TasaIVA'] = $tasa;
                     // WARNING: problema por aproximaciones al calcular el NETO e IVA a partir del BRUTO
                     //$resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntIVA'] = round($resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntNeto'] * ($tasa/100));
-                    $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntIVA'] = $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntTotal'] - $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntExe'] - $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntNeto'];
+                    $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntIVA'] = $resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntTotal']-$resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntExe']-$resumen[$d['TpoDoc']]['TotalesServicio'][$d['TpoServ']]['TotMntNeto'];
                 }
             }
             // documento anulado

Braces +3 added lines, -2 removed lines

@@ -91,8 +91,9 @@
     public function generar()
     {
         // 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 XML del envío
         $xmlEnvio = (new \sasco\LibreDTE\XML())->generate([
             'LibroBoleta' => [

lib/Sii/Folios.php

Spacing +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
             );
             return false;
         }
-        return $plain === $plain_firmado;
+        return $plain===$plain_firmado;
     }
 
     /**
@@ -234,7 +234,7 @@ 
     public function getCertificacion()
     {
         $idk = $this->getIDK();
-        return $idk ?  $idk === 100 : null;
+        return $idk ? $idk===100 : null;
     }
 
     /**

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/ConsumoFolio.php

Spacing +4 added lines, -4 removed lines

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

Braces +9 added lines, -6 removed lines

@@ -90,8 +90,9 @@ 
     public function generar()
     {
         // 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 XML del envío
         $xmlEnvio = (new \sasco\LibreDTE\XML())->generate([
             'ConsumoFolios' => [
@@ -125,8 +126,9 @@ 
     {
         $fecha = '9999-12-31';
         foreach ($this->detalles as &$d) {
-            if ($d['FchDoc'] < $fecha)
-                $fecha = $d['FchDoc'];
+            if ($d['FchDoc'] < $fecha) {
+                            $fecha = $d['FchDoc'];
+            }
         }
         return $fecha;
     }
@@ -141,8 +143,9 @@ 
     {
         $fecha = '0000-01-01';
         foreach ($this->detalles as &$d) {
-            if ($d['FchDoc'] > $fecha)
-                $fecha = $d['FchDoc'];
+            if ($d['FchDoc'] > $fecha) {
+                            $fecha = $d['FchDoc'];
+            }
         }
         return $fecha;
     }

lib/I18n.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 {
 
     private static $idioma = 'es'; ///< Idioma por defecto de los mensajes
-    private static $locales = [ ///< Mapeo de idioma a locales
+    private static $locales = [///< Mapeo de idioma a locales
         'es' => 'es_CL.utf8',
         'en' => 'en_US.utf8',
     ];

lib/PDF.php

Spacing +21 added lines, -22 removed lines

@@ -24,7 +24,7 @@ 
 namespace sasco\LibreDTE;
 
 // Directorio para imágenes (no se asume nada)
-define ('K_PATH_IMAGES', '');
+define('K_PATH_IMAGES', '');
 
 /**
  * Clase para generar PDFs
@@ -148,13 +148,13 @@ 
     {
         $widths = [];
         if (is_int($cells)) {
-            $width = floor($total/$cells);
-            for ($i=0; $i<$cells; ++$i) {
+            $width = floor($total / $cells);
+            for ($i = 0; $i<$cells; ++$i) {
                 $widths[] = $width;
             }
         }
-        else if (is_array($cells)){
-            $width = floor($total/count($cells));
+        else if (is_array($cells)) {
+            $width = floor($total / count($cells));
             foreach ($cells as $i) {
                 $widths[$i] = $width;
             }
@@ -204,7 +204,7 @@ 
                 $suma += $val;
             }
             if ($key_0!==null) {
-                $options['width'][$key_0] = 190 - $suma;
+                $options['width'][$key_0] = 190-$suma;
             }
         }
         if (isset($options['align']))
@@ -239,7 +239,7 @@ 
         $a = (isset($options['align']) and is_array($options['align'])) ? $options['align'] : [];
         $buffer = '<table style="border:1px solid #333">';
         // Definir títulos de columnas
-        $thead = isset($options['width']) and is_array($options['width']) and count($options['width']) == count($headers);
+        $thead = isset($options['width']) and is_array($options['width']) and count($options['width'])==count($headers);
         if ($thead)
             $buffer .= '<thead>';
         $buffer .= '<tr>';
@@ -293,16 +293,15 @@ 
             $options['headercolor'][1],
             $options['headercolor'][2]
         );
-        $this->SetFont($this->defaultOptions['font']['family'], 'B',  $options['fontsize']);
+        $this->SetFont($this->defaultOptions['font']['family'], 'B', $options['fontsize']);
         // Header
-        $w = is_array($options['width']) ? $options['width'] :
-            $this->getTableCellWidth($options['width'], array_keys($headers));
-        foreach($headers as $i => $header) {
-            $this->Cell ($w[$i], $options['height'], $headers[$i], 1, 0, $options['align'], 1);
+        $w = is_array($options['width']) ? $options['width'] : $this->getTableCellWidth($options['width'], array_keys($headers));
+        foreach ($headers as $i => $header) {
+            $this->Cell($w[$i], $options['height'], $headers[$i], 1, 0, $options['align'], 1);
         }
         $this->Ln();
         // Color and font restoration
-        $this->SetFillColor (
+        $this->SetFillColor(
             $options['bodybackground'][0],
             $options['bodybackground'][1],
             $options['bodybackground'][2]
@@ -318,19 +317,19 @@ 
         foreach ($data as &$row) {
             $num_pages = $this->getNumPages();
             $this->startTransaction();
-            foreach($headers as $i => $header) {
-                $this->Cell ($w[$i], $options['height'], $row[$i], 'LR', 0, $options['align'], $fill);
+            foreach ($headers as $i => $header) {
+                $this->Cell($w[$i], $options['height'], $row[$i], 'LR', 0, $options['align'], $fill);
             }
             $this->Ln();
-            if($num_pages < $this->getNumPages()) {
+            if ($num_pages<$this->getNumPages()) {
                 $this->rollbackTransaction(true);
                 $this->AddPage();
-                foreach($headers as $i => $header) {
-                    $this->Cell ($w[$i], $options['height'], $headers[$i], 1, 0, $options['align'], 1);
+                foreach ($headers as $i => $header) {
+                    $this->Cell($w[$i], $options['height'], $headers[$i], 1, 0, $options['align'], 1);
                 }
                 $this->Ln();
-                foreach($headers as $i => $header) {
-                    $this->Cell ($w[$i], $options['height'], $row[$i], 'LR', 0, $options['align'], $fill);
+                foreach ($headers as $i => $header) {
+                    $this->Cell($w[$i], $options['height'], $row[$i], 'LR', 0, $options['align'], $fill);
                 }
                 $this->Ln();
             } else {
@@ -350,7 +349,7 @@ 
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]delaf.cl)
      * @version 2014-09-20
      */
-    public function Texto($txt, $x=null, $y=null, $align='', $w=0, $link='', $border=0, $fill=false)
+    public function Texto($txt, $x = null, $y = null, $align = '', $w = 0, $link = '', $border = 0, $fill = false)
     {
         if ($x==null) $x = $this->GetX();
         if ($y==null) $y = $this->GetY();
@@ -371,7 +370,7 @@ 
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]delaf.cl)
      * @version 2014-09-20
      */
-    public function MultiTexto($txt, $x=null, $y=null, $align='', $w=0, $border=0, $fill=false)
+    public function MultiTexto($txt, $x = null, $y = null, $align = '', $w = 0, $border = 0, $fill = false)
     {
         if ($x==null) $x = $this->GetX();
         if ($y==null) $y = $this->GetY();

Braces +55 added lines, -34 removed lines

@@ -98,13 +98,16 @@ 
             $this->Line(0, $this->getY()-1, 290, $this->getY()-2, $style);
             $this->SetFont('', 'B', 6);
             if (empty($this->papelContinuo)) {
-                if (!empty($this->footer['left']))
-                    $this->Texto($this->footer['left']);
-                if (!empty($this->footer['right']))
-                    $this->Texto($this->footer['right'], null, null, 'R');
+                if (!empty($this->footer['left'])) {
+                                    $this->Texto($this->footer['left']);
+                }
+                if (!empty($this->footer['right'])) {
+                                    $this->Texto($this->footer['right'], null, null, 'R');
+                }
             } else {
-                if (!empty($this->footer['left']))
-                    $this->Texto($this->footer['left'], null, null, 'C');
+                if (!empty($this->footer['left'])) {
+                                    $this->Texto($this->footer['left'], null, null, 'C');
+                }
                 if (!empty($this->footer['right'])) {
                     $this->Ln();
                     $this->Texto($this->footer['right'], null, null, 'C');
@@ -130,8 +133,9 @@ 
                 ];
             }
             // si no es arreglo se convierte en uno
-            if (!is_array($footer))
-                $footer = ['left'=>$footer];
+            if (!is_array($footer)) {
+                            $footer = ['left'=>$footer];
+            }
             // asignar footer
             $this->footer = array_merge(['left'=>'', 'right'=>''], $footer);
         } else {
@@ -152,8 +156,7 @@ 
             for ($i=0; $i<$cells; ++$i) {
                 $widths[] = $width;
             }
-        }
-        else if (is_array($cells)){
+        } else if (is_array($cells)){
             $width = floor($total/count($cells));
             foreach ($cells as $i) {
                 $widths[$i] = $width;
@@ -174,8 +177,9 @@ 
         foreach ($data as $row) {
             foreach ($row as $col => $value) {
                 if (empty($value)) {
-                    if (!array_key_exists($col, $cols_empty))
-                        $cols_empty[$col] = 0;
+                    if (!array_key_exists($col, $cols_empty)) {
+                                            $cols_empty[$col] = 0;
+                    }
                     $cols_empty[$col]++;
                 }
             }
@@ -188,10 +192,12 @@ 
                 foreach ($data as &$row) {
                     unset($row[$col]);
                 }
-                if (isset($options['width']))
-                    unset($options['width'][$titles_keys[$col]]);
-                if (isset($options['align']))
-                    unset($options['align'][$titles_keys[$col]]);
+                if (isset($options['width'])) {
+                                    unset($options['width'][$titles_keys[$col]]);
+                }
+                if (isset($options['align'])) {
+                                    unset($options['align'][$titles_keys[$col]]);
+                }
             }
         }
         if (isset($options['width'])) {
@@ -199,16 +205,18 @@ 
             $key_0 = null;
             $suma = 0;
             foreach ($options['width'] as $key => $val) {
-                if ($val===0)
-                    $key_0 = $key;
+                if ($val===0) {
+                                    $key_0 = $key;
+                }
                 $suma += $val;
             }
             if ($key_0!==null) {
                 $options['width'][$key_0] = 190 - $suma;
             }
         }
-        if (isset($options['align']))
-            $options['align'] = array_slice($options['align'], 0);
+        if (isset($options['align'])) {
+                    $options['align'] = array_slice($options['align'], 0);
+        }
         $this->addTable($titles, $data, $options, $html);
     }
 
@@ -240,8 +248,9 @@ 
         $buffer = '<table style="border:1px solid #333">';
         // Definir títulos de columnas
         $thead = isset($options['width']) and is_array($options['width']) and count($options['width']) == count($headers);
-        if ($thead)
-            $buffer .= '<thead>';
+        if ($thead) {
+                    $buffer .= '<thead>';
+        }
         $buffer .= '<tr>';
         $i = 0;
         foreach ($headers as &$col) {
@@ -251,11 +260,13 @@ 
             $i++;
         }
         $buffer .= '</tr>';
-        if ($thead)
-            $buffer .= '</thead>';
+        if ($thead) {
+                    $buffer .= '</thead>';
+        }
         // Definir datos de la tabla
-        if ($thead)
-            $buffer .= '<tbody>';
+        if ($thead) {
+                    $buffer .= '<tbody>';
+        }
         foreach ($data as &$row) {
             $buffer .= '<tr>';
             $i = 0;
@@ -267,8 +278,9 @@ 
             }
             $buffer .= '</tr>';
         }
-        if ($thead)
-            $buffer .= '</tbody>';
+        if ($thead) {
+                    $buffer .= '</tbody>';
+        }
         // Finalizar tabla
         $buffer .= '</table>';
         // generar tabla en HTML
@@ -336,8 +348,9 @@ 
             } else {
                 $this->commitTransaction();
             }
-            if ($options['colorchange'])
-                $fill = !$fill;
+            if ($options['colorchange']) {
+                            $fill = !$fill;
+            }
         }
         $this->Cell(array_sum($w), 0, '', 'T');
         $this->Ln();
@@ -352,8 +365,12 @@ 
      */
     public function Texto($txt, $x=null, $y=null, $align='', $w=0, $link='', $border=0, $fill=false)
     {
-        if ($x==null) $x = $this->GetX();
-        if ($y==null) $y = $this->GetY();
+        if ($x==null) {
+            $x = $this->GetX();
+        }
+        if ($y==null) {
+            $y = $this->GetY();
+        }
         $textrendermode = $this->textrendermode;
         $textstrokewidth = $this->textstrokewidth;
         $this->setTextRenderingMode(0, true, false);
@@ -373,8 +390,12 @@ 
      */
     public function MultiTexto($txt, $x=null, $y=null, $align='', $w=0, $border=0, $fill=false)
     {
-        if ($x==null) $x = $this->GetX();
-        if ($y==null) $y = $this->GetY();
+        if ($x==null) {
+            $x = $this->GetX();
+        }
+        if ($y==null) {
+            $y = $this->GetY();
+        }
         $textrendermode = $this->textrendermode;
         $textstrokewidth = $this->textstrokewidth;
         $this->setTextRenderingMode(0, true, false);

lib/Chile.php

Braces +3 added lines, -2 removed lines

@@ -53,8 +53,9 @@
      */
     public static function getCiudad($comuna)
     {
-        if (!$comuna)
-            return false;
+        if (!$comuna) {
+                    return false;
+        }
         $comuna = mb_strtoupper($comuna, 'UTF-8');
         return isset(self::$ciudades[$comuna]) ? self::$ciudades[$comuna] : false;
     }

lib/Arreglo.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
     public static function mergeRecursiveDistinct(array $array1, array $array2)
     {
         $merged = $array1;
-        foreach ( $array2 as $key => &$value ) {
+        foreach ($array2 as $key => &$value) {
             if (is_array($value) && isset($merged[$key]) && is_array($merged[$key])) {
                 $merged [$key] = self::mergeRecursiveDistinct(
                     $merged [$key],

lib/FirmaElectronica.php

Spacing +3 added lines, -3 removed lines

@@ -317,10 +317,10 @@ 
      */
     public function verify($data, $signature, $pub_key = null, $signature_alg = OPENSSL_ALGO_SHA1)
     {
-        if ($pub_key === null)
+        if ($pub_key===null)
             $pub_key = $this->certs['cert'];
         $pub_key = $this->normalizeCert($pub_key);
-        return openssl_verify($data, base64_decode($signature), $pub_key, $signature_alg) == 1 ? true : false;
+        return openssl_verify($data, base64_decode($signature), $pub_key, $signature_alg)==1 ? true : false;
     }
 
     /**
@@ -451,7 +451,7 @@ 
         } else {
             $digest_calculado = base64_encode(sha1($doc->C14N(), true));
         }
-        return $digest_original == $digest_calculado;
+        return $digest_original==$digest_calculado;
     }
 
     /**

Braces +15 added lines, -10 removed lines

@@ -164,8 +164,9 @@ 
      */
     public function getName()
     {
-        if (isset($this->data['subject']['CN']))
-            return $this->data['subject']['CN'];
+        if (isset($this->data['subject']['CN'])) {
+                    return $this->data['subject']['CN'];
+        }
         return $this->error('No fue posible obtener el Name (subject.CN) de la firma');
     }
 
@@ -177,8 +178,9 @@ 
      */
     public function getEmail()
     {
-        if (isset($this->data['subject']['emailAddress']))
-            return $this->data['subject']['emailAddress'];
+        if (isset($this->data['subject']['emailAddress'])) {
+                    return $this->data['subject']['emailAddress'];
+        }
         return $this->error('No fue posible obtener el Email (subject.emailAddress) de la firma');
     }
 
@@ -317,8 +319,9 @@ 
      */
     public function verify($data, $signature, $pub_key = null, $signature_alg = OPENSSL_ALGO_SHA1)
     {
-        if ($pub_key === null)
-            $pub_key = $this->certs['cert'];
+        if ($pub_key === null) {
+                    $pub_key = $this->certs['cert'];
+        }
         $pub_key = $this->normalizeCert($pub_key);
         return openssl_verify($data, base64_decode($signature), $pub_key, $signature_alg) == 1 ? true : false;
     }
@@ -409,8 +412,9 @@ 
         // calcular SignatureValue
         $SignedInfo = $doc->saveHTML($Signature->getElementsByTagName('SignedInfo')->item(0));
         $firma = $this->sign($SignedInfo);
-        if (!$firma)
-            return false;
+        if (!$firma) {
+                    return false;
+        }
         $signature = wordwrap($firma, $this->config['wordwrap'], "\n", true);
         // reemplazar valores en la firma de
         $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue = $signature;
@@ -442,8 +446,9 @@ 
         $signature = $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue;
         $pub_key = $Signature->getElementsByTagName('X509Certificate')->item(0)->nodeValue;
         // verificar firma
-        if (!$this->verify($signed_info, $signature, $pub_key))
-            return false;
+        if (!$this->verify($signed_info, $signature, $pub_key)) {
+                    return false;
+        }
         // verificar digest
         $digest_original = $Signature->getElementsByTagName('DigestValue')->item(0)->nodeValue;
         if ($tag) {

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