libredte / libredte-lib-core

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) {

lib/CSV.php

Spacing +7 added lines, -7 removed lines

@@ -46,10 +46,10 @@ 
      */
     public static function read($archivo, $separador = ';', $delimitadortexto = '"')
     {
-        if (($handle = fopen($archivo, 'r')) !== FALSE) {
+        if (($handle = fopen($archivo, 'r'))!==FALSE) {
             $data = array();
             $i = 0;
-            while (($row = fgetcsv($handle, 0, $separador, $delimitadortexto)) !== FALSE) {
+            while (($row = fgetcsv($handle, 0, $separador, $delimitadortexto))!==FALSE) {
                 $j = 0;
                 foreach ($row as &$col) {
                     $data[$i][$j++] = $col;
@@ -77,11 +77,11 @@ 
         header('Content-Disposition: attachment; filename='.$archivo.'.csv');
         header('Pragma: no-cache');
         header('Expires: 0');
-        foreach($data as &$row) {
-            foreach($row as &$col) {
+        foreach ($data as &$row) {
+            foreach ($row as &$col) {
                 $col = $delimitadortexto.rtrim(str_replace('<br />', ', ', strip_tags($col, '<br>')), " \t\n\r\0\x0B,").$delimitadortexto;
             }
-            echo implode($separador, $row),"\r\n";
+            echo implode($separador, $row), "\r\n";
             unset($row);
         }
         unset($data);
@@ -100,8 +100,8 @@ 
     public static function save(array $data, $archivo, $separador = ';', $delimitadortexto = '"')
     {
         $fd = fopen($archivo, 'w');
-        foreach($data as &$row) {
-            foreach($row as &$col) {
+        foreach ($data as &$row) {
+            foreach ($row as &$col) {
                 $col = $delimitadortexto.rtrim(str_replace('<br />', ', ', strip_tags($col, '<br>')), " \t\n\r\0\x0B,").$delimitadortexto;
             }
             fwrite($fd, implode($separador, $row)."\r\n");

lib/Estado.php

Braces +6 added lines, -4 removed lines

@@ -174,12 +174,14 @@
     public static function get($codigo, $args = null)
     {
         // si no hay glosa asociada al código se entrega el mismo código
-        if (!isset(self::$glosas[(int)$codigo]))
-            return (int)$codigo;
+        if (!isset(self::$glosas[(int)$codigo])) {
+                    return (int)$codigo;
+        }
         // si los argumentos no son un arreglo se obtiene arreglo a partir
         // de los argumentos pasados a la función
-        if (!is_array($args))
-            $args = array_slice(func_get_args(), 1);
+        if (!is_array($args)) {
+                    $args = array_slice(func_get_args(), 1);
+        }
         // entregar glosa
         return vsprintf(I18n::translate(self::$glosas[(int)$codigo], 'estados'), $args);
     }

lib/Sii/Factoring/DteCedido.php

Doc Comments +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
 
     /**
      * Método que realiza la firma del DTE cedido
-     * @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 2016-12-10

lib/Sii/EnvioDte.php

Spacing +9 added lines, -9 removed lines

@@ -32,7 +32,7 @@ 
 {
 
     private $dtes = []; ///< Objetos con los DTE que se enviarán
-    private $config = [ // 0: DTE, 1: boleta
+    private $config = [// 0: DTE, 1: boleta
         'SubTotDTE_max' => [20, 2], ///< máxima cantidad de tipos de documentos en el envío
         'DTE_max' => [2000, 1000], ///< máxima cantidad de DTE en un envío
         'tipos' => ['EnvioDTE', 'EnvioBOLETA'], ///< Tag para el envío, según si son Boletas o no
@@ -50,11 +50,11 @@ 
     public function agregar(Dte $DTE)
     {
         // determinar el tipo del envío (DTE o boleta)
-        if ($this->tipo === null) {
+        if ($this->tipo===null) {
             $this->tipo = (int)$DTE->esBoleta();
         }
         // validar que el tipo de documento sea del tipo que se espera
-        else if ((int)$DTE->esBoleta() != $this->tipo) {
+        else if ((int)$DTE->esBoleta()!=$this->tipo) {
             return false;
         }
         //
@@ -221,7 +221,7 @@ 
                 'RutEnvia' => $Dte->getEmisor(),
                 'RutReceptor' => $Dte->getReceptor(),
                 'FchResol' => date('Y-m-d'),
-                'NroResol' => ($Dte->getCertificacion()?'0':'').'9999',
+                'NroResol' => ($Dte->getCertificacion() ? '0' : '').'9999',
             ]);
             if (!parent::loadXML($this->generar())) {
                 return false;
@@ -301,7 +301,7 @@ 
     {
         $fecha = '9999-12-31';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() < $fecha)
+            if ($Dte->getFechaEmision()<$fecha)
                 $fecha = $Dte->getFechaEmision();
         }
         return $fecha;
@@ -316,7 +316,7 @@ 
     {
         $fecha = '0000-01-01';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() > $fecha)
+            if ($Dte->getFechaEmision()>$fecha)
                 $fecha = $Dte->getFechaEmision();
         }
         return $fecha;
@@ -375,7 +375,7 @@ 
                 $e = substr($e, 0, -2);
             $d = (int)$nodo_dte->getElementsByTagName('TipoDTE')->item(0)->nodeValue;
             $f = (int)$nodo_dte->getElementsByTagName('Folio')->item(0)->nodeValue;
-            if ($folio == $f and $dte == $d and $emisor == $e) {
+            if ($folio==$f and $dte==$d and $emisor==$e) {
                 return new Dte($nodo_dte->C14N(), false); // cargar DTE sin normalizar
             }
         }
@@ -432,12 +432,12 @@ 
         if ($X509Certificate[1]==' ' or $X509Certificate[1]=="\t") {
             $lineas = explode("\n", $X509Certificate);
             $X509Certificate = '';
-            foreach($lineas as $l) {
+            foreach ($lineas as $l) {
                 $X509Certificate .= trim($l);
             }
         }
         $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 and $DigestValue===base64_encode(sha1($SetDTE, true));
     }
 

Doc Comments +2 added lines, -2 removed lines

@@ -42,7 +42,7 @@ 
 
     /**
      * Método que agrega un DTE al listado que se enviará
-     * @param DTE Objeto del DTE
+     * @param DTE Dte del DTE
      * @return =true si se pudo agregar el DTE o =false si no se agregó por exceder el límite de un envío
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-12-11
@@ -395,7 +395,7 @@ 
     /**
      * Método que determina el estado de validación sobre el envío
      * @param datos Arreglo con datos para hacer las validaciones
-     * @return Código del estado de la validación
+     * @return integer del estado de la validación
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-07-03
      */

Braces +15 added lines, -10 removed lines

@@ -135,8 +135,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 DTEs para generar entregar falso
         if (!isset($this->dtes[0])) {
             \sasco\LibreDTE\Log::write(
@@ -185,8 +186,9 @@ 
         $SubTotDTE = [];
         $subtotales = [];
         foreach ($this->dtes as &$DTE) {
-            if (!isset($subtotales[$DTE->getTipo()]))
-                $subtotales[$DTE->getTipo()] = 0;
+            if (!isset($subtotales[$DTE->getTipo()])) {
+                            $subtotales[$DTE->getTipo()] = 0;
+            }
             $subtotales[$DTE->getTipo()]++;
         }
         foreach ($subtotales as $tipo => $subtotal) {
@@ -301,8 +303,9 @@ 
     {
         $fecha = '9999-12-31';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() < $fecha)
-                $fecha = $Dte->getFechaEmision();
+            if ($Dte->getFechaEmision() < $fecha) {
+                            $fecha = $Dte->getFechaEmision();
+            }
         }
         return $fecha;
     }
@@ -316,8 +319,9 @@ 
     {
         $fecha = '0000-01-01';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() > $fecha)
-                $fecha = $Dte->getFechaEmision();
+            if ($Dte->getFechaEmision() > $fecha) {
+                            $fecha = $Dte->getFechaEmision();
+            }
         }
         return $fecha;
     }
@@ -371,8 +375,9 @@ 
         $DTEs = $this->xml->getElementsByTagName('DTE');
         foreach ($DTEs as $nodo_dte) {
             $e = $nodo_dte->getElementsByTagName('RUTEmisor')->item(0)->nodeValue;
-            if (is_numeric($emisor))
-                $e = substr($e, 0, -2);
+            if (is_numeric($emisor)) {
+                            $e = substr($e, 0, -2);
+            }
             $d = (int)$nodo_dte->getElementsByTagName('TipoDTE')->item(0)->nodeValue;
             $f = (int)$nodo_dte->getElementsByTagName('Folio')->item(0)->nodeValue;
             if ($folio == $f and $dte == $d and $emisor == $e) {

examples/016-etapa_intercambio_EnvioRecibos.php

Spacing +1 added lines, -1 removed lines

@@ -66,4 +66,4 @@
 
 // si hubo errores mostrar
 foreach (\sasco\LibreDTE\Log::readAll() as $error)
-    echo $error,"\n";
+    echo $error, "\n";

Braces +2 added lines, -1 removed lines

@@ -98,5 +98,6 @@
 }
 
 // si hubo errores mostrar
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo $error,"\n";
+}

examples/027-ventas_sin_movimiento.php

Spacing +1 added lines, -1 removed lines

@@ -66,4 +66,4 @@
 
 // si hubo errores mostrar
 foreach (\sasco\LibreDTE\Log::readAll() as $error)
-    echo $error,"\n";
+    echo $error, "\n";

Braces +2 added lines, -1 removed lines

@@ -65,5 +65,6 @@
 var_dump($track_id);
 
 // si hubo errores mostrar
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo $error,"\n";
+}