Stile del codice

2025-02-19 21:10:49 +01:00 · 2021-02-18 18:48:44 +01:00 · 2021-02-18 18:48:44 +01:00 · d8c9670d11
commit d8c9670d11
parent defd5a7358
104 changed files with 313 additions and 378 deletions
--- a/actions.php
+++ b/actions.php
@ -27,7 +27,7 @@ use Modules\Emails\Template;
 use Notifications\EmailNotification;

 if (empty($structure) || empty($structure['enabled'])) {
-    die(tr('Accesso negato'));
+    exit(tr('Accesso negato'));
 }

 $upload_dir = base_dir().'/'.Uploads::getDirectory($id_module, $id_plugin);
--- a/index.php
+++ b/index.php
@ -43,7 +43,6 @@ switch ($op) {
                $_SESSION['period_end'] = date('Y').'-12-31';
            }

-          
            // Rimozione log vecchi
            //$dbo->query('DELETE FROM `zz_operations` WHERE DATE_ADD(`created_at`, INTERVAL 30*24*60*60 SECOND) <= NOW()');
        } else {
--- a/modules/aggiornamenti/database.php
+++ b/modules/aggiornamenti/database.php
@ -128,9 +128,9 @@ if (!empty($results)) {

            foreach ($errors as $name => $diff) {
                if (count($diff) == 1 && array_key_exists('type', $diff) && str_contains($diff['type']['expected'], $diff['type']['current'])) {
-                    $class="info";
+                    $class = 'info';
                } else {
-                    $class="warning";
+                    $class = 'warning';
                }
                echo '
        <tr class="bg-'.$class.'" >
@ -161,11 +161,10 @@ if (!empty($results)) {
    <tbody>';

            foreach ($foreign_keys as $name => $diff) {
-                
                if (count($diff) == 2 && array_key_exists('current', $diff) && $diff['current'] == null) {
-                    $class="info";
+                    $class = 'info';
                } else {
-                    $class="warning";
+                    $class = 'warning';
                }
                echo '
        <tr class="bg-'.$class.'" >
--- a/modules/aggiornamenti/upload_modules.php
+++ b/modules/aggiornamenti/upload_modules.php
@ -22,7 +22,7 @@ include_once __DIR__.'/../../core.php';
 use Util\Zip;

 if (!setting('Attiva aggiornamenti')) {
-    die(tr('Accesso negato'));
+    exit(tr('Accesso negato'));
 }

 if (!extension_loaded('zip')) {
--- a/modules/anagrafiche/edit.php
+++ b/modules/anagrafiche/edit.php
@ -50,7 +50,6 @@ if (!$is_cliente) {

 $nazione_anagrafica = $anagrafica->sedeLegale->nazione;

-
 // Avvisi problemi scheda anagrafica
 $problemi_anagrafica = [];
 if ($is_cliente && empty($record['idconto_cliente'])) {
@ -62,8 +61,8 @@ if ($is_fornitore && empty($record['idconto_fornitore'])){
 }

 if (sizeof($problemi_anagrafica) > 0) {
-    echo '<div class="alert alert-warning"><i class="fa fa-warning"></i> '.tr("Attenzione: _CAMPI_" , [
-    '_CAMPI_' => implode(', ', $problemi_anagrafica)
+    echo '<div class="alert alert-warning"><i class="fa fa-warning"></i> '.tr('Attenzione: _CAMPI_', [
+    '_CAMPI_' => implode(', ', $problemi_anagrafica),
    ]).'</div>';
 }

--- a/modules/anagrafiche/validation.php
+++ b/modules/anagrafiche/validation.php
@ -129,7 +129,7 @@ switch ($name) {
        ])->count() == 0;
        $result = $disponibile;

-        $message = $disponibile ? tr("Questa email non è ancora stata utilizzata") : tr("L'email è già utilizzata in un'altra anagrafica");
+        $message = $disponibile ? tr('Questa email non è ancora stata utilizzata') : tr("L'email è già utilizzata in un'altra anagrafica");

        $errors = [];
        $check = Validate::isValidEmail($value);
--- a/modules/articoli/bulk.php
+++ b/modules/articoli/bulk.php
@ -18,12 +18,12 @@
 */

 include_once __DIR__.'/../../core.php';
-use Modules\Articoli\Articolo;
-use Modules\Preventivi\Preventivo;
-use Modules\Anagrafiche\Anagrafica;
-use Modules\Preventivi\Components\Articolo as ArticoloPreventivo;
-use Modules\TipiIntervento\Tipo as TipoSessione;
 use Modules;
+use Modules\Anagrafiche\Anagrafica;
+use Modules\Articoli\Articolo;
+use Modules\Preventivi\Components\Articolo as ArticoloPreventivo;
+use Modules\Preventivi\Preventivo;
+use Modules\TipiIntervento\Tipo as TipoSessione;
 use Prints;

 switch (post('op')) {
--- a/modules/beni/actions.php
+++ b/modules/beni/actions.php
@ -58,7 +58,6 @@ switch (post('op')) {
        break;

    case 'delete':
-
        $documenti = $dbo->fetchNum('SELECT id FROM dt_ddt WHERE idaspettobeni='.prepare($id_record).'
                     UNION SELECT id FROM co_documenti WHERE idaspettobeni='.prepare($id_record));

--- a/modules/causali/actions.php
+++ b/modules/causali/actions.php
@ -79,7 +79,6 @@ switch (filter('op')) {
        break;

    case 'delete':
-
        $documenti = $dbo->fetchNum('SELECT id FROM dt_ddt WHERE idcausalet='.prepare($id_record).'
                     UNION SELECT id FROM co_documenti WHERE idcausalet='.prepare($id_record));

--- a/modules/contratti/actions.php
+++ b/modules/contratti/actions.php
@ -108,7 +108,6 @@ switch (post('op')) {
        break;

    case 'update_rinnovo':
-
        $contratto->rinnovabile = post('rinnovabile');
        $contratto->rinnovo_automatico = post('rinnovo_automatico');
        $contratto->giorni_preavviso_rinnovo = post('giorni_preavviso_rinnovo');
@ -315,7 +314,6 @@ $riga = $contratto->getRiga($type, $id_riga);

    // eliminazione contratto
    case 'delete':
-
        // Fatture o interventi collegati a questo contratto
        $elementi = $dbo->fetchArray('SELECT 0 AS `codice`, `co_documenti`.`id` AS `id`, `co_documenti`.`numero` AS `numero`, `co_documenti`.`numero_esterno` AS `numero_esterno`,  `co_documenti`.`data`, `co_tipidocumento`.`descrizione` AS `tipo_documento`, `co_tipidocumento`.`dir` AS `dir`  FROM `co_documenti` JOIN `co_tipidocumento` ON `co_tipidocumento`.`id` = `co_documenti`.`idtipodocumento` WHERE `co_documenti`.`id` IN (SELECT `iddocumento` FROM `co_righe_documenti` WHERE `idcontratto` = '.prepare($id_record).')'.'
        UNION
@ -414,7 +412,6 @@ $riga = $contratto->getRiga($type, $id_riga);
        break;

        case 'import':
-
        $rs = $dbo->fetchArray('SELECT * FROM co_contratti_tipiintervento WHERE idcontratto = '.prepare(post('idcontratto')).' AND idtipointervento='.prepare(post('idtipointervento')));

        // Se la riga in_tipiintervento esiste, la aggiorno...
--- a/modules/contratti/src/Contratto.php
+++ b/modules/contratti/src/Contratto.php
@ -207,7 +207,6 @@ class Contratto extends Document
            $intervallo = CarbonInterval::make($this->validita.' '.$this->tipo_validita);
            $data = Carbon::make($this->data_accettazione)->add($intervallo);
            $this->data_conclusione = $data->subDays(1);
-
        }
    }

--- a/modules/dashboard/widgets/notifiche.php
+++ b/modules/dashboard/widgets/notifiche.php
@ -64,7 +64,6 @@ foreach ($moduli as $module_id => $note) {
    </tr>';

    foreach ($note as $nota) {
-
        $class = (strtotime($nota->notification_date) < strtotime(date('Y-m-d')) && !empty($nota->notification_date)) ? 'danger' : '';

        echo '
--- a/modules/ddt/crea_documento.php
+++ b/modules/ddt/crea_documento.php
@ -20,7 +20,7 @@
 include_once __DIR__.'/../../core.php';

 use Modules\DDT\DDT;
-use Modules\Fatture\Fattura;
+
 $documento = DDT::find($id_record);

 $module = Modules::get($id_module);
--- a/modules/fatture/add.php
+++ b/modules/fatture/add.php
@ -36,7 +36,6 @@ $idtipodocumento = $dbo->selectOne('co_tipidocumento', ['id'], [
    'dir' => $dir,
 ])['id'];

-
 ?>
 <form action="" method="post" id="add-form">
 	<input type="hidden" name="op" value="add">
--- a/modules/fatture/ajax/select.php
+++ b/modules/fatture/ajax/select.php
@ -141,5 +141,4 @@ switch ($resource) {
        $results = array_merge($ordini, $ddt);

        break;
-    
 }
--- a/modules/fatture/edit.php
+++ b/modules/fatture/edit.php
@ -362,7 +362,7 @@ elseif ($record['stato'] == 'Bozza') {
                if ($dir == 'uscita') {
                    echo '
                    <div class="col-md-3">
-                        {[ "type": "checkbox", "label": "'.tr("Ritenuta pagata dal fornitore").'", "name": "is_ritenuta_pagata", "value": "$is_ritenuta_pagata$" ]}
+                        {[ "type": "checkbox", "label": "'.tr('Ritenuta pagata dal fornitore').'", "name": "is_ritenuta_pagata", "value": "$is_ritenuta_pagata$" ]}
                    </div>';
                }
                if ($dir == 'entrata') {
--- a/modules/impianti/actions.php
+++ b/modules/impianti/actions.php
@ -134,7 +134,6 @@ switch ($op) {

    // Duplica impianto
    case 'copy':
-
        $dbo->query('CREATE TEMPORARY TABLE tmp SELECT * FROM my_impianti WHERE id= '.prepare($id_record));
        $dbo->query('ALTER TABLE tmp DROP id');
        $dbo->query('INSERT INTO my_impianti SELECT NULL,tmp. * FROM tmp');
--- a/modules/interventi/actions.php
+++ b/modules/interventi/actions.php
@ -108,7 +108,6 @@ switch (post('op')) {
                    }
                }
            }
-
        }
        aggiorna_sedi_movimenti('interventi', $id_record);
        flash()->info(tr('Attività modificata correttamente!'));
--- a/modules/misure/actions.php
+++ b/modules/misure/actions.php
@ -66,7 +66,6 @@ switch (filter('op')) {
        break;

    case 'delete':
-
        $righe = $dbo->fetchNum('SELECT id FROM co_righe_documenti WHERE um='.prepare($record['valore']).'
             UNION SELECT id FROM dt_righe_ddt WHERE um='.prepare($record['valore']).'
             UNION SELECT id FROM or_righe_ordini WHERE um='.prepare($record['valore']).'
--- a/modules/ordini/bulk.php
+++ b/modules/ordini/bulk.php
@ -19,14 +19,13 @@

 include_once __DIR__.'/../../core.php';

-use Modules\Ordini\Ordine;
 use Modules\Fatture\Fattura;
 use Modules\Fatture\Stato;
 use Modules\Fatture\Tipo;
+use Modules\Ordini\Ordine;

 $module_fatture = 'Fatture di vendita';

-
 // Segmenti
 $id_fatture = Modules::get($module_fatture)['id'];
 if (!isset($_SESSION['module_'.$id_fatture]['id_segment'])) {
@ -161,4 +160,5 @@ $operations['cambia_stato'] = [
        'blank' => false,
    ],
 ];
+
    return $operations;
--- a/modules/ordini/buttons.php
+++ b/modules/ordini/buttons.php
@ -36,25 +36,20 @@ echo '
            </a>
        </li>';

-
    if ($dir == 'entrata') {
-
        echo '
        <li>
            <a data-href="'.$structure->fileurl('crea_documento.php').'?id_module='.$id_module.'&id_record='.$id_record.'&documento=ordine_fornitore" data-toggle="modal" data-title="'.tr('Crea ordine fornitore').'">
                <i class="fa fa-file-o"></i> '.tr('Ordine fornitore').'
            </a>
        </li>';
-
    } else {
-
        /*echo '
        <li>
            <a data-href="'.$structure->fileurl('crea_documento.php').'?id_module='.$id_module.'&id_record='.$id_record.'&documento=ordine_cliente" data-toggle="modal" data-title="'.tr('Crea ordine cliente').'">
                <i class="fa fa-file-o"></i> '.tr('Ordine cliente').'
            </a>
        </li>';*/
-
    }

    echo '  
--- a/modules/ordini/row-list.php
+++ b/modules/ordini/row-list.php
@ -78,7 +78,6 @@ foreach ($righe as $riga) {
        <i class="fa fa-chevron-right"></i> '.tr('Riferimenti').' '.$testo_aggiuntivo.'
    </button>';

-
    // Aggiunta dei riferimenti ai documenti
    if ($riga->hasOriginalComponent()) {
        echo '
--- a/modules/relazioni_anagrafiche/actions.php
+++ b/modules/relazioni_anagrafiche/actions.php
@ -68,7 +68,6 @@ switch (filter('op')) {
        break;

    case 'delete':
-
        $righe = $dbo->fetchNum('SELECT idanagrafica FROM an_anagrafiche WHERE idrelazione='.prepare($id_record));

        if (isset($id_record) && empty($righe)) {
--- a/modules/stati_contratto/actions.php
+++ b/modules/stati_contratto/actions.php
@ -34,7 +34,6 @@ switch (post('op')) {
        break;

    case 'add':
-
        $descrizione = post('descrizione');
        $icona = post('icona');
        $is_completato = post('is_completato') ?: null;
@ -54,7 +53,6 @@ switch (post('op')) {
        break;

    case 'delete':
-
        //scelgo se settare come eliminato o cancellare direttamente la riga se non è stato utilizzato nei contratti
        if (count($dbo->fetchArray('SELECT id FROM co_contratti WHERE idstato='.prepare($id_record))) > 0) {
            $query = 'UPDATE co_staticontratti SET deleted_at = NOW() WHERE can_delete = 1 AND id='.prepare($id_record);
--- a/modules/stati_intervento/actions.php
+++ b/modules/stati_intervento/actions.php
@ -58,7 +58,6 @@ switch (post('op')) {
        break;

    case 'delete':
-
        //scelgo se settare come eliminato o cancellare direttamente la riga se non è stato utilizzato negli interventi
        if (count($dbo->fetchArray('SELECT id FROM in_interventi WHERE idstatointervento='.prepare($id_record))) > 0) {
            $query = 'UPDATE in_statiintervento SET deleted_at = NOW() WHERE idstatointervento='.prepare($id_record).' AND `can_delete`=1';
--- a/modules/stati_preventivo/actions.php
+++ b/modules/stati_preventivo/actions.php
@ -35,7 +35,6 @@ switch (post('op')) {
        break;

    case 'add':
-
        $descrizione = post('descrizione');
        $icona = post('icona');
        $is_completato = post('is_completato') ?: null;
@ -55,7 +54,6 @@ switch (post('op')) {
        break;

    case 'delete':
-
        //scelgo se settare come eliminato o cancellare direttamente la riga se non è stato utilizzato nei preventivi
        if (count($dbo->fetchArray('SELECT id FROM co_preventivi WHERE idstato='.prepare($id_record))) > 0) {
            $query = 'UPDATE co_statipreventivi SET deleted_at = NOW() WHERE can_delete = 1 AND id='.prepare($id_record);
--- a/modules/statistiche/edit.php
+++ b/modules/statistiche/edit.php
@ -332,7 +332,6 @@ $(document).ready(function() {
 });
 </script>';

-
 // Interventi per tecnico
 $tecnici = $dbo->fetchArray("SELECT an_anagrafiche.idanagrafica AS id, ragione_sociale, colore FROM an_anagrafiche
 INNER JOIN
@ -346,7 +345,6 @@ ORDER BY ragione_sociale ASC");

 $dataset = '';
 foreach ($tecnici as $tecnico) {
-
    $sessioni = $dbo->fetchArray('SELECT SUM(in_interventi_tecnici.ore) AS result, CONCAT(CAST(SUM(in_interventi_tecnici.ore) AS char(20)),\' ore\') AS ore_lavorate, YEAR(in_interventi_tecnici.orario_inizio) AS year, MONTH(in_interventi_tecnici.orario_inizio) AS month FROM in_interventi_tecnici  INNER JOIN `in_interventi` ON `in_interventi_tecnici`.`idintervento` = `in_interventi`.`id` LEFT JOIN `in_statiintervento` ON `in_interventi`.`idstatointervento`=`in_statiintervento`.`idstatointervento` WHERE in_interventi_tecnici.idtecnico = '.prepare($tecnico['id']).' AND in_interventi_tecnici.orario_inizio BETWEEN '.prepare($start).' AND '.prepare($end).' AND `in_statiintervento`.`is_completato` = 1 GROUP BY YEAR(in_interventi_tecnici.orario_inizio), MONTH(in_interventi_tecnici.orario_inizio) ORDER BY YEAR(in_interventi_tecnici.orario_inizio) ASC, MONTH(in_interventi_tecnici.orario_inizio) ASC');

    $sessioni = Stats::monthly($sessioni, $start, $end);
@ -366,8 +364,6 @@ foreach ($tecnici as $tecnico) {
        ],

    },';
-
-
 }

 echo '
--- a/modules/stato_email/actions.php
+++ b/modules/stato_email/actions.php
@ -22,7 +22,6 @@ use Notifications\EmailNotification;

 switch (post('op')) {
    case 'send':
-
        $email = EmailNotification::build($mail);

        // Invio mail
--- a/modules/stato_servizi/edit.php
+++ b/modules/stato_servizi/edit.php
@ -50,7 +50,6 @@ if (Services::isEnabled()) {
    $servizi = Cache::pool('Informazioni su Services')->content;

    if (!empty($servizi)) {
-
        // Elaborazione dei servizi in scadenza
        $limite_scadenze = (new Carbon())->addDays(60);
        $servizi_in_scadenza = [];
@ -126,12 +125,9 @@ if (Services::isEnabled()) {
            });
        });
        </script>';
-
    } else {
-
        echo '
        <div class="col-md-12 col-lg-6"><div class="alert alert-warning alert-dismissible" role="alert"><button class="close" type="button" data-dismiss="alert" aria-hidden="true"><span aria-hidden="true">×</span><span class="sr-only">'.tr('Chiudi').'</span></button><span><i class="fa fa-warning"></i> '.tr('Nessun servizio abilitato o "OSMCloud Services API Token" non valido').'.</span></div></div>';
-
    }
 }

--- a/modules/tipi_documento/actions.php
+++ b/modules/tipi_documento/actions.php
@ -27,7 +27,6 @@ switch (filter('op')) {

        if (isset($descrizione) && isset($dir) && isset($codice_tipo_documento_fe)) {
            if ($dbo->fetchNum('SELECT * FROM `co_tipidocumento` WHERE `dir`='.prepare($dir).' AND `codice_tipo_documento_fe`='.prepare($codice_tipo_documento_fe).' AND `id`!='.prepare($id_record)) == 0) {
-
                $predefined = post('predefined');
                if (!empty($predefined)) {
                    $dbo->query('UPDATE co_tipidocumento SET predefined = 0 WHERE dir = '.prepare($dir));
@ -87,7 +86,6 @@ switch (filter('op')) {
        break;

    case 'delete':
-
        $documenti = $dbo->fetchNum('SELECT id FROM co_documenti WHERE idtipodocumento ='.prepare($id_record));

        if (isset($id_record) && empty($documenti)) {
@ -96,7 +94,6 @@ switch (filter('op')) {
                '_TYPE_' => 'tipo documento',
            ]));
        } else {
-
            $dbo->update('co_tipidocumento', [
                'deleted_at' => date(),
                'predefined' => 0,
@ -107,7 +104,6 @@ switch (filter('op')) {
                '_TYPE_' => 'tipo documento',
            ]));

-
            //flash()->error(tr('Sono presenti dei documenti collegati a questo tipo documento'));
        }

--- a/modules/tipi_intervento/actions.php
+++ b/modules/tipi_intervento/actions.php
@ -66,7 +66,6 @@ switch (post('op')) {
        break;

    case 'delete':
-
        // Permetto eliminazione tipo intervento solo se questo non è utilizzado da nessun'altra parte a gestionale
        // UNION SELECT `in_tariffe`.`idtipointervento` FROM `in_tariffe` WHERE `in_tariffe`.`idtipointervento` = '.prepare($id_record).'
        // UNION SELECT `co_contratti_tipiintervento`.`idtipointervento` FROM `co_contratti_tipiintervento` WHERE `co_contratti_tipiintervento`.`idtipointervento` = '.prepare($id_record).'
--- a/modules/tipi_scadenze/actions.php
+++ b/modules/tipi_scadenze/actions.php
@ -99,7 +99,6 @@ switch (filter('op')) {
        break;

    case 'delete':
-
        $documenti = $dbo->fetchNum('SELECT id FROM co_scadenziario WHERE tipo = (SELECT nome FROM co_tipi_scadenze WHERE id = '.prepare($id_record).')');

        if (isset($id_record) && empty($documenti)) {
--- a/plugins/importFE/generate.php
+++ b/plugins/importFE/generate.php
@ -286,7 +286,7 @@ echo '
        if (!empty($ritenuta)) {
            echo '
            <div class="col-md-3">
-                {[ "type": "checkbox", "label": "'.tr('Ritenuta pagata dal fornitore').'", "name": "is_ritenuta_pagata", "value": 0, "help": "'.tr("Attivare se la ritenuta è stata pagata dal fornitore").'" ]}
+                {[ "type": "checkbox", "label": "'.tr('Ritenuta pagata dal fornitore').'", "name": "is_ritenuta_pagata", "value": 0, "help": "'.tr('Attivare se la ritenuta è stata pagata dal fornitore').'" ]}
            </div>';
        }
    echo '
--- a/plugins/importFE/src/FatturaOrdinaria.php
+++ b/plugins/importFE/src/FatturaOrdinaria.php
@ -19,13 +19,12 @@

 namespace Plugins\ImportFE;

+use Modules\Anagrafiche\Anagrafica;
 use Modules\Articoli\Articolo as ArticoloOriginale;
 use Modules\Articoli\Categoria;
 use Modules\Fatture\Components\Articolo;
 use Modules\Fatture\Components\Riga;
 use Modules\Fatture\Fattura;
-use Modules\Anagrafiche\Anagrafica;
-use Plugins\DettagliArticolo\DettaglioFornitore;
 use Plugins\DettagliArticolo\DettaglioPrezzo;
 use UnexpectedValueException;
 use Util\XML;
--- a/plugins/pianificazione_fatturazione/actions.php
+++ b/plugins/pianificazione_fatturazione/actions.php
@ -17,9 +17,9 @@
 * along with this program. If not, see <https://www.gnu.org/licenses/>.
 */

-use Modules\Contratti\Components\Riga;
-use Modules\Contratti\Components\Articolo;
 use Modules\Articoli\Articolo as ArticoloOriginale;
+use Modules\Contratti\Components\Articolo;
+use Modules\Contratti\Components\Riga;
 use Modules\Contratti\Contratto;
 use Modules\Fatture\Fattura;
 use Modules\Fatture\Tipo;
@ -99,8 +99,8 @@ switch ($operazione) {
                $qta_evasa = $r->qta_evasa;
                $data_scadenza = '';
                $inizio = $date_pianificazioni[0];
-                $fine = date("Y-m-d", strtotime($inizio.' -1 days'));
-                $fine = date("Y-m-d", strtotime($fine." ".$timeing));
+                $fine = date('Y-m-d', strtotime($inizio.' -1 days'));
+                $fine = date('Y-m-d', strtotime($fine.' '.$timeing));
                for ($rata = 1; $rata <= $numero_fatture; ++$rata) {
                    if ($qta_evasa < $r->qta) {
                        $qta_riga = ($qta[$r->id] <= ($r->qta - $qta_evasa) ? $qta[$r->id] : ($r->qta - $qta_evasa));
@ -109,8 +109,8 @@ switch ($operazione) {
                        $descrizione = variables($descrizione, $inizio, $fine)['descrizione'];

                        $inizio = $fine;
-                        $fine = date("Y-m-d", strtotime($timeing, strtotime($inizio)));
-                        $inizio = date("Y-m-d", strtotime($inizio.' +1 days'));
+                        $fine = date('Y-m-d', strtotime($timeing, strtotime($inizio)));
+                        $inizio = date('Y-m-d', strtotime($inizio.' +1 days'));

                        $prezzo_unitario = ($r->subtotale / $r->qta);

--- a/plugins/pianificazione_fatturazione/add_pianificazione.php
+++ b/plugins/pianificazione_fatturazione/add_pianificazione.php
@ -106,7 +106,7 @@ $righe = $contratto->getRighe();

 echo '
 <div class="alert alert-info">
-    <p>'.tr("Puoi utilizzare le seguenti variabili nella descrizione delle righe").':</p>'.variables()['list'].'
+    <p>'.tr('Puoi utilizzare le seguenti variabili nella descrizione delle righe').':</p>'.variables()['list'].'
 </div>';

 foreach ($righe as $riga) {
--- a/plugins/pianificazione_fatturazione/ajax_cadenza.php
+++ b/plugins/pianificazione_fatturazione/ajax_cadenza.php
@ -54,13 +54,13 @@ echo '

    while ($data_corrente->lessThanOrEqualTo($data_conclusione)) {
        $data = $data_corrente->endOfMonth()->format('Y-m-d');
-        $data_fatturazione = ($data_fatturazione ?: date("Y-m", strtotime($data)) );
+        $data_fatturazione = ($data_fatturazione ?: date('Y-m', strtotime($data)));
        unset($checked);

        if ($id_module == Modules::get('Contratti')['id']) {
-            if( $data==date("Y-m-t", strtotime($timeing, strtotime($data_fatturazione)) ) ){
+            if ($data == date('Y-m-t', strtotime($timeing, strtotime($data_fatturazione)))) {
                $checked = 'checked';
-                $data_fatturazione = date("Y-m", strtotime($data));
+                $data_fatturazione = date('Y-m', strtotime($data));
            }
        }

--- a/plugins/pianificazione_fatturazione/crea_fattura.php
+++ b/plugins/pianificazione_fatturazione/crea_fattura.php
@ -33,7 +33,7 @@ $numero_rata = $contratto->pianificazioni->search(function ($item) use ($id_pian
 $module_fattura = Modules::get('Fatture di vendita');

 $id_conto = setting('Conto predefinito fatture di vendita');
-$data = date("Y-m",strtotime($pianificazione->data_scadenza))."-".date("d",strtotime($contratto->data_accettazione));
+$data = date('Y-m', strtotime($pianificazione->data_scadenza)).'-'.date('d', strtotime($contratto->data_accettazione));

 echo '
 <form action="" method="post">
--- a/plugins/pianificazione_fatturazione/modutil.php
+++ b/plugins/pianificazione_fatturazione/modutil.php
@ -19,18 +19,18 @@
            '12' => 'Dicembre',
        ];

-        $result['list'] = "<ul>
+        $result['list'] = '<ul>
                <li><code>{periodo}</code></li>
                <li><code>{data_inizio}</code></li>
                <li><code>{data_fine}</code></li>
                <li><code>{mese_fatturazione}</code></li>
-            </ul>";
+            </ul>';

        if (!empty($descrizione)) {
-                $result['descrizione'] = str_replace("{periodo}","durata dal ".Translator::dateToLocale($inizio)." al ".Translator::dateToLocale($fine),$descrizione);
-                $result['descrizione'] = str_replace("{data_inizio}",Translator::dateToLocale($inizio), $result['descrizione']);
-                $result['descrizione'] = str_replace("{data_fine}",Translator::dateToLocale($fine), $result['descrizione']);
-                $result['descrizione'] = str_replace("{mese_fatturazione}", $mese[date('m',strtotime($inizio))], $result['descrizione']);
+            $result['descrizione'] = str_replace('{periodo}', 'durata dal '.Translator::dateToLocale($inizio).' al '.Translator::dateToLocale($fine), $descrizione);
+            $result['descrizione'] = str_replace('{data_inizio}', Translator::dateToLocale($inizio), $result['descrizione']);
+            $result['descrizione'] = str_replace('{data_fine}', Translator::dateToLocale($fine), $result['descrizione']);
+            $result['descrizione'] = str_replace('{mese_fatturazione}', $mese[date('m', strtotime($inizio))], $result['descrizione']);
        }

        return $result;
--- a/plugins/revisioni/actions.php
+++ b/plugins/revisioni/actions.php
@ -23,7 +23,6 @@ $operazione = filter('op');

 switch ($operazione) {
    case 'edit_revision':
-
        $master_revision = post('master_revision');
        $id_record = post('idrevisione');
        //Tolgo il flag default_revision da tutte le revisioni e dal record_principale
@ -34,7 +33,6 @@ switch ($operazione) {
        break;

    case 'delete_revision':
-
        $idrevisione = post('idrevisione');
        $dbo->query('DELETE FROM co_preventivi WHERE id='.prepare($idrevisione));

--- a/src/API/App/v1/Clienti.php
+++ b/src/API/App/v1/Clienti.php
@ -109,7 +109,6 @@ class Clienti extends AppResource

    public function createRecord($data)
    {
-
        $ragione_sociale = $data['ragione_sociale'];
        $id_tipo = [1];

@ -133,7 +132,6 @@ class Clienti extends AppResource
        return [
            'id' => $id_record,
        ];
-
    }

    public function updateRecord($data)
@ -163,5 +161,4 @@ class Clienti extends AppResource
        $record->cellulare = $data['cellulare'];
        $record->email = $data['email'];
    }
-
 }
--- a/src/API/App/v1/RigheInterventi.php
+++ b/src/API/App/v1/RigheInterventi.php
@ -224,7 +224,6 @@ class RigheInterventi extends AppResource
        $record->setPrezzoUnitario($data['prezzo_unitario'], $data['id_iva']);
        $record->setSconto($data['sconto_percentuale'] ?: $data['sconto_unitario'], $data['tipo_sconto']);

-
        try {
            $record->qta = $data['qta'];
        } catch (UnexpectedValueException $e) {
--- a/src/Permissions.php
+++ b/src/Permissions.php
@ -98,7 +98,7 @@ class Permissions
                }

                if (!$result && $die) {
-                    die(tr('Accesso negato'));
+                    exit(tr('Accesso negato'));
                }
            }
        }
--- a/src/Traits/RecordTrait.php
+++ b/src/Traits/RecordTrait.php
@ -39,7 +39,8 @@ trait RecordTrait
    /**
     * @param string $name
     */
-    public function customField($name) {
+    public function customField($name)
+    {
        $field = database()->table('zz_fields')
            ->leftJoin('zz_field_record', 'zz_fields.id', '=', 'zz_field_record.id_field')
                ->where('zz_fields.name', '=', $name)
--- a/src/Update.php
+++ b/src/Update.php
@ -488,8 +488,6 @@ class Update
            }
        }

-        $results;
-
        return $results;
    }

--- a/templates/barcode/body.php
+++ b/templates/barcode/body.php
@ -33,7 +33,6 @@ echo '<style>
 }
 </style>';

-
 if (!empty($_SESSION['superselect']['id_articolo_barcode'])) {
    $articoli = Articolo::whereIn('id', $_SESSION['superselect']['id_articolo_barcode'])->get();
    unset($_SESSION['superselect']['id_articolo_barcode']);
@ -51,10 +50,9 @@ foreach( $articoli as $articolo ){
        <p><b>'.$articolo->barcode.'</b></p>
    </div>';

-    $page++;
+    ++$page;

    if ($page < $pages) {
        echo '<pagebreak>';
    }
-
 }
--- a/templates/bilancio/body.php
+++ b/templates/bilancio/body.php
@ -59,16 +59,15 @@ echo '
                                        <td class="text-right">'.numberFormat($liv3_p['totale']).'</td>
                                    </tr>';
                                }
-                                
                            }
-                            if($liv2_p['descrizione']=="Crediti clienti e crediti diversi"){
+                            if ($liv2_p['descrizione'] == 'Crediti clienti e crediti diversi') {
                                echo '
                                    <tr>
                                        <td></td>
                                        <td>Clienti</td>
                                        <td class="text-right">'.numberFormat($crediti_clienti).'</td>
                                    </tr>';
-                            } elseif($liv2_p['descrizione']=="Debiti fornitori e debiti diversi") {
+                            } elseif ($liv2_p['descrizione'] == 'Debiti fornitori e debiti diversi') {
                                echo '
                                <tr>
                                    <td></td>
@ -136,14 +135,14 @@ echo '
                                    </tr>';
                                }
                            }
-                            if($liv2_p['descrizione']=="Crediti clienti e crediti diversi"){
+                            if ($liv2_p['descrizione'] == 'Crediti clienti e crediti diversi') {
                                echo '
                                    <tr>
                                        <td></td>
                                        <td>Clienti</td>
                                        <td class="text-right">'.numberFormat(abs($crediti_clienti)).'</td>
                                    </tr>';
-                            } elseif($liv2_p['descrizione']=="Debiti fornitori e debiti diversi") {
+                            } elseif ($liv2_p['descrizione'] == 'Debiti fornitori e debiti diversi') {
                                echo '
                                <tr>
                                    <td></td>
--- a/templates/bilancio/header.php
+++ b/templates/bilancio/header.php
@ -17,7 +17,6 @@
 * along with this program. If not, see <https://www.gnu.org/licenses/>.
 */

-
 echo '
 <h5 style="border-bottom:1px solid #777; display:block;">
    <div class="col-xs-5">STAMPA BILANCIO <small>'.dateFormat($date_start).' - '.dateFormat($date_end).'</small></div>
--- a/templates/bilancio/settings.php
+++ b/templates/bilancio/settings.php
@ -1,9 +1,7 @@
 <?php

 return [
-   
    'format' => 'A4',
    'orientation' => 'P',
    'font-size' => '9px',
- 
 ];
--- a/templates/ddt/init.php
+++ b/templates/ddt/init.php
@ -93,5 +93,5 @@ $custom = [
 // - utente qualsiasi con permessi almeno in lettura sul modulo
 // - admin
 if ((Auth::user()['gruppo'] == 'Clienti' && $id_cliente != Auth::user()['idanagrafica'] && !Auth::admin()) || Modules::getPermission($documento->module) == '-') {
-    die(tr('Non hai i permessi per questa stampa!'));
+    exit(tr('Non hai i permessi per questa stampa!'));
 }
--- a/templates/fatture/init.php
+++ b/templates/fatture/init.php
@ -118,7 +118,7 @@ $custom = [
 // - utente qualsiasi con permessi almeno in lettura sul modulo
 // - admin
 if ((Auth::user()['gruppo'] == 'Clienti' && $id_cliente != Auth::user()['idanagrafica'] && !Auth::admin()) || Modules::getPermission($module_name) == '-') {
-    die(tr('Non hai i permessi per questa stampa!'));
+    exit(tr('Non hai i permessi per questa stampa!'));
 }

 if ($fattura_accompagnatoria) {
--- a/templates/ordini/body.php
+++ b/templates/ordini/body.php
@ -31,17 +31,14 @@ if ($documento->direzione == 'entrata'){
    }) !== false;

    if ($has_image) {
-        $columns++;
+        ++$columns;
        $char_number = $options['pricing'] ? 26 : 63;
    }
 }

-
-
 if ($documento->direzione == 'uscita') {
-    $columns++;
+    ++$columns;
    $char_number = $options['pricing'] ? 26 : 63;
-   
 } else {
    $char_number = $options['pricing'] ? 45 : 82;
 }
@ -98,17 +95,13 @@ foreach ($righe as $riga) {
                '.$num.'
            </td>';

-
    if ($documento->direzione == 'uscita') {
-
        echo '
            <td class="text-center" style="vertical-align: middle">
                '.$riga->articolo->codice.'
            </td>';
-
    }

-
    if ($has_image) {
        if ($riga->isArticolo() && !empty($riga->articolo->image)) {
            echo '
--- a/templates/partitario_mastrino/bottom.php
+++ b/templates/partitario_mastrino/bottom.php
@ -19,8 +19,6 @@

 include_once __DIR__.'/../../core.php';

-
-
 if (get('lev') == '2' || get('lev') == '3') {
    $dare = 0;
    $avere = 0;
--- a/tests/acceptance/AnagraficheCest.php
+++ b/tests/acceptance/AnagraficheCest.php
@ -2,7 +2,7 @@

 class AnagraficheCest
 {
-    public function _before(\AcceptanceTester $t)
+    public function _before(AcceptanceTester $t)
    {
        // Effettua l'accesso con le credenziali fornite
        $t->login('admin', 'admin');
--- a/tests/acceptance/BackupCest.php
+++ b/tests/acceptance/BackupCest.php
@ -2,7 +2,7 @@

 class BackupCest
 {
-    public function _before(\AcceptanceTester $t)
+    public function _before(AcceptanceTester $t)
    {
        // Effettua l'accesso con le credenziali fornite
        $t->login('admin', 'admin');
--- a/tests/acceptance/ContrattiCest.php
+++ b/tests/acceptance/ContrattiCest.php
@ -9,7 +9,7 @@ class ContrattiCest
     */
    protected $rowHelper;

-    public function _before(\AcceptanceTester $t)
+    public function _before(AcceptanceTester $t)
    {
        // Effettua l'accesso con le credenziali fornite
        $t->login('admin', 'admin');
--- a/tests/acceptance/DDTCest.php
+++ b/tests/acceptance/DDTCest.php
@ -9,7 +9,7 @@ class DDTCest
     */
    protected $rowHelper;

-    public function _before(\AcceptanceTester $t)
+    public function _before(AcceptanceTester $t)
    {
        // Effettua l'accesso con le credenziali fornite
        $t->login('admin', 'admin');
--- a/tests/acceptance/FattureCest.php
+++ b/tests/acceptance/FattureCest.php
@ -9,7 +9,7 @@ class FattureCest
     */
    protected $rowHelper;

-    public function _before(\AcceptanceTester $t)
+    public function _before(AcceptanceTester $t)
    {
        // Effettua l'accesso con le credenziali fornite
        $t->login('admin', 'admin');
--- a/tests/acceptance/OrdiniCest.php
+++ b/tests/acceptance/OrdiniCest.php
@ -9,7 +9,7 @@ class OrdiniCest
     */
    protected $rowHelper;

-    public function _before(\AcceptanceTester $t)
+    public function _before(AcceptanceTester $t)
    {
        // Effettua l'accesso con le credenziali fornite
        $t->login('admin', 'admin');
--- a/tests/acceptance/PreventiviCest.php
+++ b/tests/acceptance/PreventiviCest.php
@ -9,7 +9,7 @@ class PreventiviCest
     */
    protected $rowHelper;

-    public function _before(\AcceptanceTester $t)
+    public function _before(AcceptanceTester $t)
    {
        // Effettua l'accesso con le credenziali fornite
        $t->login('admin', 'admin');