joenilson / fsRepublicaDominicana

Model/ImpuestoProducto.php

Spacing +4 added lines, -4 removed lines

@@ -93,10 +93,10 @@
     public function getTaxByProduct($idproducto, $rdtaxid, $use = 'venta'): ?ImpuestoProducto
     {
         $dataBase = new DataBase();
-        $sql = "SELECT * FROM impuestosproductos WHERE idproducto = " .
-                (int)$idproducto .
-                " AND codimpuesto = '" . $dataBase->escapeString($rdtaxid) . "'" .
-                " AND " . $use . " = true" . ";";
+        $sql = "SELECT * FROM impuestosproductos WHERE idproducto = ".
+                (int) $idproducto.
+                " AND codimpuesto = '".$dataBase->escapeString($rdtaxid)."'".
+                " AND ".$use." = true".";";
         $data = $dataBase->select($sql);
         if (empty($data) === true || in_array($data[0], [null, ''], true)) {
             return null;

Extension/Controller/EditCliente.php

Spacing +5 added lines, -5 removed lines

@@ -30,10 +30,10 @@ 
 {
     public function createViews(): Closure
     {
-        return function () {
-            AssetManager::add('js', \FS_ROUTE . '/Plugins/fsRepublicaDominicana/Assets/JS/CommonModals.js');
-            AssetManager::add('js', \FS_ROUTE . '/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
-            AssetManager::add('js', \FS_ROUTE . '/Plugins/fsRepublicaDominicana/Assets/JS/BusquedaRNCDGII.js');
+        return function() {
+            AssetManager::add('js', \FS_ROUTE.'/Plugins/fsRepublicaDominicana/Assets/JS/CommonModals.js');
+            AssetManager::add('js', \FS_ROUTE.'/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
+            AssetManager::add('js', \FS_ROUTE.'/Plugins/fsRepublicaDominicana/Assets/JS/BusquedaRNCDGII.js');
             $ncfTipo = new NCFTipo();
             $ncfTipos = $ncfTipo->allFor('ventas', 'suma');
             $customValues = [];
@@ -62,7 +62,7 @@ 
 
     public function execPreviousAction(): Closure
     {
-        return function ($action) {
+        return function($action) {
             switch ($action) {
                 case 'busca_rnc':
                     $this->setTemplate(false);

Lib/PlantillasPDF/PlantillaDOM.php

Spacing +87 added lines, -88 removed lines

@@ -42,7 +42,7 @@ 
         // TODO: Implement addInvoiceHeader() method.
         $html = $this->getInvoiceHeaderBilling($model)
             . $this->getInvoiceHeaderShipping($model);
-        $this->writeHTML('<table class="table-big table-border"><tr>' . $html . '</tr></table><br/>');
+        $this->writeHTML('<table class="table-big table-border"><tr>'.$html.'</tr></table><br/>');
     }
 
     public function addInvoiceLines($model)
@@ -52,7 +52,7 @@ 
 
         $tHead = '<thead><tr>';
         foreach ($this->getInvoiceLineFields() as $field) {
-            $tHead .= '<th class="' . $field['css'] . '" align="' . $field['align'] . '">' . $field['title'] . '</th>';
+            $tHead .= '<th class="'.$field['css'].'" align="'.$field['align'].'">'.$field['title'].'</th>';
         }
         $tHead .= '</tr></thead>';
 
@@ -64,20 +64,20 @@ 
             $line->numlinea = $numLinea;
             $tBody .= '<tr>';
             foreach ($this->getInvoiceLineFields() as $field) {
-                $tBody .= '<td class="' . $field['css'] . '" align="' . $field['align'] . '" valign="top">' . $this->getInvoiceLineValue($model, $line, $field) . '</td>';
+                $tBody .= '<td class="'.$field['css'].'" align="'.$field['align'].'" valign="top">'.$this->getInvoiceLineValue($model, $line, $field).'</td>';
             }
             $tBody .= '</tr>';
             $numLinea++;
 
             if (property_exists($line, 'salto_pagina') && $line->salto_pagina) {
-                $this->writeHTML('<div class="table-lines"><table class="table-big table-list">' . $tHead . $tBody . '</table></div>');
+                $this->writeHTML('<div class="table-lines"><table class="table-big table-list">'.$tHead.$tBody.'</table></div>');
                 $this->writeHTML($this->getInvoiceTotalsPartial($model, $tLines, 'mt-20'));
                 $this->mpdf->AddPage();
                 $tBody = '';
             }
         }
 
-        $this->writeHTML('<div class="table-lines"><table class="table-big table-list">' . $tHead . $tBody . '</table></div>');
+        $this->writeHTML('<div class="table-lines"><table class="table-big table-list">'.$tHead.$tBody.'</table></div>');
 
         // clonamos el documento y añadimos los totales para ver si salta de página
         $copier = new DeepCopy();
@@ -96,28 +96,28 @@ 
     protected function css(): string
     {
         return parent::css()
-            . '.title {border-bottom: 2px solid ' . $this->get('color1') . ';}'
-            . '.table-border {border-top: 1px solid ' . $this->get('color1') . '; border-bottom: 1px solid ' . $this->get('color1') . ';}'
-            . '.table-dual {border-top: 1px solid ' . $this->get('color1') . '; border-bottom: 1px solid ' . $this->get('color1') . ';}'
-            . '.table-list {border-spacing: 0px; border-top: 1px solid ' . $this->get('color1') . '; border-bottom: 1px solid ' . $this->get('color1') . ';}'
-            . '.table-list tr:nth-child(even) {background-color: ' . $this->get('color3') . ';}'
-            . '.table-list th {background-color: ' . $this->get('color1') . '; color: ' . $this->get('color2') . '; padding: 5px; text-transform: uppercase;}'
+            . '.title {border-bottom: 2px solid '.$this->get('color1').';}'
+            . '.table-border {border-top: 1px solid '.$this->get('color1').'; border-bottom: 1px solid '.$this->get('color1').';}'
+            . '.table-dual {border-top: 1px solid '.$this->get('color1').'; border-bottom: 1px solid '.$this->get('color1').';}'
+            . '.table-list {border-spacing: 0px; border-top: 1px solid '.$this->get('color1').'; border-bottom: 1px solid '.$this->get('color1').';}'
+            . '.table-list tr:nth-child(even) {background-color: '.$this->get('color3').';}'
+            . '.table-list th {background-color: '.$this->get('color1').'; color: '.$this->get('color2').'; padding: 5px; text-transform: uppercase;}'
             . '.table-list td {padding: 5px;}'
             . '.thanks-title {'
-            . 'font-size: ' . $this->get('titlefontsize') . 'px; font-weight: bold; color: ' . $this->get('color1') . '; '
-            . 'text-align: right; width: 50%; padding: 15px; border-right: 1px solid ' . $this->get('color1') . ';'
+            . 'font-size: '.$this->get('titlefontsize').'px; font-weight: bold; color: '.$this->get('color1').'; '
+            . 'text-align: right; width: 50%; padding: 15px; border-right: 1px solid '.$this->get('color1').';'
             . '}'
             . '.color-navy {color: navy;}'
             . '.color-blue {color: blue;}'
-            . '.color-template {color: ' . $this->get('color1') .';}'
+            . '.color-template {color: '.$this->get('color1').';}'
             . '.thanks-text {padding: 15px;}'
-            . '.imagetext {margin-top: 15px; text-align: ' . $this->get('endalign') . ';}'
-            . '.imagefooter {text-align: ' . $this->get('footeralign') . ';}';
+            . '.imagetext {margin-top: 15px; text-align: '.$this->get('endalign').';}'
+            . '.imagefooter {text-align: '.$this->get('footeralign').';}';
     }
 
     protected function getSubjectIdFiscalStr(BusinessDocument $model): string
     {
-        return empty($model->cifnif) ? '' : '<b>' . $model->getSubject()->tipoidfiscal . '</b>: ' . $model->cifnif;
+        return empty($model->cifnif) ? '' : '<b>'.$model->getSubject()->tipoidfiscal.'</b>: '.$model->cifnif;
     }
 
     protected function getInvoiceHeaderBilling($model): string
@@ -129,13 +129,13 @@ 
         $subject = $model->getSubject();
         $address = isset($model->codproveedor) && !isset($model->direccion) ? $subject->getDefaultAddress() : $model;
         $customerCode = $this->get('showcustomercode') ? $model->subjectColumnValue() : '';
-        $customerEmail = $this->get('showcustomeremail') && !empty($subject->email) ? '<br>' . Tools::lang()->trans('email') . ': ' . $subject->email : '';
+        $customerEmail = $this->get('showcustomeremail') && !empty($subject->email) ? '<br>'.Tools::lang()->trans('email').': '.$subject->email : '';
         $break = empty($model->cifnif) ? '' : '<br/>';
         return '<td align="left" valign="top">'
-            . '<br/><b> ' . $this->getSubjectTitle($model) . ':</b> ' . $customerCode
-            . $this->getSubjectName($model) . $break . $this->getSubjectIdFiscalStr($model)
-            . '<br/><b>' . Tools::lang()->trans('address'). ':</b> ' .$this->combineAddress($address) . $this->getInvoiceHeaderBillingPhones($subject)
-            . '<br/><b>' . Tools::lang()->trans('email'). ':</b> ' .$customerEmail
+            . '<br/><b> '.$this->getSubjectTitle($model).':</b> '.$customerCode
+            . $this->getSubjectName($model).$break.$this->getSubjectIdFiscalStr($model)
+            . '<br/><b>'.Tools::lang()->trans('address').':</b> '.$this->combineAddress($address).$this->getInvoiceHeaderBillingPhones($subject)
+            . '<br/><b>'.Tools::lang()->trans('email').':</b> '.$customerEmail
             . '<br/>'
             . '</td>';
     }
@@ -155,8 +155,8 @@ 
             return '';
         }
 
-        return '<td><b>' . Tools::lang()->trans('shipping-address') . '</b>'
-            . '<br/>' . $this->combineAddress($contacto, true) . '</td>';
+        return '<td><b>'.Tools::lang()->trans('shipping-address').'</b>'
+            . '<br/>'.$this->combineAddress($contacto, true).'</td>';
     }
 
     protected function getInvoiceHeaderBillingPhones($subject): string
@@ -170,7 +170,7 @@ 
             return '';
         }
 
-        return '<br/>' . $strPhones;
+        return '<br/>'.$strPhones;
     }
 
     protected function headerLeft(): string
@@ -183,34 +183,34 @@ 
             }
         }
 
-        $title = $this->showHeaderTitle ? '<h1 class="title">' . $this->get('headertitle') . '</h1>' . $this->spacer() : '';
+        $title = $this->showHeaderTitle ? '<h1 class="title">'.$this->get('headertitle').'</h1>'.$this->spacer() : '';
         if ($this->isSketchInvoice()) {
-            $title .= '<div class="color-red font-big font-bold">' . Tools::lang()->trans('invoice-is-sketch') . '</div>';
+            $title .= '<div class="color-red font-big font-bold">'.Tools::lang()->trans('invoice-is-sketch').'</div>';
         }
 
         $descTC = ($this->headerModel->tipocomprobante < 30) ? "<b>NCF:</b> " : "<b>e-NCF:</b> ";
-        $fechaVencimiento = ($this->headerModel->ncffechavencimiento !== null) ? '<b>Fecha Vencimiento:</b> ' . $this->headerModel->ncffechavencimiento : '';
-        $fechaEmision = '<b>Fecha Emisi&oacute;n:</b> ' . $this->headerModel->fecha;
+        $fechaVencimiento = ($this->headerModel->ncffechavencimiento !== null) ? '<b>Fecha Vencimiento:</b> '.$this->headerModel->ncffechavencimiento : '';
+        $fechaEmision = '<b>Fecha Emisi&oacute;n:</b> '.$this->headerModel->fecha;
 
         return '<table class="table-big">'
             . '<tr>'
-                . '<td valign="top"><img src="' . $this->logoPath . '" height="' . $this->get('logosize') . '"/>' . '</td>'
-                . '<td align="right" valign="top">' . $title . '</td>'
+                . '<td valign="top"><img src="'.$this->logoPath.'" height="'.$this->get('logosize').'"/>'.'</td>'
+                . '<td align="right" valign="top">'.$title.'</td>'
             . '</tr>'
             . '<tr>'
                 . '<td align="left" valign="top">'
-                . '<p><b>' . $this->empresa->nombre . '</b>'
-                . '<br/>' . $this->empresa->tipoidfiscal . ': ' . $this->empresa->cifnif
-                . '<br/>' . Tools::lang()->trans('address') . ': ' . $this->combineAddress($this->empresa) . '</p>'
-                . '<p>' . implode(' · ', $contactData) . '</p>'
+                . '<p><b>'.$this->empresa->nombre.'</b>'
+                . '<br/>'.$this->empresa->tipoidfiscal.': '.$this->empresa->cifnif
+                . '<br/>'.Tools::lang()->trans('address').': '.$this->combineAddress($this->empresa).'</p>'
+                . '<p>'.implode(' · ', $contactData).'</p>'
                 . $fechaEmision
                 . '</td>'
                 . '<td align="right" valign="top">'
-                .  '<div class="color-template font-big font-bold">' . $this->headerModel->descripcionTipoComprobante() . '</div>'
+                .  '<div class="color-template font-big font-bold">'.$this->headerModel->descripcionTipoComprobante().'</div>'
                 .  $descTC
-                .  $this->headerModel->numeroncf . '<br/>'
-                .  $fechaVencimiento . '<br/>'
-                .  '<b>' . $i18n->trans('payment-method') . ':</b> '
+                .  $this->headerModel->numeroncf.'<br/>'
+                .  $fechaVencimiento.'<br/>'
+                .  '<b>'.$i18n->trans('payment-method').':</b> '
                 .  PaymentMethodBankDataHelper::get($this->headerModel)
                 . '</td>'
             . '</tr>'
@@ -225,20 +225,20 @@ 
             }
         }
 
-        $title = $this->showHeaderTitle ? '<h1 class="title">' . $this->get('headertitle') . '</h1>' . $this->spacer() : '';
+        $title = $this->showHeaderTitle ? '<h1 class="title">'.$this->get('headertitle').'</h1>'.$this->spacer() : '';
         if ($this->isSketchInvoice()) {
-            $title .= '<div class="color-red font-big font-bold">' . Tools::lang()->trans('invoice-is-sketch') . '</div>';
+            $title .= '<div class="color-red font-big font-bold">'.Tools::lang()->trans('invoice-is-sketch').'</div>';
         }
 
         return '<table class="table-big">'
             . '<tr>'
-            . '<td> Titulo ' . $title
-            . '<p><b>' . $this->empresa->nombre . '</b>'
-            . '<br/>' . $this->empresa->tipoidfiscal . ': ' . $this->empresa->cifnif
-            . '<br/>' . $this->combineAddress($this->empresa) . '</p>' . $this->spacer()
-            . '<p>' . implode(' · ', $contactData) . '</p>'
+            . '<td> Titulo '.$title
+            . '<p><b>'.$this->empresa->nombre.'</b>'
+            . '<br/>'.$this->empresa->tipoidfiscal.': '.$this->empresa->cifnif
+            . '<br/>'.$this->combineAddress($this->empresa).'</p>'.$this->spacer()
+            . '<p>'.implode(' · ', $contactData).'</p>'
             . '</td>'
-            . '<td align="right"><img src="' . $this->logoPath . '" height="' . $this->get('logosize') . '"/></td>'
+            . '<td align="right"><img src="'.$this->logoPath.'" height="'.$this->get('logosize').'"/></td>'
             . '</tr>'
             . '</table>';
     }
@@ -247,13 +247,12 @@ 
     {
         $observations = '';
         if (!empty($this->getObservations($model))) {
-            $observations .= '<p><b>' . Tools::lang()->trans('observations') . '</b><br/>'
-                . $this->getObservations($model) . '</p>';
+            $observations .= '<p><b>'.Tools::lang()->trans('observations').'</b><br/>'
+                . $this->getObservations($model).'</p>';
         }
 
         return $this->format->hidetotals ?
-            $observations :
-            $this->getInvoiceTotalsPartial($model, [], $css) . $observations;
+            $observations : $this->getInvoiceTotalsPartial($model, [], $css).$observations;
     }
 
     protected function getInvoiceTotalsPartial($model, array $lines = [], string $css = ''): string
@@ -284,8 +283,8 @@ 
         // pintamos los irpfs
         foreach ($irpfs as $irpf) {
             $trs .= '<tr>'
-                . '<td align="right"><b>' . $irpf['name'] . '</b>:</td>'
-                . '<td class="nowrap" align="right">' . Tools::money($irpf['total'], $model->coddivisa) . '</td>'
+                . '<td align="right"><b>'.$irpf['name'].'</b>:</td>'
+                . '<td class="nowrap" align="right">'.Tools::money($irpf['total'], $model->coddivisa).'</td>'
                 . '</tr>';
         }
 
@@ -309,15 +308,15 @@ 
                 case 'dtopor1':
                 case 'dtopor2':
                     $trs .= '<tr>'
-                        . '<td align="right"><b>' . $title . '</b>:</td>'
-                        . '<td class="nowrap" align="right">' . Tools::number($model->{$key}) . '%</td>'
+                        . '<td align="right"><b>'.$title.'</b>:</td>'
+                        . '<td class="nowrap" align="right">'.Tools::number($model->{$key}).'%</td>'
                         . '</tr>';
                     break;
 
                 case 'total':
                     $trs .= '<tr>'
-                        . '<td class="text-right"><b>' . $title . '</b>:</td>'
-                        . '<td class="text-right nowrap">' . Tools::money($model->{$key}, $model->coddivisa) . '</td>'
+                        . '<td class="text-right"><b>'.$title.'</b>:</td>'
+                        . '<td class="text-right nowrap">'.Tools::money($model->{$key}, $model->coddivisa).'</td>'
                         . '</tr>';
                     break;
 
@@ -328,24 +327,24 @@ 
                 // no break
                 default:
                     $trs .= '<tr>'
-                        . '<td align="right"><b>' . $title . '</b>:</td>'
-                        . '<td class="nowrap" align="right">' . Tools::money($model->{$key}, $model->coddivisa) . '</td>'
+                        . '<td align="right"><b>'.$title.'</b>:</td>'
+                        . '<td class="nowrap" align="right">'.Tools::money($model->{$key}, $model->coddivisa).'</td>'
                         . '</tr>';
                     break;
             }
         }
 
-        return '<table class="table-big table-border ' . $css . '">'
+        return '<table class="table-big table-border '.$css.'">'
             . '<tr>'
-            . '<td> ' . $this->getInvoiceTaxes($model, $lines) . '</td>'
-            . '<td align="right" valign="top"><table>' . $trs . '</table></td>'
+            . '<td> '.$this->getInvoiceTaxes($model, $lines).'</td>'
+            . '<td align="right" valign="top"><table>'.$trs.'</table></td>'
             . '</tr>'
             . '<tr>'
             . '<td>'
             . '</td>'
             . '</tr>'
             . '</table>'
-            . '&nbsp;<b>Son:</b> ' .  $this->convert_to_words($model->total) . ' pesos dominicanos'
+            . '&nbsp;<b>Son:</b> '.$this->convert_to_words($model->total).' pesos dominicanos'
             . '<br/>';
     }
 
@@ -365,43 +364,43 @@ 
         $trs = '';
         foreach ($taxes['iva'] as $row) {
             $trs .= '<tr>'
-                . '<td class="nowrap" align="left">' . Impuestos::get($row['codimpuesto'])->descripcion . '</td>'
-                . '<td class="nowrap" align="center">' . Tools::money($row['neto'], $model->coddivisa) . '</td>'
-                . '<td class="nowrap" align="center">' . Tools::number($row['iva']) . '%</td>'
-                . '<td class="nowrap" align="center">' . Tools::money($row['totaliva'], $model->coddivisa) . '</td>';
+                . '<td class="nowrap" align="left">'.Impuestos::get($row['codimpuesto'])->descripcion.'</td>'
+                . '<td class="nowrap" align="center">'.Tools::money($row['neto'], $model->coddivisa).'</td>'
+                . '<td class="nowrap" align="center">'.Tools::number($row['iva']).'%</td>'
+                . '<td class="nowrap" align="center">'.Tools::money($row['totaliva'], $model->coddivisa).'</td>';
 
             if (empty($model->totalrecargo)) {
                 $trs .= '</tr>';
                 continue;
             }
 
-            $trs .= '<td class="nowrap" align="center">' . (empty($row['recargo']) ? '-' : Tools::number($row['recargo']) . '%') . '</td>'
-                . '<td class="nowrap" align="right">' . (empty($row['totalrecargo']) ? '-' : Tools::money($row['totalrecargo'])) . '</td>'
+            $trs .= '<td class="nowrap" align="center">'.(empty($row['recargo']) ? '-' : Tools::number($row['recargo']).'%').'</td>'
+                . '<td class="nowrap" align="right">'.(empty($row['totalrecargo']) ? '-' : Tools::money($row['totalrecargo'])).'</td>'
                 . '</tr>';
         }
 
         if (empty($model->totalrecargo)) {
-            return '<table class="' . $class . '">'
+            return '<table class="'.$class.'">'
                 . '<thead>'
                 . '<tr>'
-                . '<th align="left">' . $i18n->trans('tax') . '</th>'
-                . '<th align="center">' . $i18n->trans('tax-base') . '</th>'
-                . '<th align="center">' . $i18n->trans('percentage') . '</th>'
-                . '<th align="center">' . $i18n->trans('amount') . '</th>'
+                . '<th align="left">'.$i18n->trans('tax').'</th>'
+                . '<th align="center">'.$i18n->trans('tax-base').'</th>'
+                . '<th align="center">'.$i18n->trans('percentage').'</th>'
+                . '<th align="center">'.$i18n->trans('amount').'</th>'
                 . '</tr>'
                 . '</thead>'
                 . $trs
                 . '</table>';
         }
 
-        return '<table class="' . $class . '">'
+        return '<table class="'.$class.'">'
             . '<tr>'
-            . '<th align="left">' . $i18n->trans('tax') . '</th>'
-            . '<th align="center">' . $i18n->trans('tax-base') . '</th>'
-            . '<th align="center">' . $i18n->trans('tax') . '</th>'
-            . '<th align="center">' . $i18n->trans('amount') . '</th>'
-            . '<th align="center">' . $i18n->trans('re') . '</th>'
-            . '<th align="right">' . $i18n->trans('amount') . '</th>'
+            . '<th align="left">'.$i18n->trans('tax').'</th>'
+            . '<th align="center">'.$i18n->trans('tax-base').'</th>'
+            . '<th align="center">'.$i18n->trans('tax').'</th>'
+            . '<th align="center">'.$i18n->trans('amount').'</th>'
+            . '<th align="center">'.$i18n->trans('re').'</th>'
+            . '<th align="right">'.$i18n->trans('amount').'</th>'
             . '</tr>'
             . $trs
             . '</table>';
@@ -463,7 +462,7 @@ 
 
         if ($number < 100) {
             return ucfirst($words[10 * floor($number / 10)]
-                . ($number % 10 > 0 ? ' y ' . $words[$number % 10] : ''));
+                . ($number % 10 > 0 ? ' y '.$words[$number % 10] : ''));
         }
 
         if ($number < 1000) {
@@ -475,23 +474,23 @@ 
 
         if ($number < 1000000) {
             $thousands = floor($number / 1000);
-            return ucfirst(($thousands > 1 ? $this->convert_to_words($thousands) . ' ' : '')
+            return ucfirst(($thousands > 1 ? $this->convert_to_words($thousands).' ' : '')
                 . 'mil'
-                . ($number % 1000 > 0 ? ' ' . $this->convert_to_words($number % 1000) : ''));
+                . ($number % 1000 > 0 ? ' '.$this->convert_to_words($number % 1000) : ''));
         }
 
         if ($number < 1000000000) {
             $millions = floor($number / 1000000);
-            return ucfirst(($millions > 1 ? $this->convert_to_words($millions) . ' ' : 'un ')
+            return ucfirst(($millions > 1 ? $this->convert_to_words($millions).' ' : 'un ')
                 . 'millón'
                 . ($millions > 1 ? 'es' : '')
-                . ($number % 1000000 > 0 ? ' ' . $this->convert_to_words($number % 1000000) : ''));
+                . ($number % 1000000 > 0 ? ' '.$this->convert_to_words($number % 1000000) : ''));
         }
 
         $billions = floor($number / 1000000000);
-        return ucfirst(($billions > 1 ? $this->convert_to_words($billions) . ' ' : 'un ')
+        return ucfirst(($billions > 1 ? $this->convert_to_words($billions).' ' : 'un ')
             . 'billón'
             . ($billions > 1 ? 'es' : '')
-            . ($number % 1000000000 > 0 ? ' ' . $this->convert_to_words($number % 1000000000) : ''));
+            . ($number % 1000000000 > 0 ? ' '.$this->convert_to_words($number % 1000000000) : ''));
     }
 }
\ No newline at end of file

Model/NCFTipo.php

Spacing +33 added lines, -33 removed lines

@@ -120,16 +120,16 @@ 
             'tipocomprobante' => '17', 'descripcion' => 'COMPROBANTE PARA PAGOS AL EXTERIOR', 'clasemovimiento'=>'suma',
             'ventas'=>'N', 'compras'=>'Y', 'contribuyente'=>'Y', 'estado' => true
         ],
-        ['tipocomprobante' => '31', 'descripcion' => 'FACTURA DE CRÉDITO FISCAL ELECTRÓNICA', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-        ['tipocomprobante' => '32', 'descripcion' => 'FACTURA DE CONSUMO ELECTRÓNICA', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-        ['tipocomprobante' => '33', 'descripcion' => 'NOTA DE DÉBITO ELECTRÓNICA', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true ],
-        ['tipocomprobante' => '34', 'descripcion' => 'NOTA DE CRÉDITO ELECTRÓNICA', 'clasemovimiento' => 'resta', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true ],
-        ['tipocomprobante' => '41', 'descripcion' => 'COMPROBANTE ELECTRÓNICO DE COMPRAS', 'clasemovimiento' => 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-        ['tipocomprobante' => '43', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA GASTOS MENORES', 'clasemovimiento' => 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true ],
-        ['tipocomprobante' => '44', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA REGÍMENES ESPECIALES', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-        ['tipocomprobante' => '45', 'descripcion' => 'COMPROBANTE ELECTRÓNICO GUBERNAMENTAL', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-        ['tipocomprobante' => '46', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA EXPORTACIONES', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-        ['tipocomprobante' => '47', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA PAGOS AL EXTERIOR', 'clasemovimiento' => 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
+        ['tipocomprobante' => '31', 'descripcion' => 'FACTURA DE CRÉDITO FISCAL ELECTRÓNICA', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+        ['tipocomprobante' => '32', 'descripcion' => 'FACTURA DE CONSUMO ELECTRÓNICA', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+        ['tipocomprobante' => '33', 'descripcion' => 'NOTA DE DÉBITO ELECTRÓNICA', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true],
+        ['tipocomprobante' => '34', 'descripcion' => 'NOTA DE CRÉDITO ELECTRÓNICA', 'clasemovimiento' => 'resta', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true],
+        ['tipocomprobante' => '41', 'descripcion' => 'COMPROBANTE ELECTRÓNICO DE COMPRAS', 'clasemovimiento' => 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+        ['tipocomprobante' => '43', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA GASTOS MENORES', 'clasemovimiento' => 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true],
+        ['tipocomprobante' => '44', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA REGÍMENES ESPECIALES', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+        ['tipocomprobante' => '45', 'descripcion' => 'COMPROBANTE ELECTRÓNICO GUBERNAMENTAL', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+        ['tipocomprobante' => '46', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA EXPORTACIONES', 'clasemovimiento' => 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+        ['tipocomprobante' => '47', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA PAGOS AL EXTERIOR', 'clasemovimiento' => 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
     );
     
     /**
@@ -168,27 +168,27 @@ 
         parent::install();
         $sql = "INSERT INTO rd_ncftipo (".
             "tipocomprobante, descripcion, estado, clasemovimiento, ventas, compras, contribuyente".
-            " ) VALUES " .
-            "('01','FACTURA DE CREDITO FISCAL',true, 'suma','Y','Y','Y')," .
-            "('02','FACTURA DE CONSUMO',true, 'suma','Y','Y','Y')," .
-            "('03','NOTA DE DEBITO',true, 'suma','Y','Y','N')," .
-            "('04','NOTA DE CREDITO',true, 'resta','Y','Y','N')," .
-            "('11','COMPROBANTE DE COMPRAS',true, 'suma','N','Y','Y')," .
-            "('12','REGISTRO UNICO DE INGRESOS',true, 'suma','Y','N','N')," .
-            "('13','COMPROBANTE PARA GASTOS MENORES',true, 'suma','N','Y','N')," .
-            "('14','COMPROBANTE DE REGIMENES ESPECIALES',true, 'suma','Y','Y','Y')," .
-            "('15','COMPROBANTE GUBERNAMENTAL',true, 'suma','Y','Y','Y')," .
-            "('16','COMPROBANTE PARA EXPORTACIONES',true, 'suma','Y','N','Y')," .
-            "('17','COMPROBANTE PARA PAGOS AL EXTERIOR',true, 'suma','N', 'Y','Y')," .
-            "('31','FACTURA DE CRÉDITO FISCAL ELECTRÓNICA',true,'suma','Y','Y','Y')," .
-            "('32','FACTURA DE CONSUMO ELECTRÓNICA',true,'suma','Y','Y','Y')," .
-            "('33','NOTA DE DÉBITO ELECTRÓNICA',true,'suma','Y','Y','N')," .
-            "('34','NOTA DE CRÉDITO ELECTRÓNICA',true,'resta','Y','Y','N')," .
-            "('41','COMPROBANTE ELECTRÓNICO DE COMPRAS',true,'suma','N','Y','Y')," .
-            "('43','COMPROBANTE ELECTRÓNICO PARA GASTOS MENORES',true,'suma','N','Y','N')," .
-            "('44','COMPROBANTE ELECTRÓNICO PARA REGÍMENES ESPECIALES',true,'suma','Y','Y','Y')," .
-            "('45','COMPROBANTE ELECTRÓNICO GUBERNAMENTAL',true,'suma','Y','Y','Y')," .
-            "('46','COMPROBANTE ELECTRÓNICO PARA EXPORTACIONES',true,'suma','Y','Y','Y')," .
+            " ) VALUES ".
+            "('01','FACTURA DE CREDITO FISCAL',true, 'suma','Y','Y','Y'),".
+            "('02','FACTURA DE CONSUMO',true, 'suma','Y','Y','Y'),".
+            "('03','NOTA DE DEBITO',true, 'suma','Y','Y','N'),".
+            "('04','NOTA DE CREDITO',true, 'resta','Y','Y','N'),".
+            "('11','COMPROBANTE DE COMPRAS',true, 'suma','N','Y','Y'),".
+            "('12','REGISTRO UNICO DE INGRESOS',true, 'suma','Y','N','N'),".
+            "('13','COMPROBANTE PARA GASTOS MENORES',true, 'suma','N','Y','N'),".
+            "('14','COMPROBANTE DE REGIMENES ESPECIALES',true, 'suma','Y','Y','Y'),".
+            "('15','COMPROBANTE GUBERNAMENTAL',true, 'suma','Y','Y','Y'),".
+            "('16','COMPROBANTE PARA EXPORTACIONES',true, 'suma','Y','N','Y'),".
+            "('17','COMPROBANTE PARA PAGOS AL EXTERIOR',true, 'suma','N', 'Y','Y'),".
+            "('31','FACTURA DE CRÉDITO FISCAL ELECTRÓNICA',true,'suma','Y','Y','Y'),".
+            "('32','FACTURA DE CONSUMO ELECTRÓNICA',true,'suma','Y','Y','Y'),".
+            "('33','NOTA DE DÉBITO ELECTRÓNICA',true,'suma','Y','Y','N'),".
+            "('34','NOTA DE CRÉDITO ELECTRÓNICA',true,'resta','Y','Y','N'),".
+            "('41','COMPROBANTE ELECTRÓNICO DE COMPRAS',true,'suma','N','Y','Y'),".
+            "('43','COMPROBANTE ELECTRÓNICO PARA GASTOS MENORES',true,'suma','N','Y','N'),".
+            "('44','COMPROBANTE ELECTRÓNICO PARA REGÍMENES ESPECIALES',true,'suma','Y','Y','Y'),".
+            "('45','COMPROBANTE ELECTRÓNICO GUBERNAMENTAL',true,'suma','Y','Y','Y'),".
+            "('46','COMPROBANTE ELECTRÓNICO PARA EXPORTACIONES',true,'suma','Y','Y','Y'),".
             "('47','COMPROBANTE ELECTRÓNICO PARA PAGOS AL EXTERIOR',true,'suma','N','Y','Y');";
         return($sql);
     }
@@ -196,7 +196,7 @@ 
     public function restoreData()
     {
         $dataBase = new DataBase();
-        $sqlClean = "DELETE FROM " . $this->tableName() . ";";
+        $sqlClean = "DELETE FROM ".$this->tableName().";";
         $dataBase->exec($sqlClean);
         foreach ($this->arrayComprobantes as $arrayItem) {
             $initialData = new NCFTipo($arrayItem);
@@ -207,7 +207,7 @@ 
 
     public function allFor($type = "ventas", $movimiento = "suma")
     {
-        $where = [new DataBaseWhere($type, 'Y'),new DataBaseWhere('clasemovimiento', $movimiento)];
+        $where = [new DataBaseWhere($type, 'Y'), new DataBaseWhere('clasemovimiento', $movimiento)];
         return $this->all($where, ['tipocomprobante' => 'ASC'], 0, 50);
     }
 

Init.php

Spacing +12 added lines, -12 removed lines

@@ -64,7 +64,7 @@ 
         $this->loadExtension(new Extension\Controller\EditProducto());
         $this->loadExtension(new Extension\Controller\EditSettings());
 
-        AssetManager::add('js', \FS_ROUTE . '/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
+        AssetManager::add('js', \FS_ROUTE.'/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
         SalesLineHTML::addMod(new Mod\SalesLineMod());
         SalesFooterHTML::addMod(new Mod\SalesFooterMod());
         PurchasesFooterHTML::addMod(new Mod\PurchasesFooterMod());
@@ -145,22 +145,22 @@ 
     private function actualizarECF(): void
     {
         $arrayECF = [
-            ['tipocomprobante' => '31', 'descripcion' => 'FACTURA DE CRÉDITO FISCAL ELECTRÓNICA', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-            ['tipocomprobante' => '32', 'descripcion' => 'FACTURA DE CONSUMO ELECTRÓNICA', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-            ['tipocomprobante' => '33', 'descripcion' => 'NOTA DE DÉBITO ELECTRÓNICA', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true ],
-            ['tipocomprobante' => '34', 'descripcion' => 'NOTA DE CRÉDITO ELECTRÓNICA', 'clasemovimiento'=> 'resta', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true ],
-            ['tipocomprobante' => '41', 'descripcion' => 'COMPROBANTE ELECTRÓNICO DE COMPRAS', 'clasemovimiento'=> 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-            ['tipocomprobante' => '43', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA GASTOS MENORES', 'clasemovimiento'=> 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true ],
-            ['tipocomprobante' => '44', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA REGÍMENES ESPECIALES', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-            ['tipocomprobante' => '45', 'descripcion' => 'COMPROBANTE ELECTRÓNICO GUBERNAMENTAL', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-            ['tipocomprobante' => '46', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA EXPORTACIONES', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
-            ['tipocomprobante' => '47', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA PAGOS AL EXTERIOR', 'clasemovimiento'=> 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true ],
+            ['tipocomprobante' => '31', 'descripcion' => 'FACTURA DE CRÉDITO FISCAL ELECTRÓNICA', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+            ['tipocomprobante' => '32', 'descripcion' => 'FACTURA DE CONSUMO ELECTRÓNICA', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+            ['tipocomprobante' => '33', 'descripcion' => 'NOTA DE DÉBITO ELECTRÓNICA', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true],
+            ['tipocomprobante' => '34', 'descripcion' => 'NOTA DE CRÉDITO ELECTRÓNICA', 'clasemovimiento'=> 'resta', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true],
+            ['tipocomprobante' => '41', 'descripcion' => 'COMPROBANTE ELECTRÓNICO DE COMPRAS', 'clasemovimiento'=> 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+            ['tipocomprobante' => '43', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA GASTOS MENORES', 'clasemovimiento'=> 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'N', 'estado' => true],
+            ['tipocomprobante' => '44', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA REGÍMENES ESPECIALES', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+            ['tipocomprobante' => '45', 'descripcion' => 'COMPROBANTE ELECTRÓNICO GUBERNAMENTAL', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+            ['tipocomprobante' => '46', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA EXPORTACIONES', 'clasemovimiento'=> 'suma', 'ventas' => 'Y', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
+            ['tipocomprobante' => '47', 'descripcion' => 'COMPROBANTE ELECTRÓNICO PARA PAGOS AL EXTERIOR', 'clasemovimiento'=> 'suma', 'ventas' => 'N', 'compras' => 'Y', 'contribuyente' => 'Y', 'estado' => true],
         ];
 
         $tipoComprobantes = new NCFTipo();
         foreach ($arrayECF as $ecf) {
             $registro = $tipoComprobantes->get($ecf['tipocomprobante']);
-            if($registro === false) {
+            if ($registro === false) {
                 $registro = new NCFTipo();
                 $registro->tipocomprobante = $ecf['tipocomprobante'];
                 $registro->descripcion = $ecf['descripcion'];