From b7aa2bfb4a62998883c25e55afeae350ba4717ee Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Fri, 17 Jan 2020 17:31:46 +0100 Subject: [PATCH] Stile del codice --- include/src/Document.php | 4 +- modules/anagrafiche/ajax/select.php | 3 +- modules/ddt/row-list.php | 128 ++++++++++++------------- modules/fatture/bulk.php | 25 ++--- modules/fatture/edit.php | 2 +- modules/ordini/row-list.php | 144 ++++++++++++++-------------- templates/interventi/body.php | 4 +- templates/interventi/init.php | 4 +- 8 files changed, 153 insertions(+), 161 deletions(-) diff --git a/include/src/Document.php b/include/src/Document.php index aabfe7d71..9b606811a 100644 --- a/include/src/Document.php +++ b/include/src/Document.php @@ -30,8 +30,8 @@ abstract class Document extends Model { $righe = $this->getRighe(); - return $righe->first(function ($item) use ($type, $id){ - return $item instanceof $type && $item->id == $id ; + return $righe->first(function ($item) use ($type, $id) { + return $item instanceof $type && $item->id == $id; }); } diff --git a/modules/anagrafiche/ajax/select.php b/modules/anagrafiche/ajax/select.php index 36c66c7e8..d47e68905 100644 --- a/modules/anagrafiche/ajax/select.php +++ b/modules/anagrafiche/ajax/select.php @@ -254,8 +254,9 @@ switch ($resource) { $where[] = 'idanagrafica='.prepare($id_azienda); //admin o utente senza una sede prefissata, avrà accesso a tutte le sedi - if (!empty($user->sedi) and !$user->is_admin) + if (!empty($user->sedi) and !$user->is_admin) { $where[] = 'id IN('.implode(',', $user->sedi).')'; + } if (!empty($search)) { $search_fields[] = 'nomesede LIKE '.prepare('%'.$search.'%'); diff --git a/modules/ddt/row-list.php b/modules/ddt/row-list.php index d7f3b21b9..726e2ce4f 100644 --- a/modules/ddt/row-list.php +++ b/modules/ddt/row-list.php @@ -36,108 +36,108 @@ foreach ($righe as $riga) { } } - echo ' + echo ' '; - if (!empty($r['idarticolo'])) { - echo ' + if (!empty($r['idarticolo'])) { + echo ' '.Modules::link('Articoli', $r['idarticolo'], $riga->articolo->codice.' - '.$r['descrizione']); - if (!empty($r['abilita_serial'])) { - if (!empty($mancanti)) { - echo ' + if (!empty($r['abilita_serial'])) { + if (!empty($mancanti)) { + echo '
'.tr('_NUM_ serial mancanti', [ '_NUM_' => $mancanti, ]).''; - } - - if (!empty($serials)) { - echo ' -
'.tr('SN').': '.implode(', ', $serials); - } } - } else { - echo nl2br($r['descrizione']); + + if (!empty($serials)) { + echo ' +
'.tr('SN').': '.implode(', ', $serials); + } } + } else { + echo nl2br($r['descrizione']); + } - // Aggiunta dei riferimenti ai documenti - $ref = doc_references($r, $dir, ['idddt']); + // Aggiunta dei riferimenti ai documenti + $ref = doc_references($r, $dir, ['idddt']); - if (!empty($ref)) { - echo ' + if (!empty($ref)) { + echo '
'.Modules::link($ref['module'], $ref['id'], $ref['description'], $ref['description']); - } + } - echo ' + echo ' '; - echo ' + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' + if (empty($r['is_descrizione'])) { + echo ' '.Translator::numberToLocale($r['qta'] - $r['qta_evasa'], 'qta').' / '.Translator::numberToLocale($r['qta'], 'qta').''; - } - echo ' + } + echo ' '; - // Unità di misura - echo ' + // Unità di misura + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' - '.$r['um']; - } + if (empty($r['is_descrizione'])) { echo ' + '.$r['um']; + } + echo ' '; - // Costo unitario - echo ' + // Costo unitario + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' + if (empty($r['is_descrizione'])) { + echo ' '.moneyFormat($r['subtotale'] / $r['qta']); - if (abs($r['sconto_unitario']) > 0) { - $text = $r['sconto_unitario'] > 0 ? tr('sconto _TOT_ _TYPE_') : tr('maggiorazione _TOT_ _TYPE_'); + if (abs($r['sconto_unitario']) > 0) { + $text = $r['sconto_unitario'] > 0 ? tr('sconto _TOT_ _TYPE_') : tr('maggiorazione _TOT_ _TYPE_'); - echo ' + echo '
'.replace($text, [ '_TOT_' => Translator::numberToLocale(abs($r['sconto_unitario'])), '_TYPE_' => ($r['tipo_sconto'] == 'PRC' ? '%' : currency()), ]).''; - } } - echo ' + } + echo ' '; - // Iva - echo ' + // Iva + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' + if (empty($r['is_descrizione'])) { + echo ' '.moneyFormat($r['iva']).'
'.$r['desc_iva'].''; - } - echo ' + } + echo ' '; - // Imponibile - echo ' + // Imponibile + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' - '.moneyFormat($r['subtotale'] - $r['sconto']); - } + if (empty($r['is_descrizione'])) { echo ' + '.moneyFormat($r['subtotale'] - $r['sconto']); + } + echo ' '; - // Possibilità di rimuovere una riga solo se il ddt non è evaso - echo ' + // Possibilità di rimuovere una riga solo se il ddt non è evaso + echo ' '; - if ($record['flag_completato'] == 0) { - echo " + if ($record['flag_completato'] == 0) { + echo "
@@ -147,12 +147,12 @@ foreach ($righe as $riga) {
"; - if (!empty($r['idarticolo']) && $r['abilita_serial']) { - echo " - "; - } - + if (!empty($r['idarticolo']) && $r['abilita_serial']) { echo " + "; + } + + echo " @@ -162,14 +162,14 @@ foreach ($righe as $riga) {
"; - } + } - echo ' + echo '
'; - echo ' + echo ' '; } diff --git a/modules/fatture/bulk.php b/modules/fatture/bulk.php index 4c35528c8..9cf2873f9 100644 --- a/modules/fatture/bulk.php +++ b/modules/fatture/bulk.php @@ -185,34 +185,32 @@ switch (post('op')) { } break; - case 'copy-bulk': foreach ($id_records as $id_record) { - // Lettura dati fattura attuale $rs = $dbo->fetchOne('SELECT * FROM co_documenti WHERE id='.prepare($id_record)); - $dir = $dbo->fetchOne("SELECT dir FROM co_tipidocumento WHERE id=".prepare($rs['idtipodocumento']))['dir']; + $dir = $dbo->fetchOne('SELECT dir FROM co_tipidocumento WHERE id='.prepare($rs['idtipodocumento']))['dir']; //+ 1 settimana - if(post('skip_time')=='Giorno'){ - $data = date("Y-m-d", strtotime( '+1 day' , strtotime ( $rs['data'] )) ); + if (post('skip_time') == 'Giorno') { + $data = date('Y-m-d', strtotime('+1 day', strtotime($rs['data']))); } //+ 1 settimana - if(post('skip_time')=='Settimana'){ - $data = date("Y-m-d", strtotime( '+1 week' , strtotime ( $rs['data'] )) ); + if (post('skip_time') == 'Settimana') { + $data = date('Y-m-d', strtotime('+1 week', strtotime($rs['data']))); } //+ 1 mese - if(post('skip_time')=='Mese'){ - $data = date("Y-m-d", strtotime( '+1 month' , strtotime ( $rs['data'] )) ); + if (post('skip_time') == 'Mese') { + $data = date('Y-m-d', strtotime('+1 month', strtotime($rs['data']))); } //+ 1 anno - if(post('skip_time')=='Anno'){ - $data = date("Y-m-d", strtotime( '+1 year' , strtotime ( $rs['data'] )) ); + if (post('skip_time') == 'Anno') { + $data = date('Y-m-d', strtotime('+1 year', strtotime($rs['data']))); } // Duplicazione righe @@ -235,8 +233,7 @@ switch (post('op')) { // TODO: sistemare la duplicazione delle righe generiche e degli articoli, ignorando interventi, ddt, ordini, preventivi foreach ($righe as $riga) { - - if( !post('riferimenti') ){ + if (!post('riferimenti')) { $riga['idpreventivo'] = 0; $riga['idcontratto'] = 0; $riga['idintervento'] = 0; @@ -254,8 +251,6 @@ switch (post('op')) { // Ricalcolo inps, ritenuta e bollo (se la fattura non è stata pagata) ricalcola_costiagg_fattura($id_record); aggiorna_sedi_movimenti('documenti', $id_record); - - } flash()->info(tr('Fatture duplicate correttamente!')); diff --git a/modules/fatture/edit.php b/modules/fatture/edit.php index 310de520e..d9fd521bb 100644 --- a/modules/fatture/edit.php +++ b/modules/fatture/edit.php @@ -283,7 +283,7 @@ if (empty($record['is_fiscale'])) { '; } - echo (empty($scadenza['da_pagare']) ? ' ' : '').moneyFormat($scadenza['da_pagare']); + echo(empty($scadenza['da_pagare']) ? ' ' : '').moneyFormat($scadenza['da_pagare']); if ($scadenza['pagato'] == $scadenza['da_pagare']) { echo ' diff --git a/modules/ordini/row-list.php b/modules/ordini/row-list.php index 34162e1ab..9aebaf9e7 100644 --- a/modules/ordini/row-list.php +++ b/modules/ordini/row-list.php @@ -29,117 +29,117 @@ foreach ($righe as $riga) { // Individuazione dei seriali if (!empty($r['idarticolo']) && !empty($r['abilita_serial'])) { $serials = array_column($dbo->fetchArray('SELECT serial FROM mg_prodotti WHERE serial IS NOT NULL AND id_riga_ordine='.prepare($r['id'])), 'serial'); - $mancanti = $r['qta'] - count($serials); + $mancanti = $r['qta'] - count($serials); - if ($mancanti > 0) { - $extra = 'class="warning"'; - } else { - $mancanti = 0; - } + if ($mancanti > 0) { + $extra = 'class="warning"'; + } else { + $mancanti = 0; } + } - echo ' + echo ' '; - if (!empty($r['idarticolo'])) { - echo ' + if (!empty($r['idarticolo'])) { + echo ' '.Modules::link('Articoli', $r['idarticolo'], $riga->articolo->codice.' - '.$r['descrizione']); - if (!empty($r['abilita_serial'])) { - if (!empty($mancanti)) { - echo ' + if (!empty($r['abilita_serial'])) { + if (!empty($mancanti)) { + echo '
'.tr('_NUM_ serial mancanti', [ '_NUM_' => $mancanti, ]).''; - } - - if (!empty($serials)) { - echo ' -
'.tr('SN').': '.implode(', ', $serials); - } } - } else { - echo nl2br($r['descrizione']); + + if (!empty($serials)) { + echo ' +
'.tr('SN').': '.implode(', ', $serials); + } } + } else { + echo nl2br($r['descrizione']); + } - // Aggiunta dei riferimenti ai documenti - $ref = doc_references($r, $dir, ['idordine']); + // Aggiunta dei riferimenti ai documenti + $ref = doc_references($r, $dir, ['idordine']); - if (!empty($ref)) { - echo ' + if (!empty($ref)) { + echo '
'.Modules::link($ref['module'], $ref['id'], $ref['description'], $ref['description']); - } + } - echo ' + echo ' '; - echo ' + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' + if (empty($r['is_descrizione'])) { + echo ' '.Translator::numberToLocale($r['qta'] - $r['qta_evasa'], 'qta').' / '.Translator::numberToLocale($r['qta'], 'qta').''; - } - echo ' + } + echo ' '; - // Unità di misura - echo ' + // Unità di misura + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' - '.$r['um']; - } + if (empty($r['is_descrizione'])) { echo ' + '.$r['um']; + } + echo ' '; - // Costo unitario - echo ' + // Costo unitario + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' + if (empty($r['is_descrizione'])) { + echo ' '.moneyFormat($r['subtotale'] / $r['qta']); - if (abs($r['sconto_unitario']) > 0) { - $text = $r['sconto_unitario'] > 0 ? tr('sconto _TOT_ _TYPE_') : tr('maggiorazione _TOT_ _TYPE_'); + if (abs($r['sconto_unitario']) > 0) { + $text = $r['sconto_unitario'] > 0 ? tr('sconto _TOT_ _TYPE_') : tr('maggiorazione _TOT_ _TYPE_'); - echo ' + echo '
'.replace($text, [ '_TOT_' => Translator::numberToLocale(abs($r['sconto_unitario'])), '_TYPE_' => ($r['tipo_sconto'] == 'PRC' ? '%' : currency()), ]).''; - } } - echo ' + } + echo ' '; - // Iva - echo ' + // Iva + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' + if (empty($r['is_descrizione'])) { + echo ' '.moneyFormat($r['iva']).'
'.$r['desc_iva'].''; - } - echo ' + } + echo ' '; - // Imponibile - echo ' + // Imponibile + echo ' '; - if (empty($r['is_descrizione'])) { - echo ' - '.moneyFormat($r['subtotale'] - $r['sconto']); - } + if (empty($r['is_descrizione'])) { echo ' + '.moneyFormat($r['subtotale'] - $r['sconto']); + } + echo ' '; - // Possibilità di rimuovere una riga solo se l'ordine non è evaso - echo ' + // Possibilità di rimuovere una riga solo se l'ordine non è evaso + echo ' '; - if ($record['flag_completato'] == 0) { - echo " + if ($record['flag_completato'] == 0) { + echo "
@@ -147,33 +147,33 @@ foreach ($righe as $riga) { "; - echo " + echo "
"; - if (!empty($r['idarticolo']) && $r['abilita_serial']) { - echo " - "; - } - + if (!empty($r['idarticolo']) && $r['abilita_serial']) { echo " + "; + } + + echo "
"; - } + } - echo ' + echo '
'; - echo ' + echo ' '; - } +} echo ' '; diff --git a/templates/interventi/body.php b/templates/interventi/body.php index 32ca77fe6..8b4416bff 100644 --- a/templates/interventi/body.php +++ b/templates/interventi/body.php @@ -52,9 +52,7 @@ echo ' '.((!empty($s_provincia)) ? tr('Provincia').': '.strtoupper($s_provincia).'' : '').' '; - } - - else if (!empty($c_indirizzo) or !empty($c_cap) or !empty($c_citta) or !empty($c_provincia)) { + } elseif (!empty($c_indirizzo) or !empty($c_cap) or !empty($c_citta) or !empty($c_provincia)) { echo ' diff --git a/templates/interventi/init.php b/templates/interventi/init.php index 0687a7aa7..e62814639 100644 --- a/templates/interventi/init.php +++ b/templates/interventi/init.php @@ -12,15 +12,13 @@ $contratto = $dbo->fetchOne('SELECT nome, numero FROM co_contratti WHERE id = '. $id_cliente = $documento['idanagrafica']; $id_sede = $documento['idsede']; -if (!empty($documento['idsede_destinazione'])){ - +if (!empty($documento['idsede_destinazione'])) { $sedi = $dbo->fetchOne('SELECT nomesede, cap, citta, indirizzo, provincia FROM an_sedi WHERE id = '.prepare($documento['idsede_destinazione'])); $s_citta = $sedi['citta']; $s_indirizzo = $sedi['indirizzo']; $s_cap = $sedi['cap']; $s_provincia = $sedi['provincia']; - } //Se ho deciso di NON mostrare i prezzi al tencico mi assicuro che non li possa vedere dalla stampa