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

Doc Comments +7 added lines, -7 removed lines

@@ -140,8 +140,8 @@ 
 
     /**
      * Método para realizar una solicitud al servicio web del SII
-     * @param wsdl Nombre del WSDL que se usará
-     * @param request Nombre de la función que se ejecutará en el servicio web
+     * @param wsdl string del WSDL que se usará
+     * @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 SimpleXMLElement con la espuesta del servicio web consultado
@@ -225,7 +225,7 @@ 
      * @param usuario RUN del usuario que envía el DTE
      * @param empresa RUT de la empresa emisora del DTE
      * @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 Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
@@ -321,7 +321,7 @@ 
      * \endcode
      *
      * @param idk IDK de la clave pública del SII. Si no se indica se tratará de determinar con el ambiente que se esté usando
-     * @return Contenido del certificado
+     * @return string|false del certificado
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-16
      */
@@ -346,7 +346,7 @@ 
     /**
      * Método que asigna el ambiente que se usará por defecto (si no está
      * asignado con la constante _LibreDTE_CERTIFICACION_)
-     * @param ambiente Ambiente a usar: Sii::PRODUCCION o Sii::CERTIFICACION
+     * @param ambiente integer a usar: Sii::PRODUCCION o Sii::CERTIFICACION
      * @warning No se está verificando SSL en ambiente de certificación
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-28
@@ -381,7 +381,7 @@ 
 
     /**
      * Método que entrega la tasa de IVA vigente
-     * @return Tasa de IVA vigente
+     * @return integer de IVA vigente
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-03
      */
@@ -457,7 +457,7 @@ 
     /**
      * Método que entrega la dirección regional según la comuna que se esté
      * consultando
-     * @param comuna de la sucursal del emior o bien código de la sucursal del SII
+     * @param comuna Sii\PDF\Código|null la sucursal del emior o bien código de la sucursal del SII
      * @return Dirección regional del SII
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-06-03

Spacing +5 added lines, -5 removed lines

@@ -183,7 +183,7 @@ 
             \sasco\LibreDTE\Log::write(Estado::REQUEST_ERROR_SOAP, Estado::get(Estado::REQUEST_ERROR_SOAP, $msg));
             return false;
         }
-        for ($i=0; $i<$retry; $i++) {
+        for ($i = 0; $i<$retry; $i++) {
             try {
                 if ($args) {
                     $body = call_user_func_array([$soap, $request], $args);
@@ -281,7 +281,7 @@ 
             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')
                 break;
@@ -307,7 +307,7 @@ 
         if ($xml->STATUS!=0) {
             \sasco\LibreDTE\Log::write(
                 $xml->STATUS,
-                Estado::get($xml->STATUS).(isset($xml->DETAIL)?'. '.implode("\n", (array)$xml->DETAIL->ERROR):'')
+                Estado::get($xml->STATUS).(isset($xml->DETAIL) ? '. '.implode("\n", (array)$xml->DETAIL->ERROR) : '')
             );
         }
         return $xml;
@@ -439,12 +439,12 @@ 
         $lines = explode("\n", $response);
         $n_lines = count($lines);
         $data = [];
-        for ($i=1; $i<$n_lines; $i++) {
+        for ($i = 1; $i<$n_lines; $i++) {
             $row = str_getcsv($lines[$i], ';', '');
             unset($lines[$i]);
             if (!isset($row[5]))
                 continue;
-            for ($j=0; $j<6; $j++)
+            for ($j = 0; $j<6; $j++)
                 $row[$j] = trim($row[$j]);
             $row[1] = utf8_decode($row[1]);
             $row[4] = strtolower($row[4]);

Braces +41 added lines, -28 removed lines

@@ -124,8 +124,9 @@ 
         // entregar WSDL local (modificados para ambiente de certificación)
         if ($ambiente==self::CERTIFICACION) {
             $wsdl = dirname(dirname(__FILE__)).'/wsdl/'.self::$config['servidor'][$ambiente].'/'.$servicio.'.jws';
-            if (is_readable($wsdl))
-                return $wsdl;
+            if (is_readable($wsdl)) {
+                            return $wsdl;
+            }
         }
         // entregar WSDL oficial desde SII
         $location = isset(self::$config['wsdl'][$servicio]) ? self::$config['wsdl'][$servicio] : self::$config['wsdl']['*'];
@@ -154,10 +155,12 @@ 
             $retry = (int)$args;
             $args = null;
         }
-        if (!$retry)
-            $retry = self::$retry;
-        if ($args and !is_array($args))
-            $args = [$args];
+        if (!$retry) {
+                    $retry = self::$retry;
+        }
+        if ($args and !is_array($args)) {
+                    $args = [$args];
+        }
         if (!self::$verificar_ssl) {
             if (self::getAmbiente()==self::PRODUCCION) {
                 $msg = Estado::get(Estado::ENVIO_SSL_SIN_VERIFICAR);
@@ -255,8 +258,9 @@ 
             ),
         ];
         // definir reintentos si no se pasaron
-        if (!$retry)
-            $retry = self::$retry;
+        if (!$retry) {
+                    $retry = self::$retry;
+        }
         // crear sesión curl con sus opciones
         $curl = curl_init();
         $header = [
@@ -283,16 +287,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(Estado::ENVIO_ERROR_CURL, Estado::get(Estado::ENVIO_ERROR_CURL, curl_error($curl)));
-            if ($response=='Error 500')
-                \sasco\LibreDTE\Log::write(Estado::ENVIO_ERROR_500, Estado::get(Estado::ENVIO_ERROR_500));
+            if (!$response) {
+                            \sasco\LibreDTE\Log::write(Estado::ENVIO_ERROR_CURL, Estado::get(Estado::ENVIO_ERROR_CURL, curl_error($curl)));
+            }
+            if ($response=='Error 500') {
+                            \sasco\LibreDTE\Log::write(Estado::ENVIO_ERROR_500, Estado::get(Estado::ENVIO_ERROR_500));
+            }
             return false;
         }
         // cerrar sesión curl
@@ -330,8 +337,9 @@ 
         // si se pasó un idk y existe el archivo asociado se entrega
         if ($idk) {
             $cert = dirname(dirname(__FILE__)).'/certs/'.$idk.'.cer';
-            if (is_readable($cert))
-                return file_get_contents($cert);
+            if (is_readable($cert)) {
+                            return file_get_contents($cert);
+            }
         }
         // buscar certificado y entregar si existe o =false si no
         $ambiente = self::getAmbiente();
@@ -371,10 +379,11 @@ 
     public static function getAmbiente($ambiente = null)
     {
         if ($ambiente===null) {
-            if (defined('_LibreDTE_CERTIFICACION_'))
-                $ambiente = (int)_LibreDTE_CERTIFICACION_;
-            else
-                $ambiente = self::$ambiente;
+            if (defined('_LibreDTE_CERTIFICACION_')) {
+                            $ambiente = (int)_LibreDTE_CERTIFICACION_;
+            } else {
+                            $ambiente = self::$ambiente;
+            }
         }
         return $ambiente;
     }
@@ -400,8 +409,9 @@ 
     {
         // solicitar token
         $token = \sasco\LibreDTE\Sii\Autenticacion::getToken($Firma);
-        if (!$token)
-            return false;
+        if (!$token) {
+                    return false;
+        }
         // definir ambiente y servidor
         $ambiente = self::getAmbiente($ambiente);
         $servidor = self::$config['servidor'][$ambiente];
@@ -430,8 +440,9 @@ 
         }
         // realizar consulta curl
         $response = curl_exec($curl);
-        if (!$response)
-            return false;
+        if (!$response) {
+                    return false;
+        }
         // cerrar sesión curl
         curl_close($curl);
         // entregar datos del archivo CSV
@@ -442,10 +453,12 @@ 
         for ($i=1; $i<$n_lines; $i++) {
             $row = str_getcsv($lines[$i], ';', '');
             unset($lines[$i]);
-            if (!isset($row[5]))
-                continue;
-            for ($j=0; $j<6; $j++)
-                $row[$j] = trim($row[$j]);
+            if (!isset($row[5])) {
+                            continue;
+            }
+            for ($j=0; $j<6; $j++) {
+                            $row[$j] = trim($row[$j]);
+            }
             $row[1] = utf8_decode($row[1]);
             $row[4] = strtolower($row[4]);
             $row[5] = strtolower($row[5]);

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

Doc Comments +11 added lines, -11 removed lines

@@ -236,7 +236,7 @@ 
 
     /**
      * Método que entrega el tipo de DTE
-     * @return Tipo de dte, ej: 33 (factura electrónica)
+     * @return string de dte, ej: 33 (factura electrónica)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-02
      */
@@ -247,7 +247,7 @@ 
 
     /**
      * Método que entrega el folio del DTE
-     * @return Folio del DTE
+     * @return string del DTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-02
      */
@@ -258,7 +258,7 @@ 
 
     /**
      * Método que entrega rut del emisor del DTE
-     * @return RUT del emiro
+     * @return string del emiro
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -274,7 +274,7 @@ 
 
     /**
      * Método que entrega rut del receptor del DTE
-     * @return RUT del emiro
+     * @return string del emiro
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -290,7 +290,7 @@ 
 
     /**
      * Método que entrega fecha de emisión del DTE
-     * @return Fecha de emisión en formato AAAA-MM-DD
+     * @return string de emisión en formato AAAA-MM-DD
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -378,7 +378,7 @@ 
 
     /**
      * Método que realiza el timbrado del DTE
-     * @param Folios Objeto de los Folios con los que se desea timbrar
+     * @param Folios Folios de los Folios con los que se desea timbrar
      * @return =true si se pudo timbrar o =false en caso de error
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-09-01
@@ -464,7 +464,7 @@ 
 
     /**
      * Método que realiza la firma del DTE
-     * @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 2015-09-17
@@ -487,7 +487,7 @@ 
 
     /**
      * Método que entrega el DTE en XML
-     * @return XML con el DTE (podría: con o sin timbre y con o sin firma)
+     * @return string con el DTE (podría: con o sin timbre y con o sin firma)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-08-20
      */
@@ -1595,7 +1595,7 @@ 
      *  - Firma del DTE
      *  - RUT del emisor (si se pasó uno para comparar)
      *  - RUT del receptor (si se pasó uno para comparar)
-     * @return Código del estado de la validación
+     * @return integer del estado de la validación
      * @warning No se está validando la firma
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-08
@@ -1706,7 +1706,7 @@ 
 
     /**
      * Método que obtiene el estado del DTE
-     * @param Firma objeto que representa la Firma Electrónca
+     * @param Firma \sasco\LibreDTE\FirmaElectronica que representa la Firma Electrónca
      * @return Arreglo con el estado del DTE
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-10-24
@@ -1747,7 +1747,7 @@ 
 
     /**
      * Método que obtiene el estado avanzado del DTE
-     * @param Firma objeto que representa la Firma Electrónca
+     * @param Firma \sasco\LibreDTE\FirmaElectronica que representa la Firma Electrónca
      * @return Arreglo con el estado del DTE
      * @todo Corregir warning y también definir que se retornará (sobre todo en caso de error)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)

Spacing +16 added lines, -16 removed lines

@@ -139,7 +139,7 @@ 
                 ]
             ]);
             $parent = $this->xml->getElementsByTagName($this->tipo_general)->item(0);
-            $this->xml->generate($datos + ['TED' => null], $parent);
+            $this->xml->generate($datos+['TED' => null], $parent);
             $this->datos = $datos;
             if ($normalizar and !$this->verificarDatos()) {
                 return false;
@@ -361,7 +361,7 @@ 
         $xml->documentElement->removeAttributeNS('http://www.w3.org/2001/XMLSchema-instance', 'xsi');
         $xml->documentElement->removeAttributeNS('http://www.sii.cl/SiiDte', '');
         $TED = $xml->getFlattened('/');
-        return mb_detect_encoding($TED, ['UTF-8', 'ISO-8859-1']) != 'ISO-8859-1' ? utf8_decode($TED) : $TED;
+        return mb_detect_encoding($TED, ['UTF-8', 'ISO-8859-1'])!='ISO-8859-1' ? utf8_decode($TED) : $TED;
     }
 
     /**
@@ -374,7 +374,7 @@ 
     {
         $datos = $this->getDatos();
         $idk = !empty($datos['TED']['DD']['CAF']['DA']['IDK']) ? (int)$datos['TED']['DD']['CAF']['DA']['IDK'] : null;
-        return $idk ? $idk === 100 : null;
+        return $idk ? $idk===100 : null;
     }
 
     /**
@@ -508,7 +508,7 @@ 
     {
         $this->getDatos();
         // generar resumen
-        $resumen =  [
+        $resumen = [
             'TpoDoc' => (int)$this->datos['Encabezado']['IdDoc']['TipoDTE'],
             'NroDoc' => (int)$this->datos['Encabezado']['IdDoc']['Folio'],
             'TasaImp' => 0,
@@ -553,17 +553,17 @@ 
      */
     private function calcularNetoIVA($total, $tasa = null)
     {
-        if ($tasa === 0 or $tasa === false)
+        if ($tasa===0 or $tasa===false)
             return [0, 0];
-        if ($tasa === null)
+        if ($tasa===null)
             $tasa = \sasco\LibreDTE\Sii::getIVA();
         // WARNING: el IVA obtenido puede no ser el NETO*(TASA/100)
         // se calcula el monto neto y luego se obtiene el IVA haciendo la resta
         // entre el total y el neto, ya que hay casos de borde como:
         //  - BRUTO:   680 => NETO:   571 e IVA:   108 => TOTAL:   679
         //  - BRUTO: 86710 => NETO: 72866 e IVA: 13845 => TOTAL: 86711
-        $neto = round($total / (1+($tasa/100)));
-        $iva = $total - $neto;
+        $neto = round($total / (1+($tasa / 100)));
+        $iva = $total-$neto;
         return [$neto, $iva];
     }
 
@@ -1309,12 +1309,12 @@ 
                     );
                     // aplicar descuento
                     if ($d['DescuentoPct']) {
-                        $d['DescuentoMonto'] = round($d['MontoItem'] * (int)$d['DescuentoPct']/100);
+                        $d['DescuentoMonto'] = round($d['MontoItem'] * (int)$d['DescuentoPct'] / 100);
                     }
                     $d['MontoItem'] -= $d['DescuentoMonto'];
                     // aplicar recargo
                     if ($d['RecargoPct']) {
-                        $d['RecargoMonto'] = round($d['MontoItem'] * (int)$d['RecargoPct']/100);
+                        $d['RecargoMonto'] = round($d['MontoItem'] * (int)$d['RecargoPct'] / 100);
                     }
                     $d['MontoItem'] += $d['RecargoMonto'];
                     // aproximar monto del item
@@ -1400,7 +1400,7 @@ 
                 }
                 $valor =
                     $dr['TpoValor']=='%'
-                    ? $this->round(($dr['ValorDR']/100)*$datos['Encabezado']['Totales'][$monto], $datos['Encabezado']['Totales']['TpoMoneda'])
+                    ? $this->round(($dr['ValorDR'] / 100) * $datos['Encabezado']['Totales'][$monto], $datos['Encabezado']['Totales']['TpoMoneda'])
                     : $dr['ValorDR']
                 ;
                 // aplicar descuento
@@ -1473,7 +1473,7 @@ 
             // si el monto no existe se asigna
             if ($datos['Encabezado']['Totales']['ImptoReten'][$i]['MontoImp']===null) {
                 $datos['Encabezado']['Totales']['ImptoReten'][$i]['MontoImp'] = round(
-                    $neto * $datos['Encabezado']['Totales']['ImptoReten'][$i]['TasaImp']/100
+                    $neto * $datos['Encabezado']['Totales']['ImptoReten'][$i]['TasaImp'] / 100
                 );
             }
         }
@@ -1481,7 +1481,7 @@ 
         if (isset($datos['Encabezado']['Totales']['ImptoReten']) and is_array($datos['Encabezado']['Totales']['ImptoReten'])) {
             $codigos = array_keys($montos);
             $n_impuestos = count($datos['Encabezado']['Totales']['ImptoReten']);
-            for ($i=0; $i<$n_impuestos; $i++) {
+            for ($i = 0; $i<$n_impuestos; $i++) {
                 if (!in_array($datos['Encabezado']['Totales']['ImptoReten'][$i]['TipoImp'], $codigos)) {
                     unset($datos['Encabezado']['Totales']['ImptoReten'][$i]);
                 }
@@ -1509,7 +1509,7 @@ 
             } else {
                 if (empty($datos['Encabezado']['Totales']['IVA']) and !empty($datos['Encabezado']['Totales']['TasaIVA'])) {
                     $datos['Encabezado']['Totales']['IVA'] = round(
-                        $datos['Encabezado']['Totales']['MntNeto']*($datos['Encabezado']['Totales']['TasaIVA']/100)
+                        $datos['Encabezado']['Totales']['MntNeto'] * ($datos['Encabezado']['Totales']['TasaIVA'] / 100)
                     );
                 }
             }
@@ -1533,7 +1533,7 @@ 
                 if (ImpuestosAdicionales::getTipo($ImptoReten['TipoImp'])=='R') {
                     $datos['Encabezado']['Totales']['MntTotal'] -= $ImptoReten['MontoImp'];
                     if ($ImptoReten['MontoImp']!=$datos['Encabezado']['Totales']['IVA']) {
-                        $datos['Encabezado']['Totales']['IVANoRet'] = $datos['Encabezado']['Totales']['IVA'] - $ImptoReten['MontoImp'];
+                        $datos['Encabezado']['Totales']['IVANoRet'] = $datos['Encabezado']['Totales']['IVA']-$ImptoReten['MontoImp'];
                     }
                 }
                 // si es adicional se suma al total
@@ -1679,7 +1679,7 @@ 
         $SignatureValue = $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue;
         $X509Certificate = $Signature->getElementsByTagName('X509Certificate')->item(0)->nodeValue;
         $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;
         //return $valid and $DigestValue===base64_encode(sha1($Documento->C14N(), true));
     }

Braces +108 added lines, -72 removed lines

@@ -57,10 +57,11 @@ 
      */
     public function __construct($datos, $normalizar = true)
     {
-        if (is_array($datos))
-            $this->setDatos($datos, $normalizar);
-        else if (is_string($datos))
-            $this->loadXML($datos);
+        if (is_array($datos)) {
+                    $this->setDatos($datos, $normalizar);
+        } else if (is_string($datos)) {
+                    $this->loadXML($datos);
+        }
         $this->timestamp = date('Y-m-d\TH:i:s');
     }
 
@@ -121,8 +122,9 @@ 
             if ($normalizar) {
                 $this->normalizar($datos);
                 $method = 'normalizar_'.$this->tipo;
-                if (method_exists($this, $method))
-                    $this->$method($datos);
+                if (method_exists($this, $method)) {
+                                    $this->$method($datos);
+                }
                 $this->normalizar_final($datos);
             }
             $this->tipo_general = $this->getTipoGeneral($this->tipo);
@@ -199,8 +201,9 @@ 
      */
     public function getJSON()
     {
-        if (!$this->getDatos())
-            return false;
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return json_encode($this->datos, JSON_PRETTY_PRINT);
     }
 
@@ -225,9 +228,10 @@ 
      */
     private function getTipoGeneral($dte)
     {
-        foreach ($this->tipos as $tipo => $codigos)
-            if (in_array($dte, $codigos))
+        foreach ($this->tipos as $tipo => $codigos) {
+                    if (in_array($dte, $codigos))
                 return $tipo;
+        }
         \sasco\LibreDTE\Log::write(
             \sasco\LibreDTE\Estado::DTE_ERROR_TIPO,
             \sasco\LibreDTE\Estado::get(\sasco\LibreDTE\Estado::DTE_ERROR_TIPO, $dte)
@@ -266,10 +270,12 @@ 
     public function getEmisor()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Emisor/RUTEmisor')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Emisor']['RUTEmisor'];
     }
 
@@ -282,10 +288,12 @@ 
     public function getReceptor()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Receptor/RUTRecep')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Receptor']['RUTRecep'];
     }
 
@@ -298,10 +306,12 @@ 
     public function getFechaEmision()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/IdDoc/FchEmis')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['IdDoc']['FchEmis'];
     }
 
@@ -314,10 +324,12 @@ 
     public function getMontoTotal()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Totales/MntTotal')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Totales']['MntTotal'];
     }
 
@@ -330,10 +342,12 @@ 
     public function getMoneda()
     {
         $nodo = $this->xml->xpath('/DTE/'.$this->tipo_general.'/Encabezado/Totales/TpoMoneda')->item(0);
-        if ($nodo)
-            return $nodo->nodeValue;
-        if (!$this->getDatos())
-            return false;
+        if ($nodo) {
+                    return $nodo->nodeValue;
+        }
+        if (!$this->getDatos()) {
+                    return false;
+        }
         return $this->datos['Encabezado']['Totales']['TpoMoneda'];
     }
 
@@ -355,8 +369,9 @@ 
             return false;*/
         $xml = new \sasco\LibreDTE\XML();
         $TED = $this->xml->getElementsByTagName('TED')->item(0);
-        if (!$TED)
-            return '<TED/>';
+        if (!$TED) {
+                    return '<TED/>';
+        }
         $xml->loadXML($TED->C14N());
         $xml->documentElement->removeAttributeNS('http://www.w3.org/2001/XMLSchema-instance', 'xsi');
         $xml->documentElement->removeAttributeNS('http://www.sii.cl/SiiDte', '');
@@ -553,10 +568,12 @@ 
      */
     private function calcularNetoIVA($total, $tasa = null)
     {
-        if ($tasa === 0 or $tasa === false)
-            return [0, 0];
-        if ($tasa === null)
-            $tasa = \sasco\LibreDTE\Sii::getIVA();
+        if ($tasa === 0 or $tasa === false) {
+                    return [0, 0];
+        }
+        if ($tasa === null) {
+                    $tasa = \sasco\LibreDTE\Sii::getIVA();
+        }
         // WARNING: el IVA obtenido puede no ser el NETO*(TASA/100)
         // se calcula el monto neto y luego se obtiene el IVA haciendo la resta
         // entre el total y el neto, ya que hay casos de borde como:
@@ -665,8 +682,9 @@ 
         }
         // si existe descuento o recargo global se normalizan
         if (!empty($datos['DscRcgGlobal'])) {
-            if (!isset($datos['DscRcgGlobal'][0]))
-                $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            if (!isset($datos['DscRcgGlobal'][0])) {
+                            $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            }
             $NroLinDR = 1;
             foreach ($datos['DscRcgGlobal'] as &$dr) {
                 $dr = array_merge([
@@ -676,8 +694,9 @@ 
         }
         // si existe una o más referencias se normalizan
         if (!empty($datos['Referencia'])) {
-            if (!isset($datos['Referencia'][0]))
-                $datos['Referencia'] = [$datos['Referencia']];
+            if (!isset($datos['Referencia'][0])) {
+                            $datos['Referencia'] = [$datos['Referencia']];
+            }
             $NroLinRef = 1;
             foreach ($datos['Referencia'] as &$r) {
                 $r = array_merge([
@@ -707,8 +726,9 @@ 
         // normalizar montos de pagos programados
         if (is_array($datos['Encabezado']['IdDoc']['MntPagos'])) {
             $montos = 0;
-            if (!isset($datos['Encabezado']['IdDoc']['MntPagos'][0]))
-                $datos['Encabezado']['IdDoc']['MntPagos'] = [$datos['Encabezado']['IdDoc']['MntPagos']];
+            if (!isset($datos['Encabezado']['IdDoc']['MntPagos'][0])) {
+                            $datos['Encabezado']['IdDoc']['MntPagos'] = [$datos['Encabezado']['IdDoc']['MntPagos']];
+            }
             foreach ($datos['Encabezado']['IdDoc']['MntPagos'] as &$MntPagos) {
                 $MntPagos = array_merge([
                     'FchPago' => null,
@@ -1240,8 +1260,9 @@ 
      */
     private function normalizar_detalle(array &$datos)
     {
-        if (!isset($datos['Detalle'][0]))
-            $datos['Detalle'] = [$datos['Detalle']];
+        if (!isset($datos['Detalle'][0])) {
+                    $datos['Detalle'] = [$datos['Detalle']];
+        }
         $item = 1;
         foreach ($datos['Detalle'] as &$d) {
             $d = array_merge([
@@ -1300,8 +1321,9 @@ 
                 ];
             }
             if ($d['PrcItem']) {
-                if (!$d['QtyItem'])
-                    $d['QtyItem'] = 1;
+                if (!$d['QtyItem']) {
+                                    $d['QtyItem'] = 1;
+                }
                 if (empty($d['MontoItem'])) {
                     $d['MontoItem'] = $this->round(
                         $d['QtyItem'] * $d['PrcItem'],
@@ -1367,8 +1389,9 @@ 
     private function normalizar_aplicar_descuentos_recargos(array &$datos)
     {
         if (!empty($datos['DscRcgGlobal'])) {
-            if (!isset($datos['DscRcgGlobal'][0]))
-                $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            if (!isset($datos['DscRcgGlobal'][0])) {
+                            $datos['DscRcgGlobal'] = [$datos['DscRcgGlobal']];
+            }
             foreach ($datos['DscRcgGlobal'] as &$dr) {
                 $dr = array_merge([
                     'NroLinDR' => false,
@@ -1435,8 +1458,9 @@ 
         $montos = [];
         foreach ($datos['Detalle'] as &$d) {
             if (!empty($d['CodImpAdic'])) {
-                if (!isset($montos[$d['CodImpAdic']]))
-                    $montos[$d['CodImpAdic']] = 0;
+                if (!isset($montos[$d['CodImpAdic']])) {
+                                    $montos[$d['CodImpAdic']] = 0;
+                }
                 $montos[$d['CodImpAdic']] += $d['MontoItem'];
             }
         }
@@ -1515,10 +1539,12 @@ 
             }
             if (empty($datos['Encabezado']['Totales']['MntTotal'])) {
                 $datos['Encabezado']['Totales']['MntTotal'] = $datos['Encabezado']['Totales']['MntNeto'];
-                if (!empty($datos['Encabezado']['Totales']['IVA']))
-                    $datos['Encabezado']['Totales']['MntTotal'] += $datos['Encabezado']['Totales']['IVA'];
-                if (!empty($datos['Encabezado']['Totales']['MntExe']))
-                    $datos['Encabezado']['Totales']['MntTotal'] += $datos['Encabezado']['Totales']['MntExe'];
+                if (!empty($datos['Encabezado']['Totales']['IVA'])) {
+                                    $datos['Encabezado']['Totales']['MntTotal'] += $datos['Encabezado']['Totales']['IVA'];
+                }
+                if (!empty($datos['Encabezado']['Totales']['MntExe'])) {
+                                    $datos['Encabezado']['Totales']['MntTotal'] += $datos['Encabezado']['Totales']['MntExe'];
+                }
             }
         } else {
             if (!$datos['Encabezado']['Totales']['MntTotal'] and !empty($datos['Encabezado']['Totales']['MntExe'])) {
@@ -1544,8 +1570,10 @@ 
         }
         // si hay impuesto de crédito a constructoras del 65% se descuenta del total
         if (!empty($datos['Encabezado']['Totales']['CredEC'])) {
-            if ($datos['Encabezado']['Totales']['CredEC']===true)
-                $datos['Encabezado']['Totales']['CredEC'] = round($datos['Encabezado']['Totales']['IVA'] * 0.65); // TODO: mover a constante o método
+            if ($datos['Encabezado']['Totales']['CredEC']===true) {
+                            $datos['Encabezado']['Totales']['CredEC'] = round($datos['Encabezado']['Totales']['IVA'] * 0.65);
+            }
+            // TODO: mover a constante o método
             $datos['Encabezado']['Totales']['MntTotal'] -= $datos['Encabezado']['Totales']['CredEC'];
         }
     }
@@ -1645,10 +1673,12 @@ 
         /*if (!$this->checkFirma())
             return 1;*/
         if (is_array($datos)) {
-            if (isset($datos['RUTEmisor']) and $this->getEmisor()!=$datos['RUTEmisor'])
-                return 2;
-            if (isset($datos['RUTRecep']) and $this->getReceptor()!=$datos['RUTRecep'])
-                return 3;
+            if (isset($datos['RUTEmisor']) and $this->getEmisor()!=$datos['RUTEmisor']) {
+                            return 2;
+            }
+            if (isset($datos['RUTRecep']) and $this->getReceptor()!=$datos['RUTRecep']) {
+                            return 3;
+            }
         }
         return 0;
     }
@@ -1662,8 +1692,9 @@ 
      */
     public function checkFirma()
     {
-        if (!$this->xml)
-            return null;
+        if (!$this->xml) {
+                    return null;
+        }
         // obtener firma
         $Signature = $this->xml->documentElement->getElementsByTagName('Signature')->item(0);
         // preparar documento a validar
@@ -1755,12 +1786,14 @@ 
     {
         // solicitar token
         $token = \sasco\LibreDTE\Sii\Autenticacion::getToken($Firma);
-        if (!$token)
-            return false;
+        if (!$token) {
+                    return false;
+        }
         // consultar estado dte
         $run = $Firma->getID();
-        if ($run===false)
-            return false;
+        if ($run===false) {
+                    return false;
+        }
         list($RutConsultante, $DvConsultante) = explode('-', $run);
         list($RutCompania, $DvCompania) = explode('-', $this->getEmisor());
         list($RutReceptor, $DvReceptor) = explode('-', $this->getReceptor());
@@ -1779,8 +1812,9 @@ 
             'token'             => $token,
         ]);
         // si el estado se pudo recuperar se muestra
-        if ($xml===false)
-            return false;
+        if ($xml===false) {
+                    return false;
+        }
         // entregar estado
         return (array)$xml->xpath('/SII:RESPUESTA/SII:RESP_HDR')[0];
     }
@@ -1797,8 +1831,9 @@ 
     {
         // solicitar token
         $token = \sasco\LibreDTE\Sii\Autenticacion::getToken($Firma);
-        if (!$token)
-            return false;
+        if (!$token) {
+                    return false;
+        }
         // consultar estado dte
         list($RutEmpresa, $DvEmpresa) = explode('-', $this->getEmisor());
         list($RutReceptor, $DvReceptor) = explode('-', $this->getReceptor());
@@ -1816,8 +1851,9 @@ 
             'token'             => $token,
         ]);
         // si el estado se pudo recuperar se muestra
-        if ($xml===false)
-            return false;
+        if ($xml===false) {
+                    return false;
+        }
         // entregar estado
         return (array)$xml->xpath('/SII:RESPUESTA/SII:RESP_BODY')[0];
     }

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