Stile del codice

actions.php

@@ -241,6 +241,7 @@ elseif (post('op') == 'send-email') {
         $mail->addUpload($file);
     }
 
+    // Salvataggio email nella coda di invio
     $mail->save();
 
     // Invio mail istantaneo

editor.php

@@ -134,13 +134,11 @@ if (empty($record) || !$has_access) {
     if (!empty($record['deleted_at'])) {
         $operation = $dbo->fetchOne("SELECT zz_operations.created_at, username FROM zz_operations INNER JOIN zz_users ON zz_operations.id_utente =  zz_users.id  WHERE op='delete' AND id_module=".prepare($id_module).' AND id_record='.prepare($id_record).' ORDER BY zz_operations.created_at DESC');
 
-        
         $info = tr('Il record è stato eliminato il <b>_DATE_</b> da <b>_USER_</b>', [
             '_DATE_' => (($operation['created_at']) ? Translator::timestampToLocale($operation['created_at']) : Translator::timestampToLocale($record['deleted_at'])),
             '_USER_' => ((!empty($operation['username'])) ? $operation['username'] : 'N.D.'),
         ]).'. ';
 
-
         echo '
         <div class="alert alert-warning">
             <div class="row" >

modules/articoli/ajax/complete.php

@@ -24,7 +24,7 @@ switch ($resource) {
                     ($fatture[$i]['n2_fattura'] != '') ? $n_fattura = $fatture[$i]['n2_fattura'] : $n_fattura = $fatture[$i]['n_fattura'];
 
                     $link_id = Modules::get('Fatture di vendita')['id'];
-                    echo "<tr><td class='first_cell text-left'><a href='".ROOTDIR.'/editor.php?id_module='.$link_id.'&id_record='.$fatture[$i]['iddocumento']."'  target=\"_blank\" title=\"Apri il documento su una nuova finestra\">Fatt. n. ".$n_fattura." del ".Translator::dateToLocale($fatture[$i]['data_fattura'])." </a></td>\n";
+                    echo "<tr><td class='first_cell text-left'><a href='".ROOTDIR.'/editor.php?id_module='.$link_id.'&id_record='.$fatture[$i]['iddocumento']."'  target=\"_blank\" title=\"Apri il documento su una nuova finestra\">Fatt. n. ".$n_fattura.' del '.Translator::dateToLocale($fatture[$i]['data_fattura'])." </a></td>\n";
                     echo "<td class='table_cell text-right'>".moneyFormat($fatture[$i]['costo_unitario'])."</td></tr>\n";
                     array_push($ids, '"'.$fatture[$i]['iddocumento'].'"');
                 }
@@ -51,7 +51,7 @@ switch ($resource) {
                 ($fatture[$i]['n2_fattura'] != '') ? $n_fattura = $fatture[$i]['n2_fattura'] : $n_fattura = $fatture[$i]['n_fattura'];
 
                 $link_id = Modules::get('Fatture di vendita')['id'];
-                echo "<tr><td class='first_cell text-left'><a href='".ROOTDIR.'/editor.php?id_module='.$link_id.'&id_record='.$fatture[$i]['iddocumento']."'  target=\"_blank\" title=\"Apri il documento su una nuova finestra\">Fatt. n. ".$n_fattura." del ".Translator::dateToLocale($fatture[$i]['data_fattura'])."</a></td>\n";
+                echo "<tr><td class='first_cell text-left'><a href='".ROOTDIR.'/editor.php?id_module='.$link_id.'&id_record='.$fatture[$i]['iddocumento']."'  target=\"_blank\" title=\"Apri il documento su una nuova finestra\">Fatt. n. ".$n_fattura.' del '.Translator::dateToLocale($fatture[$i]['data_fattura'])."</a></td>\n";
                 echo "<td class='table_cell text-right'>".moneyFormat($fatture[$i]['costo_unitario'])."</td></tr>\n";
             }
             echo "</table>\n";
@@ -78,7 +78,7 @@ switch ($resource) {
                 ($fatture[$i]['n2_fattura'] != '') ? $n_fattura = $fatture[$i]['n2_fattura'] : $n_fattura = $fatture[$i]['n_fattura'];
 
                 $link_id = Modules::get('Fatture di acquisto')['id'];
-                echo "<tr><td class='first_cell text-left'><a href='".ROOTDIR.'/editor.php?id_module='.$link_id.'&id_record='.$fatture[$i]['iddocumento']."'  target=\"_blank\" title=\"Apri il documento su una nuova finestra\">Fatt. n. ".$n_fattura." del ".Translator::dateToLocale($fatture[$i]['data_fattura'])."</a></td>\n";
+                echo "<tr><td class='first_cell text-left'><a href='".ROOTDIR.'/editor.php?id_module='.$link_id.'&id_record='.$fatture[$i]['iddocumento']."'  target=\"_blank\" title=\"Apri il documento su una nuova finestra\">Fatt. n. ".$n_fattura.' del '.Translator::dateToLocale($fatture[$i]['data_fattura'])."</a></td>\n";
                 echo "<td class='table_cell text-right'>".moneyFormat($fatture[$i]['costo_unitario'])."</td></tr>\n";
             }
             echo "</table>\n";

modules/categorie_articoli/edit.php

@@ -100,13 +100,13 @@ if (!empty($elementi)) {
 
         echo '
 		<li>'.Modules::link($modulo, $id, $descrizione).'</li>';
-}
+    }
 
-echo '
+    echo '
 	</ul>
 </div>
 </div>';
-}else {
+} else {
     echo '
     <a class="btn btn-danger ask" data-backto="record-list">
         <i class="fa fa-trash"></i> '.tr('Elimina').'

modules/categorie_articoli/row-list.php

@@ -4,10 +4,6 @@ include_once __DIR__.'/../../core.php';
 
 $subcategorie = $dbo->fetchArray('SELECT * FROM `mg_categorie` WHERE `parent`='.prepare($id_record).' ORDER BY nome ASC ');
 
-
-
-
-
 foreach ($subcategorie as $sub) {
     $n_articoli = $dbo->fetchNum('SELECT * FROM `mg_articoli` WHERE `id_sottocategoria`='.prepare($sub['id']));
     echo '
@@ -17,7 +13,7 @@ foreach ($subcategorie as $sub) {
 					<td>'.$sub['nota'].'</td>
 					<td>
 						<a class="btn btn-warning btn-sm" title="Modifica riga" onclick="launch_modal(\''.tr('Modifica sottocategoria').'\', \''.$rootdir.'/add.php?id_module='.$id_module.'&id_record='.$sub['id'].'&id_original='.$id_record.'\');"><i class="fa fa-edit"></i></a>
-                        <a class="btn btn-sm btn-danger ask '.(($n_articoli>0) ? 'disabled tip' : '').'" data-backto="record-edit" data-id="'.$sub['id'].'" title="'.(($n_articoli>0) ? 'Sottocategoria collegata a '.$n_articoli.' articoli' : '' ).'">
+                        <a class="btn btn-sm btn-danger ask '.(($n_articoli > 0) ? 'disabled tip' : '').'" data-backto="record-edit" data-id="'.$sub['id'].'" title="'.(($n_articoli > 0) ? 'Sottocategoria collegata a '.$n_articoli.' articoli' : '').'">
                             <i class="fa fa-trash"></i>
                         </a>
 					</td>

modules/fatture/edit.php

@@ -615,7 +615,7 @@ if (!$block_edit) {
     $art_query = 'SELECT id FROM mg_articoli WHERE attivo = 1 AND deleted_at IS NULL';
     if ($dir == 'entrata') {
         $art_query .= ' AND (qta > 0 OR servizio = 1)';
-    }else{
+    } else {
         //Gli articoli possono essere creati al volo direttamente dal modale di aggiunta articolo
         $art_query .= ' OR 1=1';
     }

modules/interventi/src/API/v1/Impianti.php

@@ -3,12 +3,8 @@
 namespace Modules\Interventi\API\v1;
 
 use API\Interfaces\CreateInterface;
-use API\Interfaces\DeleteInterface;
 use API\Interfaces\RetrieveInterface;
-use API\Interfaces\UpdateInterface;
 use API\Resource;
-use Modules;
-use Modules\Anagrafiche\Anagrafica;
 
 class Impianti extends Resource implements RetrieveInterface, CreateInterface
 {

modules/interventi/src/API/v1/Rapportino.php

@@ -3,14 +3,10 @@
 namespace Modules\Interventi\API\v1;
 
 use API\Interfaces\CreateInterface;
-use API\Interfaces\DeleteInterface;
 use API\Interfaces\RetrieveInterface;
-use API\Interfaces\UpdateInterface;
 use API\Resource;
-use Modules;
-use Modules\Anagrafiche\Anagrafica;
-use Modules\Emails\Template;
 use Modules\Emails\Mail;
+use Modules\Emails\Template;
 
 class Rapportino extends Resource implements RetrieveInterface, CreateInterface
 {
@@ -65,7 +61,6 @@ class Rapportino extends Resource implements RetrieveInterface, CreateInterface
         $mail->save();
 
         return [
-
         ];
     }
 }

modules/my_impianti/src/API/v1/Impianti.php

@@ -2,13 +2,8 @@
 
 namespace Modules\Impianti\API\v1;
 
-use API\Interfaces\CreateInterface;
-use API\Interfaces\DeleteInterface;
 use API\Interfaces\RetrieveInterface;
-use API\Interfaces\UpdateInterface;
 use API\Resource;
-use Modules;
-use Modules\Anagrafiche\Anagrafica;
 
 class Impianti extends Resource implements RetrieveInterface
 {

modules/preventivi/buttons.php

@@ -6,7 +6,7 @@ echo'
 <button type="button" class="btn btn-primary" onclick="if( confirm(\'Duplicare questo preventivo?\') ){ $(\'#copia-preventivo\').submit(); }"> <i class="fa fa-copy"></i> '.tr('Duplica preventivo').'</button>';
 
 // Crea revisione
-if (!$record['is_completato'] || ($record['is_completato'] && $record['stato']=='Rifiutato')) {
+if (!$record['is_completato'] || ($record['is_completato'] && $record['stato'] == 'Rifiutato')) {
     echo '
 	<button type="button" class="btn btn-warning" onclick="if(confirm(\'Vuoi creare un nuova revisione?\')){$(\'#crea-revisione\').submit();}"><i class="fa fa-edit"></i> '.tr('Crea nuova revisione...').'</button>';
 }

modules/stampe/edit.php

@@ -50,7 +50,7 @@ echo '
 
 if (!empty($variables)) {
     echo '
-        <p>'.tr("Puoi utilizzare le seguenti variabili per generare il nome del file").':</p>
+        <p>'.tr('Puoi utilizzare le seguenti variabili per generare il nome del file').':</p>
         <ul>';
 
     foreach ($variables as $variable => $value) {

plugins/exportFE/src/Validator.php

@@ -592,8 +592,7 @@ class Validator
             if ($info['type'] == 'decimal') {
                 if (in_array($key, ['PrezzoUnitario', 'PrezzoTotale'])) {
                     $output = number_format($output, setting('Cifre decimali per importi'), '.', '');
-                }
+                } elseif (in_array($key, ['Quantita'])) {
-                else if (in_array($key, ['Quantita'])) {
                     $output = number_format($output, setting('Cifre decimali per quantità'), '.', '');
                 } else {
                     $output = number_format($output, 2, '.', '');

plugins/pianificazione_fatturazione/add_pianificazione.php

@@ -63,7 +63,7 @@ foreach ($iva_righe as $id_iva => $righe) {
                 <div class="row">
                     <div class="col-md-9">
                         {[ "type": "textarea", "label": "'.tr('Descrizione').'", "name": "descrizione['.$id_iva.']", "value": "'.tr('Canone contratto numero _NUM__IVA_', [
-                            '_IVA_' => (count($iva_righe) > 1 ) ? ': '.$iva->descrizione : '',
+                            '_IVA_' => (count($iva_righe) > 1) ? ': '.$iva->descrizione : '',
                             '_NUM_' => $contratto->numero,
                         ]).'" ]}
 

plugins/pianificazione_fatturazione/edit.php

@@ -6,6 +6,10 @@ use Modules\Contratti\Contratto;
 use Modules\Contratti\Stato;
 
 $contratto = Contratto::find($id_record);
+if (empty($contratto)) {
+    return;
+}
+
 $is_pianificabile = $contratto->stato->is_pianificabile && !empty($contratto['data_accettazione']); // Contratto permette la pianificazione
 $is_pianificato = false;
 $stati_pianificabili = Stato::where('is_pianificabile', 1)->get();
@@ -21,9 +25,8 @@ echo '
     <i class="fa fa-warning"></i> '.tr("Tutte le righe del contratto vengono convertite in righe generiche, rendendo impossibile risalire ad eventuali articoli utilizzati all'interno del contratto e pertanto non movimentando il magazzino").'.
 </div>';
 
-if ($contratto->pianificazioni !== NULL) {
+$pianificazioni = $contratto->pianificazioni;
-    $pianificazioni = $contratto->pianificazioni;
+if (!$pianificazioni->isEmpty()) {
-    if (!$pianificazioni->isEmpty()) {
     echo '
     <hr>
     <table class="table table-bordered table-striped table-hover table-condensed">
@@ -94,7 +97,7 @@ if ($contratto->pianificazioni !== NULL) {
     </button>';
 
     echo '<div class="clearfix"></div>';
-    } else {
+} else {
     echo '
     <div class="alert alert-info">
         <i class="fa fa-info-circle"></i> '.tr('Pianificazione della fatturazione non impostata per questo contratto').'.
@@ -104,7 +107,7 @@ if ($contratto->pianificazioni !== NULL) {
         <i class="fa fa-plus"></i> '.tr('Pianifica').'
     </button>
     <div class="clearfix"></div>';
-    }
+}
 
     echo '
     <script type="text/javascript">
@@ -116,4 +119,3 @@ if ($contratto->pianificazioni !== NULL) {
             openModal("Crea fattura", "'.$structure->fileurl('crea_fattura.php').'?id_module='.$id_module.'&id_plugin='.$id_plugin.'&id_record='.$id_record.'&rata=" + rata);
         }
     </script>';
-}

plugins/pianificazione_interventi/edit.php

@@ -6,27 +6,28 @@ use Modules\Contratti\Contratto;
 use Modules\Contratti\Stato;
 
 $contratto = Contratto::find($id_record);
+if (empty($contratto)) {
+    return;
+}
+
 $is_pianificabile = $contratto->stato->is_pianificabile && !empty($contratto['data_accettazione']); // Contratto permette la pianificazione
+$elenco_promemoria = $contratto->promemoria->sortBy('data_richiesta');
 
-if ($contratto->promemoria !== NULL) {
+$stati_pianificabili = Stato::where('is_pianificabile', 1)->get();
-    $elenco_promemoria = $contratto->promemoria->sortBy('data_richiesta');
+$elenco_stati = $stati_pianificabili->implode('descrizione', ', ');
 
-
+echo '
-    $stati_pianificabili = Stato::where('is_pianificabile', 1)->get();
+<p>'.tr('Puoi <b>pianificare dei "promemoria" o direttamente gli interventi</b> da effettuare entro determinate scadenze').'. '.tr('Per poter pianificare i promemoria, il contratto deve avere <b>data accettazione</b> e <b>data conclusione</b> definita ed essere in uno dei seguenti stati: _LINK_', [
-    $elenco_stati = $stati_pianificabili->implode('descrizione', ', ');
-
-    echo '
-    <p>'.tr('Puoi <b>pianificare dei "promemoria" o direttamente gli interventi</b> da effettuare entro determinate scadenze').'. '.tr('Per poter pianificare i promemoria, il contratto deve avere <b>data accettazione</b> e <b>data conclusione</b> definita ed essere in uno dei seguenti stati: _LINK_', [
     '_LINK_' => '<b>'.$elenco_stati.'</b>',
-    ]).'
+]).'
 
-    <span class="tip" title="'.tr("I promemoria verranno visualizzati sulla 'Dashboard' e serviranno per semplificare la pianificazione del giorno dell'intervento, ad esempio nel caso di interventi con cadenza mensile").'">
+<span class="tip" title="'.tr("I promemoria verranno visualizzati sulla 'Dashboard' e serviranno per semplificare la pianificazione del giorno dell'intervento, ad esempio nel caso di interventi con cadenza mensile").'">
     <i class="fa fa-question-circle-o"></i>
-    </span></p>';
+</span></p>';
 
-    echo '
+echo '
-    <hr>
+<hr>
-    <div class="row">
+<div class="row">
     <div class="col-md-9">
         {[ "type": "select", "placeholder": "'.tr('Tipo di promemoria').'", "name": "id_tipo_promemoria", "required": 1, "ajax-source": "tipiintervento", "class": "unblockable" ]}
     </div>
@@ -36,13 +37,13 @@ if ($contratto->promemoria !== NULL) {
             <i class="fa fa-plus"></i> '.tr('Nuovo promemoria').'
         </button>
     </div>
-    </div>
+</div>
-    <hr>';
+<hr>';
 
-    // Nessun intervento pianificato
+// Nessun intervento pianificato
-    if (!$elenco_promemoria->isEmpty()) {
+if (!$elenco_promemoria->isEmpty()) {
     echo '
-    <table class="table table-condensed table-striped table-hover">
+<table class="table table-condensed table-striped table-hover">
     <thead>
         <tr>
             <th>'.tr('Data').'</th>
@@ -152,26 +153,26 @@ if ($contratto->promemoria !== NULL) {
     }
     echo '
     </tbody>
-    </table>';
+</table>';
 
     if (!empty($promemorias)) {
         echo '
-    <br>
+<br>
-    <div class="pull-right">
+<div class="pull-right">
     <button type="button" title="Elimina tutti i promemoria non associati ad intervento" class="btn btn-danger ask tip" data-op="delete-non-associati" data-id_plugin="'.$id_plugin.'" data-backto="record-edit">
         <i class="fa fa-trash"></i> '.tr('Elimina promemoria').'
     </button>
-    </div>';
+</div>';
     }
-    } else {
+} else {
     echo '
-    <div class="alert alert-info">
+<div class="alert alert-info">
     <i class="fa fa-warning"></i> '.tr('Nessun promemoria pianificato per il contratto corrente').'.
-    </div>';
+</div>';
-    }
+}
 
-    echo '
+echo '
-    <script type="text/javascript">
+<script type="text/javascript">
     $("#add_promemoria").click(function() {
         var id_tipo = $("#id_tipo_promemoria").val();
         if (!id_tipo){
@@ -190,5 +191,4 @@ if ($contratto->promemoria !== NULL) {
             buttonRestore("#add_promemoria", restore);
         });
     });
-    </script>';
+</script>';
-}