joenilson / fsRepublicaDominicana

Extension/Controller/EditFacturaCliente.php

Spacing +6 added lines, -6 removed lines

@@ -28,16 +28,16 @@ 
 {
     public function createViews(): Closure
     {
-        return function () {
+        return function() {
             parent::createViews();
-            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/CommonModals.js');
+            AssetManager::add('js', \FS_ROUTE.'/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
         };
     }
 
     public function execPreviousAction()
     {
-        return function ($action) {
+        return function($action) {
             switch ($action) {
                 case 'busca_infocliente':
                     $this->setTemplate(false);
@@ -71,14 +71,14 @@ 
 
     public function ncftipo()
     {
-        return function () {
+        return function() {
             return NCFTipo::allVentas();
         };
     }
 
     public function ncftipoanulacion()
     {
-        return function () {
+        return function() {
             $tiposAnulacion = new NCFTipoAnulacion();
             return $tiposAnulacion->all();
         };

Extension/Controller/EditFacturaProveedor.php

Spacing +7 added lines, -7 removed lines

@@ -29,16 +29,16 @@ 
 {
     public function createViews(): Closure
     {
-        return function () {
+        return function() {
             parent::createViews();
-            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/CommonModals.js');
+            AssetManager::add('js', \FS_ROUTE.'/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
         };
     }
 
     public function execPreviousAction()
     {
-        return function ($action) {
+        return function($action) {
             switch ($action) {
                 case 'busca_tipo':
                     $this->setTemplate(false);
@@ -67,7 +67,7 @@ 
                     if (!$verificacion) {
                         echo json_encode(['success' => true], JSON_THROW_ON_ERROR);
                     } else {
-                        $message = "Factura: " . $verificacion[0]->idfactura . " Fecha: " . $verificacion[0]->fecha;
+                        $message = "Factura: ".$verificacion[0]->idfactura." Fecha: ".$verificacion[0]->fecha;
                         echo json_encode(['error' => true, 'message' => $message], JSON_THROW_ON_ERROR);
                     }
                     break;
@@ -83,14 +83,14 @@ 
 
     public function ncftipo()
     {
-        return function () {
+        return function() {
             return NCFTipo::allVentas();
         };
     }
 
     public function ncftipoanulacion()
     {
-        return function () {
+        return function() {
             $tiposAnulacion = new NCFTipoAnulacion();
             return $tiposAnulacion->all();
         };

Extension/Controller/EditCliente.php

Spacing +4 added lines, -4 removed lines

@@ -28,10 +28,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 = [];

Extension/Model/FacturaCliente.php

Spacing +3 added lines, -3 removed lines

@@ -65,7 +65,7 @@ 
 
     public function saveBefore(): Closure
     {
-        return function () {
+        return function() {
             $ncfrango = new NCFRango();
             $cliente = new Cliente();
             $appSettins = new AppSettings;
@@ -101,11 +101,11 @@ 
 
     public function all(): Closure
     {
-        return function () {
+        return function() {
             $this->facturarectnumero2 = '';
             if ($this->idfacturarect !== '') {
                 $facturaRectificativa = $this->get($this->idfacturarect);
-                $this->loadFromData(['facturarectnumero2' => 'SI' ]);
+                $this->loadFromData(['facturarectnumero2' => 'SI']);
                 $this->facturarectnumero2 = $facturaRectificativa->numero2;
             } else {
                 $this->loadFromData(['facturarectnumero2' => 'NO HAY']);

Extension/Model/FacturaProveedor.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 
     public function saveBefore(): Closure
     {
-        return function () {
+        return function() {
             $ArrayTipoNCFCompras = ['11', '12', '16', '17'];
             $ncfrango = new NCFRango();
             $cliente = new Proveedor();

Mod/PurchasesFooterMod.php

Spacing +22 added lines, -26 removed lines

@@ -29,21 +29,21 @@ 
 
 class PurchasesFooterMod implements PurchasesModInterface
 {
-    public function apply(PurchaseDocument &$model, array $formData, User $user)
+    public function apply(PurchaseDocument & $model, array $formData, User $user)
     {
         // TODO: Implement apply() method.
     }
 
-    public function applyBefore(PurchaseDocument &$model, array $formData, User $user)
+    public function applyBefore(PurchaseDocument & $model, array $formData, User $user)
     {
         // TODO: Implement applyBefore() method.
         if ($model->modelClassName() === 'FacturaProveedor') {
-            $model->numeroncf = isset($formData['numeroncf']) ? (string)$formData['numeroncf'] : $model->numeroncf;
-            $model->tipocomprobante = isset($formData['tipocomprobante']) ? (string)$formData['tipocomprobante'] : $model->tipocomprobante;
-            $model->ncffechavencimiento = isset($formData['ncffechavencimiento']) ? (string)$formData['ncffechavencimiento'] : $model->ncffechavencimiento;
-            $model->ncftipopago = isset($formData['ncftipopago']) ? (string)$formData['ncftipopago'] : $model->ncftipopago;
-            $model->ncftipomovimiento = isset($formData['ncftipomovimiento']) ? (string)$formData['ncftipomovimiento'] : $model->ncftipomovimiento;
-            $model->ncftipoanulacion = isset($formData['ncftipoanulacion']) ? (string)$formData['ncftipoanulacion'] : $model->ncftipoanulacion;
+            $model->numeroncf = isset($formData['numeroncf']) ? (string) $formData['numeroncf'] : $model->numeroncf;
+            $model->tipocomprobante = isset($formData['tipocomprobante']) ? (string) $formData['tipocomprobante'] : $model->tipocomprobante;
+            $model->ncffechavencimiento = isset($formData['ncffechavencimiento']) ? (string) $formData['ncffechavencimiento'] : $model->ncffechavencimiento;
+            $model->ncftipopago = isset($formData['ncftipopago']) ? (string) $formData['ncftipopago'] : $model->ncftipopago;
+            $model->ncftipomovimiento = isset($formData['ncftipomovimiento']) ? (string) $formData['ncftipomovimiento'] : $model->ncftipomovimiento;
+            $model->ncftipoanulacion = isset($formData['ncftipoanulacion']) ? (string) $formData['ncftipoanulacion'] : $model->ncftipoanulacion;
         }
     }
 
@@ -55,7 +55,7 @@ 
     public function newFields(): array
     {
         // TODO: Implement newFields() method.
-        return ['numeroncf','tipocomprobante', 'ncffechavencimiento', 'ncftipopago', 'ncftipomovimiento', 'ncftipoanulacion'];
+        return ['numeroncf', 'tipocomprobante', 'ncffechavencimiento', 'ncftipopago', 'ncftipomovimiento', 'ncftipoanulacion'];
     }
 
     public function renderField(Translator $i18n, PurchaseDocument $model, string $field): ?string
@@ -104,15 +104,14 @@ 
         $options = ['<option value="">------</option>'];
         foreach ($tipoComprobante as $row) {
             $options[] = ($row->tipocomprobante === $invoiceTipoComprobante) ?
-                '<option value="' . $row->tipocomprobante . '" selected="">' . $row->descripcion . '</option>' :
-                '<option value="' . $row->tipocomprobante . '">' . $row->descripcion . '</option>';
+                '<option value="'.$row->tipocomprobante.'" selected="">'.$row->descripcion.'</option>' : '<option value="'.$row->tipocomprobante.'">'.$row->descripcion.'</option>';
         }
 
         $attributes = ($model->editable || $model->numeroncf === '') ? 'name="tipocomprobante" required=""' : 'disabled=""';
         return '<div class="col-sm-3">'
             . '<div class="form-group">'
             .  $i18n->trans('tipocomprobante')
-            . '<select ' . $attributes . ' class="form-control">' . implode('', $options) . '</select>'
+            . '<select '.$attributes.' class="form-control">'.implode('', $options).'</select>'
             . '</div>'
             . '</div>';
     }
@@ -136,15 +135,14 @@ 
         $options = ['<option value="">------</option>'];
         foreach ($tipoPago as $row) {
             $options[] = ($row->codigo === $invoiceTipoPago) ?
-                '<option value="' . $row->codigo . '" selected="">' . $row->descripcion . '</option>' :
-                '<option value="' . $row->codigo . '">' . $row->descripcion . '</option>';
+                '<option value="'.$row->codigo.'" selected="">'.$row->descripcion.'</option>' : '<option value="'.$row->codigo.'">'.$row->descripcion.'</option>';
         }
 
         $attributes = $model->editable ? 'name="ncftipopago" required=""' : 'disabled=""';
         return '<div class="col-sm-2">'
             . '<div class="form-group">'
             .  $i18n->trans('ncf-payment-type')
-            . '<select ' . $attributes . ' class="form-control">' . implode('', $options) . '</select>'
+            . '<select '.$attributes.' class="form-control">'.implode('', $options).'</select>'
             . '</div>'
             . '</div>';
     }
@@ -162,15 +160,14 @@ 
         $options = ['<option value="">------</option>'];
         foreach ($tipoMovimiento as $row) {
             $options[] = ($row->codigo === $invoiceTipoMovimiento) ?
-                '<option value="' . $row->codigo . '" selected="">' . $row->descripcion . '</option>' :
-                '<option value="' . $row->codigo . '">' . $row->descripcion . '</option>';
+                '<option value="'.$row->codigo.'" selected="">'.$row->descripcion.'</option>' : '<option value="'.$row->codigo.'">'.$row->descripcion.'</option>';
         }
 
         $attributes = $model->editable ? 'name="ncftipomovimiento" required=""' : 'disabled=""';
         return '<div class="col-sm-3">'
             . '<div class="form-group">'
             .  $i18n->trans('ncf-movement-type')
-            . '<select ' . $attributes . ' class="form-control">' . implode('', $options) . '</select>'
+            . '<select '.$attributes.' class="form-control">'.implode('', $options).'</select>'
             . '</div>'
             . '</div>';
     }
@@ -188,15 +185,14 @@ 
         $options = ['<option value="">------</option>'];
         foreach ($tipoAnulacion as $row) {
             $options[] = ($row->codigo === $invoiceTipoAnulacion) ?
-                '<option value="' . $row->codigo . '" selected="">' . $row->descripcion . '</option>' :
-                '<option value="' . $row->codigo . '">' . $row->descripcion . '</option>';
+                '<option value="'.$row->codigo.'" selected="">'.$row->descripcion.'</option>' : '<option value="'.$row->codigo.'">'.$row->descripcion.'</option>';
         }
 
         $attributes = $model->editable ? 'name="ncftipoanulacion"' : 'disabled=""';
         return '<div class="col-sm-2">'
             . '<div class="form-group">'
             .  $i18n->trans('ncf-cancellation-type')
-            . '<select ' . $attributes . ' class="form-control">' . implode('', $options) . '</select>'
+            . '<select '.$attributes.' class="form-control">'.implode('', $options).'</select>'
             . '</div>'
             . '</div>';
     }
@@ -205,8 +201,8 @@ 
     {
         $attributes = ($model->editable) ? 'name="ncffechavencimiento"' : 'disabled=""';
         return '<div class="col-sm-2">'
-            . '<div class="form-group">' . $i18n->trans('due-date')
-            . '<input type="date" ' . $attributes . ' value="' . date('Y-m-d', strtotime($model->ncffechavencimiento)) . '" class="form-control"/>'
+            . '<div class="form-group">'.$i18n->trans('due-date')
+            . '<input type="date" '.$attributes.' value="'.date('Y-m-d', strtotime($model->ncffechavencimiento)).'" class="form-control"/>'
             . '</div>'
             . '</div>';
     }
@@ -219,11 +215,11 @@ 
             . '<div class="form-group">'
             . $i18n->trans('desc-numeroncf-purchases')
             . '<div class="input-group">'
-            . '<input type="text" ' . $attributes . ' value="' . $model->numeroncf . '" class="form-control"/>'
+            . '<input type="text" '.$attributes.' value="'.$model->numeroncf.'" class="form-control"/>'
             . '<div class="input-group-append">'
-            . '<button class="btn ' . $btnColor .' btn-spin-action" id="btnVerifyNCF"'
+            . '<button class="btn '.$btnColor.' btn-spin-action" id="btnVerifyNCF"'
             . 'onclick="purchasesNCFVerify()" '
-            . 'title="'. $i18n->trans('verify-numproveedor')
+            . 'title="'.$i18n->trans('verify-numproveedor')
             .'" type="button">'
             . '<i id="iconBtnVerify" class="fas fa-search fa-fw"></i>'
             . '</button>'

Translation/updater.php

Spacing +4 added lines, -4 removed lines

@@ -23,7 +23,7 @@ 
 $files = [];
 $langs = 'ca_ES,de_DE,en_EN,es_AR,es_CL,es_CO,es_CR,es_DO,es_EC,es_ES,es_GT,es_MX,es_PA,es_PE,es_UY,eu_ES,fr_FR,gl_ES,it_IT,pt_PT,va_ES';
 foreach (explode(',', $langs) as $lang) {
-    $files[] = $lang . '.json';
+    $files[] = $lang.'.json';
 }
 foreach (scandir(__DIR__, SCANDIR_SORT_ASCENDING) as $filename) {
     if (is_file($filename) && substr($filename, -5) === '.json' && false === in_array($filename, $files, true)) {
@@ -34,14 +34,14 @@ 
 // download json from facturascripts.com
 foreach ($files as $filename) {
     $url = "https://facturascripts.com/EditLanguage?action=json&idproject=176&code=";
-    $json = file_get_contents($url . substr($filename, 0, -5));
+    $json = file_get_contents($url.substr($filename, 0, -5));
     if (empty($json) || strlen($json) < 10) {
         unlink($filename);
-        echo "Remove " . $filename . "\n";
+        echo "Remove ".$filename."\n";
         continue;
     }
 
-    echo "Download " . $filename . "\n";
+    echo "Download ".$filename."\n";
     file_put_contents($filename, $json);
 }
 

Init.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
         $this->loadExtension(new Extension\Controller\EditProveedor());
         $this->loadExtension(new Extension\Controller\EditFacturaCliente());
         $this->loadExtension(new Extension\Controller\EditFacturaProveedor());
-        AssetManager::add('js', \FS_ROUTE . '/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
+        AssetManager::add('js', \FS_ROUTE.'/Plugins/fsRepublicaDominicana/Assets/JS/CommonDomFunctions.js');
         SalesFooterHTML::addMod(new Mod\SalesFooterMod());
         PurchasesFooterHTML::addMod(new Mod\PurchasesFooterMod());
     }

Model/NCFTipoMovimiento.php

Spacing +19 added lines, -19 removed lines

@@ -61,23 +61,23 @@ 
      * @var array
      */
     private $arrayTiposMovimiento = [
-        ['tipomovimiento'=>'COM','codigo' => '01', 'descripcion' => 'GASTOS DE PERSONAL', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '02', 'descripcion' => 'GASTOS POR TRABAJOS, SUMINISTROS Y SERVICIOS', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '03', 'descripcion' => 'ARRENDAMIENTOS', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '04', 'descripcion' => 'GASTOS DE ACTIVOS FIJOS', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '05', 'descripcion' => 'GASTOS DE REPRESENTACIÓN', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '06', 'descripcion' => 'OTRAS DEDUCCIONES ADMITIDAS', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '07', 'descripcion' => 'GASTOS FINANCIEROS', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '08', 'descripcion' => 'GASTOS EXTRAORDINARIOS', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '09', 'descripcion' => 'COMPRAS Y GASTOS QUE FORMARÁN PARTE DEL COSTO DE VENTA', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '10', 'descripcion' => 'ADQUISICIONES DE ACTIVOS', 'estado' => true],
-        ['tipomovimiento'=>'COM','codigo' => '11', 'descripcion' => 'GASTOS DE SEGUROS', 'estado' => true],
-        ['tipomovimiento'=>'VEN','codigo' => '1', 'descripcion' => 'Ingresos por operaciones (No financieros)', 'estado' => true],
-        ['tipomovimiento'=>'VEN','codigo' => '2', 'descripcion' => 'Ingresos Financieros', 'estado' => true],
-        ['tipomovimiento'=>'VEN','codigo' => '3', 'descripcion' => 'Ingresos Extraordinarios', 'estado' => true],
-        ['tipomovimiento'=>'VEN','codigo' => '4', 'descripcion' => 'Ingresos por Arrendamientos', 'estado' => true],
-        ['tipomovimiento'=>'VEN','codigo' => '5', 'descripcion' => 'Ingresos por Venta de Activo Depreciable', 'estado' => true],
-        ['tipomovimiento'=>'VEN','codigo' => '6', 'descripcion' => 'Otros Ingresos', 'estado' => true]
+        ['tipomovimiento'=>'COM', 'codigo' => '01', 'descripcion' => 'GASTOS DE PERSONAL', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '02', 'descripcion' => 'GASTOS POR TRABAJOS, SUMINISTROS Y SERVICIOS', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '03', 'descripcion' => 'ARRENDAMIENTOS', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '04', 'descripcion' => 'GASTOS DE ACTIVOS FIJOS', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '05', 'descripcion' => 'GASTOS DE REPRESENTACIÓN', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '06', 'descripcion' => 'OTRAS DEDUCCIONES ADMITIDAS', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '07', 'descripcion' => 'GASTOS FINANCIEROS', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '08', 'descripcion' => 'GASTOS EXTRAORDINARIOS', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '09', 'descripcion' => 'COMPRAS Y GASTOS QUE FORMARÁN PARTE DEL COSTO DE VENTA', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '10', 'descripcion' => 'ADQUISICIONES DE ACTIVOS', 'estado' => true],
+        ['tipomovimiento'=>'COM', 'codigo' => '11', 'descripcion' => 'GASTOS DE SEGUROS', 'estado' => true],
+        ['tipomovimiento'=>'VEN', 'codigo' => '1', 'descripcion' => 'Ingresos por operaciones (No financieros)', 'estado' => true],
+        ['tipomovimiento'=>'VEN', 'codigo' => '2', 'descripcion' => 'Ingresos Financieros', 'estado' => true],
+        ['tipomovimiento'=>'VEN', 'codigo' => '3', 'descripcion' => 'Ingresos Extraordinarios', 'estado' => true],
+        ['tipomovimiento'=>'VEN', 'codigo' => '4', 'descripcion' => 'Ingresos por Arrendamientos', 'estado' => true],
+        ['tipomovimiento'=>'VEN', 'codigo' => '5', 'descripcion' => 'Ingresos por Venta de Activo Depreciable', 'estado' => true],
+        ['tipomovimiento'=>'VEN', 'codigo' => '6', 'descripcion' => 'Otros Ingresos', 'estado' => true]
     ];
     
     /**
@@ -102,7 +102,7 @@ 
     public function install(): string
     {
         parent::install();
-        return "INSERT INTO rd_ncftipomovimiento (tipomovimiento, codigo, descripcion, estado) VALUES " .
+        return "INSERT INTO rd_ncftipomovimiento (tipomovimiento, codigo, descripcion, estado) VALUES ".
             "('COM','01','GASTOS DE PERSONAL',true), ".
             "('COM','02','GASTOS POR TRABAJOS, SUMINISTROS Y SERVICIOS',true), ".
             "('COM','03','ARRENDAMIENTOS',true), ".
@@ -125,7 +125,7 @@ 
     public function restoreData()
     {
         $dataBase = new DataBase();
-        $sqlClean = "DELETE FROM " . $this->tableName() . ";";
+        $sqlClean = "DELETE FROM ".$this->tableName().";";
         $dataBase->exec($sqlClean);
         foreach ($this->arrayTiposMovimiento as $arrayItem) {
             $initialData = new NCFTipoMovimiento($arrayItem);