libredte / libredte-lib-core

examples/011-set_pruebas_factura_exenta.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 +10 added lines, -6 removed lines

@@ -411,17 +411,20 @@ 
 // Objetos de Firma, Folios y EnvioDTE
 $Firma = new \sasco\LibreDTE\FirmaElectronica($config['firma']);
 $Folios = [];
-foreach ($folios as $tipo => $cantidad)
+foreach ($folios as $tipo => $cantidad) {
     $Folios[$tipo] = new \sasco\LibreDTE\Sii\Folios(file_get_contents('xml/folios/'.$tipo.'.xml'));
+}
 $EnvioDTE = new \sasco\LibreDTE\Sii\EnvioDte();
 
 // generar cada DTE, timbrar, firmar y agregar al sobre de EnvioDTE
 foreach ($set_pruebas as $documento) {
     $DTE = new \sasco\LibreDTE\Sii\Dte($documento);
-    if (!$DTE->timbrar($Folios[$DTE->getTipo()]))
-        break;
-    if (!$DTE->firmar($Firma))
-        break;
+    if (!$DTE->timbrar($Folios[$DTE->getTipo()])) {
+            break;
+    }
+    if (!$DTE->firmar($Firma)) {
+            break;
+    }
     $EnvioDTE->agregar($DTE);
 }
 
@@ -433,5 +436,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";
+}

examples/015-etapa_intercambio_RecepcionDTE.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/017-etapa_intercambio_ResultadoDTE.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/012-set_pruebas_ventas.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";
+}

examples/031-libro_boleta.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/022-libro_ventas_csv.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";
+}

lib/Sii/RegistroCompraVenta.php

Doc Comments +6 added lines, -4 removed lines

@@ -55,7 +55,7 @@ 
 
     /**
      * Constructor, obtiene el token de la sesión y lo guarda
-     * @param Firma Objeto con la firma electrónica
+     * @param Firma \sasco\LibreDTE\FirmaElectronica con la firma electrónica
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-08-28
      */
@@ -97,6 +97,8 @@ 
      * Método que entrega los eventos asociados a un DTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-08-28
+     * @param string $dte
+     * @param string $folio
      */
     public function listarEventosHistDoc($rut, $dv, $dte, $folio)
     {
@@ -184,10 +186,10 @@ 
 
     /**
      * Método para realizar una solicitud al servicio web del SII
-     * @param request Nombre de la función que se ejecutará en el servicio web
+     * @param request string de la función que se ejecutará en el servicio web
      * @param args Argumentos que se pasarán al servicio web
-     * @param retry Intentos que se realizarán como máximo para obtener respuesta
-     * @return Objeto o String con la respuesta (depende servicio web)
+     * @param retry integer que se realizarán como máximo para obtener respuesta
+     * @return string o String con la respuesta (depende servicio web)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-08-28
      */

Spacing +1 added lines, -1 removed lines

@@ -220,7 +220,7 @@
             \sasco\LibreDTE\Log::write(\sasco\LibreDTE\Estado::REQUEST_ERROR_SOAP, \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::REQUEST_ERROR_SOAP, $msg));
             return false;
         }
-        for ($i=0; $i<$retry; $i++) {
+        for ($i = 0; $i<$retry; $i++) {
             try {
                 $body = call_user_func_array([$soap, $request], $args);
                 break;

lib/XML.php

Doc Comments +6 added lines, -5 removed lines

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

Braces +42 added lines, -30 removed lines

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

Spacing +3 added lines, -3 removed lines

@@ -197,7 +197,7 @@ 
      */
     private function utf2iso($string)
     {
-        return mb_detect_encoding($string, ['UTF-8', 'ISO-8859-1']) != 'ISO-8859-1' ? utf8_decode($string) : $string;
+        return mb_detect_encoding($string, ['UTF-8', 'ISO-8859-1'])!='ISO-8859-1' ? utf8_decode($string) : $string;
     }
 
     /**
@@ -237,7 +237,7 @@ 
         }
         // agregar nodos hijos
         if ($dom->hasChildNodes()) {
-            foreach($dom->childNodes as $child) {
+            foreach ($dom->childNodes as $child) {
                 if ($child instanceof \DOMText) {
                     $textContent = trim($child->textContent);
                     if ($textContent!="") {
@@ -362,7 +362,7 @@ 
         $newXML = '';
         $n_letras = strlen($xml);
         $convertir = false;
-        for ($i=0; $i<$n_letras; ++$i) {
+        for ($i = 0; $i<$n_letras; ++$i) {
             if ($xml[$i]=='>')
                 $convertir = true;
             if ($xml[$i]=='<')

lib/Sii/Folios.php

Doc Comments +1 added lines, -1 removed lines

@@ -254,7 +254,7 @@
 
     /**
      * Método que indica si el CAF es de certificación o no
-     * @return bool =true si los folios son del ambiente de certificación, =null si no se pudo determinar
+     * @return boolean|null =true si los folios son del ambiente de certificación, =null si no se pudo determinar
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-30
      */

Spacing +8 added lines, -8 removed lines

@@ -63,11 +63,11 @@ 
         // validar firma del SII sobre los folios
         $firma = $this->getFirma();
         $idk = $this->getIDK();
-        if ($firma === false || $idk === false) {
+        if ($firma===false || $idk===false) {
             return false;
         }
         $pub_key = \sasco\LibreDTE\Sii::cert($idk);
-        if ($pub_key === false || openssl_verify($this->xml->getFlattened('/AUTORIZACION/CAF/DA'), base64_decode($firma), $pub_key)!==1) {
+        if ($pub_key===false || openssl_verify($this->xml->getFlattened('/AUTORIZACION/CAF/DA'), base64_decode($firma), $pub_key)!==1) {
             \sasco\LibreDTE\Log::write(
                 \sasco\LibreDTE\Estado::FOLIOS_ERROR_FIRMA,
                 \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::FOLIOS_ERROR_FIRMA)
@@ -98,7 +98,7 @@ 
             );
             return false;
         }
-        return $plain === $plain_firmado;
+        return $plain===$plain_firmado;
     }
 
     /**
@@ -261,7 +261,7 @@ 
     public function getCertificacion()
     {
         $idk = $this->getIDK();
-        return $idk ?  $idk === 100 : null;
+        return $idk ? $idk===100 : null;
     }
 
     /**
@@ -277,14 +277,14 @@ 
         }
         $fecha_autorizacion = $this->getFechaAutorizacion();
         $hoy = date('Y-m-d');
-        if ($fecha_autorizacion < '2018-07-01' and $hoy > '2018-12-31') {
+        if ($fecha_autorizacion<'2018-07-01' and $hoy>'2018-12-31') {
             return false;
         }
-        $vigencia = $fecha_autorizacion >= '2018-07-01' ? 6 : 18;
+        $vigencia = $fecha_autorizacion>='2018-07-01' ? 6 : 18;
         $d1 = new \DateTime($fecha_autorizacion);
         $d2 = new \DateTime($hoy);
-        $meses = $d1->diff($d2)->m + ($d1->diff($d2)->y*12);
-        return $meses <= $vigencia;
+        $meses = $d1->diff($d2)->m+($d1->diff($d2)->y * 12);
+        return $meses<=$vigencia;
     }
 
     /**