libredte / libredte-lib-core

lib/FirmaElectronica.php

Braces +12 added lines, -8 removed lines

@@ -168,8 +168,9 @@ 
      */
     public function getName()
     {
-        if (isset($this->data['subject']['CN']))
-            return $this->data['subject']['CN'];
+        if (isset($this->data['subject']['CN'])) {
+                    return $this->data['subject']['CN'];
+        }
         return $this->error('No fue posible obtener el Name (subject.CN) de la firma');
     }
 
@@ -353,8 +354,9 @@ 
      */
     public function verify($data, $signature, $pub_key = null, $signature_alg = OPENSSL_ALGO_SHA1)
     {
-        if ($pub_key === null)
-            $pub_key = $this->certs['cert'];
+        if ($pub_key === null) {
+                    $pub_key = $this->certs['cert'];
+        }
         $pub_key = $this->normalizeCert($pub_key);
         return openssl_verify($data, base64_decode($signature), $pub_key, $signature_alg) == 1 ? true : false;
     }
@@ -453,8 +455,9 @@ 
         // calcular SignatureValue
         $SignedInfo = $doc->saveHTML($Signature->getElementsByTagName('SignedInfo')->item(0));
         $firma = $this->sign($SignedInfo);
-        if (!$firma)
-            return false;
+        if (!$firma) {
+                    return false;
+        }
         $signature = wordwrap($firma, $this->config['wordwrap'], "\n", true);
         // reemplazar valores en la firma de
         $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue = $signature;
@@ -486,8 +489,9 @@ 
         $signature = $Signature->getElementsByTagName('SignatureValue')->item(0)->nodeValue;
         $pub_key = $Signature->getElementsByTagName('X509Certificate')->item(0)->nodeValue;
         // verificar firma
-        if (!$this->verify($signed_info, $signature, $pub_key))
-            return false;
+        if (!$this->verify($signed_info, $signature, $pub_key)) {
+                    return false;
+        }
         // verificar digest
         $digest_original = $Signature->getElementsByTagName('DigestValue')->item(0)->nodeValue;
         if ($tag) {

Spacing +4 added lines, -4 removed lines

@@ -262,7 +262,7 @@ 
         if (!$when) {
             $when = date('Y-m-d').' 00:00:00';
         }
-        return $this->getTo() > $when;
+        return $this->getTo()>$when;
     }
 
     /**
@@ -378,10 +378,10 @@ 
      */
     public function verify($data, $signature, $pub_key = null, $signature_alg = OPENSSL_ALGO_SHA1)
     {
-        if ($pub_key === null)
+        if ($pub_key===null)
             $pub_key = $this->certs['cert'];
         $pub_key = $this->normalizeCert($pub_key);
-        return openssl_verify($data, base64_decode($signature), $pub_key, $signature_alg) == 1 ? true : false;
+        return openssl_verify($data, base64_decode($signature), $pub_key, $signature_alg)==1 ? true : false;
     }
 
     /**
@@ -520,7 +520,7 @@ 
         } else {
             $digest_calculado = base64_encode(sha1($doc->C14N(), true));
         }
-        return $digest_original == $digest_calculado;
+        return $digest_original==$digest_calculado;
     }
 
     /**

lib/Sii/Factoring/Aec.php

Spacing +8 added lines, -8 removed lines

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

Braces +30 added lines, -20 removed lines

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

lib/PDF.php

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

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;'.$width_table.'">';
         // 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();

lib/Sii.php

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

Braces +15 added lines, -10 removed lines

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

Spacing +8 added lines, -8 removed lines

@@ -32,7 +32,7 @@ 
 {
 
     private $dtes = []; ///< Objetos con los DTE que se enviarán
-    private $config = [ // 0: DTE, 1: boleta
+    private $config = [// 0: DTE, 1: boleta
         'SubTotDTE_max' => [20, 2], ///< máxima cantidad de tipos de documentos en el envío
         'DTE_max' => [2000, 1000], ///< máxima cantidad de DTE en un envío
         'tipos' => ['EnvioDTE', 'EnvioBOLETA'], ///< Tag para el envío, según si son Boletas o no
@@ -50,11 +50,11 @@ 
     public function agregar(Dte $DTE)
     {
         // determinar el tipo del envío (DTE o boleta)
-        if ($this->tipo === null) {
+        if ($this->tipo===null) {
             $this->tipo = (int)$DTE->esBoleta();
         }
         // validar que el tipo de documento sea del tipo que se espera
-        else if ((int)$DTE->esBoleta() != $this->tipo) {
+        else if ((int)$DTE->esBoleta()!=$this->tipo) {
             return false;
         }
         //
@@ -239,7 +239,7 @@ 
                 'RutEnvia' => $Dte->getEmisor(),
                 'RutReceptor' => $Dte->getReceptor(),
                 'FchResol' => date('Y-m-d'),
-                'NroResol' => ($Dte->getCertificacion()?'0':'').'9999',
+                'NroResol' => ($Dte->getCertificacion() ? '0' : '').'9999',
             ]);
             // cargar nuevo XML con datos completos
             if (!parent::loadXML($this->generar())) {
@@ -320,7 +320,7 @@ 
     {
         $fecha = '9999-12-31';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() < $fecha)
+            if ($Dte->getFechaEmision()<$fecha)
                 $fecha = $Dte->getFechaEmision();
         }
         return $fecha;
@@ -335,7 +335,7 @@ 
     {
         $fecha = '0000-01-01';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() > $fecha)
+            if ($Dte->getFechaEmision()>$fecha)
                 $fecha = $Dte->getFechaEmision();
         }
         return $fecha;
@@ -394,7 +394,7 @@ 
                 $e = substr($e, 0, -2);
             $d = (int)$nodo_dte->getElementsByTagName('TipoDTE')->item(0)->nodeValue;
             $f = (int)$nodo_dte->getElementsByTagName('Folio')->item(0)->nodeValue;
-            if ($folio == $f and $dte == $d and $emisor == $e) {
+            if ($folio==$f and $dte==$d and $emisor==$e) {
                 return new Dte($nodo_dte->C14N(), false); // cargar DTE sin normalizar
             }
         }
@@ -452,7 +452,7 @@ 
         $SignatureValue = trim(str_replace(["\n", ' ', "\t"], '', $Signatures->item($Signatures->length-1)->getElementsByTagName('SignatureValue')->item(0)->nodeValue));
         $X509Certificate = trim(str_replace(["\n", ' ', "\t"], '', $Signatures->item($Signatures->length-1)->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 and $DigestValue===base64_encode(sha1($SetDTE, true));
     }
 

lib/XML.php

Spacing +3 added lines, -3 removed lines

@@ -197,7 +197,7 @@ 
      */
     private function utf2iso($string)
     {
-        return mb_detect_encoding($string, ['UTF-8', 'ISO-8859-1']) != 'ISO-8859-1' ? utf8_decode($string) : $string;
+        return mb_detect_encoding($string, ['UTF-8', 'ISO-8859-1'])!='ISO-8859-1' ? utf8_decode($string) : $string;
     }
 
     /**
@@ -240,7 +240,7 @@ 
         }
         // si existen nodos hijos se agregan
         if ($dom->hasChildNodes()) {
-            foreach($dom->childNodes as $child) {
+            foreach ($dom->childNodes as $child) {
                 if ($child instanceof \DOMText) {
                     $textContent = trim($child->textContent);
                     if ($textContent!="") {
@@ -372,7 +372,7 @@ 
         $newXML = '';
         $n_letras = strlen($xml);
         $convertir = false;
-        for ($i=0; $i<$n_letras; ++$i) {
+        for ($i = 0; $i<$n_letras; ++$i) {
             if ($xml[$i]=='>')
                 $convertir = true;
             if ($xml[$i]=='<')

Braces +19 added lines, -14 removed lines

@@ -120,8 +120,9 @@ 
     private function sanitize($txt)
     {
         // si no se paso un texto o bien es un número no se hace nada
-        if (!$txt or is_numeric($txt))
-            return $txt;
+        if (!$txt or is_numeric($txt)) {
+                    return $txt;
+        }
         // convertir "predefined entities" de XML
         $txt = str_replace(
             ['&', '&', '<', '<', '>', '&#62', '"', '"', ''', '''],
@@ -174,8 +175,9 @@ 
     {
         if ($xpath) {
             $node = $this->xpath($xpath)->item(0);
-            if (!$node)
-                return false;
+            if (!$node) {
+                            return false;
+            }
             $xml = $this->utf2iso($node->C14N());
             $xml = $this->fixEntities($xml);
         } else {
@@ -250,8 +252,7 @@ 
                             $array[$dom->tagName]['@value'] = $textContent;
                         }
                     }
-                }
-                else if ($child instanceof \DOMElement) {
+                } else if ($child instanceof \DOMElement) {
                     $nodos_gemelos = $this->countTwins($dom, $child->tagName);
                     // agregar nodo hijo directamente, ya que es el único nodo hijo con el mismo nombre de tag
                     if ($nodos_gemelos==1) {
@@ -307,8 +308,9 @@ 
     public function getErrors()
     {
         $errors = [];
-        foreach (libxml_get_errors() as $e)
-            $errors[] = $e->message;
+        foreach (libxml_get_errors() as $e) {
+                    $errors[] = $e->message;
+        }
         return $errors;
     }
 
@@ -331,8 +333,9 @@ 
     public function getSchema()
     {
         $schemaLocation = $this->documentElement->getAttribute('xsi:schemaLocation');
-        if (!$schemaLocation or strpos($schemaLocation, ' ')===false)
-            return false;
+        if (!$schemaLocation or strpos($schemaLocation, ' ')===false) {
+                    return false;
+        }
         list($uri, $xsd) = explode(' ', $schemaLocation);
         return $xsd;
     }
@@ -373,10 +376,12 @@ 
         $n_letras = strlen($xml);
         $convertir = false;
         for ($i=0; $i<$n_letras; ++$i) {
-            if ($xml[$i]=='>')
-                $convertir = true;
-            if ($xml[$i]=='<')
-                $convertir = false;
+            if ($xml[$i]=='>') {
+                            $convertir = true;
+            }
+            if ($xml[$i]=='<') {
+                            $convertir = false;
+            }
             if ($convertir) {
                 $l = $xml[$i]=='\'' ? '&apos;' : ($xml[$i]=='"' ? '&quot;' : $xml[$i]);
             } else {

lib/Sii/Aduana.php

Braces +1 added lines, -2 removed lines

@@ -852,8 +852,7 @@
             $valor = $codigo['CantBultos'].' '.$valor;
             if (!empty($codigo['IdContainer'])) {
                 $valor .= ' ('.$codigo['IdContainer'].' / '.$codigo['Sello'].' / '.$codigo['EmisorSello'].')';
-            }
-            else if (!empty($codigo['Marcas'])) {
+            } else if (!empty($codigo['Marcas'])) {
                 $valor .= ' ('.$codigo['Marcas'].')';
             }
         } else {

lib/Sii/ConsumoFolio.php

Spacing +4 added lines, -4 removed lines

@@ -126,7 +126,7 @@ 
     {
         $fecha = '9999-12-31';
         foreach ($this->detalles as &$d) {
-            if ($d['FchDoc'] < $fecha) {
+            if ($d['FchDoc']<$fecha) {
                 $fecha = $d['FchDoc'];
             }
         }
@@ -143,7 +143,7 @@ 
     {
         $fecha = '0000-01-01';
         foreach ($this->detalles as &$d) {
-            if ($d['FchDoc'] > $fecha) {
+            if ($d['FchDoc']>$fecha) {
                 $fecha = $d['FchDoc'];
             }
         }
@@ -202,7 +202,7 @@ 
         // ajustes post agregar detalles
         foreach ($Resumen as &$r) {
             // obtener folios utilizados = emitidos + anulados
-            $r['FoliosUtilizados'] = $r['FoliosEmitidos'] + $r['FoliosAnulados'];
+            $r['FoliosUtilizados'] = $r['FoliosEmitidos']+$r['FoliosAnulados'];
             $r['RangoUtilizados'] = $this->getRangos($RangoUtilizados[$r['TipoDocumento']]);
         }
         // completar con los resumenes que no se colocaron
@@ -234,7 +234,7 @@ 
         $aux = [];
         $inicial = $folios[0];
         $i = $inicial;
-        foreach($folios as $f) {
+        foreach ($folios as $f) {
             if ($i!=$f) {
                 $inicial = $f;
                 $i = $inicial;

lib/Sii/Dte/PDF/Dte.php

Braces +18 added lines, -12 removed lines

@@ -770,10 +770,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);
@@ -823,10 +825,12 @@ 
             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'];
@@ -889,8 +893,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'])) {
@@ -1124,8 +1129,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) {

Spacing +53 added lines, -53 removed lines

@@ -74,7 +74,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;
     }
 
     /**
@@ -193,7 +193,7 @@ 
         $this->Ln();
         // datos del documento
         $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(
@@ -242,7 +242,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);
@@ -354,15 +354,15 @@ 
             $dte['Encabezado']['IdDoc']['TipoDTE'],
             $dte['Encabezado']['IdDoc']['Folio'],
             isset($dte['Encabezado']['Emisor']['CmnaOrigen']) ? $dte['Encabezado']['Emisor']['CmnaOrigen'] : 'Sin comuna', // siempre debería tener comuna
-            $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,
@@ -433,14 +433,14 @@ 
             2,
             45,
             9,
-            [0,0,0]
+            [0, 0, 0]
         );
-        $y[] = $this->agregarEmisor($dte['Encabezado']['Emisor'], 1, 2, 20, 30, 9, [0,0,0], $y[0]);
+        $y[] = $this->agregarEmisor($dte['Encabezado']['Emisor'], 1, 2, 20, 30, 9, [0, 0, 0], $y[0]);
         $this->SetY(max($y));
         $this->Ln();
         // datos del documento
         $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,
@@ -473,7 +473,7 @@ 
         if ($this->cedible and !in_array($dte['Encabezado']['IdDoc']['TipoDTE'], $this->sinAcuseRecibo)) {
             $this->agregarAcuseRecibo(63, $y+6, 45, 40, 15);
             $this->setFont('', 'B', 8);
-            $this->Texto('CEDIBLE'.($dte['Encabezado']['IdDoc']['TipoDTE']==52?' CON SU FACTURA':''), $x_start, $this->y+1, 'L');
+            $this->Texto('CEDIBLE'.($dte['Encabezado']['IdDoc']['TipoDTE']==52 ? ' CON SU FACTURA' : ''), $x_start, $this->y+1, 'L');
         }
         // si el alto no se pasó, entonces es con autocálculo, se elimina esta página y se pasa el alto
         // que se logró determinar para crear la página con el alto correcto
@@ -504,16 +504,16 @@ 
         // logo del documento
         if (isset($this->logo)) {
             // logo centrado (papel continuo)
-            if (!empty($this->logo['posicion']) and $this->logo['posicion'] == 'C') {
+            if (!empty($this->logo['posicion']) and $this->logo['posicion']=='C') {
                 $logo_w = null;
                 $logo_y = null;
                 $logo_position = 'C';
                 $logo_next_pointer = 'N';
             }
             // logo a la derecha (posicion=0) o arriba (posicion=1)
-            else if (empty($this->logo['posicion']) or $this->logo['posicion'] == 1) {
+            else if (empty($this->logo['posicion']) or $this->logo['posicion']==1) {
                 $logo_w = !$this->logo['posicion'] ? $w_img : null;
-                $logo_y = $this->logo['posicion'] ? $w_img/2 : null;
+                $logo_y = $this->logo['posicion'] ? $w_img / 2 : null;
                 $logo_position = '';
                 $logo_next_pointer = 'T';
             }
@@ -532,17 +532,17 @@ 
                 $logo_w,
                 $logo_y,
                 'PNG',
-                (isset($emisor['url'])?$emisor['url']:''),
+                (isset($emisor['url']) ? $emisor['url'] : ''),
                 $logo_next_pointer,
                 2,
                 300,
                 $logo_position
             );
-            if (!empty($this->logo['posicion']) and $this->logo['posicion'] == 'C') {
+            if (!empty($this->logo['posicion']) and $this->logo['posicion']=='C') {
                 $w += 40;
             } else {
                 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;
@@ -555,20 +555,20 @@ 
         // agregar datos del emisor
         if ($agregarDatosEmisor) {
             $this->setFont('', 'B', $font_size ? $font_size : 14);
-            $this->SetTextColorArray($color===null?[32, 92, 144]:$color);
-            $this->MultiTexto(!empty($emisor['RznSoc']) ? $emisor['RznSoc'] : $emisor['RznSocEmisor'], $x, $this->y+2, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+            $this->SetTextColorArray($color===null ? [32, 92, 144] : $color);
+            $this->MultiTexto(!empty($emisor['RznSoc']) ? $emisor['RznSoc'] : $emisor['RznSocEmisor'], $x, $this->y+2, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             $this->setFont('', 'B', $font_size ? $font_size : 9);
-            $this->SetTextColorArray([0,0,0]);
-            $this->MultiTexto(!empty($emisor['GiroEmis']) ? $emisor['GiroEmis'] : $emisor['GiroEmisor'], $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+            $this->SetTextColorArray([0, 0, 0]);
+            $this->MultiTexto(!empty($emisor['GiroEmis']) ? $emisor['GiroEmis'] : $emisor['GiroEmisor'], $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             $direccion = !empty($emisor['DirOrigen']) ? $emisor['DirOrigen'] : null;
             $comuna = !empty($emisor['CmnaOrigen']) ? $emisor['CmnaOrigen'] : null;
             $ciudad = !empty($emisor['CiudadOrigen']) ? $emisor['CiudadOrigen'] : \sasco\LibreDTE\Chile::getCiudad($comuna);
-            $this->MultiTexto($direccion.($comuna?(', '.$comuna):'').($ciudad?(', '.$ciudad):''), $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+            $this->MultiTexto($direccion.($comuna ? (', '.$comuna) : '').($ciudad ? (', '.$ciudad) : ''), $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             if (!empty($emisor['Sucursal'])) {
-                $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+                $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             }
             if (!empty($this->casa_matriz)) {
-                $this->MultiTexto('Casa matriz: '.$this->casa_matriz, $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+                $this->MultiTexto('Casa matriz: '.$this->casa_matriz, $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             }
             $contacto = [];
             if (!empty($emisor['Telefono'])) {
@@ -583,7 +583,7 @@ 
                 $contacto[] = $emisor['CorreoEmisor'];
             }
             if ($contacto) {
-                $this->MultiTexto(implode(' / ', $contacto), $x, $this->y, 'L', ($h_folio and $h_folio < $this->getY()) ? $w_all : $w);
+                $this->MultiTexto(implode(' / ', $contacto), $x, $this->y, 'L', ($h_folio and $h_folio<$this->getY()) ? $w_all : $w);
             }
         }
         return $this->y;
@@ -613,12 +613,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, $folio), $x, null, 'C', $w);
@@ -631,7 +631,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;
     }
@@ -745,10 +745,10 @@ 
         }
         if (!empty($receptor['RznSocRecep'])) {
             $this->setFont('', 'B', null);
-            $this->Texto(in_array($this->dte, [39, 41]) ? 'Nombre' : ($x==10?'Razón social':'Razón soc.'), $x);
+            $this->Texto(in_array($this->dte, [39, 41]) ? 'Nombre' : ($x==10 ? 'Razón social' : 'Razón soc.'), $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);
@@ -765,7 +765,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);
@@ -806,7 +806,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();
     }
@@ -903,7 +903,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']).' ';
@@ -964,7 +964,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") : ': ';
                     $item['DscItem'] = $html ? str_replace("\n", '<br/>', $item['DscItem']) : str_replace(['<br/>', '<br>'], "\n", $item['DscItem']);
                     if ($html) {
                         $item['NmbItem'] .= '<span style="font-size:0.7em">'.$item['DscItem'].'</span>';
@@ -1067,9 +1067,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+$offsets[0], $this->y, ucfirst($this->detalle_cols['NmbItem']['align'][0]), $this->detalle_cols['NmbItem']['width']);
         $this->Texto($this->detalle_cols['PrcItem']['title'], $x+$offsets[1], $this->y, ucfirst($this->detalle_cols['PrcItem']['align'][0]), $this->detalle_cols['PrcItem']['width']);
@@ -1081,7 +1081,7 @@ 
         }
         // mostrar items
         $this->SetY($this->getY()+2);
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             // nombre y descripción del item
             $item = $d['NmbItem'];
             if ($this->papel_continuo_item_detalle and !empty($d['DscItem'])) {
@@ -1122,7 +1122,7 @@ 
         if (!isset($detalle[0])) {
             $detalle = [$detalle];
         }
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             if (!empty($d['MontoItem'])) {
                 $subtotal += $d['MontoItem'];
             }
@@ -1149,18 +1149,18 @@ 
         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) {
-                $glosa = $tipo.' global'.(!empty($dr['GlosaDR'])?(' ('.$dr['GlosaDR'].')'):'').':';
+                $glosa = $tipo.' global'.(!empty($dr['GlosaDR']) ? (' ('.$dr['GlosaDR'].')') : '').':';
                 $this->Texto($glosa, $x, null, 'R', 30);
                 $this->Texto($valor, $x+$offset, null, 'R', 30);
             } 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();
@@ -1180,8 +1180,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();
         }
     }
@@ -1215,7 +1215,7 @@ 
             'TpoMoneda' => 'Moneda',
             'MntNeto' => 'Neto $',
             'MntExe' => 'Exento $',
-            'IVA' => 'IVA '.(!empty($totales['TasaIVA'])?(' ('.$totales['TasaIVA'].'%) '):'').'$',
+            'IVA' => 'IVA '.(!empty($totales['TasaIVA']) ? (' ('.$totales['TasaIVA'].'%) ') : '').'$',
             'IVANoRet' => 'IVA no retenido $',
             'CredEC' => 'Desc. 65% IVA $',
             'MntTotal' => 'Total $',
@@ -1232,7 +1232,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'].'%) $';
@@ -1348,9 +1348,9 @@ 
                 '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' => $position === null ? ($this->papelContinuo ? 'C' : 'S') : $position,
+                'position' => $position===null ? ($this->papelContinuo ? 'C' : 'S') : $position,
             ];
             $ecl = version_compare(phpversion(), '7.0.0', '<') ? -1 : $this->ecl;
             $this->write2DBarcode($timbre, 'PDF417,,'.$ecl, $x_timbre, $y, $w, 0, $style, 'B');
@@ -1381,7 +1381,7 @@ 
     protected function agregarAcuseRecibo($x = 83, $y = 190, $w = 60, $h = 40, $line = 25)
     {
         $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);
@@ -1411,9 +1411,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);
@@ -1443,7 +1443,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');
     }
 
     /**
@@ -1454,7 +1454,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');
     }
 
 }