libredte / libredte-lib-core

lib/Sii.php

Spacing +6 added lines, -6 removed lines

@@ -197,7 +197,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);
@@ -265,7 +265,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);
@@ -314,7 +314,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;
@@ -343,7 +343,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;
@@ -475,12 +475,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]);

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

Braces +20 added lines, -14 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;
     }
@@ -777,8 +779,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];
@@ -807,8 +810,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
@@ -819,10 +823,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/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/PDF.php

Braces +52 added lines, -32 removed lines

@@ -101,13 +101,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');
@@ -133,8 +136,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 {
@@ -155,8 +159,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;
@@ -177,8 +180,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]++;
                 }
             }
@@ -191,10 +195,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'])) {
@@ -202,16 +208,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);
     }
 
@@ -243,8 +251,9 @@ 
         $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);
-        if ($thead)
-            $buffer .= '<thead>';
+        if ($thead) {
+                    $buffer .= '<thead>';
+        }
         $buffer .= '<tr>';
         $i = 0;
         foreach ($headers as &$col) {
@@ -254,11 +263,13 @@ 
             $i++;
         }
         $buffer .= '</tr>';
-        if ($thead)
-            $buffer .= '</thead>';
+        if ($thead) {
+                    $buffer .= '</thead>';
+        }
         // Definir datos de la tabla
-        if ($thead)
-            $buffer .= '<tbody>';
+        if ($thead) {
+                    $buffer .= '<tbody>';
+        }
         foreach ($data as &$row) {
             $buffer .= '<tr>';
             $i = 0;
@@ -270,8 +281,9 @@ 
             }
             $buffer .= '</tr>';
         }
-        if ($thead)
-            $buffer .= '</tbody>';
+        if ($thead) {
+                    $buffer .= '</tbody>';
+        }
         // Finalizar tabla
         $buffer .= '</table>';
         // generar tabla en HTML
@@ -397,8 +409,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);
@@ -418,8 +434,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);

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>';
         $buffer .= '<tr>';
@@ -303,7 +303,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'])) {
@@ -325,13 +325,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]
@@ -356,7 +356,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]);
@@ -366,28 +366,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]);
@@ -397,15 +397,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();
             }
@@ -434,7 +434,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();
@@ -455,7 +455,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();

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

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

Doc Comments +31 added lines, -28 removed lines

@@ -77,7 +77,7 @@ 
     /**
      * Método que asigna la ubicación del logo de la empresa
      * @param logo URI del logo (puede ser local o en una URL)
-     * @param posicion Posición respecto a datos del emisor (=0 izq, =1 arriba)
+     * @param posicion integer respecto a datos del emisor (=0 izq, =1 arriba)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-04
      */
@@ -221,7 +221,7 @@ 
      * contínuo
      * @param dte Arreglo con los datos del XML (tag Documento)
      * @param timbre String XML con el tag TED del DTE
-     * @param width Ancho del papel contínuo en mm
+     * @param width boolean del papel contínuo en mm
      * @author Pablo Reyes (https://github.com/pabloxp)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-10-24
@@ -383,10 +383,11 @@ 
      *  - Dirección casa central del emisor
      *  - Dirección sucursales
      * @param emisor Arreglo con los datos del emisor (tag Emisor del XML)
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho de la información del emisor
-     * @param w_img Ancho máximo de la imagen
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer de la información del emisor
+     * @param w_img integer máximo de la imagen
+     * @param integer $font_size
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2018-06-15
      */
@@ -460,11 +461,13 @@ 
      * @param rut RUT del emisor
      * @param tipo Código o glosa del tipo de documento
      * @param sucursal_sii Código o glosa de la sucursal del SII del Emisor
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho de la información del emisor
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer de la información del emisor
+     * @param integer $font_size
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-12-02
+     * @return integer
      */
     protected function agregarFolio($rut, $tipo, $folio, $sucursal_sii = null, $x = 130, $y = 15, $w = 70, $font_size = null, array $color = null)
     {
@@ -496,7 +499,7 @@ 
      * Método que agrega los datos de la emisión del DTE que no son los dato del
      * receptor
      * @param IdDoc Información general del documento
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-06-15
      */
@@ -584,7 +587,7 @@ 
     /**
      * Método que agrega los datos del receptor
      * @param receptor Arreglo con los datos del receptor (tag Receptor del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2019-02-13
      */
@@ -671,7 +674,7 @@ 
      * Método que agrega los datos del traslado
      * @param IndTraslado
      * @param Transporte
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-03
      */
@@ -742,7 +745,7 @@ 
     /**
      * Método que agrega las referencias del documento
      * @param referencias Arreglo con las referencias del documento (tag Referencia del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-09-25
      */
@@ -779,7 +782,7 @@ 
     /**
      * Método que agrega el detalle del documento
      * @param detalle Arreglo con el detalle del documento (tag Detalle del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @param y Posición vertical de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2018-11-04
@@ -859,7 +862,7 @@ 
     /**
      * Método que agrega el detalle del documento
      * @param detalle Arreglo con el detalle del documento (tag Detalle del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @param y Posición vertical de inicio en el PDF
      * @author Pablo Reyes (https://github.com/pabloxp)
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
@@ -895,7 +898,7 @@ 
      * Método que agrega el subtotal del DTE
      * @param detalle Arreglo con los detalles del documentos para poder
      * calcular subtotal
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-08-17
      */
@@ -921,7 +924,7 @@ 
     /**
      * Método que agrega los descuentos y/o recargos globales del documento
      * @param descuentosRecargos Arreglo con los descuentos y/o recargos del documento (tag DscRcgGlobal del XML)
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2018-05-29
      */
@@ -949,7 +952,7 @@ 
     /**
      * Método que agrega los pagos del documento
      * @param pagos Arreglo con los pagos del documento
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-07-24
      */
@@ -1107,9 +1110,9 @@ 
      *  - Se imprime en el tamaño mínimo: 2x5 cms
      *  - En el lado de abajo con margen izquierdo mínimo de 2 cms
      * @param timbre String con los datos del timbre
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho del timbre
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer del timbre
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2018-04-12
      */
@@ -1147,10 +1150,10 @@ 
 
     /**
      * Método que agrega el acuse de rebido
-     * @param x Posición horizontal de inicio en el PDF
-     * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho del acuse de recibo
-     * @param h Alto del acuse de recibo
+     * @param x integer horizontal de inicio en el PDF
+     * @param y integer vertical de inicio en el PDF
+     * @param w integer del acuse de recibo
+     * @param h integer del acuse de recibo
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2018-04-13
      */
@@ -1178,10 +1181,10 @@ 
 
     /**
      * Método que agrega el acuse de rebido
-     * @param x Posición horizontal de inicio en el PDF
+     * @param x integer horizontal de inicio en el PDF
      * @param y Posición vertical de inicio en el PDF
-     * @param w Ancho del acuse de recibo
-     * @param h Alto del acuse de recibo
+     * @param w integer del acuse de recibo
+     * @param h integer del acuse de recibo
      * @author Pablo Reyes (https://github.com/pabloxp)
      * @version 2015-11-17
      */

Spacing +40 added lines, -40 removed lines

@@ -71,7 +71,7 @@ 
     {
         parent::__construct();
         $this->SetTitle('Documento Tributario Electrónico (DTE) de Chile by LibreDTE');
-        $this->papelContinuo = $papelContinuo === true ? 80 : $papelContinuo;
+        $this->papelContinuo = $papelContinuo===true ? 80 : $papelContinuo;
     }
 
     /**
@@ -181,7 +181,7 @@ 
         $this->setY(max($y));
         $this->Ln();
         $y = [];
-        $y[] = $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor'])?$dte['Encabezado']['Emisor']['CdgVendedor']:null);
+        $y[] = $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor']) ? $dte['Encabezado']['Emisor']['CdgVendedor'] : null);
         $y[] = $this->agregarReceptor($dte['Encabezado']);
         $this->setY(max($y));
         $this->agregarTraslado(
@@ -241,15 +241,15 @@ 
             $dte['Encabezado']['IdDoc']['TipoDTE'],
             $dte['Encabezado']['IdDoc']['Folio'],
             $dte['Encabezado']['Emisor']['CmnaOrigen'],
-            $x_start, $y_start, $width-($x_start*4), 10,
-            [0,0,0]
+            $x_start, $y_start, $width-($x_start * 4), 10,
+            [0, 0, 0]
         );
-        $y = $this->agregarEmisor($dte['Encabezado']['Emisor'], $x_start, $y+2, $width-($x_start*45), 8, 9, [0,0,0]);
+        $y = $this->agregarEmisor($dte['Encabezado']['Emisor'], $x_start, $y+2, $width-($x_start * 45), 8, 9, [0, 0, 0]);
         // datos del documento
         $this->SetY($y);
         $this->Ln();
         $this->setFont('', '', 8);
-        $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor'])?$dte['Encabezado']['Emisor']['CdgVendedor']:null, $x_start, $offset, false);
+        $this->agregarDatosEmision($dte['Encabezado']['IdDoc'], !empty($dte['Encabezado']['Emisor']['CdgVendedor']) ? $dte['Encabezado']['Emisor']['CdgVendedor'] : null, $x_start, $offset, false);
         $this->agregarReceptor($dte['Encabezado'], $x_start, $offset);
         $this->agregarTraslado(
             !empty($dte['Encabezado']['IdDoc']['IndTraslado']) ? $dte['Encabezado']['IdDoc']['IndTraslado'] : null,
@@ -302,7 +302,7 @@ 
         $this->AddPage('P', [$height ? $height : $this->papel_continuo_alto, $width]);
         $x = 1;
         $y = 5;
-        $this->SetXY($x,$y);
+        $this->SetXY($x, $y);
         // agregar datos del documento
         $this->setFont('', '', 8);
         $this->MultiTexto(!empty($dte['Encabezado']['Emisor']['RznSoc']) ? $dte['Encabezado']['Emisor']['RznSoc'] : $dte['Encabezado']['Emisor']['RznSocEmisor'], $x, null, '', $width-2);
@@ -398,13 +398,13 @@ 
                 $this->logo['uri'],
                 $x,
                 $y,
-                !$this->logo['posicion']?$w_img:null, $this->logo['posicion']?($w_img/2):null,
+                !$this->logo['posicion'] ? $w_img : null, $this->logo['posicion'] ? ($w_img / 2) : null,
                 'PNG',
-                (isset($emisor['url'])?$emisor['url']:''),
+                (isset($emisor['url']) ? $emisor['url'] : ''),
                 'T'
             );
             if ($this->logo['posicion']) {
-                $this->SetY($this->y + ($w_img/2));
+                $this->SetY($this->y+($w_img / 2));
                 $w += 40;
             } else {
                 $x = $this->x+3;
@@ -415,14 +415,14 @@ 
         }
         // agregar datos del emisor
         $this->setFont('', 'B', $font_size ? $font_size : 14);
-        $this->SetTextColorArray($color===null?[32, 92, 144]:$color);
+        $this->SetTextColorArray($color===null ? [32, 92, 144] : $color);
         $this->MultiTexto(!empty($emisor['RznSoc']) ? $emisor['RznSoc'] : $emisor['RznSocEmisor'], $x, $this->y+2, 'L', $w);
         $this->setFont('', 'B', $font_size ? $font_size : 9);
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->MultiTexto(!empty($emisor['GiroEmis']) ? $emisor['GiroEmis'] : $emisor['GiroEmisor'], $x, $this->y, 'L', $w);
         $comuna = !empty($emisor['CmnaOrigen']) ? $emisor['CmnaOrigen'] : null;
         $ciudad = !empty($emisor['CiudadOrigen']) ? $emisor['CiudadOrigen'] : \sasco\LibreDTE\Chile::getCiudad($comuna);
-        $this->MultiTexto($emisor['DirOrigen'].($comuna?(', '.$comuna):'').($ciudad?(', '.$ciudad):''), $x, $this->y, 'L', $w);
+        $this->MultiTexto($emisor['DirOrigen'].($comuna ? (', '.$comuna) : '').($ciudad ? (', '.$ciudad) : ''), $x, $this->y, 'L', $w);
         if (!empty($emisor['Sucursal'])) {
             $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', $w);
         }
@@ -469,12 +469,12 @@ 
     protected function agregarFolio($rut, $tipo, $folio, $sucursal_sii = null, $x = 130, $y = 15, $w = 70, $font_size = null, array $color = null)
     {
         if ($color===null) {
-            $color = $tipo ? ($tipo==52 ? [0,172,140] : [255,0,0]) : [0,0,0];
+            $color = $tipo ? ($tipo==52 ? [0, 172, 140] : [255, 0, 0]) : [0, 0, 0];
         }
         $this->SetTextColorArray($color);
         // colocar rut emisor, glosa documento y folio
         list($rut, $dv) = explode('-', $rut);
-        $this->setFont ('', 'B', $font_size ? $font_size : 15);
+        $this->setFont('', 'B', $font_size ? $font_size : 15);
         $this->MultiTexto('R.U.T.: '.$this->num($rut).'-'.$dv, $x, $y+4, 'C', $w);
         $this->setFont('', 'B', $font_size ? $font_size : 12);
         $this->MultiTexto($this->getTipo($tipo), $x, null, 'C', $w);
@@ -487,7 +487,7 @@ 
         if ($tipo) {
             $this->Texto('S.I.I. - '.\sasco\LibreDTE\Sii::getDireccionRegional($sucursal_sii), $x, $this->getY()+4, 'C', $w);
         }
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->Ln();
         return $this->y;
     }
@@ -604,7 +604,7 @@ 
             $this->Texto(in_array($this->dte, [39, 41]) ? 'Nombre' : 'Razón social', $x);
             $this->Texto(':', $x+$offset);
             $this->setFont('', '', null);
-            $this->MultiTexto($receptor['RznSocRecep'], $x+$offset+2, null, '', $x==10?105:0);
+            $this->MultiTexto($receptor['RznSocRecep'], $x+$offset+2, null, '', $x==10 ? 105 : 0);
         }
         if (!empty($receptor['GiroRecep'])) {
             $this->setFont('', 'B', null);
@@ -621,7 +621,7 @@ 
             $ciudad = !empty($receptor['CiudadRecep']) ? $receptor['CiudadRecep'] : (
                 !empty($receptor['CmnaRecep']) ? \sasco\LibreDTE\Chile::getCiudad($receptor['CmnaRecep']) : ''
             );
-            $this->MultiTexto($receptor['DirRecep'].(!empty($receptor['CmnaRecep'])?(', '.$receptor['CmnaRecep']):'').($ciudad?(', '.$ciudad):''), $x+$offset+2);
+            $this->MultiTexto($receptor['DirRecep'].(!empty($receptor['CmnaRecep']) ? (', '.$receptor['CmnaRecep']) : '').($ciudad ? (', '.$ciudad) : ''), $x+$offset+2);
         }
         if (!empty($receptor['Extranjero']['Nacionalidad'])) {
             $this->setFont('', 'B', null);
@@ -662,7 +662,7 @@ 
             $this->Texto('Cód. recep.', $x);
             $this->Texto(':', $x+$offset);
             $this->setFont('', '', null);
-            $this->MultiTexto($receptor['CdgIntRecep'], $x+$offset+2, null, '', $x==10?105:0);
+            $this->MultiTexto($receptor['CdgIntRecep'], $x+$offset+2, null, '', $x==10 ? 105 : 0);
         }
         return $this->GetY();
     }
@@ -750,7 +750,7 @@ 
     {
         if (!isset($referencias[0]))
             $referencias = [$referencias];
-        foreach($referencias as $r) {
+        foreach ($referencias as $r) {
             $texto = $r['NroLinRef'].' - ';
             if (!empty($r['TpoDocRef'])) {
                 $texto .= $this->getTipo($r['TpoDocRef']).' ';
@@ -801,7 +801,7 @@ 
             // quitar columnas
             foreach ($item as $col => $valor) {
                 if ($col=='DscItem' and !empty($item['DscItem'])) {
-                    $item['NmbItem'] .= !$this->item_detalle_posicion ? ($html?'<br/>':"\n") : ': ';
+                    $item['NmbItem'] .= !$this->item_detalle_posicion ? ($html ? '<br/>' : "\n") : ': ';
                     if ($html) {
                         $item['NmbItem'] .= '<span style="font-size:0.7em">'.$item['DscItem'].'</span>';
                     } else {
@@ -831,7 +831,7 @@ 
                 $item_default[$key] = false;
             $item = array_merge($item_default, $item);
             // si hay código de item se extrae su valor
-            if (!empty($item['CdgItem']['VlrCodigo'])){
+            if (!empty($item['CdgItem']['VlrCodigo'])) {
                 $item['CdgItem'] = $item['CdgItem']['VlrCodigo'];
             }
             // dar formato a números
@@ -870,9 +870,9 @@ 
         $this->SetY($this->getY()+1);
         $p1x = $x;
         $p1y = $this->y;
-        $p2x = $this->getPageWidth() - 2;
-        $p2y = $p1y;  // Use same y for a straight line
-        $style = array('width' => 0.2,'color' => array(0, 0, 0));
+        $p2x = $this->getPageWidth()-2;
+        $p2y = $p1y; // Use same y for a straight line
+        $style = array('width' => 0.2, 'color' => array(0, 0, 0));
         $this->Line($p1x, $p1y, $p2x, $p2y, $style);
         $this->Texto($this->detalle_cols['NmbItem']['title'], $x+1, $this->y, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
         $this->Texto($this->detalle_cols['PrcItem']['title'], $x+15, $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
@@ -882,9 +882,9 @@ 
         if (!isset($detalle[0]))
             $detalle = [$detalle];
         $this->SetY($this->getY()+2);
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             $this->MultiTexto($d['NmbItem'], $x+1, $this->y+4, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
-            $this->Texto(number_format($d['PrcItem'],0,',','.'), $x+15, $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
+            $this->Texto(number_format($d['PrcItem'], 0, ',', '.'), $x+15, $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
             $this->Texto($this->num($d['QtyItem']), $x+35, $this->y, ucfirst($this->detalle_cols['QtyItem']['align'][0]), $this->detalle_cols['QtyItem']['width']);
             $this->Texto($this->num($d['MontoItem']), $x+45, $this->y, ucfirst($this->detalle_cols['MontoItem']['align'][0]), $this->detalle_cols['MontoItem']['width']);
         }
@@ -904,7 +904,7 @@ 
         if (!isset($detalle[0])) {
             $detalle = [$detalle];
         }
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             if (!empty($d['MontoItem'])) {
                 $subtotal += $d['MontoItem'];
             }
@@ -930,16 +930,16 @@ 
         if (!isset($descuentosRecargos[0])) {
             $descuentosRecargos = [$descuentosRecargos];
         }
-        foreach($descuentosRecargos as $dr) {
+        foreach ($descuentosRecargos as $dr) {
             $tipo = $dr['TpoMov']=='D' ? 'Descuento' : 'Recargo';
             if (!empty($dr['IndExeDR'])) {
                 $tipo .= ' EX';
             }
             $valor = $dr['TpoValor']=='%' ? $dr['ValorDR'].'%' : $this->num($dr['ValorDR']);
             if ($this->papelContinuo) {
-                $this->Texto($tipo.' global: '.$valor.(!empty($dr['GlosaDR'])?(' ('.$dr['GlosaDR'].')'):''), $x);
+                $this->Texto($tipo.' global: '.$valor.(!empty($dr['GlosaDR']) ? (' ('.$dr['GlosaDR'].')') : ''), $x);
             } else {
-                $this->Texto($tipo.(!empty($dr['GlosaDR'])?(' ('.$dr['GlosaDR'].')'):'').':', 77, null, 'R', 100);
+                $this->Texto($tipo.(!empty($dr['GlosaDR']) ? (' ('.$dr['GlosaDR'].')') : '').':', 77, null, 'R', 100);
                 $this->Texto($valor, 177, null, 'R', 22);
             }
             $this->Ln();
@@ -959,8 +959,8 @@ 
             $pagos = [$pagos];
         $this->Texto('Pago(s) programado(s):', $x);
         $this->Ln();
-        foreach($pagos as $p) {
-            $this->Texto('  - '.$this->date($p['FchPago'], false).': $'.$this->num($p['MntPago']).'.-'.(!empty($p['GlosaPagos'])?(' ('.$p['GlosaPagos'].')'):''), $x);
+        foreach ($pagos as $p) {
+            $this->Texto('  - '.$this->date($p['FchPago'], false).': $'.$this->num($p['MntPago']).'.-'.(!empty($p['GlosaPagos']) ? (' ('.$p['GlosaPagos'].')') : ''), $x);
             $this->Ln();
         }
     }
@@ -1011,7 +1011,7 @@ 
             if (!isset($ImptoReten[0])) {
                 $ImptoReten = [$ImptoReten];
             }
-            foreach($ImptoReten as $i) {
+            foreach ($ImptoReten as $i) {
                 $totales['ImptoReten_'.$i['TipoImp']] = $i['MontoImp'];
                 if (!empty($i['TasaImp'])) {
                     $glosas['ImptoReten_'.$i['TipoImp']] = \sasco\LibreDTE\Sii\ImpuestosAdicionales::getGlosa($i['TipoImp']).' ('.$i['TasaImp'].'%) $';
@@ -1124,7 +1124,7 @@ 
                 'vpadding' => 0,
                 'module_width' => 1, // width of a single module in points
                 'module_height' => 1, // height of a single module in points
-                'fgcolor' => [0,0,0],
+                'fgcolor' => [0, 0, 0],
                 'bgcolor' => false, // [255,255,255]
                 'position' => $this->papelContinuo ? 'C' : 'S',
             ];
@@ -1157,7 +1157,7 @@ 
     protected function agregarAcuseRecibo($x = 83, $y = 190, $w = 60, $h = 40)
     {
         $y = (!$this->papelContinuo and !$this->timbre_pie) ? $this->x_fin_datos : $y;
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->Rect($x, $y, $w, $h, 'D', ['all' => ['width' => 0.1, 'color' => [0, 0, 0]]]);
         $this->setFont('', 'B', 10);
         $this->Texto('Acuse de recibo', $x, $y+1, 'C', $w);
@@ -1187,9 +1187,9 @@ 
      */
     protected function agregarAcuseReciboContinuo($x = 3, $y = null, $w = 68, $h = 40)
     {
-        $this->SetTextColorArray([0,0,0]);
+        $this->SetTextColorArray([0, 0, 0]);
         $this->Rect($x, $y, $w, $h, 'D', ['all' => ['width' => 0.1, 'color' => [0, 0, 0]]]);
-        $style = array('width' => 0.2,'color' => array(0, 0, 0));
+        $style = array('width' => 0.2, 'color' => array(0, 0, 0));
         $this->Line($x, $y+22, $w+3, $y+22, $style);
         //$this->setFont('', 'B', 10);
         //$this->Texto('Acuse de recibo', $x, $y+1, 'C', $w);
@@ -1219,7 +1219,7 @@ 
         $y = (!$this->papelContinuo and !$this->timbre_pie and $this->x_fin_datos<=$y) ? $this->x_fin_datos : $y;
         $y += 48;
         $this->setFont('', 'B', $font_size);
-        $this->Texto('CEDIBLE'.($tipo==52?' CON SU FACTURA':''), null, $y, 'R');
+        $this->Texto('CEDIBLE'.($tipo==52 ? ' CON SU FACTURA' : ''), null, $y, 'R');
     }
 
     /**
@@ -1230,7 +1230,7 @@ 
     protected function agregarLeyendaDestinoContinuo($tipo)
     {
         $this->setFont('', 'B', 8);
-        $this->Texto('CEDIBLE'.($tipo==52?' CON SU FACTURA':''), null, $this->y+6, 'R');
+        $this->Texto('CEDIBLE'.($tipo==52 ? ' CON SU FACTURA' : ''), null, $this->y+6, 'R');
     }
 
 }

Braces +59 added lines, -40 removed lines

@@ -431,10 +431,12 @@ 
         }
         $contacto = [];
         if (!empty($emisor['Telefono'])) {
-            if (!is_array($emisor['Telefono']))
-                $emisor['Telefono'] = [$emisor['Telefono']];
-            foreach ($emisor['Telefono'] as $t)
-                $contacto[] = $t;
+            if (!is_array($emisor['Telefono'])) {
+                            $emisor['Telefono'] = [$emisor['Telefono']];
+            }
+            foreach ($emisor['Telefono'] as $t) {
+                            $contacto[] = $t;
+            }
         }
         if (!empty($emisor['CorreoEmisor'])) {
             $contacto[] = $emisor['CorreoEmisor'];
@@ -638,10 +640,12 @@ 
             $this->MultiTexto($receptor['Extranjero']['NumId'], $x+$offset+2);
         }
         $contacto = [];
-        if (!empty($receptor['Contacto']))
-            $contacto[] = $receptor['Contacto'];
-        if (!empty($receptor['CorreoRecep']))
-            $contacto[] = $receptor['CorreoRecep'];
+        if (!empty($receptor['Contacto'])) {
+                    $contacto[] = $receptor['Contacto'];
+        }
+        if (!empty($receptor['CorreoRecep'])) {
+                    $contacto[] = $receptor['CorreoRecep'];
+        }
         if (!empty($contacto)) {
             $this->setFont('', 'B', null);
             $this->Texto('Contacto', $x);
@@ -691,15 +695,19 @@ 
             if (!empty($Transporte['DirDest']) and !empty($Transporte['CmnaDest'])) {
                 $transporte .= 'a '.$Transporte['DirDest'].', '.$Transporte['CmnaDest'];
             }
-            if (!empty($Transporte['RUTTrans']))
-                $transporte .= ' por '.$Transporte['RUTTrans'];
-            if (!empty($Transporte['Patente']))
-                $transporte .= ' en vehículo '.$Transporte['Patente'];
+            if (!empty($Transporte['RUTTrans'])) {
+                            $transporte .= ' por '.$Transporte['RUTTrans'];
+            }
+            if (!empty($Transporte['Patente'])) {
+                            $transporte .= ' en vehículo '.$Transporte['Patente'];
+            }
             if (isset($Transporte['Chofer']) and is_array($Transporte['Chofer'])) {
-                if (!empty($Transporte['Chofer']['NombreChofer']))
-                    $transporte .= ' con chofer '.$Transporte['Chofer']['NombreChofer'];
-                if (!empty($Transporte['Chofer']['RUTChofer']))
-                    $transporte .= ' ('.$Transporte['Chofer']['RUTChofer'].')';
+                if (!empty($Transporte['Chofer']['NombreChofer'])) {
+                                    $transporte .= ' con chofer '.$Transporte['Chofer']['NombreChofer'];
+                }
+                if (!empty($Transporte['Chofer']['RUTChofer'])) {
+                                    $transporte .= ' ('.$Transporte['Chofer']['RUTChofer'].')';
+                }
             }
             if ($transporte) {
                 $this->setFont('', 'B', null);
@@ -713,8 +721,9 @@ 
         if (!empty($Transporte['Aduana']) and is_array($Transporte['Aduana'])) {
             $col = 0;
             foreach ($Transporte['Aduana'] as $tag => $codigo) {
-                if ($codigo===false)
-                    continue;
+                if ($codigo===false) {
+                                    continue;
+                }
                 $glosa = \sasco\LibreDTE\Sii\Aduana::getGlosa($tag);
                 $valor = \sasco\LibreDTE\Sii\Aduana::getValor($tag, $codigo);
                 if ($glosa!==false and $valor!==false) {
@@ -727,15 +736,18 @@ 
                     $this->Texto(':', $x+$offset+$col);
                     $this->setFont('', '', null);
                     $this->Texto($valor, $x+$offset+2+$col);
-                    if ($tag=='TipoBultos')
-                        $col = abs($col-110);
-                    if ($col)
-                        $this->Ln();
+                    if ($tag=='TipoBultos') {
+                                            $col = abs($col-110);
+                    }
+                    if ($col) {
+                                            $this->Ln();
+                    }
                     $col = abs($col-110);
                 }
             }
-            if ($col)
-                $this->Ln();
+            if ($col) {
+                            $this->Ln();
+            }
         }
     }
 
@@ -748,8 +760,9 @@ 
      */
     protected function agregarReferencia($referencias, $x = 10, $offset = 22)
     {
-        if (!isset($referencias[0]))
-            $referencias = [$referencias];
+        if (!isset($referencias[0])) {
+                    $referencias = [$referencias];
+        }
         foreach($referencias as $r) {
             $texto = $r['NroLinRef'].' - ';
             if (!empty($r['TpoDocRef'])) {
@@ -786,8 +799,9 @@ 
      */
     protected function agregarDetalle($detalle, $x = 10, $html = true)
     {
-        if (!isset($detalle[0]))
-            $detalle = [$detalle];
+        if (!isset($detalle[0])) {
+                    $detalle = [$detalle];
+        }
         $this->setFont('', '', $this->detalle_fuente);
         // titulos
         $titulos = [];
@@ -820,15 +834,17 @@ 
             }
             // ajustes a IndExe
             if (isset($item['IndExe'])) {
-                if ($item['IndExe']==1)
-                    $item['IndExe'] = 'EX';
-                else if ($item['IndExe']==2)
-                    $item['IndExe'] = 'NF';
+                if ($item['IndExe']==1) {
+                                    $item['IndExe'] = 'EX';
+                } else if ($item['IndExe']==2) {
+                                    $item['IndExe'] = 'NF';
+                }
             }
             // agregar todas las columnas que se podrían imprimir en la tabla
             $item_default = [];
-            foreach ($this->detalle_cols as $key => $info)
-                $item_default[$key] = false;
+            foreach ($this->detalle_cols as $key => $info) {
+                            $item_default[$key] = false;
+            }
             $item = array_merge($item_default, $item);
             // si hay código de item se extrae su valor
             if (!empty($item['CdgItem']['VlrCodigo'])){
@@ -845,8 +861,9 @@ 
         $options = ['align'=>[]];
         $i = 0;
         foreach ($this->detalle_cols as $info) {
-            if (isset($info['width']))
-                $options['width'][$i] = $info['width'];
+            if (isset($info['width'])) {
+                            $options['width'][$i] = $info['width'];
+            }
             $options['align'][$i] = $info['align'];
             $i++;
         }
@@ -879,8 +896,9 @@ 
         $this->Texto($this->detalle_cols['QtyItem']['title'], $x+35, $this->y, ucfirst($this->detalle_cols['QtyItem']['align'][0]), $this->detalle_cols['QtyItem']['width']);
         $this->Texto($this->detalle_cols['MontoItem']['title'], $x+45, $this->y, ucfirst($this->detalle_cols['MontoItem']['align'][0]), $this->detalle_cols['MontoItem']['width']);
         $this->Line($p1x, $p1y+4, $p2x, $p2y+4, $style);
-        if (!isset($detalle[0]))
-            $detalle = [$detalle];
+        if (!isset($detalle[0])) {
+                    $detalle = [$detalle];
+        }
         $this->SetY($this->getY()+2);
         foreach($detalle as  &$d) {
             $this->MultiTexto($d['NmbItem'], $x+1, $this->y+4, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
@@ -955,8 +973,9 @@ 
      */
     protected function agregarPagos(array $pagos, $x = 10)
     {
-        if (!isset($pagos[0]))
-            $pagos = [$pagos];
+        if (!isset($pagos[0])) {
+                    $pagos = [$pagos];
+        }
         $this->Texto('Pago(s) programado(s):', $x);
         $this->Ln();
         foreach($pagos as $p) {