libredte / libredte-lib-core

examples/031-libro_boleta.php

Spacing +1 added lines, -1 removed lines

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

Braces +2 added lines, -1 removed lines

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

examples/022-libro_ventas_csv.php

Spacing +1 added lines, -1 removed lines

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

Braces +2 added lines, -1 removed lines

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

lib/Sii/Folios.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +8 added lines, -8 removed lines

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

lib/Sii.php

Doc Comments +7 added lines, -7 removed lines

@@ -492,8 +492,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
@@ -588,7 +588,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 gzip Permite enviar el archivo XML comprimido al servidor
      * @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
@@ -696,7 +696,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
      */
@@ -722,7 +722,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
@@ -757,7 +757,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
      */
@@ -833,7 +833,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\Dte\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 2017-11-07

Spacing +4 added lines, -4 removed lines

@@ -536,7 +536,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);
@@ -605,7 +605,7 @@ 
             $dte = '<?xml version="1.0" encoding="ISO-8859-1"?>'."\n".$dte;
         }
         do {
-            $file = sys_get_temp_dir().'/dte_'.md5(microtime().$token.$dte).'.'.($gzip?'gz':'xml');
+            $file = sys_get_temp_dir().'/dte_'.md5(microtime().$token.$dte).'.'.($gzip ? 'gz' : 'xml');
         } while (file_exists($file));
         if ($gzip) {
             $dte = gzencode($dte);
@@ -654,7 +654,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;
@@ -683,7 +683,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;

Braces +8 added lines, -6 removed lines

@@ -476,8 +476,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']['*'];
@@ -748,10 +749,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;
     }

lib/Sii/Dte/Base/DteImpreso.php

Spacing +1 added lines, -1 removed lines

@@ -236,7 +236,7 @@
         $dias = ['Domingo', 'Lunes', 'Martes', 'Miércoles', 'Jueves', 'Viernes', 'Sábado'];
         $meses = ['enero', 'febrero', 'marzo', 'abril', 'mayo', 'junio', 'julio', 'agosto', 'septiembre', 'octubre', 'noviembre', 'diciembre'];
         $unixtime = strtotime($date);
-        $fecha = date(($mostrar_dia?'\D\I\A ':'').'j \d\e \M\E\S \d\e\l Y', $unixtime);
+        $fecha = date(($mostrar_dia ? '\D\I\A ' : '').'j \d\e \M\E\S \d\e\l Y', $unixtime);
         $dia = $dias[date('w', $unixtime)];
         $mes = $meses[date('n', $unixtime)-1];
         return str_replace(array('DIA', 'MES'), array($dia, $mes), $fecha);

Doc Comments +1 added lines, -1 removed lines

@@ -198,7 +198,7 @@
     /**
      * Método que entrega la glosa del tipo de documento
      * @param tipo Código del tipo de documento
-     * @param folio Folio del tipo de documento (usado al ser generados a partir de borradores en formato libredte)
+     * @param folio integer del tipo de documento (usado al ser generados a partir de borradores en formato libredte)
      * @return Glosa del tipo de documento
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-08-05

lib/Sii/Dte/PDF/LibroCompraVenta.php

Spacing +1 added lines, -1 removed lines

@@ -138,7 +138,7 @@
                 ];
                 // agregar otros impuestos adicionales
                 if ($n_OtrosImp>1) {
-                    for ($i=1; $i<$n_OtrosImp; $i++) {
+                    for ($i = 1; $i<$n_OtrosImp; $i++) {
                         $detalle[] = [
                             '',
                             '',

Braces +10 added lines, -7 removed lines

@@ -57,8 +57,9 @@ 
     {
         $this->startPageGroup();
         $this->AddPage();
-        if (isset($libro['LibroCompraVenta']))
-            $libro = $libro['LibroCompraVenta'];
+        if (isset($libro['LibroCompraVenta'])) {
+                    $libro = $libro['LibroCompraVenta'];
+        }
         // título del libro
         $this->SetFont('helvetica', 'B', 16);
         $this->Texto('Libro de '.ucfirst(strtolower($libro['EnvioLibro']['Caratula']['TipoOperacion'])), null, null, 'C');
@@ -109,8 +110,7 @@ 
                     $total_resumen['TotIVARetTotal'] += !empty($total['TotIVARetTotal']) ? $total['TotIVARetTotal'] : 0;
                     $total_resumen['TotIVANoRetenido'] += !empty($total['TotIVANoRetenido']) ? $total['TotIVANoRetenido'] : 0;
                     $total_resumen['TotMntTotal'] += $total['TotMntTotal'];
-                }
-                else if (in_array($total['TpoDoc'], $this->dte_tipo_operacion['resta'])) {
+                } else if (in_array($total['TpoDoc'], $this->dte_tipo_operacion['resta'])) {
                     $total_resumen['TotMntExe'] -= !empty($total['TotMntExe']) ? $total['TotMntExe'] : 0;
                     $total_resumen['TotMntNeto'] -= !empty($total['TotMntNeto']) ? $total['TotMntNeto'] : 0;
                     $total_resumen['TotMntIVA'] -= !empty($total['TotMntIVA']) ? $total['TotMntIVA'] : 0;
@@ -171,10 +171,13 @@ 
             foreach ($libro['EnvioLibro']['Detalle'] as $d) {
                 // impuesto adicional
                 if (!empty($d['OtrosImp'])) {
-                    if (!isset($d['OtrosImp'][0]))
-                        $d['OtrosImp'] = [$d['OtrosImp']];
+                    if (!isset($d['OtrosImp'][0])) {
+                                            $d['OtrosImp'] = [$d['OtrosImp']];
+                    }
                     $n_OtrosImp = count($d['OtrosImp']);
-                } else $n_OtrosImp = 0;
+                } else {
+                    $n_OtrosImp = 0;
+                }
                 // se agrega detalle
                 $detalle[] = [
                     $d['TpoDoc'],

examples/018-generar_pdf.php

Braces +7 added lines, -4 removed lines

@@ -52,15 +52,18 @@
 
 // directorio temporal para guardar los PDF
 $dir = sys_get_temp_dir().'/dte_'.$Caratula['RutEmisor'].'_'.$Caratula['RutReceptor'].'_'.str_replace(['-', ':', 'T'], '', $Caratula['TmstFirmaEnv']);
-if (is_dir($dir))
+if (is_dir($dir)) {
     \sasco\LibreDTE\File::rmdir($dir);
-if (!mkdir($dir))
+}
+if (!mkdir($dir)) {
     die('No fue posible crear directorio temporal para DTEs');
+}
 
 // procesar cada DTEs e ir agregándolo al PDF
 foreach ($Documentos as $DTE) {
-    if (!$DTE->getDatos())
-        die('No se pudieron obtener los datos del DTE');
+    if (!$DTE->getDatos()) {
+            die('No se pudieron obtener los datos del DTE');
+    }
     $pdf = new \sasco\LibreDTE\Sii\Dte\PDF\Dte(false); // =false hoja carta, =true papel contínuo (false por defecto si no se pasa)
     $pdf->setFooterText();
     $pdf->setLogo('/home/delaf/www/localhost/dev/pages/sasco/website/webroot/img/logo_mini.png'); // debe ser PNG!

lib/PDF.php

Spacing +24 added lines, -24 removed lines

@@ -24,7 +24,7 @@ 
 namespace sasco\LibreDTE;
 
 // Directorio para imágenes (no se asume nada)
-define ('K_PATH_IMAGES', '');
+define('K_PATH_IMAGES', '');
 
 /**
  * Clase para generar PDFs
@@ -152,13 +152,13 @@ 
     {
         $widths = [];
         if (is_int($cells)) {
-            $width = floor($total/$cells);
-            for ($i=0; $i<$cells; ++$i) {
+            $width = floor($total / $cells);
+            for ($i = 0; $i<$cells; ++$i) {
                 $widths[] = $width;
             }
         }
-        else if (is_array($cells)){
-            $width = floor($total/count($cells));
+        else if (is_array($cells)) {
+            $width = floor($total / count($cells));
             foreach ($cells as $i) {
                 $widths[$i] = $width;
             }
@@ -208,7 +208,7 @@ 
                 $suma += $val;
             }
             if ($key_0!==null) {
-                $options['width'][$key_0] = 190 - $suma;
+                $options['width'][$key_0] = 190-$suma;
             }
         }
         if (isset($options['align']))
@@ -243,7 +243,7 @@ 
         $a = (isset($options['align']) and is_array($options['align'])) ? $options['align'] : [];
         $buffer = '<table style="border:1px solid #333">';
         // Definir títulos de columnas
-        $thead = isset($options['width']) and is_array($options['width']) and count($options['width']) == count($headers);
+        $thead = isset($options['width']) and is_array($options['width']) and count($options['width'])==count($headers);
         if ($thead) {
             $buffer .= '<thead>';
         }
@@ -307,7 +307,7 @@ 
             $options['bordercolor'][2]
         );
         $this->SetLineWidth($options['borderwidth']);
-        $this->SetFont($this->defaultOptions['font']['family'], 'B',  $options['fontsize']);
+        $this->SetFont($this->defaultOptions['font']['family'], 'B', $options['fontsize']);
         // corregir indices
         $headers_keys = array_keys($headers);
         if (is_array($options['width'])) {
@@ -329,13 +329,13 @@ 
         }
         // Header
         $x = $this->GetX();
-        foreach($headers as $i => $header) {
+        foreach ($headers as $i => $header) {
             $this->Cell($options['width'][$i], $options['height'], $headers[$i], 1, 0, $options['align'][$i], 1);
         }
         $this->Ln();
         $y = $this->GetY();
         // Color and font restoration
-        $this->SetFillColor (
+        $this->SetFillColor(
             $options['bodybackground'][0],
             $options['bodybackground'][1],
             $options['bodybackground'][2]
@@ -360,7 +360,7 @@ 
             $this->SetX($x);
             $y_0 = $this->GetY();
             $y_s = [];
-            foreach($headers as $i => $header) {
+            foreach ($headers as $i => $header) {
                 $x_0 = $this->GetX();
                 $this->SetXY($x_0, $y_0);
                 $aux = explode("\n", $row[$i]);
@@ -370,28 +370,28 @@ 
                 if ($value2) {
                     $this->Ln();
                     $this->SetX($x_0);
-                    $this->SetFont($this->defaultOptions['font']['family'], '',  $options['fontsize']-2);
+                    $this->SetFont($this->defaultOptions['font']['family'], '', $options['fontsize']-2);
                     $y_2 = $this->MultiCell($options['width'][$i], $options['height'], $value2, $options['tdborder'], $options['align'][$i], false, 0);
-                    $this->SetFont($this->defaultOptions['font']['family'], '',  $options['fontsize']);
-                    $y_s[] = $y_1 + $y_2*0.9;
+                    $this->SetFont($this->defaultOptions['font']['family'], '', $options['fontsize']);
+                    $y_s[] = $y_1+$y_2 * 0.9;
                 } else {
                     $y_s[] = $y_1;
                 }
             }
-            $this->Ln(max($y_s)*5);
+            $this->Ln(max($y_s) * 5);
             // si se pasó a página siguiente se hace rollback y se crea nueva página con cabecera nuevamente en la tabla
-            if($num_pages < $this->getNumPages()) {
+            if ($num_pages<$this->getNumPages()) {
                 $this->rollbackTransaction(true);
                 $this->AddPage();
                 $this->SetX($x);
-                foreach($headers as $i => $header) {
+                foreach ($headers as $i => $header) {
                     $this->Cell($options['width'][$i], $options['height'], $headers[$i], 1, 0, $options['align'][$i], 1);
                 }
                 $this->Ln();
                 $this->SetX($x);
                 $y_0 = $this->GetY();
                 $y_s = [];
-                foreach($headers as $i => $header) {
+                foreach ($headers as $i => $header) {
                     $x_0 = $this->GetX();
                     $this->SetXY($x_0, $y_0);
                     $aux = explode("\n", $row[$i]);
@@ -401,15 +401,15 @@ 
                     if ($value2) {
                         $this->Ln();
                         $this->SetX($x_0);
-                        $this->SetFont($this->defaultOptions['font']['family'], '',  $options['fontsize']-2);
+                        $this->SetFont($this->defaultOptions['font']['family'], '', $options['fontsize']-2);
                         $y_2 = $this->MultiCell($options['width'][$i], $options['height'], $value2, $options['tdborder'], $options['align'][$i], false, 0);
-                        $this->SetFont($this->defaultOptions['font']['family'], '',  $options['fontsize']);
-                        $y_s[] = $y_1 + $y_2*0.9;
+                        $this->SetFont($this->defaultOptions['font']['family'], '', $options['fontsize']);
+                        $y_s[] = $y_1+$y_2 * 0.9;
                     } else {
                         $y_s[] = $y_1;
                     }
                 }
-                $this->Ln(max($y_s)*5);
+                $this->Ln(max($y_s) * 5);
             } else {
                 $this->commitTransaction();
             }
@@ -438,7 +438,7 @@ 
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]delaf.cl)
      * @version 2014-09-20
      */
-    public function Texto($txt, $x=null, $y=null, $align='', $w=0, $link='', $border=0, $fill=false)
+    public function Texto($txt, $x = null, $y = null, $align = '', $w = 0, $link = '', $border = 0, $fill = false)
     {
         if ($x==null) $x = $this->GetX();
         if ($y==null) $y = $this->GetY();
@@ -459,7 +459,7 @@ 
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]delaf.cl)
      * @version 2014-09-20
      */
-    public function MultiTexto($txt, $x=null, $y=null, $align='', $w=0, $border=0, $fill=false)
+    public function MultiTexto($txt, $x = null, $y = null, $align = '', $w = 0, $border = 0, $fill = false)
     {
         if ($x==null) $x = $this->GetX();
         if ($y==null) $y = $this->GetY();

Braces +40 added lines, -24 removed lines

@@ -102,13 +102,16 @@ 
             $this->Line(0, $this->getY()-1, 290, $this->getY()-2, $style);
             $this->SetFont('', 'B', 6);
             if (empty($this->papelContinuo)) {
-                if (!empty($this->footer['left']))
-                    $this->Texto($this->footer['left']);
-                if (!empty($this->footer['right']))
-                    $this->Texto($this->footer['right'], null, null, 'R');
+                if (!empty($this->footer['left'])) {
+                                    $this->Texto($this->footer['left']);
+                }
+                if (!empty($this->footer['right'])) {
+                                    $this->Texto($this->footer['right'], null, null, 'R');
+                }
             } else {
-                if (!empty($this->footer['left']))
-                    $this->Texto($this->footer['left'], null, null, 'C');
+                if (!empty($this->footer['left'])) {
+                                    $this->Texto($this->footer['left'], null, null, 'C');
+                }
                 if (!empty($this->footer['right'])) {
                     $this->Ln();
                     $this->Texto($this->footer['right'], null, null, 'C');
@@ -134,8 +137,9 @@ 
                 ];
             }
             // si no es arreglo se convierte en uno
-            if (!is_array($footer))
-                $footer = ['left'=>$footer];
+            if (!is_array($footer)) {
+                            $footer = ['left'=>$footer];
+            }
             // asignar footer
             $this->footer = array_merge(['left'=>'', 'right'=>''], $footer);
         } else {
@@ -156,8 +160,7 @@ 
             for ($i=0; $i<$cells; ++$i) {
                 $widths[] = $width;
             }
-        }
-        else if (is_array($cells)){
+        } else if (is_array($cells)){
             $width = floor($total/count($cells));
             foreach ($cells as $i) {
                 $widths[$i] = $width;
@@ -178,8 +181,9 @@ 
         foreach ($data as $row) {
             foreach ($row as $col => $value) {
                 if (empty($value)) {
-                    if (!array_key_exists($col, $cols_empty))
-                        $cols_empty[$col] = 0;
+                    if (!array_key_exists($col, $cols_empty)) {
+                                            $cols_empty[$col] = 0;
+                    }
                     $cols_empty[$col]++;
                 }
             }
@@ -192,10 +196,12 @@ 
                 foreach ($data as &$row) {
                     unset($row[$col]);
                 }
-                if (isset($options['width']))
-                    unset($options['width'][$titles_keys[$col]]);
-                if (isset($options['align']))
-                    unset($options['align'][$titles_keys[$col]]);
+                if (isset($options['width'])) {
+                                    unset($options['width'][$titles_keys[$col]]);
+                }
+                if (isset($options['align'])) {
+                                    unset($options['align'][$titles_keys[$col]]);
+                }
             }
         }
         if (isset($options['width'])) {
@@ -203,16 +209,18 @@ 
             $key_0 = null;
             $suma = 0;
             foreach ($options['width'] as $key => $val) {
-                if ($val===0)
-                    $key_0 = $key;
+                if ($val===0) {
+                                    $key_0 = $key;
+                }
                 $suma += $val;
             }
             if ($key_0!==null) {
                 $options['width'][$key_0] = 190 - $suma;
             }
         }
-        if (isset($options['align']))
-            $options['align'] = array_slice($options['align'], 0);
+        if (isset($options['align'])) {
+                    $options['align'] = array_slice($options['align'], 0);
+        }
         $this->addTable($titles, $data, $options, $html);
     }
 
@@ -440,8 +448,12 @@ 
      */
     public function Texto($txt, $x=null, $y=null, $align='', $w=0, $link='', $border=0, $fill=false)
     {
-        if ($x==null) $x = $this->GetX();
-        if ($y==null) $y = $this->GetY();
+        if ($x==null) {
+            $x = $this->GetX();
+        }
+        if ($y==null) {
+            $y = $this->GetY();
+        }
         $textrendermode = $this->textrendermode;
         $textstrokewidth = $this->textstrokewidth;
         $this->setTextRenderingMode(0, true, false);
@@ -461,8 +473,12 @@ 
      */
     public function MultiTexto($txt, $x=null, $y=null, $align='', $w=0, $border=0, $fill=false)
     {
-        if ($x==null) $x = $this->GetX();
-        if ($y==null) $y = $this->GetY();
+        if ($x==null) {
+            $x = $this->GetX();
+        }
+        if ($y==null) {
+            $y = $this->GetY();
+        }
         $textrendermode = $this->textrendermode;
         $textstrokewidth = $this->textstrokewidth;
         $this->setTextRenderingMode(0, true, false);

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)

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], null, $parent);
+            $this->xml->generate($datos+['TED' => null], null, $parent);
             $this->datos = $datos;
             if ($normalizar and !$this->verificarDatos()) {
                 return false;
@@ -382,7 +382,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;
     }
 
     /**
@@ -395,7 +395,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;
     }
 
     /**
@@ -529,7 +529,7 @@ 
     {
         $this->getDatos();
         // generar resumen
-        $resumen =  [
+        $resumen = [
             'TpoDoc' => (int)$this->datos['Encabezado']['IdDoc']['TipoDTE'],
             'NroDoc' => (int)$this->datos['Encabezado']['IdDoc']['Folio'],
             'TasaImp' => 0,
@@ -574,17 +574,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];
     }
 
@@ -1429,12 +1429,12 @@ 
                     );
                     // aplicar descuento
                     if ($d['DescuentoPct']) {
-                        $d['DescuentoMonto'] = round($d['MontoItem'] * (float)$d['DescuentoPct']/100);
+                        $d['DescuentoMonto'] = round($d['MontoItem'] * (float)$d['DescuentoPct'] / 100);
                     }
                     $d['MontoItem'] -= $d['DescuentoMonto'];
                     // aplicar recargo
                     if ($d['RecargoPct']) {
-                        $d['RecargoMonto'] = round($d['MontoItem'] * (float)$d['RecargoPct']/100);
+                        $d['RecargoMonto'] = round($d['MontoItem'] * (float)$d['RecargoPct'] / 100);
                     }
                     $d['MontoItem'] += $d['RecargoMonto'];
                     // aproximar monto del item
@@ -1520,7 +1520,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
@@ -1593,7 +1593,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
                 );
             }
         }
@@ -1601,7 +1601,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]);
                 }
@@ -1629,7 +1629,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)
                     );
                 }
             }
@@ -1653,7 +1653,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
@@ -1813,7 +1813,7 @@ 
         $SignatureValue = trim(str_replace(["\n", ' ', "\t"], '', $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue));
         $X509Certificate = trim(str_replace(["\n", ' ', "\t"], '', $Signature->getElementsByTagName('X509Certificate')->item(0)->nodeValue));
         $X509Certificate = '-----BEGIN CERTIFICATE-----'."\n".wordwrap($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 +78 added lines, -52 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(
                         (float)$d['QtyItem'] * (float)$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'];
         }
     }