libredte / libredte-lib-core

lib/Log.php

Braces +15 added lines, -10 removed lines

@@ -65,8 +65,9 @@ 
     public static function write($code, $msg = null, $severity = LOG_ERR)
     {
         // si no existe la bitácora para la gravedad se crea
-        if (!isset(self::$bitacora[$severity]))
-            self::$bitacora[$severity] = [];
+        if (!isset(self::$bitacora[$severity])) {
+                    self::$bitacora[$severity] = [];
+        }
         // si el código es un string se copia a msg
         if (is_string($code)) {
             $msg = $code;
@@ -97,8 +98,9 @@ 
      */
     public static function read($severity = LOG_ERR)
     {
-        if (!isset(self::$bitacora[$severity]))
-            return false;
+        if (!isset(self::$bitacora[$severity])) {
+                    return false;
+        }
         return array_pop(self::$bitacora[$severity]);
     }
 
@@ -112,11 +114,13 @@ 
      */
     public static function readAll($severity = LOG_ERR, $new_first = true)
     {
-        if (!isset(self::$bitacora[$severity]))
-            return [];
+        if (!isset(self::$bitacora[$severity])) {
+                    return [];
+        }
         $bitacora = self::$bitacora[$severity];
-        if ($new_first)
-            krsort($bitacora);
+        if ($new_first) {
+                    krsort($bitacora);
+        }
         self::$bitacora[$severity] = [];
         return $bitacora;
     }
@@ -163,8 +167,9 @@ 
     public function __toString()
     {
         $msg = $this->msg ? $this->msg : 'Error código '.$this->code;
-        if (!$this->file)
-            return $msg;
+        if (!$this->file) {
+                    return $msg;
+        }
         return $msg.' (by '.$this->class.$this->type.$this->function.'() in '.$this->file.' on line '.$this->line.')';
     }
 

lib/Sii/EnvioRecibos.php

Braces +6 added lines, -4 removed lines

@@ -98,8 +98,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;
+        }
         // si no hay respuestas para generar entregar falso
         if (!isset($this->recibos[0])) {
             \sasco\LibreDTE\Log::write(
@@ -173,8 +174,9 @@ 
             }
             // crear recibos a partir del arreglo
             $Recibos = $this->arreglo['EnvioRecibos']['SetRecibos']['Recibo'];
-            if (!isset($Recibos[0]))
-                $Recibos = [$Recibos];
+            if (!isset($Recibos[0])) {
+                            $Recibos = [$Recibos];
+            }
             foreach ($Recibos as $Recibo) {
                 $this->recibos[] = $Recibo;
             }

lib/Sii/LibroGuia.php

Braces +9 added lines, -6 removed lines

@@ -40,8 +40,9 @@ 
      */
     public function agregar(array $detalle, $normalizar = true)
     {
-        if ($normalizar)
-            $this->normalizarDetalle($detalle);
+        if ($normalizar) {
+                    $this->normalizarDetalle($detalle);
+        }
         $this->detalles[] = $detalle;
         return true;
     }
@@ -138,8 +139,9 @@ 
             'TipoEnvio' => 'TOTAL',
             'FolioNotificacion' => null,
         ], $caratula);
-        if ($this->caratula['TipoEnvio']=='ESPECIAL')
-            $this->caratula['FolioNotificacion'] = null;
+        if ($this->caratula['TipoEnvio']=='ESPECIAL') {
+                    $this->caratula['FolioNotificacion'] = null;
+        }
         $this->id = 'LibreDTE_LIBRO_GUIA_'.str_replace('-', '', $this->caratula['RutEmisorLibro']).'_'.str_replace('-', '', $this->caratula['PeriodoTributario']).'_'.date('U');
     }
 
@@ -153,8 +155,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 XML del envío
         $xmlEnvio = (new \sasco\LibreDTE\XML())->generate([
             'LibroGuia' => [

Spacing +7 added lines, -7 removed lines

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

lib/Sii/Autenticacion.php

Braces +12 added lines, -5 removed lines

@@ -84,8 +84,9 @@ 
      */
     private static function getTokenRequest($seed, $Firma = [])
     {
-        if (is_array($Firma))
-            $Firma = new \sasco\LibreDTE\FirmaElectronica($Firma);
+        if (is_array($Firma)) {
+                    $Firma = new \sasco\LibreDTE\FirmaElectronica($Firma);
+        }
         $seedSigned = $Firma->signXML(
             (new \sasco\LibreDTE\XML())->generate([
                 'getToken' => [
@@ -119,11 +120,17 @@ 
      */
     public static function getToken($Firma = [])
     {
-        if (!$Firma) return false;
+        if (!$Firma) {
+            return false;
+        }
         $semilla = self::getSeed();
-        if (!$semilla) return false;
+        if (!$semilla) {
+            return false;
+        }
         $requestFirmado = self::getTokenRequest($semilla, $Firma);
-        if (!$requestFirmado) return false;
+        if (!$requestFirmado) {
+            return false;
+        }
         $xml = \sasco\LibreDTE\Sii::request('GetTokenFromSeed', 'getToken', $requestFirmado);
         if ($xml===false or (string)$xml->xpath('/SII:RESPUESTA/SII:RESP_HDR/ESTADO')[0]!=='00') {
             \sasco\LibreDTE\Log::write(

lib/Sii/RespuestaEnvio.php

Braces +27 added lines, -18 removed lines

@@ -73,10 +73,12 @@ 
      */
     public function agregarRespuestaEnvio(array $datos)
     {
-        if (isset($this->respuesta_documentos[0]))
-            return false;
-        if (isset($this->respuesta_envios[$this->config['respuesta_envios_max']-1]))
-            return false;
+        if (isset($this->respuesta_documentos[0])) {
+                    return false;
+        }
+        if (isset($this->respuesta_envios[$this->config['respuesta_envios_max']-1])) {
+                    return false;
+        }
         $this->respuesta_envios[] = array_merge([
             'NmbEnvio' => false,
             'FchRecep' => date('Y-m-d\TH:i:s'),
@@ -102,10 +104,12 @@ 
      */
     public function agregarRespuestaDocumento(array $datos)
     {
-        if (isset($this->respuesta_envios[0]))
-            return false;
-        if (isset($this->respuesta_documentos[$this->config['respuesta_documentos_max']-1]))
-            return false;
+        if (isset($this->respuesta_envios[0])) {
+                    return false;
+        }
+        if (isset($this->respuesta_documentos[$this->config['respuesta_documentos_max']-1])) {
+                    return false;
+        }
         $this->respuesta_documentos[] = array_merge([
             'TipoDTE' => false,
             'Folio' => false,
@@ -156,8 +160,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;
+        }
         // si no hay respuestas para generar entregar falso
         if (!isset($this->respuesta_envios[0]) and !isset($this->respuesta_documentos[0])) {
             \sasco\LibreDTE\Log::write(
@@ -230,8 +235,9 @@ 
      */
     public function getRecepciones()
     {
-        if (!$this->esRecepcionEnvio())
-            return false;
+        if (!$this->esRecepcionEnvio()) {
+                    return false;
+        }
         // si no hay respustas se deben crear
         if (!$this->respuesta_envios) {
             // si no está creado el arrelgo con los datos error
@@ -242,8 +248,9 @@ 
             if (isset($this->arreglo['RespuestaDTE']['Resultado']['RecepcionEnvio']['RecepcionDTE'])) {
                 // crear repuestas a partir del arreglo
                 $Recepciones = $this->arreglo['RespuestaDTE']['Resultado']['RecepcionEnvio']['RecepcionDTE'];
-                if (!isset($Recepciones[0]))
-                    $Recepciones = [$Recepciones];
+                if (!isset($Recepciones[0])) {
+                                    $Recepciones = [$Recepciones];
+                }
                 foreach ($Recepciones as $Recepcion) {
                     $this->respuesta_envios[] = $Recepcion;
                 }
@@ -260,8 +267,9 @@ 
      */
     public function getResultados()
     {
-        if (!$this->esResultadoDTE())
-            return false;
+        if (!$this->esResultadoDTE()) {
+                    return false;
+        }
         // si no hay respustas se deben crear
         if (!$this->respuesta_documentos) {
             // si no está creado el arrelgo con los datos error
@@ -270,8 +278,9 @@ 
             }
             // crear repuestas a partir del arreglo
             $Resultados = $this->arreglo['RespuestaDTE']['Resultado']['ResultadoDTE'];
-            if (!isset($Resultados[0]))
-                $Resultados = [$Resultados];
+            if (!isset($Resultados[0])) {
+                            $Resultados = [$Resultados];
+            }
             foreach ($Resultados as $Resultado) {
                 $this->respuesta_documentos[] = $Resultado;
             }

lib/Sii/ImpuestosAdicionales.php

Braces +9 added lines, -6 removed lines

@@ -162,8 +162,9 @@ 
      */
     public static function getTipo($codigo)
     {
-        if (isset(self::$impuestos[$codigo]))
-            return self::$impuestos[$codigo]['tipo'];
+        if (isset(self::$impuestos[$codigo])) {
+                    return self::$impuestos[$codigo]['tipo'];
+        }
         return false;
     }
 
@@ -175,8 +176,9 @@ 
      */
     public static function getGlosa($codigo)
     {
-        if (isset(self::$impuestos[$codigo]))
-            return self::$impuestos[$codigo]['glosa'];
+        if (isset(self::$impuestos[$codigo])) {
+                    return self::$impuestos[$codigo]['glosa'];
+        }
         return 'Impto. cód. '.$codigo;
     }
 
@@ -188,8 +190,9 @@ 
      */
     public static function getTasa($codigo)
     {
-        if (isset(self::$impuestos[$codigo]['tasa']))
-            return self::$impuestos[$codigo]['tasa'];
+        if (isset(self::$impuestos[$codigo]['tasa'])) {
+                    return self::$impuestos[$codigo]['tasa'];
+        }
         return false;
     }
 

lib/Sii/Dte/Formatos.php

Spacing +2 added lines, -2 removed lines

@@ -76,12 +76,12 @@
         if (!self::$formatos) {
             $dir = dirname(__FILE__).'/Formatos';
             $formatos = scandir($dir);
-            foreach($formatos as &$formato) {
+            foreach ($formatos as &$formato) {
                 if ($formato[0]=='.')
                     continue;
                 if (is_dir($dir.'/'.$formato)) {
                     $subformatos = scandir($dir.'/'.$formato);
-                    foreach($subformatos as &$subformato) {
+                    foreach ($subformatos as &$subformato) {
                         if ($subformato[0]=='.')
                             continue;
                         self::$formatos[] = $formato.'.'.substr($subformato, 0, -4);

Braces +6 added lines, -4 removed lines

@@ -77,13 +77,15 @@
             $dir = dirname(__FILE__).'/Formatos';
             $formatos = scandir($dir);
             foreach($formatos as &$formato) {
-                if ($formato[0]=='.')
-                    continue;
+                if ($formato[0]=='.') {
+                                    continue;
+                }
                 if (is_dir($dir.'/'.$formato)) {
                     $subformatos = scandir($dir.'/'.$formato);
                     foreach($subformatos as &$subformato) {
-                        if ($subformato[0]=='.')
-                            continue;
+                        if ($subformato[0]=='.') {
+                                                    continue;
+                        }
                         self::$formatos[] = $formato.'.'.substr($subformato, 0, -4);
                     }
                 } else {

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