Stile del codice

config/csrf_config.php

@@ -17,7 +17,7 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-/**
+/*
  * Configuration file for CSRF Protector.
  */
 return [

cron.php

@@ -17,7 +17,7 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-/**
+/*
  * Script dedicato alla gestione delle operazioni di cron ricorrenti del gestionale.
  * Una volta attivato, questo script rimane attivo in background per gestire l'esecuzione delle diverse operazioni come pianificate nella tabella zz_tasks.
  *

editor.php

@@ -118,9 +118,9 @@ if (empty($record) || !$has_access) {
                 <script>
                 $( document ).ready(function() {';
 
-                    if (!empty($hide_left_sidebar)){
+    if (!empty($hide_left_sidebar)) {
-    echo '              $(".control-sidebar").removeClass("control-sidebar-shown");  $("aside.content-wrapper, .main-footer").toggleClass("with-control-sidebar");';
+        echo '              $(".control-sidebar").removeClass("control-sidebar-shown");  $("aside.content-wrapper, .main-footer").toggleClass("with-control-sidebar");';
-                    }
+    }
 
     echo '
                     $(".control-sidebar-toggle").bind("click", function() {

lib/common.php

@@ -17,7 +17,7 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-/**
+/*
  * Funzioni globali utilizzate per il funzionamento dei componenti indipendenti del progetto (moduli, plugin, stampe, ...).
  *
  * @since 2.4.2

lib/helpers.php

@@ -17,7 +17,7 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-/**
+/*
  * Funzioni di aiuto per la semplificazione del codice.
  *
  * @since 2.4.2

modules/articoli/actions.php

@@ -149,7 +149,7 @@ switch (post('op')) {
             foreach ($contenuto_componente as $key => $value) {
                 //Fix per nomi con spazi che vengono tradotti con "_" (es. Data_di_installazione)
                 $key = preg_replace('/\s+/', '_', $key);
-                
+
                 $valore = $contenuto_precedente_esistente ? filter($key) : $value['valore'];
 
                 $campi_componente[$key] = $valore;

modules/articoli/ajax/select.php

@@ -60,11 +60,11 @@ switch ($resource) {
             NULL AS id_dettaglio_fornitore,';
         }
 
-        if ($usare_iva_anagrafica){
+        if ($usare_iva_anagrafica) {
             $query .= '
             IFNULL(iva_anagrafica.id, IFNULL(iva_predefinita.id, iva_articolo.id)) AS idiva_vendita,
             IFNULL(iva_anagrafica.descrizione, IFNULL(iva_predefinita.descrizione, iva_articolo.descrizione)) AS iva_vendita,';
-        }else{
+        } else {
             $query .= '
             IFNULL(iva_predefinita.id, iva_articolo.id) AS idiva_vendita,
             IFNULL(iva_predefinita.descrizione, iva_articolo.descrizione) AS iva_vendita,';
@@ -95,9 +95,9 @@ switch ($resource) {
             LEFT JOIN co_iva AS iva_predefinita ON iva_predefinita.id = (SELECT valore FROM zz_settings WHERE nome = 'Iva predefinita')";
 
         if ($usare_iva_anagrafica) {
-            $query .= "
+            $query .= '
-            LEFT JOIN co_iva AS iva_anagrafica ON iva_anagrafica.id = (SELECT idiva_vendite FROM an_anagrafiche WHERE idanagrafica = ".prepare($superselect['idanagrafica']).')';
+            LEFT JOIN co_iva AS iva_anagrafica ON iva_anagrafica.id = (SELECT idiva_vendite FROM an_anagrafiche WHERE idanagrafica = '.prepare($superselect['idanagrafica']).')';
-            }
+        }
 
             $query .= '
 
@@ -151,13 +151,13 @@ switch ($resource) {
         // IVA da impostazioni
         foreach ($rs as $k => $r) {
             // Lettura movimenti delle mie sedi
-            $qta_azienda = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_azienda WHERE mg_movimenti.idarticolo = ".prepare($r['id']).' AND idsede_azienda = '.prepare($superselect['idsede_partenza']));
+            $qta_azienda = $dbo->fetchOne('SELECT SUM(mg_movimenti.qta) AS qta FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_azienda WHERE mg_movimenti.idarticolo = '.prepare($r['id']).' AND idsede_azienda = '.prepare($superselect['idsede_partenza']));
 
             // Lettura eventuali movimenti ad una propria sede (nel caso di movimenti fra sedi della mia azienda) per il calcolo corretto delle quantità
             if ($superselect['idsede_partenza'] != 0) {
-                $qta_controparte = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte WHERE mg_movimenti.idarticolo = ".prepare($r['id']).' AND idsede_controparte = '.prepare($superselect['idsede_partenza']));
+                $qta_controparte = $dbo->fetchOne('SELECT SUM(mg_movimenti.qta) AS qta FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte WHERE mg_movimenti.idarticolo = '.prepare($r['id']).' AND idsede_controparte = '.prepare($superselect['idsede_partenza']));
             } else {
-                $qta_controparte = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta FROM ((( mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte ) LEFT JOIN dt_ddt ON mg_movimenti.idddt = dt_ddt.id ) LEFT JOIN co_documenti ON mg_movimenti.iddocumento = co_documenti.id ) WHERE mg_movimenti.idarticolo = ".prepare($r['id']).' AND idsede_controparte = '.prepare($superselect['idsede_partenza']).' AND IFNULL(dt_ddt.idanagrafica, co_documenti.idanagrafica) = '.prepare(setting('Azienda predefinita')));
+                $qta_controparte = $dbo->fetchOne('SELECT SUM(mg_movimenti.qta) AS qta FROM ((( mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte ) LEFT JOIN dt_ddt ON mg_movimenti.idddt = dt_ddt.id ) LEFT JOIN co_documenti ON mg_movimenti.iddocumento = co_documenti.id ) WHERE mg_movimenti.idarticolo = '.prepare($r['id']).' AND idsede_controparte = '.prepare($superselect['idsede_partenza']).' AND IFNULL(dt_ddt.idanagrafica, co_documenti.idanagrafica) = '.prepare(setting('Azienda predefinita')));
             }
 
             $qta = $qta_azienda['qta'] - $qta_controparte['qta'];

modules/articoli/edit.php

@@ -175,7 +175,7 @@ include_once __DIR__.'/../../core.php';
                         </div>
 
                         <div class="col-md-4">
-                            {[ "type": "number", "label": "<?php echo tr('Q.tà multipla'); ?>", "name": "qta_multipla", "value": "$qta_multipla$", "help": "<?php echo tr("Quantità multipla di scorta da tenere a magazzino. Se lasciato a 0, la quantità proposta da riordinare verrà gestita con la semplice differenza fra quantità richiesta e quantità disponibile."); ?>" ]}
+                            {[ "type": "number", "label": "<?php echo tr('Q.tà multipla'); ?>", "name": "qta_multipla", "value": "$qta_multipla$", "help": "<?php echo tr('Quantità multipla di scorta da tenere a magazzino. Se lasciato a 0, la quantità proposta da riordinare verrà gestita con la semplice differenza fra quantità richiesta e quantità disponibile.'); ?>" ]}
                         </div>
                     </div>
                 </div>

modules/banche/add.php

@@ -21,8 +21,7 @@ include_once __DIR__.'/../../core.php';
 
 $id_anagrafica = filter('id_anagrafica');
 
-$help_codice_bic = tr("Il codice BIC (o SWIFT) è composto da 8 a 11 caratteri (lettere e numeri) ed è suddiviso in:").'<br><br><ul><li>'.tr('AAAA - codice bancario').'</li><li>'.tr('BB - codice ISO della nazione').'</li><li>'.tr('CC - codice città presso la quale è ubicata la banca').'</li><li>'.tr('DD - codice della filiale (opzionale)').'</li></ul>';
+$help_codice_bic = tr('Il codice BIC (o SWIFT) è composto da 8 a 11 caratteri (lettere e numeri) ed è suddiviso in:').'<br><br><ul><li>'.tr('AAAA - codice bancario').'</li><li>'.tr('BB - codice ISO della nazione').'</li><li>'.tr('CC - codice città presso la quale è ubicata la banca').'</li><li>'.tr('DD - codice della filiale (opzionale)').'</li></ul>';
-
 
 echo '
 <form action="" method="post" id="add-form">

modules/banche/edit.php

@@ -68,7 +68,7 @@ include_once __DIR__.'/../../core.php';
                 </div>
 
                 <div class="col-md-6">
-                    {[ "type": "text", "label": "<?php echo tr('Codice SIA'); ?>", "name": "codice_sia", "class": "alphanumeric-mask", "maxlength": 5, "value": "$codice_sia$", "help":"<?php echo tr('Questo campo è necessario per la generazione delle Ri.Ba.');?>" ]}
+                    {[ "type": "text", "label": "<?php echo tr('Codice SIA'); ?>", "name": "codice_sia", "class": "alphanumeric-mask", "maxlength": 5, "value": "$codice_sia$", "help":"<?php echo tr('Questo campo è necessario per la generazione delle Ri.Ba.'); ?>" ]}
                 </div>
             </div>
 

modules/ddt/edit.php

@@ -171,9 +171,10 @@ if ($module['name'] == 'Ddt di vendita') {
 				</div>
 
 				<div class="col-md-3">
-                    <?php 
+                    <?php
-                        if (!empty($record['idvettore']))
+                        if (!empty($record['idvettore'])) {
                             echo Modules::link('Anagrafiche', $record['idvettore'], null, null, 'class="pull-right"');
+                        }
                     ?>
 					{[ "type": "select", "label": "<?php echo tr('Vettore'); ?>", "name": "idvettore", "ajax-source": "vettori", "value": "$idvettore$", "disabled": <?php echo intval($record['idspedizione'] == 3); ?>, "required": <?php echo (!empty($record['idspedizione'])) ? intval($record['idspedizione'] != 3) : 0; ?>, "icon-after": "add|<?php echo Modules::get('Anagrafiche')['id']; ?>|tipoanagrafica=Vettore&readonly_tipo=1|btn_idvettore|<?php echo (($record['idspedizione'] != 3 and intval(!$record['flag_completato']))) ? '' : 'disabled'; ?>" ]}
 				</div>

modules/fatture/row-list.php

@@ -81,7 +81,7 @@ foreach ($righe as $riga) {
             '_RITENUTA_ACCONTO_' => $riga->ritenuta_acconto ? '<br>Ritenuta acconto: '.moneyFormat(abs($riga->ritenuta_acconto)) : null,
             '_RITENUTA_CONTRIBUTI_' => $riga->ritenuta_contributi ? '<br>Ritenuta contributi: '.moneyFormat(abs($riga->ritenuta_contributi)) : null,
             '_DESCRIZIONE_CONTO_' => $descrizione_conto ?: '<span class="label label-danger" ><i class="fa fa-exclamation-triangle"></i>
-            '.tr("Conto mancante").'</span>',
+            '.tr('Conto mancante').'</span>',
             '_ID_DOCUMENTO_' => $id_documento_fe ? ' - DOC: '.$id_documento_fe : null,
             '_NUMERO_RIGA_' => $num_item ? ', NRI: '.$num_item : null,
             '_CODICE_CIG_' => $codice_cig ? ', CIG: '.$codice_cig : null,

modules/impostazioni/sezione.php

@@ -33,23 +33,19 @@ foreach ($impostazioni as $impostazione) {
 
     <script>';
 
-    if ($impostazione->tipo == 'time'){
+    if ($impostazione->tipo == 'time') {
-    echo '
+        echo '
     input("setting['.$impostazione->id.']");
     $(document).on("blur", "#setting'.$impostazione->id.'", function (e) {
       salvaImpostazione('.$impostazione->id.', $("#setting'.$impostazione->id.'").val());
     });
     ';
-
+    } else {
-    }
+        echo '
-    else{
-
-    echo '
 
     input("setting['.$impostazione->id.']").change(function (){
         salvaImpostazione('.$impostazione->id.', input(this).get());
     });';
-
     }
 
     echo '

modules/interventi/add.php

@@ -305,7 +305,6 @@ echo '
 	</div>
 </form>';
 
-
 if (!empty($id_intervento)) {
     echo '
 <script type="text/javascript">

modules/interventi/modutil.php

@@ -20,6 +20,7 @@
 include_once __DIR__.'/../../core.php';
 
 use Modules\Anagrafiche\Anagrafica;
+use Modules\Articoli\Articolo as ArticoloOriginale;
 use Modules\Emails\Mail;
 use Modules\Emails\Template;
 use Modules\Fatture\Components\Descrizione;
@@ -27,7 +28,6 @@ use Modules\Fatture\Components\Riga;
 use Modules\Fatture\Fattura;
 use Modules\Interventi\Components\Sessione;
 use Modules\Interventi\Intervento;
-use Modules\Articoli\Articolo as ArticoloOriginale;
 use Util\Ini;
 
 /**
@@ -140,7 +140,7 @@ function aggiungi_intervento_in_fattura($id_intervento, $id_fattura, $descrizion
     $fattura = Fattura::find($id_fattura);
     $intervento = Intervento::find($id_intervento);
 
-    if( !empty($fattura->anagrafica->idiva_vendite) ){
+    if (!empty($fattura->anagrafica->idiva_vendite)) {
         $id_iva = $fattura->anagrafica->idiva_vendite;
     }
 

modules/preventivi/edit.php

@@ -105,9 +105,10 @@ echo '
                 </div>
 
                 <div class="col-md-3">
-                    <?php 
+                    <?php
-                        if (!empty($record['idpagamento']))
+                        if (!empty($record['idpagamento'])) {
                             echo Modules::link('Pagamenti', $record['idpagamento'], null, null, 'class="pull-right"');
+                        }
                     ?>
 
                     {[ "type": "select", "label": "<?php echo tr('Pagamento'); ?>", "name": "idpagamento", "values": "query=SELECT id, descrizione FROM co_pagamenti GROUP BY descrizione ORDER BY descrizione", "value": "$idpagamento$" ]}

modules/primanota/add.php

@@ -253,11 +253,10 @@ if (!empty($id_records) && get('origine') == 'fatture' && !empty($counter)) {
         ]).'.</b></p>
 </div>';
 }
-if(!empty(get('id_anagrafica'))){
+if (!empty(get('id_anagrafica'))) {
     $id_anagrafica = get('id_anagrafica');
-} else{
+} else {
-    $id_anagrafica = $dbo->fetchOne('SELECT idanagrafica FROM co_documenti WHERE id IN('.( get('id_documenti') ?: '0' ).')')['idanagrafica'];
+    $id_anagrafica = $dbo->fetchOne('SELECT idanagrafica FROM co_documenti WHERE id IN('.(get('id_documenti') ?: '0').')')['idanagrafica'];
-
 }
 echo '
 <form action="'.base_path().'/controller.php?id_module='.$module->id.'" method="post" id="add-form">
@@ -288,7 +287,7 @@ echo '
 		</div>
     </div>';
 
-if(!empty($id_anagrafica)){
+if (!empty($id_anagrafica)) {
     $id_conto_anticipo_fornitori = setting('Conto anticipo fornitori');
     $id_conto_anticipo_clienti = setting('Conto anticipo clienti');
 
@@ -296,24 +295,23 @@ if(!empty($id_anagrafica)){
 
     $anticipo_fornitore = $dbo->fetchOne('SELECT ABS(SUM(totale)) AS totale FROM co_movimenti WHERE  co_movimenti.id_anagrafica='.prepare($id_anagrafica).' AND  co_movimenti.idconto='.prepare($id_conto_anticipo_fornitori));
 
-
+    if ($anticipo_fornitore['totale'] != 0) {
-    if($anticipo_fornitore['totale'] != 0){
         echo '
         <div class="alert alert-warning">
             '.tr('Attenzione: è stato anticipato al fornitore un importo di _TOTALE_',
                 [
-                    '_TOTALE_' => moneyFormat($anticipo_fornitore['totale'])
+                    '_TOTALE_' => moneyFormat($anticipo_fornitore['totale']),
                 ]
             ).'
         </div>';
     }
 
-    if($anticipo_cliente['totale'] != 0){
+    if ($anticipo_cliente['totale'] != 0) {
         echo '
         <div class="alert alert-warning">
             '.tr('Attenzione: è stato ricevuto un anticipo dal cliente di _TOTALE_',
                 [
-                    '_TOTALE_' => moneyFormat($anticipo_cliente['totale'])
+                    '_TOTALE_' => moneyFormat($anticipo_cliente['totale']),
                 ]
             ).'
         </div>';

modules/primanota/src/Mastrino.php

@@ -55,6 +55,7 @@ class Mastrino extends Model
         $model->is_insoluto = $is_insoluto;
         $model->primanota = $contabile;
         $model->id_anagrafica = $id_anagrafica;
+
         return $model;
     }
 

modules/primanota/src/Movimento.php

@@ -48,7 +48,7 @@ class Movimento extends Model
         $model->primanota = $mastrino->primanota;
         $model->is_insoluto = $mastrino->is_insoluto;
         $model->id_anagrafica = $mastrino->id_anagrafica;
-        
+
         // Conto associato
         $model->idconto = $id_conto;
 

modules/utenti/actions.php

@@ -137,8 +137,7 @@ switch (filter('op')) {
 
         $already_token = $dbo->fetchOne('SELECT `id` FROM `zz_tokens` WHERE `id_utente` = '.prepare($id_utente))['id'];
 
-        if (empty($already_token)){
+        if (empty($already_token)) {
-
             //Quando richiamo getApiTokens,  non trovando nessun token abilitato ne crea uno nuovo
             $tokens = $utente->getApiTokens();
 
@@ -146,12 +145,10 @@ switch (filter('op')) {
                 $dbo->query('UPDATE zz_tokens SET enabled = 1 WHERE id = '.prepare($token['id']));
                 flash()->info(tr('Token creato!'));
             }
-
+        } elseif ($dbo->query('UPDATE zz_tokens SET enabled = 1 WHERE id_utente = '.prepare($id_utente))) {
-        }else if ($dbo->query('UPDATE zz_tokens SET enabled = 1 WHERE id_utente = '.prepare($id_utente))) {
             flash()->info(tr('Token abilitato!'));
         }
 
-       
         break;
 
     // Disabilita API utente

modules/utenti/modutil.php

@@ -33,7 +33,7 @@ function menuSelection($element, $group_id, $depth, $permessi_disponibili)
     // Permessi impostati per il gruppo
     $permesso_salvato = $dbo->fetchOne('SELECT permessi FROM zz_permissions WHERE idgruppo = '.prepare($group_id).' AND idmodule = '.prepare($element['id']));
     $permessi = $permesso_salvato ? $permesso_salvato['permessi'] : '-';
-    foreach ($permessi_disponibili as $id => $nome){
+    foreach ($permessi_disponibili as $id => $nome) {
         $attr = ($id == $permessi) ? ' selected="selected"' : '';
 
         $result .= '

plugins/receiptFE/edit.php

@@ -41,7 +41,7 @@ if (Interaction::isEnabled()) {
         ->orderBy('data_stato_fe')
         ->get();
 
-    if (!empty($fatture_generate)){
+    if (!empty($fatture_generate)) {
         echo '
     <div class="alert alert-warning">
         <p><i class="fa fa-warning"></i> '.tr('Attenzione: le seguenti fatture sono in attesa di una ricevuta').'.</p>

plugins/receiptFE/src/ReceiptTask.php

@@ -25,7 +25,7 @@ class ReceiptTask extends Manager
 {
     public function execute()
     {
-        if (!Interaction::isEnabled()){
+        if (!Interaction::isEnabled()) {
             return;
         }
 

templates/base/footer.php

@@ -17,7 +17,7 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-/**
+/*
  * Footer di default.
  * I contenuti di questo file vengono utilizzati per generare il footer delle stampe nel caso non esista un file footer.php all'interno della stampa.
  *

templates/base/header.php

@@ -17,7 +17,7 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-/**
+/*
  * Header di default.
  * I contenuti di questo file vengono utilizzati per generare l'header delle stampe nel caso non esista un file header.php all'interno della stampa.
  *

templates/partitario_mastrino/header.php

@@ -17,7 +17,7 @@
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
-/**
+/*
  * Header di default.
  * I contenuti di questo file vengono utilizzati per generare l'header delle stampe nel caso non esista un file header.php all'interno della stampa.
  *

templates/preventivi/body.php

@@ -37,9 +37,9 @@ $anagrafica = Anagrafica::find($documento['idanagrafica']);
 $pagamento = $dbo->fetchOne('SELECT * FROM co_pagamenti WHERE id = '.$documento['idpagamento']);
 
 //Verifico se c'è una banca predefinita per il mio cliente
-if (!empty($anagrafica->idbanca_vendite)){
+if (!empty($anagrafica->idbanca_vendite)) {
     $banca = $dbo->fetchOne('SELECT co_banche.nome, co_banche.iban, co_banche.bic FROM co_banche INNER JOIN co_pianodeiconti3 ON co_banche.id_pianodeiconti3 = co_pianodeiconti3.id WHERE co_pianodeiconti3.id = '.$pagamento['idconto_vendite'].' AND co_banche.id ='.$anagrafica->idbanca_vendite);
-}else if (!empty($pagamento['idconto_vendite'])) {
+} elseif (!empty($pagamento['idconto_vendite'])) {
     //Altrimenti prendo quella associata la metodo di pagamento selezionato
     $banca = $dbo->fetchOne('SELECT co_banche.nome, co_banche.iban, co_banche.bic FROM co_banche INNER JOIN co_pianodeiconti3 ON co_banche.id_pianodeiconti3 = co_pianodeiconti3.id WHERE co_pianodeiconti3.id = '.$pagamento['idconto_vendite'].' AND co_banche.deleted_at IS NULL');
 }