libredte / libredte-lib-core

lib/Sii/Dte/Formatos/XML.php

Braces +7 added lines, -6 removed lines

@@ -47,12 +47,13 @@
         if (!isset($datos['DTE'])) {
             throw new \Exception('El nodo raíz del string XML debe ser el tag DTE');
         }
-        if (isset($datos['DTE']['Documento']))
-            $dte = $datos['DTE']['Documento'];
-        else if (isset($datos['DTE']['Exportaciones']))
-            $dte = $datos['DTE']['Exportaciones'];
-        else if (isset($datos['DTE']['Liquidacion']))
-            $dte = $datos['DTE']['Liquidacion'];
+        if (isset($datos['DTE']['Documento'])) {
+                    $dte = $datos['DTE']['Documento'];
+        } else if (isset($datos['DTE']['Exportaciones'])) {
+                    $dte = $datos['DTE']['Exportaciones'];
+        } else if (isset($datos['DTE']['Liquidacion'])) {
+                    $dte = $datos['DTE']['Liquidacion'];
+        }
         unset($dte['@attributes']);
         return $dte;
     }

lib/Sii/Factoring/Cesion.php

Braces +3 added lines, -2 removed lines

@@ -194,8 +194,9 @@
             return false;
         }
         $this->xml = new \sasco\LibreDTE\XML();
-        if (!$this->xml->loadXML($xml) or !$this->schemaValidate())
-            return false;
+        if (!$this->xml->loadXML($xml) or !$this->schemaValidate()) {
+                    return false;
+        }
         return true;
     }
 

Doc Comments +3 added lines, -3 removed lines

@@ -39,8 +39,8 @@ 
 
     /**
      * Constructor de la clase Cesion
-     * @param DTECedido Objeto DteCedido
-     * @param Seq secuencia de la cesión
+     * @param DTECedido DteCedido DteCedido
+     * @param Seq integer de la cesión
      * @author Adonias Vasquez (adonias.vasquez[at]epys.cl)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2020-07-28
@@ -195,7 +195,7 @@ 
 
     /**
      * Método que realiza la firma de cesión
-     * @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 Adonias Vasquez (adonias.vasquez[at]epys.cl)
      * @version 2020-07-28

lib/Sii/LibroCompraVenta.php

Spacing +15 added lines, -15 removed lines

@@ -167,9 +167,9 @@ 
         // calcular valores que no se hayan entregado
         if (isset($detalle['FctProp'])) {
             if ($detalle['IVAUsoComun']===false)
-                $detalle['IVAUsoComun'] = round($detalle['MntNeto'] * ($detalle['TasaImp']/100));
+                $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));
+            $detalle['MntIVA'] = round($detalle['MntNeto'] * ($detalle['TasaImp'] / 100));
         }
         // si el monto total es 0 pero no se asigno neto ni exento se coloca
         if ($detalle['MntExe']===false and $detalle['MntNeto']===false and $detalle['MntTotal']===0) {
@@ -185,7 +185,7 @@ 
                 $MntIVANoRec = explode(',', $detalle['IVANoRec'][0]['MntIVANoRec']);
                 $detalle['IVANoRec'] = [];
                 $n_inr = count($CodIVANoRec);
-                for ($i=0; $i<$n_inr; $i++) {
+                for ($i = 0; $i<$n_inr; $i++) {
                     $detalle['IVANoRec'][] = [
                         'CodIVANoRec' => $CodIVANoRec[$i],
                         'MntIVANoRec' => $MntIVANoRec[$i],
@@ -204,7 +204,7 @@ 
                 $MntImp = explode(',', $detalle['OtrosImp'][0]['MntImp']);
                 $detalle['OtrosImp'] = [];
                 $n_impuestos = count($CodImp);
-                for ($i=0; $i<$n_impuestos; $i++) {
+                for ($i = 0; $i<$n_impuestos; $i++) {
                     $detalle['OtrosImp'][] = [
                         'CodImp' => $CodImp[$i],
                         'TasaImp' => !empty($TasaImp[$i]) ? $TasaImp[$i] : false,
@@ -216,20 +216,20 @@ 
             $retenido = ImpuestosAdicionales::getRetenido($detalle['OtrosImp']);
             if ($retenido) {
                 // si el iva retenido es total
-                if ($retenido == $detalle['MntIVA']) {
+                if ($retenido==$detalle['MntIVA']) {
                     $detalle['IVARetTotal'] = $retenido;
                 }
                 // si el iva retenido es parcial
                 else {
                     $detalle['IVARetParcial'] = $retenido;
-                    $detalle['IVANoRetenido'] = $detalle['MntIVA'] - $retenido;
+                    $detalle['IVANoRetenido'] = $detalle['MntIVA']-$retenido;
                 }
             }
         }
         // calcular monto total si no se especificó
         if ($detalle['MntTotal']===false) {
             // calcular monto total inicial
-            $detalle['MntTotal'] = $detalle['MntExe'] + $detalle['MntNeto'] + (int)$detalle['MntIVA'];
+            $detalle['MntTotal'] = $detalle['MntExe']+$detalle['MntNeto']+(int)$detalle['MntIVA'];
             // agregar iva no recuperable al monto total
             if (!empty($detalle['IVANoRec'])) {
                 foreach ($detalle['IVANoRec'] as $IVANoRec) {
@@ -249,7 +249,7 @@ 
                 }
             }
             // agregar otro montos e impuestos al total
-            $detalle['MntTotal'] += (int)$detalle['MntSinCred'] + (int)$detalle['TabPuros'] + (int)$detalle['TabCigarrillos'] + (int)$detalle['TabElaborado'] + (int)$detalle['ImpVehiculo'];
+            $detalle['MntTotal'] += (int)$detalle['MntSinCred']+(int)$detalle['TabPuros']+(int)$detalle['TabCigarrillos']+(int)$detalle['TabElaborado']+(int)$detalle['ImpVehiculo'];
         }
         // si no hay no hay monto neto, no se crean campos para IVA
         if ($detalle['MntNeto']===false) {
@@ -309,7 +309,7 @@ 
         $data = \sasco\LibreDTE\CSV::read($archivo);
         $n_data = count($data);
         $detalles = [];
-        for ($i=1; $i<$n_data; $i++) {
+        for ($i = 1; $i<$n_data; $i++) {
             // detalle genérico
             $detalle = [
                 'TpoDoc' => $data[$i][0],
@@ -344,7 +344,7 @@ 
             if (!empty($data[$i][11])) {
                 $detalle['IVANoRec'] = [
                     'CodIVANoRec' => $data[$i][11],
-                    'MntIVANoRec' => !empty($data[$i][12]) ? $data[$i][12] : round($detalle['MntNeto'] * ($detalle['TasaImp']/100)),
+                    'MntIVANoRec' => !empty($data[$i][12]) ? $data[$i][12] : round($detalle['MntNeto'] * ($detalle['TasaImp'] / 100)),
                 ];
             }
             // agregar código y monto de otros impuestos
@@ -352,7 +352,7 @@ 
                 $detalle['OtrosImp'] = [
                     'CodImp' => $data[$i][14],
                     'TasaImp' => !empty($data[$i][15]) ? $data[$i][15] : 0,
-                    'MntImp' => !empty($data[$i][16]) ? $data[$i][16] : round($detalle['MntNeto'] * ($data[$i][15]/100)),
+                    'MntImp' => !empty($data[$i][16]) ? $data[$i][16] : round($detalle['MntNeto'] * ($data[$i][15] / 100)),
                 ];
             }
             // si hay monto total se agrega
@@ -427,7 +427,7 @@ 
         $data = \sasco\LibreDTE\CSV::read($archivo);
         $n_data = count($data);
         $detalles = [];
-        for ($i=1; $i<$n_data; $i++) {
+        for ($i = 1; $i<$n_data; $i++) {
             // detalle genérico
             $detalle = [
                 'TpoDoc' => $data[$i][0],
@@ -466,7 +466,7 @@ 
                 $detalle['OtrosImp'] = [
                     'CodImp' => $data[$i][11],
                     'TasaImp' => !empty($data[$i][12]) ? $data[$i][12] : false,
-                    'MntImp' => !empty($data[$i][13]) ? $data[$i][13] : round($detalle['MntNeto'] * ($data[$i][12]/100)),
+                    'MntImp' => !empty($data[$i][13]) ? $data[$i][13] : round($detalle['MntNeto'] * ($data[$i][12] / 100)),
                 ];
             }
             // agregar datos extranjeros
@@ -628,8 +628,8 @@ 
             // si hay IVA de uso común se contabiliza
             if (!empty($d['FctProp'])) {
                 $totales[$d['TpoDoc']]['TotIVAUsoComun'] += $d['IVAUsoComun'];
-                $totales[$d['TpoDoc']]['FctProp'] = $d['FctProp']/100;
-                $totales[$d['TpoDoc']]['TotCredIVAUsoComun'] += round($d['IVAUsoComun'] * ($d['FctProp']/100));
+                $totales[$d['TpoDoc']]['FctProp'] = $d['FctProp'] / 100;
+                $totales[$d['TpoDoc']]['TotCredIVAUsoComun'] += round($d['IVAUsoComun'] * ($d['FctProp'] / 100));
                 unset($d['FctProp']); // se quita el factor de proporcionalidad del detalle ya que no es parte del XML
             }
             // contabilizar IVA fuera de plazo

Braces +27 added lines, -18 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;
     }
@@ -183,8 +185,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));
         }
@@ -194,8 +197,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']);
@@ -212,8 +216,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']);
@@ -531,8 +536,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');
     }
 
@@ -546,8 +552,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;
@@ -712,8 +719,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);
@@ -733,8 +741,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

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 +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;
                                     }
                                 }

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

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

Braces +12 added lines, -8 removed lines

@@ -168,8 +168,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');
     }
 
@@ -353,8 +354,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;
     }
@@ -453,8 +455,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;
@@ -486,8 +489,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) {