libredte / libredte-lib-core

lib/XML.php

Doc Comments +6 added lines, -5 removed lines

@@ -51,8 +51,8 @@ 
     /**
      * Método que genera nodos XML a partir de un arreglo
      * @param array Arreglo con los datos que se usarán para generar XML
-     * @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 2016-04-03
      */
@@ -140,7 +140,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
      */
@@ -261,7 +261,7 @@ 
      * Método que cuenta los nodos con el mismo nombre hijos deun 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
      */
@@ -301,7 +301,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
      */
@@ -343,6 +343,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 +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
      * @author Esteban De La Fuente Rubio, DeLaF (esteban[at]sasco.cl)
      * @version 2016-04-03
      */
-    public function generate(array $array, \DOMElement &$parent = null)
+    public function generate(array $array, \DOMElement & $parent = null)
     {
         if ($parent===null)
             $parent = &$this;
@@ -184,7 +184,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;
     }
 
     /**
@@ -224,7 +224,7 @@ 
         }
         // agregar nodos hijos
         if ($dom->hasChildNodes()) {
-            foreach($dom->childNodes as $child) {
+            foreach ($dom->childNodes as $child) {
                 if ($child instanceof \DOMText) {
                     $textContent = trim($child->textContent);
                     if ($textContent!="") {
@@ -349,7 +349,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 +48 added lines, -34 removed lines

@@ -58,13 +58,15 @@ 
      */
     public function generate(array $array, \DOMElement &$parent = null)
     {
-        if ($parent===null)
-            $parent = &$this;
+        if ($parent===null) {
+                    $parent = &$this;
+        }
         foreach ($array as $key => $value) {
             if ($key=='@attributes') {
                 foreach ($value as $attr => $val) {
-                    if ($val!==false)
-                        $parent->setAttribute($attr, $val);
+                    if ($val!==false) {
+                                            $parent->setAttribute($attr, $val);
+                    }
                 }
             } else if ($key=='@value') {
                 $parent->nodeValue = $this->sanitize($value);
@@ -107,8 +109,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', '"', '"', ''', '''],
@@ -161,8 +164,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 {
@@ -209,12 +213,15 @@ 
     public function toArray(\DOMElement $dom = null, array &$array = null, $arregloNodos = false)
     {
         // determinar valores de parámetros
-        if (!$dom)
-            $dom = $this->documentElement;
-        if (!$dom)
-            return false;
-        if ($array===null)
-            $array = [$dom->tagName => null];
+        if (!$dom) {
+                    $dom = $this->documentElement;
+        }
+        if (!$dom) {
+                    return false;
+        }
+        if ($array===null) {
+                    $array = [$dom->tagName => null];
+        }
         // agregar atributos del nodo
         if ($dom->hasAttributes()) {
             $array[$dom->tagName]['@attributes'] = [];
@@ -230,22 +237,24 @@ 
                     if ($textContent!="") {
                         if ($dom->childNodes->length==1 and empty($array[$dom->tagName])) {
                             $array[$dom->tagName] = $textContent;
-                        } else
-                            $array[$dom->tagName]['@value'] = $textContent;
+                        } else {
+                                                    $array[$dom->tagName]['@value'] = $textContent;
+                        }
                     }
-                }
-                else if ($child instanceof \DOMElement) {
+                } else if ($child instanceof \DOMElement) {
                     $nodos_gemelos = $this->countTwins($dom, $child->tagName);
                     if ($nodos_gemelos==1) {
-                        if ($arregloNodos)
-                            $this->toArray($child, $array);
-                        else
-                            $this->toArray($child, $array[$dom->tagName]);
+                        if ($arregloNodos) {
+                                                    $this->toArray($child, $array);
+                        } else {
+                                                    $this->toArray($child, $array[$dom->tagName]);
+                        }
                     }
                     // crear arreglo con nodos hijos que tienen el mismo nombre de tag
                     else {
-                        if (!isset($array[$dom->tagName][$child->tagName]))
-                            $array[$dom->tagName][$child->tagName] = [];
+                        if (!isset($array[$dom->tagName][$child->tagName])) {
+                                                    $array[$dom->tagName][$child->tagName] = [];
+                        }
                         $siguiente = count($array[$dom->tagName][$child->tagName]);
                         $array[$dom->tagName][$child->tagName][$siguiente] = [];
                         $this->toArray($child, $array[$dom->tagName][$child->tagName][$siguiente], true);
@@ -269,8 +278,9 @@ 
     {
         $twins = 0;
         foreach ($dom->childNodes as $child) {
-            if ($child instanceof \DOMElement and $child->tagName==$tagName)
-                $twins++;
+            if ($child instanceof \DOMElement and $child->tagName==$tagName) {
+                            $twins++;
+            }
         }
         return $twins;
     }
@@ -284,8 +294,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;
     }
 
@@ -308,8 +319,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;
     }
@@ -350,10 +362,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/PDF/Dte.php

Spacing +34 added lines, -34 removed lines

@@ -167,7 +167,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;
     }
 
     /**
@@ -342,7 +342,7 @@ 
         // determinar alto de la página y agregarla
         $height = 145;
         $n_detalle = count($dte['Detalle']);
-        if ($n_detalle>1) $height += $n_detalle*20;
+        if ($n_detalle>1) $height += $n_detalle * 20;
         if ($this->cedible) $height += 50;
         $this->AddPage('P', [$height, $width]);
         // agregar cabecera del documento
@@ -351,10 +351,10 @@ 
             $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, 40, 8, 9, [0,0,0]);
+        $y = $this->agregarEmisor($dte['Encabezado']['Emisor'], $x_start, $y+2, 40, 8, 9, [0, 0, 0]);
         // datos del documento
         $this->SetY($y);
         $this->Ln();
@@ -416,13 +416,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;
@@ -433,13 +433,13 @@ 
         }
         // 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);
         $ciudad = !empty($emisor['CiudadOrigen']) ? $emisor['CiudadOrigen'] : \sasco\LibreDTE\Chile::getCiudad($emisor['CmnaOrigen']);
-        $this->MultiTexto($emisor['DirOrigen'].', '.$emisor['CmnaOrigen'].($ciudad?(', '.$ciudad):''), $x, $this->y, 'L', $w);
+        $this->MultiTexto($emisor['DirOrigen'].', '.$emisor['CmnaOrigen'].($ciudad ? (', '.$ciudad) : ''), $x, $this->y, 'L', $w);
         if (!empty($emisor['Sucursal'])) {
             $this->MultiTexto('Sucursal: '.$emisor['Sucursal'], $x, $this->y, 'L', $w);
         }
@@ -483,12 +483,12 @@ 
     private 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);
@@ -501,7 +501,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;
     }
@@ -628,7 +628,7 @@ 
             $this->Texto('Señor(es)', $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);
@@ -645,7 +645,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);
@@ -760,7 +760,7 @@ 
     {
         if (!isset($referencias[0]))
             $referencias = [$referencias];
-        foreach($referencias as $r) {
+        foreach ($referencias as $r) {
             $texto = $r['NroLinRef'].' - '.$this->getTipo($r['TpoDocRef']).' N° '.$r['FolioRef'].' del '.$r['FchRef'];
             if (isset($r['RazonRef']) and $r['RazonRef']!==false)
                 $texto = $texto.': '.$r['RazonRef'];
@@ -853,9 +853,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']);
@@ -865,9 +865,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']);
         }
@@ -887,7 +887,7 @@ 
         if (!isset($detalle[0])) {
             $detalle = [$detalle];
         }
-        foreach($detalle as  &$d) {
+        foreach ($detalle as  &$d) {
             if (!empty($d['MontoItem'])) {
                 $subtotal += $d['MontoItem'];
             }
@@ -907,10 +907,10 @@ 
     {
         if (!isset($descuentosRecargos[0]))
             $descuentosRecargos = [$descuentosRecargos];
-        foreach($descuentosRecargos as $dr) {
+        foreach ($descuentosRecargos as $dr) {
             $tipo = $dr['TpoMov']=='D' ? 'Descuento' : 'Recargo';
             $valor = $dr['TpoValor']=='%' ? $dr['ValorDR'].'%' : '$'.$this->num($dr['ValorDR']).'.-';
-            $this->Texto($tipo.' global: '.$valor.(!empty($dr['GlosaDR'])?(' ('.$dr['GlosaDR'].')'):''), $x);
+            $this->Texto($tipo.' global: '.$valor.(!empty($dr['GlosaDR']) ? (' ('.$dr['GlosaDR'].')') : ''), $x);
             $this->Ln();
         }
     }
@@ -928,8 +928,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();
         }
     }
@@ -979,7 +979,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'].'%) $';
@@ -1044,7 +1044,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',
             ];
@@ -1070,7 +1070,7 @@ 
      */
     private function agregarAcuseRecibo($x = 93, $y = 190, $w = 50, $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]]]);
         $this->setFont('', 'B', 10);
         $this->Texto('Acuse de recibo', $x, $y+1, 'C', $w);
@@ -1100,9 +1100,9 @@ 
      */
     private 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);
@@ -1130,7 +1130,7 @@ 
     private function agregarLeyendaDestino($tipo, $y = 254, $font_size = 10)
     {
         $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');
     }
 
     /**
@@ -1161,7 +1161,7 @@ 
         $dias = ['Domingo', 'Lunes', 'Martes', 'Miércoles', 'Jueves', 'Viernes', 'Sábado'];
         $meses = ['enero', 'febrero', 'marzo', 'abril', 'mayo', 'junio', 'julio', 'agosto', 'septiembre', 'octubre', 'noviembre', 'diciembre'];
         $unixtime = strtotime($date);
-        $fecha = date(($mostrar_dia?'\D\I\A ':'').'j \d\e \M\E\S \d\e\l Y', $unixtime);
+        $fecha = date(($mostrar_dia ? '\D\I\A ' : '').'j \d\e \M\E\S \d\e\l Y', $unixtime);
         $dia = $dias[date('w', $unixtime)];
         $mes = $meses[date('n', $unixtime)-1];
         return str_replace(array('DIA', 'MES'), array($dia, $mes), $fecha);

lib/Sii/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[$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()
     {
         // 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/Sii/EnvioDte.php

Spacing +9 added lines, -9 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;
         }
         //
@@ -221,7 +221,7 @@ 
                 'RutEnvia' => $Dte->getEmisor(),
                 'RutReceptor' => $Dte->getReceptor(),
                 'FchResol' => date('Y-m-d'),
-                'NroResol' => ($Dte->getCertificacion()?'0':'').'9999',
+                'NroResol' => ($Dte->getCertificacion() ? '0' : '').'9999',
             ]);
             if (!parent::loadXML($this->generar())) {
                 return false;
@@ -301,7 +301,7 @@ 
     {
         $fecha = '9999-12-31';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() < $fecha)
+            if ($Dte->getFechaEmision()<$fecha)
                 $fecha = $Dte->getFechaEmision();
         }
         return $fecha;
@@ -316,7 +316,7 @@ 
     {
         $fecha = '0000-01-01';
         foreach ($this->getDocumentos() as $Dte) {
-            if ($Dte->getFechaEmision() > $fecha)
+            if ($Dte->getFechaEmision()>$fecha)
                 $fecha = $Dte->getFechaEmision();
         }
         return $fecha;
@@ -375,7 +375,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
             }
         }
@@ -432,12 +432,12 @@ 
         if ($X509Certificate[1]==' ' or $X509Certificate[1]=="\t") {
             $lineas = explode("\n", $X509Certificate);
             $X509Certificate = '';
-            foreach($lineas as $l) {
+            foreach ($lineas as $l) {
                 $X509Certificate .= trim($l);
             }
         }
         $X509Certificate = '-----BEGIN CERTIFICATE-----'."\n".wordwrap(trim($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));
     }
 

examples/034-xml_entities.php

Braces +2 added lines, -1 removed lines

@@ -83,5 +83,6 @@
 echo $XML2->getFlattened('/nodo1/nodo2')."\n\n";
 
 // si hubo errores mostrar
-foreach (\sasco\LibreDTE\Log::readAll() as $error)
+foreach (\sasco\LibreDTE\Log::readAll() as $error) {
     echo $error, "\n";
+}