libredte / libredte-lib-core

lib/Sii/Dte/PDF/Dte.php

Doc Comments +31 added lines, -28 removed lines

@@ -78,7 +78,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
      */
@@ -222,7 +222,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
@@ -384,10 +384,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
      */
@@ -461,11 +462,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)
     {
@@ -497,7 +500,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
      */
@@ -585,7 +588,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
      */
@@ -672,7 +675,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
      */
@@ -743,7 +746,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
      */
@@ -780,7 +783,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 2019-07-09
@@ -880,7 +883,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)
@@ -916,7 +919,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
      */
@@ -942,7 +945,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
      */
@@ -970,7 +973,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
      */
@@ -1128,9 +1131,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
      */
@@ -1168,10 +1171,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
      */
@@ -1199,10 +1202,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

@@ -72,7 +72,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;
     }
 
     /**
@@ -182,7 +182,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(
@@ -242,15 +242,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,
@@ -303,7 +303,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);
@@ -399,13 +399,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;
@@ -416,14 +416,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);
         }
@@ -470,12 +470,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);
@@ -488,7 +488,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;
     }
@@ -605,7 +605,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);
@@ -622,7 +622,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);
@@ -663,7 +663,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();
     }
@@ -751,7 +751,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']).' ';
@@ -803,7 +803,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 {
@@ -851,7 +851,7 @@ 
             }
             $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
@@ -891,9 +891,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']);
@@ -903,9 +903,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']);
         }
@@ -925,7 +925,7 @@ 
         if (!isset($detalle[0])) {
             $detalle = [$detalle];
         }
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             if (!empty($d['MontoItem'])) {
                 $subtotal += $d['MontoItem'];
             }
@@ -951,16 +951,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();
@@ -980,8 +980,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();
         }
     }
@@ -1032,7 +1032,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'].'%) $';
@@ -1145,7 +1145,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',
             ];
@@ -1178,7 +1178,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);
@@ -1208,9 +1208,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);
@@ -1240,7 +1240,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');
     }
 
     /**
@@ -1251,7 +1251,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 +45 added lines, -30 removed lines

@@ -432,10 +432,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'];
@@ -639,10 +641,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);
@@ -692,15 +696,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);
@@ -714,8 +722,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) {
@@ -728,15 +737,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();
+            }
         }
     }
 
@@ -749,8 +761,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'])) {
@@ -900,8 +913,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']);
@@ -976,8 +990,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) {

lib/XML.php

Doc Comments +6 added lines, -5 removed lines

@@ -52,8 +52,8 @@ 
      * Método que genera nodos XML a partir de un arreglo
      * @param data Arreglo con los datos que se usarán para generar XML
      * @param namespace Arreglo con el espacio de nombres para el XML que se generará (URI y prefijo)
-     * @param parent DOMElement padre para los elementos, o =null para que sea la raíz
-     * @return Objeto \sasco\LibreDTE\XML
+     * @param parent \DOMElement padre para los elementos, o =null para que sea la raíz
+     * @return XML \sasco\LibreDTE\XML
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-10-22
      */
@@ -153,7 +153,7 @@ 
     /**
      * Método para realizar consultas XPATH al documento XML
      * @param expression Expresión XPath a ejecutar
-     * @return DOMNodeList
+     * @return \DOMNodeList
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-08-05
      */
@@ -283,7 +283,7 @@ 
      * Método que cuenta los nodos con el mismo nombre hijos de un DOMElement
      * No sirve usar: $dom->getElementsByTagName($tagName)->length ya que esto
      * entrega todos los nodos con el nombre, sean hijos, nietos, etc.
-     * @return Cantidad de nodos hijos con el mismo nombre en el DOMElement
+     * @return integer de nodos hijos con el mismo nombre en el DOMElement
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-09-07
      */
@@ -324,7 +324,7 @@ 
 
     /**
      * Método que entrega el nombre del archivo del schema del XML
-     * @return Nombre del schema o bien =false si no se encontró
+     * @return string del schema o bien =false si no se encontró
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2015-12-14
      */
@@ -366,6 +366,7 @@ 
      * respeta el estándar y las requiere convertidas
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2017-01-20
+     * @param string $xml
      */
     private function fixEntities($xml)
     {

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 {