libredte / libredte-lib-core

lib/File.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 
     /**
      * Método que entrega el mimetype de un archivo
-     * @param file Ruta hacia el fichero
+     * @param file string hacia el fichero
      * @return Mimetype del fichero o =false si no se pudo determinar
      * @author http://stackoverflow.com/a/23287361
      * @version 2015-11-03

Spacing +8 added lines, -8 removed lines

@@ -49,18 +49,18 @@ 
     public static function rmdir($dir)
     {
         // List the contents of the directory table
-        $dir_content = scandir ($dir);
+        $dir_content = scandir($dir);
         // Is it a directory?
         if ($dir_content!==false) {
             // For each directory entry
             foreach ($dir_content as &$entry) {
                 // Unix symbolic shortcuts, we go
-                if (!in_array ($entry, array ('.','..'))) {
+                if (!in_array($entry, array('.', '..'))) {
                     // We find the path from the beginning
-                    $entry = $dir.DIRECTORY_SEPARATOR. $entry;
+                    $entry = $dir.DIRECTORY_SEPARATOR.$entry;
                     // This entry is not an issue: it clears
                     if (!is_dir($entry)) {
-                        unlink ($entry);
+                        unlink($entry);
                     } else { // This entry is a folder, it again on this issue
                         self::rmdir($entry);
                     }
@@ -68,7 +68,7 @@ 
             }
         }
         // It has erased all entries in the folder, we can now erase
-        rmdir ($dir);
+        rmdir($dir);
     }
 
     /**
@@ -160,11 +160,11 @@ 
         // enviar archivo
         if ($options['download']) {
             ob_clean();
-            header ('Content-Disposition: attachment; filename='.$file_compressed);
+            header('Content-Disposition: attachment; filename='.$file_compressed);
             $mimetype = self::mimetype($dir.DIRECTORY_SEPARATOR.$file_compressed);
             if ($mimetype)
-                header ('Content-Type: '.$mimetype);
-            header ('Content-Length: '.filesize($dir.DIRECTORY_SEPARATOR.$file_compressed));
+                header('Content-Type: '.$mimetype);
+            header('Content-Length: '.filesize($dir.DIRECTORY_SEPARATOR.$file_compressed));
             readfile($dir.DIRECTORY_SEPARATOR.$file_compressed);
             unlink($dir.DIRECTORY_SEPARATOR.$file_compressed);
         }

Braces +16 added lines, -8 removed lines

@@ -80,8 +80,9 @@ 
      */
     public static function mimetype($file)
     {
-        if (!function_exists('finfo_open'))
-            return false;
+        if (!function_exists('finfo_open')) {
+                    return false;
+        }
         $finfo = finfo_open(FILEINFO_MIME_TYPE);
         $mimetype = finfo_file($finfo, $file);
         finfo_close($finfo);
@@ -121,8 +122,11 @@ 
         }
         // si es formato gz y es directorio se cambia a tgz
         if (is_dir($file)) {
-            if ($options['format']=='gz') $options['format'] = 'tar.gz';
-            else if ($options['format']=='bz2') $options['format'] = 'tar.bz2';
+            if ($options['format']=='gz') {
+                $options['format'] = 'tar.gz';
+            } else if ($options['format']=='bz2') {
+                $options['format'] = 'tar.bz2';
+            }
         }
         // obtener directorio que contiene al archivo/directorio y el nombre de este
         $filepath = $file;
@@ -162,8 +166,9 @@ 
             ob_clean();
             header ('Content-Disposition: attachment; filename='.$file_compressed);
             $mimetype = self::mimetype($dir.DIRECTORY_SEPARATOR.$file_compressed);
-            if ($mimetype)
-                header ('Content-Type: '.$mimetype);
+            if ($mimetype) {
+                            header ('Content-Type: '.$mimetype);
+            }
             header ('Content-Length: '.filesize($dir.DIRECTORY_SEPARATOR.$file_compressed));
             readfile($dir.DIRECTORY_SEPARATOR.$file_compressed);
             unlink($dir.DIRECTORY_SEPARATOR.$file_compressed);
@@ -171,8 +176,11 @@ 
         // borrar directorio o archivo que se está comprimiendo si así se ha
         // solicitado
         if ($options['delete']) {
-            if (is_dir($filepath)) self::rmdir($filepath);
-            else unlink($filepath);
+            if (is_dir($filepath)) {
+                self::rmdir($filepath);
+            } else {
+                unlink($filepath);
+            }
         }
     }
 

lib/Log.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
      * Método que escribe un mensaje en la bitácora
      * @param code Código del mensaje que se desea escribir
      * @param msg Mensaje que se desea escribir
-     * @param severity Gravedad del mensaje, por defecto LOG_ERR (puede ser cualquiera de las constantes PHP de syslog)
+     * @param severity integer del mensaje, por defecto LOG_ERR (puede ser cualquiera de las constantes PHP de syslog)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-16
      */

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

Doc Comments +4 added lines, -4 removed lines

@@ -57,7 +57,7 @@ 
      * WSDL producción: https://palena.sii.cl/DTEWS/CrSeed.jws?WSDL
      * WSDL certificación: https://maullin.sii.cl/DTEWS/CrSeed.jws?WSDL
      *
-     * @return Semilla obtenida desde SII
+     * @return false|string obtenida desde SII
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-17
      */
@@ -76,9 +76,9 @@ 
 
     /**
      * Método que firma una semilla previamente obtenida
-     * @param seed Semilla obtenida desde SII
+     * @param seed string obtenida desde SII
      * @param Firma objeto de la Firma electrónica o arreglo con configuración de la misma
-     * @return Solicitud de token con la semilla incorporada y firmada
+     * @return \sasco\LibreDTE\Argumentos|null de token con la semilla incorporada y firmada
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-17
      */
@@ -113,7 +113,7 @@ 
      * WSDL certificación: https://maullin.sii.cl/DTEWS/GetTokenFromSeed.jws?WSDL
      *
      * @param Firma objeto de la Firma electrónica o arreglo con configuración de la misma
-     * @return Token para autenticación en SII o =false si no se pudo obtener
+     * @return false|string para autenticación en SII o =false si no se pudo obtener
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-11-03
      */

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/Base/Documento.php

Doc Comments +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
 
     /**
      * Método que entrega el ID del documento
-     * @return ID del libro
+     * @return string del libro
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-12-14
      */
@@ -70,7 +70,7 @@ 
 
     /**
      * Método para asignar la caratula
-     * @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 2015-12-14
      */

Braces +6 added lines, -4 removed lines

@@ -169,10 +169,12 @@
      */
     public function toArray()
     {
-        if (!$this->xml)
-            return false;
-        if (!$this->arreglo)
-            $this->arreglo = $this->xml->toArray();
+        if (!$this->xml) {
+                    return false;
+        }
+        if (!$this->arreglo) {
+                    $this->arreglo = $this->xml->toArray();
+        }
         return $this->arreglo;
     }
 

lib/Sii/EnvioDte.php

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
      * @warning No se está validando la firma del EnvioDTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07

Braces +24 added lines, -16 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) {
@@ -402,12 +407,14 @@ 
      */
     public function getEstadoValidacion(array $datos = null)
     {
-        if (!$this->schemaValidate())
-            return 1;
+        if (!$this->schemaValidate()) {
+                    return 1;
+        }
         /*if (!$this->checkFirma())
             return 2;*/
-        if ($datos and $this->getReceptor()!=$datos['RutReceptor'])
-            return 3;
+        if ($datos and $this->getReceptor()!=$datos['RutReceptor']) {
+                    return 3;
+        }
         return 0;
     }
 
@@ -419,8 +426,9 @@ 
      */
     public function checkFirma()
     {
-        if (!$this->xml)
-            return null;
+        if (!$this->xml) {
+                    return null;
+        }
         // listado de firmas del XML
         $Signatures = $this->xml->documentElement->getElementsByTagName('Signature');
         // verificar firma de SetDTE

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

lib/Sii/Factoring/Aec.php

Doc Comments +3 added lines, -3 removed lines

@@ -48,7 +48,7 @@ 
 
     /**
      * Método que agrega el objeto Cesion
-     * @param Cesion Objeto de \sasco\LibreDTE\Sii\Factoring\Cesion
+     * @param Cesion Cesion de \sasco\LibreDTE\Sii\Factoring\Cesion
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-12-10
      */
@@ -80,7 +80,7 @@ 
 
     /**
      * Método que genera el XML del AEC
-     * @return XML AEC con DTE y Cesion
+     * @return boolean|string AEC con DTE y Cesion
      * @author Adonias Vasquez (adonias.vasquez[at]epys.cl)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-12-10
@@ -173,7 +173,7 @@ 
      * @param email del usuario que envía el AEC
      * @param empresa RUT de la empresa emisora del AEC
      * @param dte Documento XML con el DTE que se desea enviar a SII
-     * @param token Token de autenticación automática ante el SII
+     * @param token string de autenticación automática ante el SII
      * @param retry Intentos que se realizarán como máximo para obtener respuesta
      * @return Respuesta XML desde SII o bien null si no se pudo obtener respuesta
      * @author Adonias Vasquez (adonias.vasquez[at]epys.cl)

Spacing +8 added lines, -8 removed lines

@@ -184,11 +184,11 @@ 
     {
         // definir datos que se usarán en el envío
         list($rutCompany, $dvCompany) = explode('-', str_replace('.', '', $empresa));
-        if (strpos($dte, '<?xml') === false) {
-            $dte = '<?xml version="1.0" encoding="ISO-8859-1"?>' . "\n" . $dte;
+        if (strpos($dte, '<?xml')===false) {
+            $dte = '<?xml version="1.0" encoding="ISO-8859-1"?>'."\n".$dte;
         }
         do {
-            $file = sys_get_temp_dir() . '/aec_' . md5(microtime() . $token . $dte) . '.xml';
+            $file = sys_get_temp_dir().'/aec_'.md5(microtime().$token.$dte).'.xml';
         } while (file_exists($file));
         file_put_contents($file, $dte);
         $data = [
@@ -228,17 +228,17 @@ 
             curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);
         }
         // enviar XML al SII
-        for ($i = 0; $i < $retry; $i++) {
+        for ($i = 0; $i<$retry; $i++) {
             $response = curl_exec($curl);
-            if ($response and $response != 'Error 500')
+            if ($response and $response!='Error 500')
                 break;
         }
         unlink($file);
         // verificar respuesta del envío y entregar error en caso que haya uno
-        if (!$response or $response == 'Error 500') {
+        if (!$response or $response=='Error 500') {
             if (!$response)
                 \sasco\LibreDTE\Log::write(\sasco\LibreDTE\Estado::ENVIO_ERROR_CURL, \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::ENVIO_ERROR_CURL, curl_error($curl)));
-            if ($response == 'Error 500')
+            if ($response=='Error 500')
                 \sasco\LibreDTE\Log::write(\sasco\LibreDTE\Estado::ENVIO_ERROR_500, \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::ENVIO_ERROR_500));
             return false;
         }
@@ -270,7 +270,7 @@ 
             9 => 'Error Interno',
             10 => 'Error Interno'
         ];
-        if ($xml->STATUS != 0) {
+        if ($xml->STATUS!=0) {
             \sasco\LibreDTE\Log::write(
                 $xml->STATUS,
                 $error[$xml->STATUS]

Braces +30 added lines, -20 removed lines

@@ -87,10 +87,12 @@ 
      */
     public function generar()
     {
-        if (!isset($this->cedido) or !isset($this->cesiones[0]))
-            return false;
-        if (!isset($this->caratula))
-            $this->setCaratula();
+        if (!isset($this->cedido) or !isset($this->cesiones[0])) {
+                    return false;
+        }
+        if (!isset($this->caratula)) {
+                    $this->setCaratula();
+        }
         // genear XML del envío
         $xmlEnvio = (new \sasco\LibreDTE\XML())->generate([
             'AEC' => [
@@ -137,8 +139,9 @@ 
     public function enviar()
     {
         // generar XML que se enviará
-        if (!$this->xml_data)
-            $this->xml_data = $this->generar();
+        if (!$this->xml_data) {
+                    $this->xml_data = $this->generar();
+        }
         if (!$this->xml_data) {
             \sasco\LibreDTE\Log::write(
                 \sasco\LibreDTE\Estado::DOCUMENTO_ERROR_GENERAR_XML,
@@ -150,20 +153,24 @@ 
             return false;
         }
         // validar schema del documento antes de enviar
-        if (!$this->schemaValidate())
-            return false;
+        if (!$this->schemaValidate()) {
+                    return false;
+        }
         // solicitar token
         $token = \sasco\LibreDTE\Sii\Autenticacion::getToken($this->Firma);
-        if (!$token)
-            return false;
+        if (!$token) {
+                    return false;
+        }
         // enviar AEC
         $email = $this->caratula['MailContacto'];
         $emisor = $this->caratula['RutCedente'];
         $result = $this->enviarRTC($email, $emisor, $this->xml_data, $token, 10);
-        if ($result===false)
-            return false;
-        if (!is_numeric((string)$result->TRACKID))
-            return false;
+        if ($result===false) {
+                    return false;
+        }
+        if (!is_numeric((string)$result->TRACKID)) {
+                    return false;
+        }
         return (int)(string)$result->TRACKID;
     }
 
@@ -230,16 +237,19 @@ 
         // enviar XML al SII
         for ($i = 0; $i < $retry; $i++) {
             $response = curl_exec($curl);
-            if ($response and $response != 'Error 500')
-                break;
+            if ($response and $response != 'Error 500') {
+                            break;
+            }
         }
         unlink($file);
         // verificar respuesta del envío y entregar error en caso que haya uno
         if (!$response or $response == 'Error 500') {
-            if (!$response)
-                \sasco\LibreDTE\Log::write(\sasco\LibreDTE\Estado::ENVIO_ERROR_CURL, \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::ENVIO_ERROR_CURL, curl_error($curl)));
-            if ($response == 'Error 500')
-                \sasco\LibreDTE\Log::write(\sasco\LibreDTE\Estado::ENVIO_ERROR_500, \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::ENVIO_ERROR_500));
+            if (!$response) {
+                            \sasco\LibreDTE\Log::write(\sasco\LibreDTE\Estado::ENVIO_ERROR_CURL, \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::ENVIO_ERROR_CURL, curl_error($curl)));
+            }
+            if ($response == 'Error 500') {
+                            \sasco\LibreDTE\Log::write(\sasco\LibreDTE\Estado::ENVIO_ERROR_500, \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::ENVIO_ERROR_500));
+            }
             return false;
         }
         // cerrar sesión curl

lib/Sii/Factoring/Cesion.php

Doc Comments +3 added lines, -3 removed lines

@@ -38,8 +38,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 2016-12-10
@@ -177,7 +177,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 2016-08-10

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

examples/020-log.php

Spacing +8 added lines, -8 removed lines

@@ -65,7 +65,7 @@ 
     }
     public function caso2()
     {
-        for ($i=0; $i<5; $i++)
+        for ($i = 0; $i<5; $i++)
             \sasco\LibreDTE\Log::write('Soy el error '.($i+1));
     }
     public function caso3()
@@ -82,11 +82,11 @@ 
 $GeneradorErrores->caso1();
 
 // obtener el error más reciente y usar como string
-echo \sasco\LibreDTE\Log::read(),"\n\n";
+echo \sasco\LibreDTE\Log::read(), "\n\n";
 
 // mostrar el resto de errores y usar como string
 foreach (\sasco\LibreDTE\Log::readAll() as $error)
-    echo $error,"\n";
+    echo $error, "\n";
 echo "\n\n\n";
 
 // ejecutar caso 2
@@ -94,27 +94,27 @@ 
 
 // obtener el error más reciente y usar como objeto
 $error = \sasco\LibreDTE\Log::read();
-echo 'error código: ',$error->code,' y mensaje ',$error->msg,"\n\n";
+echo 'error código: ', $error->code, ' y mensaje ', $error->msg, "\n\n";
 
 // mostrar el resto de errores y usar como objeto
 foreach (\sasco\LibreDTE\Log::readAll() as $error)
-    echo 'error código: ',$error->code,' y mensaje ',$error->msg,"\n";
+    echo 'error código: ', $error->code, ' y mensaje ', $error->msg, "\n";
 echo "\n\n\n";
 
 // ejecutar caso 3 y mostrar en español todos los mensajes
 $GeneradorErrores->caso3();
 foreach (\sasco\LibreDTE\Log::readAll() as $error)
-    echo $error,"\n";
+    echo $error, "\n";
 echo "\n\n\n";
 
 // ejecutar caso 3 y mostrar en español todos los mensajes pero como objetos
 $GeneradorErrores->caso3();
 foreach (\sasco\LibreDTE\Log::readAll() as $error)
-    echo 'error código: ',$error->code,' y mensaje ',$error->msg,"\n";
+    echo 'error código: ', $error->code, ' y mensaje ', $error->msg, "\n";
 echo "\n\n\n";
 
 // ejecutar caso 3 y mostrar en inglés todos los mensajes
 \sasco\LibreDTE\I18n::setIdioma('en'); // idioma se debe asignar antes que se registre cualquier mensaje en el Log
 $GeneradorErrores->caso3();
 foreach (\sasco\LibreDTE\Log::readAll() as $error)
-    echo $error,"\n";
+    echo $error, "\n";

Braces +13 added lines, -7 removed lines

@@ -65,8 +65,9 @@ 
     }
     public function caso2()
     {
-        for ($i=0; $i<5; $i++)
-            \sasco\LibreDTE\Log::write('Soy el error '.($i+1));
+        for ($i=0; $i<5; $i++) {
+                    \sasco\LibreDTE\Log::write('Soy el error '.($i+1));
+        }
     }
     public function caso3()
     {
@@ -85,8 +86,9 @@ 
 echo \sasco\LibreDTE\Log::read(),"\n\n";
 
 // mostrar el resto de errores y usar como string
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo $error,"\n";
+}
 echo "\n\n\n";
 
 // ejecutar caso 2
@@ -97,24 +99,28 @@ 
 echo 'error código: ',$error->code,' y mensaje ',$error->msg,"\n\n";
 
 // mostrar el resto de errores y usar como objeto
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo 'error código: ',$error->code,' y mensaje ',$error->msg,"\n";
+}
 echo "\n\n\n";
 
 // ejecutar caso 3 y mostrar en español todos los mensajes
 $GeneradorErrores->caso3();
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo $error,"\n";
+}
 echo "\n\n\n";
 
 // ejecutar caso 3 y mostrar en español todos los mensajes pero como objetos
 $GeneradorErrores->caso3();
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo 'error código: ',$error->code,' y mensaje ',$error->msg,"\n";
+}
 echo "\n\n\n";
 
 // ejecutar caso 3 y mostrar en inglés todos los mensajes
 \sasco\LibreDTE\I18n::setIdioma('en'); // idioma se debe asignar antes que se registre cualquier mensaje en el Log
 $GeneradorErrores->caso3();
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo $error,"\n";
+}

examples/inc.php

Braces +6 added lines, -4 removed lines

@@ -38,18 +38,20 @@
 date_default_timezone_set('America/Santiago');
 
 // incluir autocarga de composer
-if (is_readable(dirname(dirname(__FILE__)).'/vendor/autoload.php'))
+if (is_readable(dirname(dirname(__FILE__)).'/vendor/autoload.php')) {
     include dirname(dirname(__FILE__)).'/vendor/autoload.php';
-else
+} else {
     die('Para probar los ejemplos debes ejecutar primero "composer install" en el directorio '.dirname(dirname(__FILE__))."\n");
+}
 
 // todos los ejemplos se ejecutan con backtrace activado, esto para ayudar al
 // debug de los mismos
 \sasco\LibreDTE\Log::setBacktrace(true);
 
 // incluir configuración específica de los ejemplos
-if (is_readable('config.php'))
+if (is_readable('config.php')) {
     include 'config.php';
-else
+} else {
     die('Debes crear config.php a partir de config-dist.php'."\n");
+}