From 1f141a5dc94a413dd3608fd348370df014db392c Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Mon, 18 Oct 2021 10:23:39 +0200
Subject: [PATCH] Stile del codice

---
 include/init/update.php                       |  2 +-
 modules/articoli/bulk.php                     |  2 +-
 modules/articoli/src/Import/CSV.php           | 12 +++----
 .../fatture/src/Components/RelationTrait.php  |  2 +-
 modules/interventi/modutil.php                |  8 ++---
 modules/movimenti/add.php                     |  2 +-
 modules/stato_servizi/actions.php             |  3 +-
 modules/stato_servizi/edit.php                | 34 +++++++------------
 plugins/importFE/generate.php                 | 10 +++---
 plugins/importFE/src/FatturaElettronica.php   |  2 +-
 plugins/importFE/src/FatturaOrdinaria.php     | 19 +++++------
 plugins/statistiche_anagrafiche/info.php      |  6 ++--
 src/API/Services.php                          |  2 +-
 src/Common/Components/Component.php           |  2 +-
 src/Models/Upload.php                         |  2 +-
 15 files changed, 49 insertions(+), 59 deletions(-)

diff --git a/include/init/update.php b/include/init/update.php
index 5e1ef168c..2d7fb3272 100755
--- a/include/init/update.php
+++ b/include/init/update.php
@@ -214,7 +214,7 @@ if (filter('action') == 'do_update') {
             $total += $scriptValue;
         }
     }
-    
+
     echo '
                 <script>
                     $(document).ready(function(){
diff --git a/modules/articoli/bulk.php b/modules/articoli/bulk.php
index 07c2c03ee..903d6767e 100644
--- a/modules/articoli/bulk.php
+++ b/modules/articoli/bulk.php
@@ -54,7 +54,7 @@ switch (post('op')) {
 
         foreach ($id_records as $id) {
             $articolo = Articolo::find($id);
-            $prezzo_partenza = post('prezzo_partenza')=='vendita' ? $articolo->prezzo_vendita : $articolo->prezzo_acquisto;
+            $prezzo_partenza = post('prezzo_partenza') == 'vendita' ? $articolo->prezzo_vendita : $articolo->prezzo_acquisto;
 
             $new_prezzo_vendita = $prezzo_partenza + ($prezzo_partenza * $percentuale / 100);
             $articolo->setPrezzoVendita($new_prezzo_vendita, $articolo->idiva_vendita);
diff --git a/modules/articoli/src/Import/CSV.php b/modules/articoli/src/Import/CSV.php
index 3cdcd653b..931be7bf4 100644
--- a/modules/articoli/src/Import/CSV.php
+++ b/modules/articoli/src/Import/CSV.php
@@ -19,17 +19,17 @@
 
 namespace Modules\Articoli\Import;
 
-use Uploads;
-use Modules;
 use Carbon\Carbon;
-use Models\Upload;
 use Importer\CSVImporter;
+use Models\Upload;
+use Modules;
 use Modules\Anagrafiche\Anagrafica;
 use Modules\Anagrafiche\Sede;
 use Modules\Articoli\Articolo;
 use Modules\Articoli\Categoria;
 use Modules\Iva\Aliquota;
 use Plugins\ListinoClienti\DettaglioPrezzo;
+use Uploads;
 
 /**
  * Struttura per la gestione delle operazioni di importazione (da CSV) degli Articoli.
@@ -331,11 +331,11 @@ class CSV extends CSVImporter
         ]);
 
         //Gestione immagine
-        if ( !empty($url) ) {
+        if (!empty($url)) {
             $file_content = file_get_contents($url);
 
-            if( !empty($file_content) ){
-                $name = "immagine_".$articolo->id.'.'.Upload::getExtensionFromMimeType($file_content);
+            if (!empty($file_content)) {
+                $name = 'immagine_'.$articolo->id.'.'.Upload::getExtensionFromMimeType($file_content);
 
                 $upload = Uploads::upload($file_content, [
                     'name' => 'Immagine',
diff --git a/modules/fatture/src/Components/RelationTrait.php b/modules/fatture/src/Components/RelationTrait.php
index 39106d420..ecc0a5587 100755
--- a/modules/fatture/src/Components/RelationTrait.php
+++ b/modules/fatture/src/Components/RelationTrait.php
@@ -182,7 +182,7 @@ trait RelationTrait
     {
         $result = parent::delete();
 
-        if (!empty($this->idintervento) && $this->fattura->getRighe()->where('idintervento', $this->idintervento)->count()==1) {
+        if (!empty($this->idintervento) && $this->fattura->getRighe()->where('idintervento', $this->idintervento)->count() == 1) {
             database()->query("UPDATE in_interventi SET idstatointervento = (SELECT idstatointervento FROM in_statiintervento WHERE codice = 'OK') WHERE id=".prepare($this->idintervento));
         }
 
diff --git a/modules/interventi/modutil.php b/modules/interventi/modutil.php
index 50d1759ac..5bc18ac81 100755
--- a/modules/interventi/modutil.php
+++ b/modules/interventi/modutil.php
@@ -186,10 +186,10 @@ function aggiungi_intervento_in_fattura($id_intervento, $id_fattura, $descrizion
             $riga->prezzo_unitario = $sessione->prezzo_orario;
 
             //Calcolo lo sconto unitario della sessione in base all'impostazione sui prezzi ivati
-            $iva = $dbo->table('co_iva')->where('id',$id_iva)->first();
-            if( $sessione->tipo_sconto=='UNT' && setting('Utilizza prezzi di vendita comprensivi di IVA') ){
-                $sconto_unitario = $sessione->sconto_unitario + (($sessione->sconto_unitario*$iva->percentuale)/100);
-            }else{
+            $iva = $dbo->table('co_iva')->where('id', $id_iva)->first();
+            if ($sessione->tipo_sconto == 'UNT' && setting('Utilizza prezzi di vendita comprensivi di IVA')) {
+                $sconto_unitario = $sessione->sconto_unitario + (($sessione->sconto_unitario * $iva->percentuale) / 100);
+            } else {
                 $sconto_unitario = $sessione->sconto_unitario;
             }
 
diff --git a/modules/movimenti/add.php b/modules/movimenti/add.php
index 3000371a8..8dd0d79d5 100755
--- a/modules/movimenti/add.php
+++ b/modules/movimenti/add.php
@@ -345,4 +345,4 @@ echo '
         $("#idarticolo").selectReset();
     });
 
-</script>';
\ No newline at end of file
+</script>';
diff --git a/modules/stato_servizi/actions.php b/modules/stato_servizi/actions.php
index c1104fe47..58979b695 100755
--- a/modules/stato_servizi/actions.php
+++ b/modules/stato_servizi/actions.php
@@ -317,10 +317,9 @@ switch (filter('op')) {
         $max_number = $informazioni['maxNumber'];
         $avviso_numero = !empty($max_number) && floatval($history[0]['number']) > 0.9 * $max_number;
 
-
         // Formattazione dei contenuti dello storico
         foreach ($history as $key => $value) {
-            $history[$key]['size'] = (($history[$key]['size'])? Filesystem::formatBytes($value['size']) : '-' );
+            $history[$key]['size'] = (($history[$key]['size']) ? Filesystem::formatBytes($value['size']) : '-');
             //$history[$key]['invoices_size'] = Filesystem::formatBytes($value['invoices_size']);
             //$history[$key]['notifies_size'] = Filesystem::formatBytes($value['notifies_size']);
         }
diff --git a/modules/stato_servizi/edit.php b/modules/stato_servizi/edit.php
index fb9a6ff66..abf7a3ed5 100755
--- a/modules/stato_servizi/edit.php
+++ b/modules/stato_servizi/edit.php
@@ -94,22 +94,17 @@ if (Services::isEnabled()) {
     $risorse_attive = Services::getRisorseAttive();
     if (!$risorse_attive->isEmpty()) {
         $risorse_in_scadenza = Services::getRisorseInScadenza($limite_scadenze);
-        
+
         if (!$risorse_in_scadenza->isEmpty()) {
             echo '
                 <div class="alert alert-warning" role="alert"> <i class="fa fa-warning"></i> '.tr('Attenzione, _NUM_ risorse sono in scadenza:', [
-
-                    '_NUM_' =>  $risorse_in_scadenza->count()
-
+                    '_NUM_' => $risorse_in_scadenza->count(),
                 ]).'</div>';
-               
-        }else{
-
-        echo '
+        } else {
+            echo '
             <div class="alert alert-success" role="alert"> <i class="fa fa-check"></i> '.tr('Bene, tutte le risorse sono attive e non presentano avvisi:', [
-                '_NUM_' => $risorse_attive->count()
+                '_NUM_' => $risorse_attive->count(),
             ]).'</div>';
-
         }
 
         echo '
@@ -124,25 +119,23 @@ if (Services::isEnabled()) {
                 
                 <tbody>';
 
-            foreach ($risorse_attive as $servizio) {
-                $scadenza = Carbon::parse($servizio['expiration_at']);
-                echo '
+        foreach ($risorse_attive as $servizio) {
+            $scadenza = Carbon::parse($servizio['expiration_at']);
+            echo '
                 <tr>
                     <td>'.$servizio['name'].'</td>
-                    <td>'.(($servizio['credits']<100 && $servizio['credits'])? '<b><i class="fa fa-icon fa-warning" ></i>': '' ).(($servizio['credits'])? $servizio['credits']: '-' ).(($servizio['credits']<100 && $servizio['credits'])? '</b>': '' ).'</td>
-                    <td>'.((Carbon::now()->diffInDays($scadenza, false)<60 && $scadenza)? '<b><i class="fa fa-icon fa-warning" ></i>': '' ).dateFormat($scadenza).' ('.$scadenza->diffForHumans().')'.((Carbon::now()->diffInDays($scadenza, false)<60 && $scadenza)? '</b>': '' ).'</td>
+                    <td>'.(($servizio['credits'] < 100 && $servizio['credits']) ? '<b><i class="fa fa-icon fa-warning" ></i>' : '').(($servizio['credits']) ? $servizio['credits'] : '-').(($servizio['credits'] < 100 && $servizio['credits']) ? '</b>' : '').'</td>
+                    <td>'.((Carbon::now()->diffInDays($scadenza, false) < 60 && $scadenza) ? '<b><i class="fa fa-icon fa-warning" ></i>' : '').dateFormat($scadenza).' ('.$scadenza->diffForHumans().')'.((Carbon::now()->diffInDays($scadenza, false) < 60 && $scadenza) ? '</b>' : '').'</td>
                 </tr>';
-            }
+        }
 
         echo '
                 </tbody>
             </table><hr>';
-    
 
-       
         //Il servizio Fatturazione Elettronica deve essere presente per visualizzare le Statistiche su Fatture Elettroniche
-        if (Services::getRisorseAttive()->where('name','Fatturazione Elettronica')->count()){
-        echo '
+        if (Services::getRisorseAttive()->where('name', 'Fatturazione Elettronica')->count()) {
+            echo '
 
                 <div class="panel panel-info">
 
@@ -203,7 +196,6 @@ if (Services::isEnabled()) {
                     aggiornaStatisticheFE();
                 });
                 </script>';
-
         }
     } else {
         echo '
diff --git a/plugins/importFE/generate.php b/plugins/importFE/generate.php
index 464c1d856..b414d736f 100755
--- a/plugins/importFE/generate.php
+++ b/plugins/importFE/generate.php
@@ -294,11 +294,11 @@ echo '
     </div>';
 
 // Righe
-if (setting('Aggiorna info di acquisto')=='Non aggiornare'){
+if (setting('Aggiorna info di acquisto') == 'Non aggiornare') {
     $update_info = 'update_not';
-} elseif (setting('Aggiorna info di acquisto')=='Aggiorna prezzo di listino'){ 
-    $update_info ='update_price'; 
-} else{
+} elseif (setting('Aggiorna info di acquisto') == 'Aggiorna prezzo di listino') {
+    $update_info = 'update_price';
+} else {
     $update_info = 'update_all';
 }
 
@@ -358,7 +358,7 @@ if (!empty($righe)) {
                     $id_articolo = $database->fetchOne('SELECT id_articolo AS id FROM mg_fornitore_articolo WHERE REPLACE(codice_fornitore, " ", "") = '.prepare($codice_principale).' AND id_fornitore = '.prepare($anagrafica->id))['id'];
                 }
             }
-            
+
             if (empty($id_articolo)) {
                 $id_articolo = $database->fetchOne('SELECT id FROM mg_articoli WHERE codice = '.prepare($codice_principale))['id'];
                 if (empty($id_articolo)) {
diff --git a/plugins/importFE/src/FatturaElettronica.php b/plugins/importFE/src/FatturaElettronica.php
index a6bef5108..313e964f3 100755
--- a/plugins/importFE/src/FatturaElettronica.php
+++ b/plugins/importFE/src/FatturaElettronica.php
@@ -21,9 +21,9 @@ namespace Plugins\ImportFE;
 
 use Modules;
 use Modules\Anagrafiche\Anagrafica;
-use Modules\Banche\Banca;
 use Modules\Anagrafiche\Nazione;
 use Modules\Anagrafiche\Tipo as TipoAnagrafica;
+use Modules\Banche\Banca;
 use Modules\Fatture\Fattura;
 use Modules\Fatture\Stato as StatoFattura;
 use Modules\Fatture\Tipo as TipoFattura;
diff --git a/plugins/importFE/src/FatturaOrdinaria.php b/plugins/importFE/src/FatturaOrdinaria.php
index 4d97b7b08..352556185 100755
--- a/plugins/importFE/src/FatturaOrdinaria.php
+++ b/plugins/importFE/src/FatturaOrdinaria.php
@@ -155,7 +155,7 @@ class FatturaOrdinaria extends FatturaElettronica
             if (!empty($articolo)) {
                 $articolo->idconto_acquisto = $conto[$key];
                 $articolo->save();
-                
+
                 $obj = Articolo::build($fattura, $articolo);
 
                 $obj->movimentazione($movimentazione);
@@ -242,11 +242,11 @@ class FatturaOrdinaria extends FatturaElettronica
                     if ($tipo_sconto == 'PRC') {
                         $sconto_calcolato = calcola_sconto([
                             'sconto' => $sconto_riga,
-                            'prezzo' => $sconto_unitario ? $obj->prezzo_unitario-($sconto_calcolato/$obj->qta) : $obj->prezzo_unitario,
+                            'prezzo' => $sconto_unitario ? $obj->prezzo_unitario - ($sconto_calcolato / $obj->qta) : $obj->prezzo_unitario,
                             'tipo' => 'PRC',
                             'qta' => $obj->qta,
                         ]);
-    
+
                         if ($tipo == 'PRC') {
                             $tot_sconto = $sconto_calcolato * 100 / $obj->imponibile;
                         } else {
@@ -256,15 +256,15 @@ class FatturaOrdinaria extends FatturaElettronica
                         $tot_sconto = $sconto_riga;
                     }
 
-                    $sconto_unitario += $tot_sconto;    
+                    $sconto_unitario += $tot_sconto;
                 }
 
-                $obj->setSconto($sconto_unitario, $tipo);   
+                $obj->setSconto($sconto_unitario, $tipo);
             }
 
             // Aggiornamento prezzo di acquisto e fornitore predefinito in base alle impostazioni
             if (!empty($articolo)) {
-                if ($update_info[$key]=='update_price' || $update_info[$key]=='update_all') {
+                if ($update_info[$key] == 'update_price' || $update_info[$key] == 'update_all') {
                     $dettaglio_predefinito = DettaglioPrezzo::dettaglioPredefinito($articolo->id, $anagrafica->idanagrafica, $direzione)
                     ->first();
 
@@ -272,7 +272,7 @@ class FatturaOrdinaria extends FatturaElettronica
                     if (empty($dettaglio_predefinito)) {
                         $dettaglio_predefinito = DettaglioPrezzo::build($articolo, $anagrafica, $direzione);
                     }
-                    
+
                     // Imposto lo sconto nel listino solo se è una percentuale, se è un importo lo sottraggo dal prezzo
                     if ($tipo == 'PRC') {
                         $dettaglio_predefinito->sconto_percentuale = $sconto_unitario;
@@ -288,7 +288,7 @@ class FatturaOrdinaria extends FatturaElettronica
                     $dettaglio_predefinito->save();
 
                     // Aggiornamento fornitore predefinito
-                    if ($update_info[$key]=='update_all') {
+                    if ($update_info[$key] == 'update_all') {
                         // Aggiornamento prezzo di acquisto e fornitore predefinito
                         $articolo->prezzo_acquisto = $prezzo_acquisto;
                         $articolo->id_fornitore = $anagrafica->idanagrafica;
@@ -296,11 +296,10 @@ class FatturaOrdinaria extends FatturaElettronica
                     }
                 }
             }
-            
+
             $tipo = null;
             $sconto_unitario = null;
             $obj->save();
-
         }
 
         // Ricaricamento della fattura
diff --git a/plugins/statistiche_anagrafiche/info.php b/plugins/statistiche_anagrafiche/info.php
index 1844de015..67124b9d5 100755
--- a/plugins/statistiche_anagrafiche/info.php
+++ b/plugins/statistiche_anagrafiche/info.php
@@ -189,13 +189,13 @@ echo '
 
             <div class="col-md-4">
                 <div class="info-box">
-                    <span class="info-box-icon bg-'.($fatture_vendita->count()+$note_credito->count() == 0 ? 'gray' : 'green').'"><i class="fa fa-money"></i></span>
+                    <span class="info-box-icon bg-'.($fatture_vendita->count() + $note_credito->count() == 0 ? 'gray' : 'green').'"><i class="fa fa-money"></i></span>
                     <div class="info-box-content">
                         <span class="info-box-text pull-left">'.tr('Fatture').'</span>
-                        '.($fatture_vendita->count()+$note_credito->count() > 0 ? '<span class="info-box-text pull-right"><a href="'.base_path().'/controller.php?id_module='.Modules::get('Fatture di vendita')['id'].'&search_Ragione-sociale='.rawurlencode($anagrafica['ragione_sociale']).'">'.tr('Visualizza').' <i class="fa fa-chevron-circle-right"></i></a></span>' : '').'
+                        '.($fatture_vendita->count() + $note_credito->count() > 0 ? '<span class="info-box-text pull-right"><a href="'.base_path().'/controller.php?id_module='.Modules::get('Fatture di vendita')['id'].'&search_Ragione-sociale='.rawurlencode($anagrafica['ragione_sociale']).'">'.tr('Visualizza').' <i class="fa fa-chevron-circle-right"></i></a></span>' : '').'
                         <br class="clearfix">
                         <span class="info-box-number">
-                            <big>'.($fatture_vendita->count()+$note_credito->count()).'</big><br>
+                            <big>'.($fatture_vendita->count() + $note_credito->count()).'</big><br>
                             <small class="help-block">'.moneyFormat($totale_fatture_vendita).'</small>
                         </span>
                     </div>
diff --git a/src/API/Services.php b/src/API/Services.php
index 64ab8552c..a591fd31c 100755
--- a/src/API/Services.php
+++ b/src/API/Services.php
@@ -86,7 +86,7 @@ class Services
         return self::getServiziAttivi()
             ->flatten(1)
             ->filter(function ($item) use ($limite_scadenze) {
-                return (isset($item['data_conclusione']) && Carbon::parse($item['data_conclusione'])->lessThan($limite_scadenze));
+                return isset($item['data_conclusione']) && Carbon::parse($item['data_conclusione'])->lessThan($limite_scadenze);
             });
     }
 
diff --git a/src/Common/Components/Component.php b/src/Common/Components/Component.php
index 772c5dfe7..eaa825294 100644
--- a/src/Common/Components/Component.php
+++ b/src/Common/Components/Component.php
@@ -417,7 +417,7 @@ abstract class Component extends Model
      */
     protected function customBeforeDataCopiaIn($original)
     {
-        if(!empty($original->idiva)){
+        if (!empty($original->idiva)) {
             $this->idiva = $original->idiva;
         }
     }
diff --git a/src/Models/Upload.php b/src/Models/Upload.php
index f6daecb1c..26ac0a6a3 100755
--- a/src/Models/Upload.php
+++ b/src/Models/Upload.php
@@ -222,7 +222,7 @@ class Upload extends Model
 
     /**
      * @return string
-    */
+     */
     public static function getExtensionFromMimeType($file_content)
     {
         $finfo = finfo_open();