libredte / libredte-lib-core

lib/Sii/Dte.php

Doc Comments +11 added lines, -11 removed lines

@@ -237,7 +237,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
      */
@@ -248,7 +248,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
      */
@@ -259,7 +259,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
      */
@@ -275,7 +275,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
      */
@@ -291,7 +291,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
      */
@@ -400,7 +400,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
@@ -486,7 +486,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 2017-10-22
@@ -509,7 +509,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
      */
@@ -1767,7 +1767,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
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-07-03
      */
@@ -1880,7 +1880,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
@@ -1921,7 +1921,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)

Braces +93 added lines, -62 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'];
     }
 
@@ -376,8 +390,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', '');
@@ -574,10 +589,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:
@@ -693,8 +710,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([
@@ -1357,8 +1375,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([
@@ -1420,8 +1439,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'],
@@ -1487,8 +1507,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,
@@ -1555,8 +1576,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'];
             }
         }
@@ -1635,10 +1657,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'])) {
@@ -1664,8 +1688,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'];
         }
     }
@@ -1889,12 +1915,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());
@@ -1913,8 +1941,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];
     }
@@ -1931,8 +1960,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());
@@ -1950,8 +1980,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
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-07-03
      */

Braces +15 added lines, -10 removed lines

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