From 82aff077cd96927c7eb1723c6b3101dfb2980d62 Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Thu, 19 Jul 2018 15:33:32 +0200 Subject: [PATCH] Utilizzo delle funzioni get() e post() --- actions.php | 16 +- add.php | 4 +- ajax_complete.php | 4 +- ajax_search.php | 2 +- bug.php | 10 +- core.php | 2 +- mail.php | 2 +- modules/anagrafiche/actions.php | 161 +++++++++--------- modules/automezzi/actions.php | 4 +- modules/automezzi/add_articolo.php | 2 +- modules/automezzi/add_tecnico.php | 2 +- modules/banche/actions.php | 10 +- modules/contratti/actions.php | 54 +++--- modules/contratti/ajax.php | 4 +- modules/contratti/plugins/actions.php | 16 +- modules/contratti/plugins/addfattura.php | 8 +- .../contratti/plugins/addpianficazione.php | 4 +- modules/contratti/plugins/ajax_articoli.php | 4 +- modules/contratti/plugins/ajax_righe.php | 4 +- .../contratti.fatturaordiniservizio.php | 16 +- .../contratti.ordiniservizio.interventi.php | 14 +- .../plugins/contratti.ordiniservizio.php | 8 +- .../contratti.pianificazioneinterventi.php | 4 +- modules/contratti/row-add.php | 4 +- modules/contratti/row-edit.php | 2 +- modules/ddt/actions.php | 74 ++++---- modules/ddt/row-add.php | 4 +- modules/ddt/row-edit.php | 2 +- modules/emails/actions.php | 28 +-- modules/fatture/actions.php | 154 ++++++++--------- modules/fatture/crea_documento.php | 8 +- modules/fatture/row-add.php | 4 +- modules/fatture/row-edit.php | 2 +- modules/interventi/actions.php | 34 ++-- modules/modelli_primanota/actions.php | 4 +- .../plugins/my_impianti.interventi.php | 4 +- modules/ordini/actions.php | 42 ++--- modules/ordini/crea_documento.php | 6 +- modules/ordini/row-add.php | 4 +- modules/ordini/row-edit.php | 2 +- modules/pagamenti/actions.php | 20 +-- modules/partitario/actions.php | 2 +- modules/partitario/dettagli_movimento.php | 4 +- modules/preventivi/actions.php | 34 ++-- modules/preventivi/row-add.php | 4 +- modules/preventivi/row-edit.php | 2 +- modules/primanota/actions.php | 18 +- modules/scadenzario/actions.php | 4 +- modules/smtp/actions.php | 32 ++-- modules/viste/actions.php | 58 +++---- plugins/referenti/actions.php | 20 +-- plugins/sedi/actions.php | 56 +++--- 52 files changed, 495 insertions(+), 492 deletions(-) diff --git a/actions.php b/actions.php index 48e440332..69df6d94d 100644 --- a/actions.php +++ b/actions.php @@ -90,7 +90,7 @@ if (filter('op') == 'link_file' || filter('op') == 'unlink_file') { download($upload_dir.'/'.$rs[0]['filename'], $rs[0]['original']); } elseif (post('op') == 'send-email') { - $template = Mail::getTemplate($post['template']); + $template = Mail::getTemplate(post('template')); $id_account = $template['id_smtp']; // Informazioni di log @@ -99,7 +99,7 @@ if (filter('op') == 'link_file' || filter('op') == 'unlink_file') { $mail = new Mail($id_account); // Conferma di lettura - if (!empty($post['read_notify'])) { + if (!empty(post('read_notify'))) { $mail->ConfirmReadingTo = $mail->From; } @@ -119,16 +119,16 @@ if (filter('op') == 'link_file' || filter('op') == 'unlink_file') { } // Destinatari - $mail->addReceivers($post['destinatari'], $post['tipo_destinatari']); + $mail->addReceivers(post('destinatari'), post('tipo_destinatari')); // Oggetto - $mail->Subject = $post['subject']; + $mail->Subject = post('subject'); // Allegati - $mail->attach($post['prints'], $post['attachments']); + $mail->attach(post('prints'), post('attachments')); // Contenuto - $mail->Body = $post['body']; + $mail->Body = post('body'); // Invio mail if (!$mail->send()) { @@ -202,8 +202,8 @@ if (Modules::getPermission($id_module) == 'rw') { if (!starts_with(post('op'), 'delete')) { $values = []; foreach ($customs as $custom) { - if (isset($post[$custom['name']])) { - $values[$custom['id']] = $post[$custom['name']]; + if (post($custom['name']) !== null) { + $values[$custom['id']] = post($custom['name']); } } diff --git a/add.php b/add.php index 679313029..bc08cd716 100644 --- a/add.php +++ b/add.php @@ -75,9 +75,9 @@ $(document).ready(function(){ success: function(data){ data = data.trim(); - if(data && $("#'.$get['select'].'").val() !== undefined ) { + if(data && $("#'.get('select').'").val() !== undefined ) { result = JSON.parse(data); - $("#'.$get['select'].'").selectSetNew(result.id, result.text); + $("#'.get('select').'").selectSetNew(result.id, result.text); } $("#bs-popup2").modal("hide"); diff --git a/ajax_complete.php b/ajax_complete.php index 580e19b0d..d17e7a6f0 100644 --- a/ajax_complete.php +++ b/ajax_complete.php @@ -3,8 +3,8 @@ include_once __DIR__.'/core.php'; if (!isset($resource)) { - $module = $get['module']; - $op = $get['op']; + $module = get('module'); + $op = get('op'); $result = AJAX::complete($op); diff --git a/ajax_search.php b/ajax_search.php index f653a0903..2ea553d61 100644 --- a/ajax_search.php +++ b/ajax_search.php @@ -9,7 +9,7 @@ if (!isset($term)) { Il risultato è in json */ - $term = $get['term']; + $term = get('term'); $term = str_replace('/', '\\/', $term); $results = AJAX::search($term); diff --git a/bug.php b/bug.php index 1111cacfc..b1ebc57d1 100644 --- a/bug.php +++ b/bug.php @@ -18,12 +18,12 @@ if (filter('op') == 'send') { $mail->Subject = 'Segnalazione bug OSM '.$version; // Aggiunta dei file di log (facoltativo) - if (!empty($post['log']) && file_exists($docroot.'/logs/error.log')) { + if (!empty(post('log')) && file_exists($docroot.'/logs/error.log')) { $mail->AddAttachment($docroot.'/logs/error.log'); } // Aggiunta della copia del database (facoltativo) - if (!empty($post['sql'])) { + if (!empty(post('sql'))) { $backup_file = $docroot.'/Backup OSM '.date('Y-m-d').' '.date('H_i_s').'.sql'; Backup::database($backup_file); @@ -41,12 +41,12 @@ if (filter('op') == 'send') { ]; // Aggiunta delle informazioni sul sistema (facoltativo) - if (!empty($post['info'])) { + if (!empty(post('info'))) { $infos['Sistema'] = $_SERVER['HTTP_USER_AGENT'].' - '.getOS(); } // Completamento del body - $body = $post['body'].'
'; + $body = post('body').'
'; foreach ($infos as $key => $value) { $body .= '

'.$key.': '.$value.'

'; } @@ -63,7 +63,7 @@ if (filter('op') == 'send') { } // Rimozione del dump del database - if (!empty($post['sql'])) { + if (!empty(post('sql'))) { delete($backup_file); } diff --git a/core.php b/core.php index 27fab142f..53d8a5e99 100644 --- a/core.php +++ b/core.php @@ -227,7 +227,7 @@ if (!API::isAPIRequest()) { Permissions::check(); } - // Variabili GET e POST + // Retrocompatibilità $post = Filter::getPOST(); $get = Filter::getGET(); } diff --git a/mail.php b/mail.php index 83a07bab0..c8aac7e89 100644 --- a/mail.php +++ b/mail.php @@ -2,7 +2,7 @@ include_once __DIR__.'/core.php'; -$template = Mail::getTemplate($get['id']); +$template = Mail::getTemplate(get('id')); $module = Modules::get($id_module); $smtp = Mail::get($template['id_smtp']); diff --git a/modules/anagrafiche/actions.php b/modules/anagrafiche/actions.php index ca0b31b9c..8b5e759b1 100644 --- a/modules/anagrafiche/actions.php +++ b/modules/anagrafiche/actions.php @@ -9,104 +9,104 @@ $id_tecnico = $dbo->fetchArray("SELECT idtipoanagrafica FROM an_tipianagrafiche switch (post('op')) { case 'update': - $post['piva'] = trim(strtoupper($post['piva'])); - $post['codice_fiscale'] = trim(strtoupper($post['codice_fiscale'])); + $partita_iva = trim(strtoupper(post('piva'))); + $codice_fiscale = trim(strtoupper(post('codice_fiscale'))); // Leggo tutti i valori passati dal POST e li salvo in un array $dbo->update('an_anagrafiche', [ - 'ragione_sociale' => $post['ragione_sociale'], - 'tipo' => $post['tipo'], - 'piva' => $post['piva'], - 'codice_fiscale' => $post['codice_fiscale'], - 'data_nascita' => $post['data_nascita'], - 'luogo_nascita' => $post['luogo_nascita'], - 'sesso' => $post['sesso'], - 'capitale_sociale' => $post['capitale_sociale'], - 'indirizzo' => $post['indirizzo'], - 'indirizzo2' => $post['indirizzo2'], - 'citta' => $post['citta'], - 'cap' => $post['cap'], - 'provincia' => $post['provincia'], - 'km' => $post['km'], - 'id_nazione' => !empty($post['id_nazione']) ? $post['id_nazione'] : null, - 'telefono' => $post['telefono'], - 'cellulare' => $post['cellulare'], - 'fax' => $post['fax'], - 'email' => $post['email'], - 'pec' => $post['pec'], - 'idsede_fatturazione' => $post['idsede_fatturazione'], - 'note' => $post['note'], - 'codiceri' => $post['codiceri'], - 'codicerea' => $post['codicerea'], - 'appoggiobancario' => $post['appoggiobancario'], - 'filiale' => $post['filiale'], - 'codiceiban' => $post['codiceiban'], - 'bic' => $post['bic'], - 'diciturafissafattura' => $post['diciturafissafattura'], - 'idpagamento_acquisti' => $post['idpagamento_acquisti'], - 'idpagamento_vendite' => $post['idpagamento_vendite'], - 'idlistino_acquisti' => $post['idlistino_acquisti'], - 'idlistino_vendite' => $post['idlistino_vendite'], - 'idiva_acquisti' => $post['idiva_acquisti'], - 'idiva_vendite' => $post['idiva_vendite'], - 'idbanca_acquisti' => $post['idbanca_acquisti'], - 'idbanca_vendite' => $post['idbanca_vendite'], - 'settore' => $post['settore'], - 'marche' => $post['marche'], - 'dipendenti' => $post['dipendenti'], - 'macchine' => $post['macchine'], - 'idagente' => $post['idagente'], - 'idrelazione' => $post['idrelazione'], - 'sitoweb' => $post['sitoweb'], - 'idzona' => $post['idzona'], - 'nome_cognome' => $post['nome_cognome'], - 'iscrizione_tribunale' => $post['iscrizione_tribunale'], - 'cciaa' => $post['cciaa'], - 'cciaa_citta' => $post['cciaa_citta'], - 'n_alboartigiani' => $post['n_alboartigiani'], - 'foro_competenza' => $post['foro_competenza'], - 'colore' => $post['colore'], - 'idtipointervento_default' => $post['idtipointervento_default'], - 'gaddress' => $post['gaddress'], - 'lat' => $post['lat'], - 'lng' => $post['lng'], + 'ragione_sociale' => post('ragione_sociale'), + 'tipo' => post('tipo'), + 'piva' => $partita_iva, + 'codice_fiscale' => $codice_fiscale, + 'data_nascita' => post('data_nascita'), + 'luogo_nascita' => post('luogo_nascita'), + 'sesso' => post('sesso'), + 'capitale_sociale' => post('capitale_sociale'), + 'indirizzo' => post('indirizzo'), + 'indirizzo2' => post('indirizzo2'), + 'citta' => post('citta'), + 'cap' => post('cap'), + 'provincia' => post('provincia'), + 'km' => post('km'), + 'id_nazione' => !empty(post('id_nazione')) ? post('id_nazione') : null, + 'telefono' => post('telefono'), + 'cellulare' => post('cellulare'), + 'fax' => post('fax'), + 'email' => post('email'), + 'pec' => post('pec'), + 'idsede_fatturazione' => post('idsede_fatturazione'), + 'note' => post('note'), + 'codiceri' => post('codiceri'), + 'codicerea' => post('codicerea'), + 'appoggiobancario' => post('appoggiobancario'), + 'filiale' => post('filiale'), + 'codiceiban' => post('codiceiban'), + 'bic' => post('bic'), + 'diciturafissafattura' => post('diciturafissafattura'), + 'idpagamento_acquisti' => post('idpagamento_acquisti'), + 'idpagamento_vendite' => post('idpagamento_vendite'), + 'idlistino_acquisti' => post('idlistino_acquisti'), + 'idlistino_vendite' => post('idlistino_vendite'), + 'idiva_acquisti' => post('idiva_acquisti'), + 'idiva_vendite' => post('idiva_vendite'), + 'idbanca_acquisti' => post('idbanca_acquisti'), + 'idbanca_vendite' => post('idbanca_vendite'), + 'settore' => post('settore'), + 'marche' => post('marche'), + 'dipendenti' => post('dipendenti'), + 'macchine' => post('macchine'), + 'idagente' => post('idagente'), + 'idrelazione' => post('idrelazione'), + 'sitoweb' => post('sitoweb'), + 'idzona' => post('idzona'), + 'nome_cognome' => post('nome_cognome'), + 'iscrizione_tribunale' => post('iscrizione_tribunale'), + 'cciaa' => post('cciaa'), + 'cciaa_citta' => post('cciaa_citta'), + 'n_alboartigiani' => post('n_alboartigiani'), + 'foro_competenza' => post('foro_competenza'), + 'colore' => post('colore'), + 'idtipointervento_default' => post('idtipointervento_default'), + 'gaddress' => post('gaddress'), + 'lat' => post('lat'), + 'lng' => post('lng'), ], ['idanagrafica' => $id_record]); - App::flash()->info(str_replace('_NAME_', '"'.$post['ragione_sociale'].'"', "Informazioni per l'anagrafica _NAME_ salvate correttamente!")); + App::flash()->info(str_replace('_NAME_', '"'.post('ragione_sociale').'"', "Informazioni per l'anagrafica _NAME_ salvate correttamente!")); // Validazione della Partita IVA - $check_vat_number = Validate::isValidVatNumber(strtoupper($post['piva'])); + $check_vat_number = Validate::isValidVatNumber(strtoupper($partita_iva)); if (empty($check_vat_number)) { App::flash()->error(tr('Attenzione: la partita IVA _IVA_ sembra non essere valida', [ - '_IVA_' => $post['piva'], + '_IVA_' => $partita_iva, ])); } // Aggiorno il codice anagrafica se non è già presente, altrimenti lo ignoro - $esiste = $dbo->fetchNum('SELECT idanagrafica FROM an_anagrafiche WHERE codice='.prepare($post['codice']).' AND NOT idanagrafica='.prepare($id_record)); + $esiste = $dbo->fetchNum('SELECT idanagrafica FROM an_anagrafiche WHERE codice='.prepare(post('codice')).' AND NOT idanagrafica='.prepare($id_record)); // Verifica dell'esistenza codice anagrafica if ($esiste) { App::flash()->error(tr("Il codice anagrafica inserito esiste già! Inserirne un'altro...")); } else { - $dbo->query('UPDATE an_anagrafiche SET codice='.prepare($post['codice']).' WHERE idanagrafica='.prepare($id_record)); + $dbo->query('UPDATE an_anagrafiche SET codice='.prepare(post('codice')).' WHERE idanagrafica='.prepare($id_record)); } // Aggiorno gli agenti collegati - $dbo->sync('an_anagrafiche_agenti', ['idanagrafica' => $id_record], ['idagente' => (array) $post['idagenti']]); + $dbo->sync('an_anagrafiche_agenti', ['idanagrafica' => $id_record], ['idagente' => (array) post('idagenti')]); // Se l'agente di default è stato elencato anche tra gli agenti secondari lo rimuovo - if (!empty($post['idagente'])) { - $dbo->query('DELETE FROM an_anagrafiche_agenti WHERE idanagrafica='.prepare($id_record).' AND idagente='.prepare($post['idagente'])); + if (!empty(post('idagente'))) { + $dbo->query('DELETE FROM an_anagrafiche_agenti WHERE idanagrafica='.prepare($id_record).' AND idagente='.prepare(post('idagente'))); } // Aggiorno le tipologie di anagrafica - $post['idtipoanagrafica'] = (array) $post['idtipoanagrafica']; + $idtipoanagrafica = (array) post('idtipoanagrafica'); if (in_array($id_azienda, $tipi_anagrafica)) { - $post['idtipoanagrafica'][] = $id_azienda; + $idtipoanagrafica[] = $id_azienda; } - $dbo->sync('an_tipianagrafiche_anagrafiche', ['idanagrafica' => $id_record], ['idtipoanagrafica' => $post['idtipoanagrafica']]); + $dbo->sync('an_tipianagrafiche_anagrafiche', ['idanagrafica' => $id_record], ['idtipoanagrafica' => $idtipoanagrafica]); // Verifico se esiste già l'associazione dell'anagrafica a conti del partitario $rs = $dbo->fetchArray('SELECT idconto_cliente, idconto_fornitore FROM an_anagrafiche WHERE idanagrafica='.prepare($id_record)); @@ -114,26 +114,26 @@ switch (post('op')) { $idconto_fornitore = $rs[0]['idconto_fornitore']; // Creo il relativo conto nel partitario se non esiste - if (empty($idconto_cliente) && in_array($id_cliente, $post['idtipoanagrafica'])) { + if (empty($idconto_cliente) && in_array($id_cliente, $idtipoanagrafica)) { // Calcolo prossimo numero cliente $rs = $dbo->fetchArray("SELECT MAX(CAST(co_pianodeiconti3.numero AS UNSIGNED)) AS max_numero FROM co_pianodeiconti3 INNER JOIN co_pianodeiconti2 ON co_pianodeiconti3.idpianodeiconti2=co_pianodeiconti2.id WHERE co_pianodeiconti2.descrizione='Crediti clienti e crediti diversi'"); $new_numero = $rs[0]['max_numero'] + 1; $new_numero = str_pad($new_numero, 6, '0', STR_PAD_LEFT); - $dbo->query('INSERT INTO co_pianodeiconti3(numero, descrizione, idpianodeiconti2, can_delete, can_edit) VALUES('.prepare($new_numero).', '.prepare($post['ragione_sociale']).", (SELECT id FROM co_pianodeiconti2 WHERE descrizione='Crediti clienti e crediti diversi'), 1, 1)"); + $dbo->query('INSERT INTO co_pianodeiconti3(numero, descrizione, idpianodeiconti2, can_delete, can_edit) VALUES('.prepare($new_numero).', '.prepare(post('ragione_sociale')).", (SELECT id FROM co_pianodeiconti2 WHERE descrizione='Crediti clienti e crediti diversi'), 1, 1)"); $idconto = $dbo->lastInsertedID(); // Collegamento conto $dbo->query('UPDATE an_anagrafiche SET idconto_cliente='.prepare($idconto).' WHERE idanagrafica='.prepare($id_record)); } - if (empty($idconto_fornitore) && in_array($id_fornitore, $post['idtipoanagrafica'])) { + if (empty($idconto_fornitore) && in_array($id_fornitore, $idtipoanagrafica)) { // Calcolo prossimo numero cliente $rs = $dbo->fetchArray("SELECT MAX(CAST(co_pianodeiconti3.numero AS UNSIGNED)) AS max_numero FROM co_pianodeiconti3 INNER JOIN co_pianodeiconti2 ON co_pianodeiconti3.idpianodeiconti2=co_pianodeiconti2.id WHERE co_pianodeiconti2.descrizione='Debiti fornitori e debiti diversi'"); $new_numero = $rs[0]['max_numero'] + 1; $new_numero = str_pad($new_numero, 6, '0', STR_PAD_LEFT); - $dbo->query('INSERT INTO co_pianodeiconti3(numero, descrizione, idpianodeiconti2, can_delete, can_edit) VALUES('.prepare($new_numero).', '.prepare($post['ragione_sociale']).", (SELECT id FROM co_pianodeiconti2 WHERE descrizione='Debiti fornitori e debiti diversi'), 1, 1)"); + $dbo->query('INSERT INTO co_pianodeiconti3(numero, descrizione, idpianodeiconti2, can_delete, can_edit) VALUES('.prepare($new_numero).', '.prepare(post('ragione_sociale')).", (SELECT id FROM co_pianodeiconti2 WHERE descrizione='Debiti fornitori e debiti diversi'), 1, 1)"); $idconto = $dbo->lastInsertedID(); // Collegamento conto @@ -164,14 +164,17 @@ switch (post('op')) { } } - $idagente = ($agente_is_logged && in_array($id_cliente, $post['idtipoanagrafica'])) ? $user['idanagrafica'] : 0; + $idagente = ($agente_is_logged && in_array($id_cliente, $idtipoanagrafica)) ? $user['idanagrafica'] : 0; + + $partita_iva = trim(strtoupper(post('piva'))); + $codice_fiscale = trim(strtoupper(post('codice_fiscale'))); // Inserisco l'anagrafica $dbo->insert('an_anagrafiche', [ 'ragione_sociale' => $ragione_sociale, 'codice' => $codice, - 'piva' => post('piva'), - 'codice_fiscale' => post('codice_fiscale'), + 'piva' => $partita_iva, + 'codice_fiscale' => $codice_fiscale, 'indirizzo' => post('indirizzo'), 'citta' => post('citta'), 'cap' => post('cap'), @@ -195,7 +198,7 @@ switch (post('op')) { } //se sto inserendo un tecnico, mi copio già le tariffe per le varie attività - if (in_array($id_tecnico, $post['idtipoanagrafica'])) { + if (in_array($id_tecnico, $idtipoanagrafica)) { //per ogni tipo di attività $rs_tipiintervento = $dbo->fetchArray('SELECT * FROM in_tipiintervento'); @@ -209,7 +212,7 @@ switch (post('op')) { } // Creo il relativo conto nel partitario (cliente) - if (in_array($id_cliente, $post['idtipoanagrafica'])) { + if (in_array($id_cliente, $idtipoanagrafica)) { // Calcolo prossimo numero cliente $rs = $dbo->fetchArray("SELECT MAX(CAST(co_pianodeiconti3.numero AS UNSIGNED)) AS max_numero FROM co_pianodeiconti3 INNER JOIN co_pianodeiconti2 ON co_pianodeiconti3.idpianodeiconti2=co_pianodeiconti2.id WHERE co_pianodeiconti2.descrizione='Crediti clienti e crediti diversi'"); $new_numero = $rs[0]['max_numero'] + 1; @@ -224,7 +227,7 @@ switch (post('op')) { } // Creo il relativo conto nel partitario (fornitore) - if (in_array($id_fornitore, $post['idtipoanagrafica'])) { + if (in_array($id_fornitore, $idtipoanagrafica)) { // Calcolo prossimo numero cliente $rs = $dbo->fetchArray("SELECT MAX(CAST(co_pianodeiconti3.numero AS UNSIGNED)) AS max_numero FROM co_pianodeiconti3 INNER JOIN co_pianodeiconti2 ON co_pianodeiconti3.idpianodeiconti2=co_pianodeiconti2.id WHERE co_pianodeiconti2.descrizione='Debiti fornitori e debiti diversi'"); $new_numero = $rs[0]['max_numero'] + 1; diff --git a/modules/automezzi/actions.php b/modules/automezzi/actions.php index 74162d742..b821c5a4b 100644 --- a/modules/automezzi/actions.php +++ b/modules/automezzi/actions.php @@ -47,7 +47,7 @@ switch (post('op')) { $data_fine = null; // Controllo sull'effettivo inserimento di una data di fine successiva a quella di inizio - if (!empty($post['data_fine'])) { + if (!empty(post('data_fine'))) { if (new DateTime(post('data_fine')) >= new DateTime($data_inizio)) { $data_fine = post('data_fine'); } @@ -71,7 +71,7 @@ switch (post('op')) { $data_fine = null; // Controllo sull'effettivo inserimento di una data di fine successiva a quella di inizio - if (!empty($post['data_fine'][$idautomezzotecnico])) { + if (!empty(post('data_fine')[$idautomezzotecnico])) { if (new DateTime(post('data_fine')[$idautomezzotecnico]) >= new DateTime($data_inizio)) { $data_fine = post('data_fine')[$idautomezzotecnico]; } diff --git a/modules/automezzi/add_articolo.php b/modules/automezzi/add_articolo.php index ebaa244b0..d5b5f8be8 100644 --- a/modules/automezzi/add_articolo.php +++ b/modules/automezzi/add_articolo.php @@ -2,7 +2,7 @@ include_once __DIR__.'/../../core.php'; -$idautomezzo = $get['idautomezzo']; +$idautomezzo = get('idautomezzo'); /* Form di inserimento riga documento diff --git a/modules/automezzi/add_tecnico.php b/modules/automezzi/add_tecnico.php index b57d5518a..d8f43c38d 100644 --- a/modules/automezzi/add_tecnico.php +++ b/modules/automezzi/add_tecnico.php @@ -2,7 +2,7 @@ include_once __DIR__.'/../../core.php'; -$id_record = $get['idautomezzo']; +$id_record = get('idautomezzo'); // Form di inserimento responsabili automezzo echo ' diff --git a/modules/banche/actions.php b/modules/banche/actions.php index 92a2c6e0d..591f59654 100644 --- a/modules/banche/actions.php +++ b/modules/banche/actions.php @@ -10,11 +10,11 @@ switch (filter('op')) { if (isset($nome)) { $array = [ 'nome' => $nome, - 'filiale' => $post['filiale'], - 'iban' => $post['iban'], - 'bic' => $post['bic'], - 'id_pianodeiconti3' => $post['id_pianodeiconti3'], - 'note' => $post['note'], + 'filiale' => post('filiale'), + 'iban' => post('iban'), + 'bic' => post('bic'), + 'id_pianodeiconti3' => post('id_pianodeiconti3'), + 'note' => post('note'), ]; if (!empty($id_record)) { diff --git a/modules/contratti/actions.php b/modules/contratti/actions.php index dc91c1a54..988158930 100644 --- a/modules/contratti/actions.php +++ b/modules/contratti/actions.php @@ -23,7 +23,7 @@ switch (post('op')) { $idpagamento = (!empty($rsa[0]['idpagamento'])) ? $rsa[0]['idpagamento'] : setting('Tipo di pagamento predefinito'); - if (isset($post['idanagrafica'])) { + if (post('idanagrafica') !== null) { $dbo->query('INSERT INTO co_contratti(idanagrafica, nome, numero, idagente, idpagamento, idstato, data_bozza) VALUES ('.prepare($idanagrafica).', '.prepare($nome).', '.prepare($numero).', '.prepare($idagente).', '.prepare($idpagamento).", (SELECT `id` FROM `co_staticontratti` WHERE `descrizione`='Bozza'), NOW())"); $id_record = $dbo->lastInsertedID(); @@ -42,7 +42,7 @@ switch (post('op')) { break; case 'update': - if (isset($post['id_record'])) { + if (post('id_record') !== null) { $idstato = post('idstato'); $idanagrafica = post('idanagrafica'); $idsede = post('idsede'); @@ -60,11 +60,11 @@ switch (post('op')) { $budget = $rs[0]['budget']; } - $data_bozza = $post['data_bozza']; - $data_accettazione = $post['data_accettazione']; - $data_rifiuto = $post['data_rifiuto']; - $data_conclusione = $post['data_conclusione']; - $rinnovabile = $post['rinnovabile']; + $data_bozza = post('data_bozza'); + $data_accettazione = post('data_accettazione'); + $data_rifiuto = post('data_rifiuto'); + $data_conclusione = post('data_conclusione'); + $rinnovabile = post('rinnovabile'); $giorni_preavviso_rinnovo = post('giorni_preavviso_rinnovo'); $validita = post('validita'); @@ -83,8 +83,8 @@ switch (post('op')) { $dbo->query($query); // Aggiornamento sconto - $tipo_sconto = $post['tipo_sconto_generico']; - $sconto = $post['sconto_generico']; + $tipo_sconto = post('tipo_sconto_generico'); + $sconto = post('sconto_generico'); $dbo->update('co_contratti', [ 'tipo_sconto_globale' => $tipo_sconto, @@ -100,27 +100,27 @@ switch (post('op')) { ], $id_record); $dbo->query('DELETE FROM my_impianti_contratti WHERE idcontratto='.prepare($id_record)); - foreach ((array) $post['matricolaimpianto'] as $matricolaimpianto) { + foreach ((array) post('matricolaimpianto') as $matricolaimpianto) { $dbo->query('INSERT INTO my_impianti_contratti(idcontratto,idimpianto) VALUES('.prepare($id_record).', '.prepare($matricolaimpianto).')'); } // Salvataggio costi attività unitari del contratto - foreach ($post['costo_ore'] as $idtipointervento => $valore) { + foreach (post('costo_ore') as $idtipointervento => $valore) { $rs = $dbo->fetchArray('SELECT * FROM co_contratti_tipiintervento WHERE idcontratto='.prepare($id_record).' AND idtipointervento='.prepare($idtipointervento)); // Se non esiste il record lo inserisco... if (sizeof($rs) == 0) { // Se almeno un valore è diverso da 0 inserisco l'importo... - if ($post['costo_ore'][$idtipointervento] != 0 || $post['costo_km'][$idtipointervento] != 0 || $post['costo_dirittochiamata'][$idtipointervento] != 0) { - $dbo->query('INSERT INTO co_contratti_tipiintervento(idcontratto, idtipointervento, costo_ore, costo_km, costo_dirittochiamata, costo_ore_tecnico, costo_km_tecnico, costo_dirittochiamata_tecnico) VALUES('.prepare($id_record).', '.prepare($idtipointervento).', '.prepare($post['costo_ore'][$idtipointervento]).', '.prepare($post['costo_km'][$idtipointervento]).', '.prepare($post['costo_dirittochiamata'][$idtipointervento]).', '.prepare($post['costo_ore_tecnico'][$idtipointervento]).', '.prepare($post['costo_km_tecnico'][$idtipointervento]).', '.prepare($post['costo_dirittochiamata_tecnico'][$idtipointervento]).')'); + if (post('costo_ore')[$idtipointervento] != 0 || post('costo_km')[$idtipointervento] != 0 || post('costo_dirittochiamata')[$idtipointervento] != 0) { + $dbo->query('INSERT INTO co_contratti_tipiintervento(idcontratto, idtipointervento, costo_ore, costo_km, costo_dirittochiamata, costo_ore_tecnico, costo_km_tecnico, costo_dirittochiamata_tecnico) VALUES('.prepare($id_record).', '.prepare($idtipointervento).', '.prepare(post('costo_ore')[$idtipointervento]).', '.prepare(post('costo_km')[$idtipointervento]).', '.prepare(post('costo_dirittochiamata')[$idtipointervento]).', '.prepare(post('costo_ore_tecnico')[$idtipointervento]).', '.prepare(post('costo_km_tecnico')[$idtipointervento]).', '.prepare(post('costo_dirittochiamata_tecnico')[$idtipointervento]).')'); } } // ...altrimenti... else { // Aggiorno il nuovo valore se è diverso da 0... - if ($post['costo_ore'][$idtipointervento] != 0 || $post['costo_km'][$idtipointervento] != 0 || $post['costo_dirittochiamata'][$idtipointervento] != 0) { - $dbo->query('UPDATE co_contratti_tipiintervento SET costo_ore='.prepare($post['costo_ore'][$idtipointervento]).', costo_km='.prepare($post['costo_km'][$idtipointervento]).', costo_dirittochiamata='.prepare($post['costo_dirittochiamata'][$idtipointervento]).', costo_ore_tecnico='.prepare($post['costo_ore_tecnico'][$idtipointervento]).', costo_km_tecnico='.prepare($post['costo_km_tecnico'][$idtipointervento]).', costo_dirittochiamata_tecnico='.prepare($post['costo_dirittochiamata_tecnico'][$idtipointervento]).' WHERE idcontratto='.prepare($id_record).' AND idtipointervento='.prepare($idtipointervento)); + if (post('costo_ore')[$idtipointervento] != 0 || post('costo_km')[$idtipointervento] != 0 || post('costo_dirittochiamata')[$idtipointervento] != 0) { + $dbo->query('UPDATE co_contratti_tipiintervento SET costo_ore='.prepare(post('costo_ore')[$idtipointervento]).', costo_km='.prepare(post('costo_km')[$idtipointervento]).', costo_dirittochiamata='.prepare(post('costo_dirittochiamata')[$idtipointervento]).', costo_ore_tecnico='.prepare(post('costo_ore_tecnico')[$idtipointervento]).', costo_km_tecnico='.prepare(post('costo_km_tecnico')[$idtipointervento]).', costo_dirittochiamata_tecnico='.prepare(post('costo_dirittochiamata_tecnico')[$idtipointervento]).' WHERE idcontratto='.prepare($id_record).' AND idtipointervento='.prepare($idtipointervento)); } // ...altrimenti cancello l'eventuale riga @@ -141,12 +141,12 @@ switch (post('op')) { $idarticolo = post('idarticolo'); $descrizione = post('descrizione'); - $qta = $post['qta']; - $prezzo = $post['prezzo']; + $qta = post('qta'); + $prezzo = post('prezzo'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -186,13 +186,13 @@ switch (post('op')) { $idarticolo = post('idarticolo'); $descrizione = post('descrizione'); - $qta = $post['qta']; - $prezzo = $post['prezzo']; + $qta = post('qta'); + $prezzo = post('prezzo'); $subtot = $prezzo * $qta; // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -224,7 +224,7 @@ switch (post('op')) { // Eliminazione riga case 'delriga': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { $idriga = post('idriga'); $query = 'DELETE FROM `co_righe_contratti` WHERE idcontratto='.prepare($id_record).' AND id='.prepare($idriga); @@ -241,9 +241,9 @@ switch (post('op')) { // Scollegamento intervento da contratto case 'unlink': - if (isset($get['idcontratto']) && isset($get['idintervento'])) { - $idcontratto = $get['idcontratto']; - $idintervento = $get['idintervento']; + if (get('idcontratto') !== null && get('idintervento') !== null) { + $idcontratto = get('idcontratto'); + $idintervento = get('idintervento'); $query = 'DELETE FROM `co_contratti_promemoria` WHERE idcontratto='.prepare($idcontratto).' AND idintervento='.prepare($idintervento); $dbo->query($query); diff --git a/modules/contratti/ajax.php b/modules/contratti/ajax.php index 08ff70195..15c4f2624 100644 --- a/modules/contratti/ajax.php +++ b/modules/contratti/ajax.php @@ -11,8 +11,8 @@ if (get('op') == 'get_costo_orario') { // Copia ordine di servizio da un impianto ad un altro elseif (get('op') == 'get_pianificazione_os') { - $idcontratto = $get['idcontratto']; - $matricola_src = $get['matricola_src']; + $idcontratto = get('idcontratto'); + $matricola_src = get('matricola_src'); $ordiniservizio = []; diff --git a/modules/contratti/plugins/actions.php b/modules/contratti/plugins/actions.php index 036f99c8c..542ffa550 100644 --- a/modules/contratti/plugins/actions.php +++ b/modules/contratti/plugins/actions.php @@ -51,8 +51,8 @@ switch (post('op')) { $prezzo_vendita = post('prezzo_vendita'); $idiva = post('idiva'); - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo_vendita, @@ -60,7 +60,7 @@ switch (post('op')) { 'qta' => $qta, ]); - $idcontratto_riga = $post['idcontratto_riga']; + $idcontratto_riga = post('idcontratto_riga'); // Decremento la quantità //add_movimento_magazzino($idarticolo, -$qta, ['idautomezzo' => $idautomezzo, 'idintervento' => $id_record]); @@ -135,8 +135,8 @@ switch (post('op')) { $prezzo_vendita = post('prezzo_vendita'); $prezzo_acquisto = post('prezzo_acquisto'); - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo_vendita, @@ -150,7 +150,7 @@ switch (post('op')) { $iva = (($prezzo_vendita * $qta) - $sconto) * $rs_iva[0]['percentuale'] / 100; - $idcontratto_riga = $post['idcontratto_riga']; + $idcontratto_riga = post('idcontratto_riga'); $dbo->query('INSERT INTO co_righe_contratti_materiali(descrizione, qta, um, prezzo_vendita, prezzo_acquisto, idiva, desc_iva, iva, sconto, sconto_unitario, tipo_sconto, id_riga_contratto) VALUES ('.prepare($descrizione).', '.prepare($qta).', '.prepare($um).', '.prepare($prezzo_vendita).', '.prepare($prezzo_acquisto).', '.prepare($idiva).', '.prepare($desc_iva).', '.prepare($iva).', '.prepare($sconto).', '.prepare($sconto_unitario).', '.prepare($tipo_sconto).', '.prepare($idcontratto_riga).')'); @@ -166,8 +166,8 @@ switch (post('op')) { $prezzo_vendita = post('prezzo_vendita'); $prezzo_acquisto = post('prezzo_acquisto'); - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo_vendita, diff --git a/modules/contratti/plugins/addfattura.php b/modules/contratti/plugins/addfattura.php index 6cb43126b..d96f937fd 100644 --- a/modules/contratti/plugins/addfattura.php +++ b/modules/contratti/plugins/addfattura.php @@ -2,10 +2,10 @@ include_once __DIR__.'/../../../core.php'; -$idcontratto = $get['idcontratto']; -$idpianificazione = $get['idpianificazione']; -$importo = $get['importo']; -$n_rata = $get['n_rata']; +$idcontratto = get('idcontratto'); +$idpianificazione = get('idpianificazione'); +$importo = get('importo'); +$n_rata = get('n_rata'); // Lettura numero contratto e nome zona $rs = $dbo->fetchArray('SELECT numero, (SELECT descrizione FROM an_zone WHERE id=(SELECT idzona FROM co_ordiniservizio_pianificazionefatture WHERE id='.prepare($idpianificazione).')) AS zona FROM co_contratti WHERE id='.prepare($idcontratto)); diff --git a/modules/contratti/plugins/addpianficazione.php b/modules/contratti/plugins/addpianficazione.php index 6cba09618..6bd53d206 100644 --- a/modules/contratti/plugins/addpianficazione.php +++ b/modules/contratti/plugins/addpianficazione.php @@ -15,8 +15,8 @@ $idanagrafica = $dbo->fetchArray('SELECT `idanagrafica` FROM `co_contratti` WHER $list = '\"1\":\"'.tr('Pianificare a partire da oggi ').date('d/m/Y').'\"'; //promemoria esistente -if (!empty($get['idcontratto_riga'])) { - $idcontratto_riga = $get['idcontratto_riga']; +if (!empty(get('idcontratto_riga'))) { + $idcontratto_riga = get('idcontratto_riga'); $qp = 'SELECT *, (SELECT descrizione FROM in_tipiintervento WHERE idtipointervento=co_contratti_promemoria.idtipointervento) AS tipointervento, (SELECT tempo_standard FROM in_tipiintervento WHERE idtipointervento = co_contratti_promemoria.idtipointervento) AS tempo_standard FROM co_contratti_promemoria WHERE id = '.$idcontratto_riga; $rsp = $dbo->fetchArray($qp); diff --git a/modules/contratti/plugins/ajax_articoli.php b/modules/contratti/plugins/ajax_articoli.php index 6c84b72f3..7e0dbb5ef 100644 --- a/modules/contratti/plugins/ajax_articoli.php +++ b/modules/contratti/plugins/ajax_articoli.php @@ -7,8 +7,8 @@ include_once Modules::filepath('Articoli', 'modutil.php'); //$query = 'SELECT *, (SELECT codice FROM mg_articoli WHERE id=mg_articoli_interventi.idarticolo) AS codice, mg_articoli_interventi.id AS idriga, (SELECT prc_guadagno FROM mg_listini WHERE id=(SELECT idlistino_vendite FROM an_anagrafiche WHERE idanagrafica=(SELECT idanagrafica FROM in_interventi WHERE id=mg_articoli_interventi.idintervento) ) ) AS prc_guadagno FROM mg_articoli_interventi WHERE idintervento='.prepare($id_record).' '.Modules::getAdditionalsQuery('Magazzino'); //$rs = $dbo->fetchArray($query); -if (!empty($get['idcontratto_riga'])) { - $idcontratto_riga = $get['idcontratto_riga']; +if (!empty(get('idcontratto_riga'))) { + $idcontratto_riga = get('idcontratto_riga'); } $query = 'SELECT * FROM co_righe_contratti_articoli WHERE id_riga_contratto='.prepare($idcontratto_riga).' '.Modules::getAdditionalsQuery('Magazzino').' ORDER BY id ASC'; diff --git a/modules/contratti/plugins/ajax_righe.php b/modules/contratti/plugins/ajax_righe.php index 4fe1b8360..00d88bdcf 100644 --- a/modules/contratti/plugins/ajax_righe.php +++ b/modules/contratti/plugins/ajax_righe.php @@ -2,8 +2,8 @@ include_once __DIR__.'/../../../core.php'; -if (!empty($get['idcontratto_riga'])) { - $idcontratto_riga = $get['idcontratto_riga']; +if (!empty(get('idcontratto_riga'))) { + $idcontratto_riga = get('idcontratto_riga'); } $query = 'SELECT * FROM co_righe_contratti_materiali WHERE id_riga_contratto='.prepare($idcontratto_riga).' '.Modules::getAdditionalsQuery('Magazzino').' ORDER BY id ASC'; diff --git a/modules/contratti/plugins/contratti.fatturaordiniservizio.php b/modules/contratti/plugins/contratti.fatturaordiniservizio.php index 657e660cc..d5a4ec247 100644 --- a/modules/contratti/plugins/contratti.fatturaordiniservizio.php +++ b/modules/contratti/plugins/contratti.fatturaordiniservizio.php @@ -28,18 +28,18 @@ if (get('op') == 'add_fatturazione') { // Azzero la pianificazione zone se era già stata fatta, per poter sostituire la pianificazione, // mantenendo però le pianificazioni già fatturate - foreach ($post['zona'] as $data_scadenza => $zone) { + foreach (post('zona') as $data_scadenza => $zone) { foreach ($zone as $n => $idzona) { $dbo->query('DELETE FROM co_ordiniservizio_pianificazionefatture WHERE idzona='.prepare($idzona).' AND iddocumento=0 AND idcontratto='.prepare($id_record)); } } // Ciclo fra le voci in arrivo dal form - foreach ($post['zona'] as $data_scadenza => $zone) { + foreach (post('zona') as $data_scadenza => $zone) { // Ogni data può avere più zone da pianificare foreach ($zone as $n => $idzona) { // Aggiunta pianificazione solo se la zona è spuntata - if (in_array($idzona, $post['idzona'])) { + if (in_array($idzona, post('idzona'))) { // Creazione pianificazione $dbo->query('INSERT INTO co_ordiniservizio_pianificazionefatture(idcontratto, data_scadenza, idzona, iddocumento) VALUES('.prepare($id_record).', '.prepare($data_scadenza).', '.prepare($idzona).', 0)'); } @@ -53,7 +53,7 @@ if (get('op') == 'add_fatturazione') { // Eliminazione pianificazione specifica elseif (get('op') == 'del_pianificazione') { - $idpianificazione = $get['idpianificazione']; + $idpianificazione = get('idpianificazione'); $n = $dbo->fetchNum('SELECT id FROM co_ordiniservizio_pianificazionefatture WHERE id='.prepare($idpianificazione)); @@ -67,11 +67,11 @@ elseif (get('op') == 'del_pianificazione') { // Creazione fattura pianificata elseif (get('op') == 'addfattura') { - $idpianificazione = $get['idpianificazione']; + $idpianificazione = get('idpianificazione'); $descrizione = post('note'); - $data = $post['data']; - $idtipodocumento = $post['idtipodocumento']; - $note = $post['note']; + $data = post('data'); + $idtipodocumento = post('idtipodocumento'); + $note = post('note'); // Lettura idanagrafica $rs = $dbo->fetchArray('SELECT idanagrafica FROM co_contratti WHERE id='.prepare($id_record)); diff --git a/modules/contratti/plugins/contratti.ordiniservizio.interventi.php b/modules/contratti/plugins/contratti.ordiniservizio.interventi.php index 618c0b374..2191fecc9 100644 --- a/modules/contratti/plugins/contratti.ordiniservizio.interventi.php +++ b/modules/contratti/plugins/contratti.ordiniservizio.interventi.php @@ -8,13 +8,13 @@ include_once __DIR__.'/../../../core.php'; if (filter('op') == 'save_ordineservizio') { $n_errors = 0; - if (isset($post['eseguito'])) { - foreach ($post['eseguito'] as $idvoceservizio => $eseguito) { - $presenza = $post['presenza'][$idvoceservizio]; - $esito = $post['esito'][$idvoceservizio]; - $priorita = $post['priorita'][$idvoceservizio]; + if (post('eseguito') !== null) { + foreach (post('eseguito') as $idvoceservizio => $eseguito) { + $presenza = post('presenza')[$idvoceservizio]; + $esito = post('esito')[$idvoceservizio]; + $priorita = post('priorita')[$idvoceservizio]; - if (!$dbo->query('UPDATE co_ordiniservizio_vociservizio SET eseguito='.prepare($eseguito).', presenza='.prepare($presenza).', esito='.prepare($esito).', priorita='.prepare($priorita).', note='.prepare($post['note_ods'][$idvoceservizio]).' WHERE id='.prepare($idvoceservizio))) { + if (!$dbo->query('UPDATE co_ordiniservizio_vociservizio SET eseguito='.prepare($eseguito).', presenza='.prepare($presenza).', esito='.prepare($esito).', priorita='.prepare($priorita).', note='.prepare(post('note_ods')[$idvoceservizio]).' WHERE id='.prepare($idvoceservizio))) { ++$n_errors; } } @@ -27,7 +27,7 @@ if (filter('op') == 'save_ordineservizio') { } // Aggiornamento 4 spunte - $dbo->query('UPDATE co_ordiniservizio SET copia_centrale='.prepare($post['copia_centrale']).', copia_cliente='.prepare($post['copia_cliente']).', copia_amministratore='.prepare($post['copia_amministratore']).'", funzionamento_in_sicurezza='.prepare($post['funzionamento_in_sicurezza']).' WHERE idintervento='.prepare($id_record)); + $dbo->query('UPDATE co_ordiniservizio SET copia_centrale='.prepare(post('copia_centrale')).', copia_cliente='.prepare(post('copia_cliente')).', copia_amministratore='.prepare(post('copia_amministratore')).'", funzionamento_in_sicurezza='.prepare(post('funzionamento_in_sicurezza')).' WHERE idintervento='.prepare($id_record)); } /* diff --git a/modules/contratti/plugins/contratti.ordiniservizio.php b/modules/contratti/plugins/contratti.ordiniservizio.php index 675556a0a..0d4ef4e90 100644 --- a/modules/contratti/plugins/contratti.ordiniservizio.php +++ b/modules/contratti/plugins/contratti.ordiniservizio.php @@ -25,16 +25,16 @@ if (get('op') == 'add_ordineservizio') { $prev_data = ''; // Ciclo fra le voci in arrivo dal form - foreach ($post['voce'] as $data_scadenza => $ordiniservizio) { + foreach (post('voce') as $data_scadenza => $ordiniservizio) { $data_scadenza = date_create_from_format('Ym', $data_scadenza)->format(Intl\Formatter::getStandardFormats()['date']); // Ogni data può avere più voci di servizio da salvare foreach ($ordiniservizio as $n => $idvoce) { // Aggiunta ordine di servizio solo se la voce è spuntata - if (in_array($idvoce, $post['idvoce'])) { + if (in_array($idvoce, post('idvoce'))) { // Creazione ordine di servizio per data di scadenza if ($prev_data != $data_scadenza) { - $dbo->query('INSERT INTO co_ordiniservizio(idcontratto, data_scadenza, idimpianto, stato) VALUES('.prepare($id_record).', '.prepare($data_scadenza).', '.prepare($post['matricola']).", 'aperto')"); + $dbo->query('INSERT INTO co_ordiniservizio(idcontratto, data_scadenza, idimpianto, stato) VALUES('.prepare($id_record).', '.prepare($data_scadenza).', '.prepare(post('matricola')).", 'aperto')"); $idordineservizio = $dbo->lastInsertedID(); } @@ -50,7 +50,7 @@ if (get('op') == 'add_ordineservizio') { // Eliminazione pianificazione specifica elseif (get('op') == 'del_ordineservizio') { - $idordineservizio = $get['idordineservizio']; + $idordineservizio = get('idordineservizio'); $n = $dbo->fetchNum('SELECT id FROM co_ordiniservizio WHERE id='.prepare($idordineservizio)." AND stato='aperto'"); diff --git a/modules/contratti/plugins/contratti.pianificazioneinterventi.php b/modules/contratti/plugins/contratti.pianificazioneinterventi.php index 56a718d43..657ffd712 100644 --- a/modules/contratti/plugins/contratti.pianificazioneinterventi.php +++ b/modules/contratti/plugins/contratti.pianificazioneinterventi.php @@ -26,7 +26,7 @@ switch (filter('op')) { $idtipointervento = filter('idtipointervento'); $richiesta = filter('richiesta'); $idsede = filter('idsede_c'); - $idimpianti = implode(',', $post['idimpianti']); + $idimpianti = implode(',', post('idimpianti')); $query = 'UPDATE co_contratti_promemoria SET idtipointervento='.prepare($idtipointervento).', data_richiesta='.prepare($data_richiesta).', richiesta='.prepare($richiesta).', idsede='.prepare($idsede).', idimpianti='.prepare($idimpianti).' WHERE id = '.prepare($idcontratto_riga); @@ -130,7 +130,7 @@ switch (filter('op')) { App::flash()->info(tr('Promemoria intervento pianificato!')); //pianificare anche l' intervento? - if ($post['pianifica_intervento']) { + if (post('pianifica_intervento')) { /*$orario_inizio = post('orario_inizio'); $orario_fine = post('orario_fine');*/ diff --git a/modules/contratti/row-add.php b/modules/contratti/row-add.php index 22e2f771a..4250f8bc5 100644 --- a/modules/contratti/row-add.php +++ b/modules/contratti/row-add.php @@ -38,9 +38,9 @@ if (!empty($rss)) { // Importazione della gestione dedicata $file = 'riga'; -if (isset($get['is_descrizione'])) { +if (get('is_descrizione') !== null) { $file = 'descrizione'; -} elseif (isset($get['is_articolo'])) { +} elseif (get('is_articolo') !== null) { $file = 'articolo'; } diff --git a/modules/contratti/row-edit.php b/modules/contratti/row-edit.php index 0a2365975..169c58212 100644 --- a/modules/contratti/row-edit.php +++ b/modules/contratti/row-edit.php @@ -15,7 +15,7 @@ $options = [ ]; // Dati della riga -$rsr = $dbo->fetchArray('SELECT * FROM co_righe_contratti WHERE idcontratto='.prepare($id_record).' AND id='.prepare($get['idriga'])); +$rsr = $dbo->fetchArray('SELECT * FROM co_righe_contratti WHERE idcontratto='.prepare($id_record).' AND id='.prepare(get('idriga'))); $result = $rsr[0]; $result['prezzo'] = $rsr[0]['subtotale'] / $rsr[0]['qta']; diff --git a/modules/ddt/actions.php b/modules/ddt/actions.php index 08146ccd9..8b346b91a 100644 --- a/modules/ddt/actions.php +++ b/modules/ddt/actions.php @@ -18,11 +18,11 @@ if ($module['name'] == 'Ddt di vendita') { switch (post('op')) { case 'add': $idanagrafica = post('idanagrafica'); - $data = $post['data']; - $dir = $post['dir']; + $data = post('data'); + $dir = post('dir'); $idtipoddt = post('idtipoddt'); - if (isset($post['idanagrafica'])) { + if (post('idanagrafica') !== null) { $numero = get_new_numeroddt($data); $numero_esterno = ($dir == 'entrata') ? get_new_numerosecondarioddt($data) : ''; @@ -68,8 +68,8 @@ switch (post('op')) { $bollo = 0; } - $tipo_sconto = $post['tipo_sconto_generico']; - $sconto = $post['sconto_generico']; + $tipo_sconto = post('tipo_sconto_generico'); + $sconto = post('sconto_generico'); // Leggo la descrizione del pagamento $query = 'SELECT descrizione FROM co_pagamenti WHERE id='.prepare($idpagamento); @@ -135,19 +135,19 @@ switch (post('op')) { break; case 'addarticolo': - if (isset($post['idarticolo'])) { + if (post('idarticolo') !== null) { $dir = post('dir'); $idarticolo = post('idarticolo'); $descrizione = post('descrizione'); $idiva = post('idiva'); - $qta = $post['qta']; - $prezzo = $post['prezzo']; + $qta = post('qta'); + $prezzo = post('prezzo'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -155,7 +155,7 @@ switch (post('op')) { 'qta' => $qta, ]); - add_articolo_inddt($id_record, $idarticolo, $descrizione, $idiva, $qta, $post['um'], $prezzo * $qta, $sconto, $sconto_unitario, $tipo_sconto); + add_articolo_inddt($id_record, $idarticolo, $descrizione, $idiva, $qta, post('um'), $prezzo * $qta, $sconto, $sconto_unitario, $tipo_sconto); // Ricalcolo inps, ritenuta e bollo ricalcola_costiagg_ddt($id_record); @@ -170,12 +170,12 @@ switch (post('op')) { $idiva = post('idiva'); $um = post('um'); - $prezzo = $post['prezzo']; - $qta = $post['qta']; + $prezzo = post('prezzo'); + $qta = post('qta'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -215,7 +215,7 @@ switch (post('op')) { // Creazione ddt da ordine case 'ddt_da_ordine': $totale_ordine = 0.00; - $data = $post['data']; + $data = post('data'); $idanagrafica = post('idanagrafica'); $idpagamento = post('idpagamento'); $idconto = post('idconto'); @@ -233,22 +233,22 @@ switch (post('op')) { $id_record = $dbo->lastInsertedID(); // Lettura di tutte le righe della tabella in arrivo - foreach ($post['qta_da_evadere'] as $idriga => $value) { + foreach (post('qta_da_evadere') as $idriga => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$idriga] == 'on') { + if (post('evadere')[$idriga] == 'on') { $idarticolo = post('idarticolo')[$idriga]; $descrizione = post('descrizione')[$idriga]; - $qta = $post['qta_da_evadere'][$idriga]; + $qta = post('qta_da_evadere')[$idriga]; $um = post('um')[$idriga]; $abilita_serial = post('abilita_serial')[$idriga]; - $subtot = $post['subtot'][$idriga] * $qta; - $sconto = $post['sconto'][$idriga]; + $subtot = post('subtot')[$idriga] * $qta; + $sconto = post('sconto')[$idriga]; $sconto = $sconto * $qta; $idiva = post('idiva')[$idriga]; - $iva = $post['iva'][$idriga] * $qta; + $iva = post('iva')[$idriga] * $qta; $qprc = 'SELECT tipo_sconto, sconto_unitario FROM or_righe_ordini WHERE id='.prepare($idriga); $rsprc = $dbo->fetchArray($qprc); @@ -266,7 +266,7 @@ switch (post('op')) { $riga = $dbo->lastInsertedID(); // Aggiornamento seriali dalla riga dell'ordine - $serials = is_array($post['serial'][$idriga]) ? $post['serial'][$idriga] : []; + $serials = is_array(post('serial')[$idriga]) ? post('serial')[$idriga] : []; $serials = array_filter($serials, function ($value) { return !empty($value); }); $dbo->sync('mg_prodotti', ['id_riga_ddt' => $riga, 'dir' => $dir, 'id_articolo' => $idarticolo], ['serial' => $serials]); @@ -349,17 +349,17 @@ switch (post('op')) { // Modifica riga case 'editriga': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { // Selezione costi da intervento $idriga = post('idriga'); $descrizione = post('descrizione'); - $prezzo = $post['prezzo']; - $qta = $post['qta']; + $prezzo = post('prezzo'); + $qta = post('qta'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -480,10 +480,10 @@ switch (post('op')) { break; case 'add_serial': - $idriga = $post['idriga']; - $idarticolo = $post['idarticolo']; + $idriga = post('idriga'); + $idarticolo = post('idarticolo'); - $serials = (array) $post['serial']; + $serials = (array) post('serial'); foreach ($serials as $key => $value) { if (empty($value)) { unset($serials[$key]); @@ -511,21 +511,21 @@ switch (post('op')) { // aggiungi righe da ordine case 'add_ordine': - $idordine = $post['iddocumento']; + $idordine = post('iddocumento'); // Lettura di tutte le righe della tabella in arrivo - foreach ($post['qta_da_evadere'] as $i => $value) { + foreach (post('qta_da_evadere') as $i => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$i] == 'on') { + if (post('evadere')[$i] == 'on') { $idrigaordine = $i; $idarticolo = post('idarticolo')[$i]; $descrizione = post('descrizione')[$i]; - $qta = $post['qta_da_evadere'][$i]; + $qta = post('qta_da_evadere')[$i]; $um = post('um')[$i]; - $subtot = $post['subtot'][$i] * $qta; - $sconto = $post['sconto'][$i]; + $subtot = post('subtot')[$i] * $qta; + $sconto = post('sconto')[$i]; $sconto = $sconto * $qta; $qprc = 'SELECT tipo_sconto, sconto_unitario FROM or_righe_ordini WHERE id='.prepare($idrigaordine); diff --git a/modules/ddt/row-add.php b/modules/ddt/row-add.php index 3fe2967a3..fe4affd93 100644 --- a/modules/ddt/row-add.php +++ b/modules/ddt/row-add.php @@ -46,9 +46,9 @@ if (!empty($rss)) { // Importazione della gestione dedicata $file = 'riga'; -if (isset($get['is_descrizione'])) { +if (get('is_descrizione') !== null) { $file = 'descrizione'; -} elseif (isset($get['is_articolo'])) { +} elseif (get('is_articolo') !== null) { $file = 'articolo'; $options['op'] = 'addarticolo'; diff --git a/modules/ddt/row-edit.php b/modules/ddt/row-edit.php index 09de156e7..872f49324 100644 --- a/modules/ddt/row-edit.php +++ b/modules/ddt/row-edit.php @@ -22,7 +22,7 @@ $options = [ ]; // Dati della riga -$rsr = $dbo->fetchArray('SELECT * FROM dt_righe_ddt WHERE idddt='.prepare($id_record).' AND id='.prepare($get['idriga'])); +$rsr = $dbo->fetchArray('SELECT * FROM dt_righe_ddt WHERE idddt='.prepare($id_record).' AND id='.prepare(get('idriga'))); $result = $rsr[0]; $result['prezzo'] = $rsr[0]['subtotale'] / $rsr[0]['qta']; diff --git a/modules/emails/actions.php b/modules/emails/actions.php index c296273b1..fdd43740c 100644 --- a/modules/emails/actions.php +++ b/modules/emails/actions.php @@ -5,10 +5,10 @@ include_once __DIR__.'/../../core.php'; switch (post('op')) { case 'add': $dbo->insert('zz_emails', [ - 'name' => $post['name'], - 'id_module' => $post['module'], - 'id_smtp' => $post['smtp'], - 'subject' => $post['subject'], + 'name' => post('name'), + 'id_module' => post('module'), + 'id_smtp' => post('smtp'), + 'subject' => post('subject'), ]); $id_record = $dbo->lastInsertedID(); @@ -19,18 +19,18 @@ switch (post('op')) { case 'update': $dbo->update('zz_emails', [ - 'name' => $post['name'], - 'id_smtp' => $post['smtp'], - 'icon' => $post['icon'], - 'subject' => $post['subject'], - 'reply_to' => $post['reply_to'], - 'cc' => $post['cc'], - 'bcc' => $post['bcc'], - 'body' => $_POST['body'], // $post['body'], - 'read_notify' => $post['read_notify'], + 'name' => post('name'), + 'id_smtp' => post('smtp'), + 'icon' => post('icon'), + 'subject' => post('subject'), + 'reply_to' => post('reply_to'), + 'cc' => post('cc'), + 'bcc' => post('bcc'), + 'body' => $_POST['body'], // post('body'), + 'read_notify' => post('read_notify'), ], ['id' => $id_record]); - $dbo->sync('zz_email_print', ['id_email' => $id_record], ['id_print' => (array) $post['prints']]); + $dbo->sync('zz_email_print', ['id_email' => $id_record], ['id_print' => (array) post('prints')]); App::flash()->info(tr('Informazioni salvate correttamente!')); diff --git a/modules/fatture/actions.php b/modules/fatture/actions.php index c40aa787a..217d4d1a2 100644 --- a/modules/fatture/actions.php +++ b/modules/fatture/actions.php @@ -19,8 +19,8 @@ if ($module['name'] == 'Fatture di vendita') { switch (post('op')) { case 'add': $idanagrafica = post('idanagrafica'); - $data = $post['data']; - $dir = $post['dir']; + $data = post('data'); + $dir = post('dir'); $idtipodocumento = post('idtipodocumento'); $id_segment = post('id_segment'); @@ -68,7 +68,7 @@ switch (post('op')) { break; case 'update': - if (isset($post['id_record'])) { + if (post('id_record') !== null) { $idstatodocumento = post('idstatodocumento'); $idpagamento = post('idpagamento'); @@ -272,7 +272,7 @@ switch (post('op')) { break; case 'addintervento': - if (!empty($id_record) && isset($post['idintervento'])) { + if (!empty($id_record) && post('idintervento') !== null) { $idintervento = post('idintervento'); $descrizione = post('descrizione'); $idiva = post('idiva'); @@ -495,18 +495,18 @@ switch (post('op')) { break; case 'addpreventivo': - if (!empty($id_record) && isset($post['idpreventivo'])) { + if (!empty($id_record) && post('idpreventivo') !== null) { $idpreventivo = post('idpreventivo'); $descrizione = post('descrizione'); $idiva = post('idiva'); $idconto = post('idconto'); - $prezzo = $post['prezzo']; + $prezzo = post('prezzo'); $qta = 1; // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -515,7 +515,7 @@ switch (post('op')) { ]); $subtot = 0; - $aggiorna_budget = ($post['aggiorna_budget'] == 'on') ? 1 : 0; + $aggiorna_budget = (post('aggiorna_budget') == 'on') ? 1 : 0; // Leggo l'anagrafica del cliente $rs = $dbo->fetchArray('SELECT idanagrafica, numero FROM `co_preventivi` WHERE id='.prepare($idpreventivo)); @@ -543,7 +543,7 @@ switch (post('op')) { $ritenutaacconto = ($prezzo - $sconto + $rivalsainps) / 100 * $rs[0]['percentuale']; } - if (!empty($post['import'])) { + if (!empty(post('import'))) { // Replicazione delle righe del preventivo sul documento $righe = $dbo->fetchArray('SELECT idarticolo, idiva, desc_iva, iva, iva_indetraibile, descrizione, subtotale, um, qta, sconto, sconto_unitario, tipo_sconto, IFNULL( (SELECT mg_articoli.abilita_serial FROM mg_articoli WHERE mg_articoli.id=co_righe_preventivi.idarticolo), 0 ) AS abilita_serial FROM co_righe_preventivi WHERE idpreventivo='.prepare($idpreventivo)); @@ -623,18 +623,18 @@ switch (post('op')) { break; case 'addcontratto': - if (!empty($id_record) && isset($post['idcontratto'])) { + if (!empty($id_record) && post('idcontratto') !== null) { $idcontratto = post('idcontratto'); $descrizione = post('descrizione'); $idiva = post('idiva'); $idconto = post('idconto'); - $prezzo = $post['prezzo']; + $prezzo = post('prezzo'); $qta = 1; // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -643,7 +643,7 @@ switch (post('op')) { ]); $subtot = 0; - $aggiorna_budget = ($post['aggiorna_budget'] == 'on') ? 1 : 0; + $aggiorna_budget = (post('aggiorna_budget') == 'on') ? 1 : 0; // Leggo l'anagrafica del cliente $rs = $dbo->fetchArray('SELECT idanagrafica, numero FROM `co_contratti` WHERE id='.prepare($idcontratto)); @@ -696,7 +696,7 @@ switch (post('op')) { break; case 'addarticolo': - if (!empty($id_record) && isset($post['idarticolo'])) { + if (!empty($id_record) && post('idarticolo') !== null) { $idarticolo = post('idarticolo'); $descrizione = post('descrizione'); @@ -704,16 +704,16 @@ switch (post('op')) { $idconto = post('idconto'); $idum = post('um'); - $qta = $post['qta']; + $qta = post('qta'); if (!empty($record['is_reversed'])) { $qta = -$qta; } - $prezzo = $post['prezzo']; + $prezzo = post('prezzo'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -736,16 +736,16 @@ switch (post('op')) { $um = post('um'); $calcolo_ritenutaacconto = post('calcolo_ritenutaacconto'); - $qta = $post['qta']; + $qta = post('qta'); if (!empty($record['is_reversed'])) { $qta = -$qta; } - $prezzo = $post['prezzo']; + $prezzo = post('prezzo'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -799,7 +799,7 @@ switch (post('op')) { break; case 'editriga': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { // Selezione costi da intervento $idriga = post('idriga'); $descrizione = post('descrizione'); @@ -808,16 +808,16 @@ switch (post('op')) { $um = post('um'); $calcolo_ritenutaacconto = post('calcolo_ritenutaacconto'); - $qta = $post['qta']; + $qta = post('qta'); if (!empty($record['is_reversed'])) { $qta = -$qta; } - $prezzo = $post['prezzo']; + $prezzo = post('prezzo'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -907,11 +907,11 @@ switch (post('op')) { // Creazione fattura da ddt case 'fattura_da_ddt': $totale_fattura = 0.00; - $data = $post['data']; - $idanagrafica = $post['idanagrafica']; - $idarticolo = $post['idarticolo']; - $idpagamento = $post['idpagamento']; - $idddt = $post['idddt']; + $data = post('data'); + $idanagrafica = post('idanagrafica'); + $idarticolo = post('idarticolo'); + $idpagamento = post('idpagamento'); + $idddt = post('idddt'); $id_segment = post('id_segment'); $numero = get_new_numerofattura($data); @@ -935,16 +935,16 @@ switch (post('op')) { $id_record = $dbo->lastInsertedID(); // Lettura di tutte le righe della tabella in arrivo - foreach ($post['qta_da_evadere'] as $i => $value) { + foreach (post('qta_da_evadere') as $i => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$i] == 'on') { + if (post('evadere')[$i] == 'on') { $idrigaddt = $i; $idarticolo = post('idarticolo')[$i]; $descrizione = post('descrizione')[$i]; - $qta = $post['qta_da_evadere'][$i]; - $um = $post['um'][$i]; - $subtot = $post['subtot'][$i] * $qta; - $sconto = $post['sconto'][$i]; + $qta = post('qta_da_evadere')[$i]; + $um = post('um')[$i]; + $subtot = post('subtot')[$i] * $qta; + $sconto = post('sconto')[$i]; $sconto = $sconto * $qta; $idiva = post('idiva')[$i]; @@ -973,7 +973,7 @@ switch (post('op')) { $idriga = add_articolo_infattura($id_record, $idarticolo, $descrizione, $idiva_acquisto, $qta, $prezzo_acquisto, $sconto, $sconto_unitario, $tipo_sconto); // Aggiornamento seriali dalla riga dell'ordine - $serials = is_array($post['serial'][$i]) ? $post['serial'][$i] : []; + $serials = is_array(post('serial')[$i]) ? post('serial')[$i] : []; $serials = array_filter($serials, function ($value) { return !empty($value); }); $dbo->sync('mg_prodotti', ['id_riga_documento' => $idriga, 'dir' => $dir, 'id_articolo' => $idarticolo], ['serial' => $serials]); @@ -1000,12 +1000,12 @@ switch (post('op')) { // Creazione fattura da ordine case 'fattura_da_ordine': $totale_fattura = 0.00; - $data = $post['data']; - $idanagrafica = $post['idanagrafica']; - $idarticolo = $post['idarticolo']; - $idpagamento = $post['idpagamento']; - $idconto = $post['idconto']; - $idordine = $post['idordine']; + $data = post('data'); + $idanagrafica = post('idanagrafica'); + $idarticolo = post('idarticolo'); + $idpagamento = post('idpagamento'); + $idconto = post('idconto'); + $idordine = post('idordine'); $id_segment = post('id_segment'); $numero = get_new_numerofattura($data); $numero_esterno = get_new_numerosecondariofattura($data); @@ -1017,17 +1017,17 @@ switch (post('op')) { $id_record = $dbo->lastInsertedID(); // Lettura di tutte le righe della tabella in arrivo - foreach ($post['qta_da_evadere'] as $i => $value) { + foreach (post('qta_da_evadere') as $i => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$i] == 'on') { + if (post('evadere')[$i] == 'on') { $idriga = $i; $idarticolo = post('idarticolo')[$i]; $descrizione = post('descrizione')[$i]; $qta = post('qta_da_evadere')[$i]; $um = post('um')[$i]; - $subtot = $post['subtot'][$i] * $qta; + $subtot = post('subtot')[$i] * $qta; $idiva = post('idiva')[$i]; - $iva = $post['iva'][$i] * $qta; + $iva = post('iva')[$i] * $qta; $sconto = post('sconto')[$i]; $sconto = $sconto * $qta; @@ -1054,7 +1054,7 @@ switch (post('op')) { $riga = add_articolo_infattura($id_record, $idarticolo, $descrizione, $idiva_acquisto, $qta, $prezzo_acquisto, $sconto, $sconto_unitario, $tipo_sconto); // Aggiornamento seriali dalla riga dell'ordine - $serials = is_array($post['serial'][$i]) ? $post['serial'][$i] : []; + $serials = is_array(post('serial')[$i]) ? post('serial')[$i] : []; $serials = array_filter($serials, function ($value) { return !empty($value); }); $dbo->sync('mg_prodotti', ['id_riga_documento' => $riga, 'dir' => $dir, 'id_articolo' => $idarticolo], ['serial' => $serials]); @@ -1111,24 +1111,24 @@ switch (post('op')) { // aggiungi righe da ddt case 'add_ddt': - $idddt = $post['iddocumento']; + $idddt = post('iddocumento'); $rs = $dbo->fetchArray('SELECT * FROM co_documenti WHERE id='.prepare($id_record)); $idconto = $rs[0]['idconto']; // Lettura di tutte le righe della tabella in arrivo - foreach ($post['qta_da_evadere'] as $i => $value) { + foreach (post('qta_da_evadere') as $i => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$i] == 'on') { + if (post('evadere')[$i] == 'on') { $idrigaddt = $i; $idarticolo = post('idarticolo')[$i]; $descrizione = post('descrizione')[$i]; - $qta = $post['qta_da_evadere'][$i]; + $qta = post('qta_da_evadere')[$i]; $um = post('um')[$i]; - $subtot = $post['subtot'][$i] * $qta; - $sconto = $post['sconto'][$i]; + $subtot = post('subtot')[$i] * $qta; + $sconto = post('sconto')[$i]; $sconto = $sconto * $qta; $qprc = 'SELECT tipo_sconto, sconto_unitario FROM dt_righe_ddt WHERE id='.prepare($idrigaddt); @@ -1179,7 +1179,7 @@ switch (post('op')) { // Scollegamento intervento da documento case 'unlink_intervento': - if (!empty($id_record) && isset($post['idriga'])) { + if (!empty($id_record) && post('idriga') !== null) { $idriga = post('idriga'); // Lettura preventivi collegati @@ -1244,7 +1244,7 @@ switch (post('op')) { // Scollegamento preventivo da documento case 'unlink_preventivo': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { $idriga = post('idriga'); // Lettura preventivi collegati @@ -1294,7 +1294,7 @@ switch (post('op')) { // Scollegamento contratto da documento case 'unlink_contratto': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { $idriga = post('idriga'); // Lettura contratti collegati @@ -1344,7 +1344,7 @@ switch (post('op')) { // Scollegamento riga generica da documento case 'unlink_riga': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { $idriga = post('idriga'); rimuovi_riga_fattura($id_record, $idriga, $dir); @@ -1361,10 +1361,10 @@ switch (post('op')) { break; case 'add_serial': - $idriga = $post['idriga']; - $idarticolo = $post['idarticolo']; + $idriga = post('idriga'); + $idarticolo = post('idarticolo'); - $serials = (array) $post['serial']; + $serials = (array) post('serial'); foreach ($serials as $key => $value) { if (empty($value)) { unset($serials[$key]); @@ -1392,21 +1392,21 @@ switch (post('op')) { // aggiungi righe da ordine case 'add_ordine': - $idordine = $post['iddocumento']; + $idordine = post('iddocumento'); // Lettura di tutte le righe della tabella in arrivo - foreach ($post['qta_da_evadere'] as $i => $value) { + foreach (post('qta_da_evadere') as $i => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$i] == 'on') { + if (post('evadere')[$i] == 'on') { $idriga = $i; $idarticolo = post('idarticolo')[$i]; $descrizione = post('descrizione')[$i]; - $qta = $post['qta_da_evadere'][$i]; + $qta = post('qta_da_evadere')[$i]; $um = post('um')[$i]; - $subtot = $post['subtot'][$i] * $qta; - $sconto = $post['sconto'][$i]; + $subtot = post('subtot')[$i] * $qta; + $sconto = post('sconto')[$i]; $sconto = $sconto * $qta; $qprc = 'SELECT tipo_sconto, sconto_unitario FROM or_righe_ordini WHERE id='.prepare($idriga); @@ -1473,18 +1473,18 @@ switch (post('op')) { $id_record = $dbo->lastInsertedID(); // Lettura di tutte le righe della tabella in arrivo - foreach ($post['qta_da_evadere'] as $i => $value) { + foreach (post('qta_da_evadere') as $i => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$i] == 'on') { + if (post('evadere')[$i] == 'on') { $idriga = $i; $idarticolo = post('idarticolo')[$i]; $descrizione = post('descrizione')[$i]; - $qta = -$post['qta_da_evadere'][$i]; + $qta = -post('qta_da_evadere')[$i]; $um = post('um')[$i]; - $subtot = $post['subtot'][$i] * $qta; - $sconto = $post['sconto'][$i]; + $subtot = post('subtot')[$i] * $qta; + $sconto = post('sconto')[$i]; $sconto = $sconto * $qta; $qprc = 'SELECT tipo_sconto, sconto_unitario FROM co_righe_documenti WHERE id='.prepare($idriga); @@ -1513,7 +1513,7 @@ switch (post('op')) { $riga = add_articolo_infattura($id_record, $idarticolo, $descrizione, $idiva_acquisto, $qta, $prezzo_acquisto, $sconto, $sconto_unitario, $tipo_sconto); // Aggiornamento seriali dalla riga dell'ordine - $serials = is_array($post['serial'][$i]) ? $post['serial'][$i] : []; + $serials = is_array(post('serial')[$i]) ? post('serial')[$i] : []; $serials = array_filter($serials, function ($value) { return !empty($value); }); $dbo->sync('mg_prodotti', ['id_riga_documento' => $riga, 'dir' => 'uscita', 'id_articolo' => $idarticolo], ['serial' => $serials]); diff --git a/modules/fatture/crea_documento.php b/modules/fatture/crea_documento.php index 9b281cd1e..85e552576 100644 --- a/modules/fatture/crea_documento.php +++ b/modules/fatture/crea_documento.php @@ -66,10 +66,10 @@ if ($module['name'] == 'Ordini cliente') { $module_name = 'Fatture di vendita'; } -$op = !empty($get['op']) ? $get['op'] : $op; +$op = !empty(get('op')) ? get('op') : $op; $button = ($documento == 'ddt') ? tr('Crea ddt') : tr('Crea fattura'); -$button = !empty($get['op']) ? tr('Aggiungi') : $button; +$button = !empty(get('op')) ? tr('Aggiungi') : $button; // Info documento $rs = $dbo->fetchArray('SELECT * FROM '.$table.' WHERE id='.prepare($id_record)); @@ -95,7 +95,7 @@ if (!empty($rs)) { echo '

'.tr('Seleziona le righe e le relative quantità da inserire nel documento').'.

-
+ @@ -106,7 +106,7 @@ if (!empty($rs)) { '; - if (empty($get['op'])) { + if (empty(get('op'))) { echo '
diff --git a/modules/fatture/row-add.php b/modules/fatture/row-add.php index 78ffc72f3..9920e9287 100644 --- a/modules/fatture/row-add.php +++ b/modules/fatture/row-add.php @@ -55,9 +55,9 @@ if (!empty($rss)) { // Importazione della gestione dedicata $file = 'riga'; -if (isset($get['is_descrizione'])) { +if (get('is_descrizione') !== null) { $file = 'descrizione'; -} elseif (isset($get['is_articolo'])) { +} elseif (get('is_articolo') !== null) { $file = 'articolo'; $options['op'] = 'addarticolo'; diff --git a/modules/fatture/row-edit.php b/modules/fatture/row-edit.php index d56099e4a..9e6bf5585 100644 --- a/modules/fatture/row-edit.php +++ b/modules/fatture/row-edit.php @@ -25,7 +25,7 @@ $options = [ ]; // Dati della riga -$rsr = $dbo->fetchArray('SELECT * FROM co_righe_documenti WHERE iddocumento='.prepare($id_record).' AND id='.prepare($get['idriga'])); +$rsr = $dbo->fetchArray('SELECT * FROM co_righe_documenti WHERE iddocumento='.prepare($id_record).' AND id='.prepare(get('idriga'))); $result = $rsr[0]; $result['prezzo'] = $rsr[0]['subtotale'] / $rsr[0]['qta']; diff --git a/modules/interventi/actions.php b/modules/interventi/actions.php index 21d5332e1..6344004ea 100644 --- a/modules/interventi/actions.php +++ b/modules/interventi/actions.php @@ -84,11 +84,11 @@ switch (post('op')) { $km = post('km')[$idriga]; // Lettura tariffe in base al tipo di intervento ed al tecnico - $idtipointervento_tecnico = $post['idtipointerventot'][$idriga]; - $rs = $dbo->fetchArray('SELECT * FROM in_interventi_tecnici WHERE idtecnico='.prepare($post['idtecnico'][$idriga]).' AND idintervento='.prepare($id_record)); + $idtipointervento_tecnico = post('idtipointerventot')[$idriga]; + $rs = $dbo->fetchArray('SELECT * FROM in_interventi_tecnici WHERE idtecnico='.prepare(post('idtecnico')[$idriga]).' AND idintervento='.prepare($id_record)); if ($idtipointervento_tecnico != $rs[0]['idtipointervento']) { - $rsc = $dbo->fetchArray('SELECT * FROM in_tariffe WHERE idtecnico='.prepare($post['idtecnico'][$idriga]).' AND idtipointervento='.prepare($idtipointervento_tecnico)); + $rsc = $dbo->fetchArray('SELECT * FROM in_tariffe WHERE idtecnico='.prepare(post('idtecnico')[$idriga]).' AND idtipointervento='.prepare($idtipointervento_tecnico)); if ($rsc[0]['costo_ore'] != 0 || $rsc[0]['costo_km'] != 0 || $rsc[0]['costo_dirittochiamata'] != 0 || $rsc[0]['costo_ore_tecnico'] != 0 || $rsc[0]['costo_km_tecnico'] != 0 || $rsc[0]['costo_dirittochiamata_tecnico'] != 0) { $prezzo_ore_unitario = $rsc[0]['costo_ore']; @@ -174,8 +174,8 @@ switch (post('op')) { } } - $tipo_sconto = $post['tipo_sconto_globale']; - $sconto = $post['sconto_globale']; + $tipo_sconto = post('tipo_sconto_globale'); + $sconto = post('sconto_globale'); // Salvataggio modifiche intervento $dbo->update('in_interventi', [ @@ -225,7 +225,7 @@ switch (post('op')) { $richiesta = post('richiesta'); $idautomezzo = null; - if (!empty($codice) && !empty($post['idanagrafica']) && !empty($post['idtipointervento'])) { + if (!empty($codice) && !empty(post('idanagrafica')) && !empty(post('idtipointervento'))) { // Salvataggio modifiche intervento $dbo->insert('in_interventi', [ 'idanagrafica' => post('idanagrafica'), @@ -311,8 +311,8 @@ switch (post('op')) { } } - if (!empty($post['idordineservizio'])) { - $dbo->query('UPDATE co_ordiniservizio SET idintervento='.prepare($id_record).' WHERE id='.prepare($post['idordineservizio'])); + if (!empty(post('idordineservizio'))) { + $dbo->query('UPDATE co_ordiniservizio SET idintervento='.prepare($id_record).' WHERE id='.prepare(post('idordineservizio'))); } // Collegamenti tecnici/interventi @@ -432,8 +432,8 @@ switch (post('op')) { $prezzo_vendita = post('prezzo_vendita'); $prezzo_acquisto = post('prezzo_acquisto'); - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo_vendita, @@ -460,8 +460,8 @@ switch (post('op')) { $prezzo_vendita = post('prezzo_vendita'); $prezzo_acquisto = post('prezzo_acquisto'); - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo_vendita, @@ -540,8 +540,8 @@ switch (post('op')) { $prezzo_vendita = post('prezzo_vendita'); $idiva = post('idiva'); - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo_vendita, @@ -608,10 +608,10 @@ switch (post('op')) { break; case 'add_serial': - $idriga = $post['idriga']; - $idarticolo = $post['idarticolo']; + $idriga = post('idriga'); + $idarticolo = post('idarticolo'); - $serials = (array) $post['serial']; + $serials = (array) post('serial'); foreach ($serials as $key => $value) { if (empty($value)) { unset($serials[$key]); diff --git a/modules/modelli_primanota/actions.php b/modules/modelli_primanota/actions.php index 0b09f91f2..433c0a7d1 100644 --- a/modules/modelli_primanota/actions.php +++ b/modules/modelli_primanota/actions.php @@ -9,7 +9,7 @@ switch (post('op')) { $idmastrino = get_new_idmastrino('co_movimenti_modelli'); $descrizione = post('descrizione'); - for ($i = 0; $i < sizeof($post['idconto']); ++$i) { + for ($i = 0; $i < sizeof(post('idconto')); ++$i) { $idconto = post('idconto')[$i]; $query = 'INSERT INTO co_movimenti_modelli(idmastrino, descrizione, idconto) VALUES('.prepare($idmastrino).', '.prepare($descrizione).', '.prepare($idconto).')'; if ($dbo->query($query)) { @@ -26,7 +26,7 @@ switch (post('op')) { // Eliminazione prima nota $dbo->query('DELETE FROM co_movimenti_modelli WHERE idmastrino='.prepare($idmastrino)); - for ($i = 0; $i < sizeof($post['idconto']); ++$i) { + for ($i = 0; $i < sizeof(post('idconto')); ++$i) { $idconto = post('idconto')[$i]; $query = 'INSERT INTO co_movimenti_modelli(idmastrino, descrizione, idconto) VALUES('.prepare($idmastrino).', '.prepare($descrizione).', '.prepare($idconto).')'; if ($dbo->query($query)) { diff --git a/modules/my_impianti/plugins/my_impianti.interventi.php b/modules/my_impianti/plugins/my_impianti.interventi.php index 965c9e6cc..9aa8054f3 100644 --- a/modules/my_impianti/plugins/my_impianti.interventi.php +++ b/modules/my_impianti/plugins/my_impianti.interventi.php @@ -31,9 +31,9 @@ if (filter('op') == 'link_myimpianti') { App::flash()->info(tr('Informazioni impianti salvate!')); } elseif (filter('op') == 'link_componenti') { - $components = (array) $post['componenti']; + $components = (array) post('componenti'); - $list = (!empty($post['list'])) ? explode(',', $post['list']) : []; + $list = (!empty(post('list'))) ? explode(',', post('list')) : []; foreach ($list as $delete) { if (!in_array($delete, $components)) { $dbo->query('DELETE FROM my_componenti_interventi WHERE id_componente = '.prepare($delete).' AND id_intervento = '.prepare($id_record)); diff --git a/modules/ordini/actions.php b/modules/ordini/actions.php index 50e03c348..fc58b1724 100644 --- a/modules/ordini/actions.php +++ b/modules/ordini/actions.php @@ -17,13 +17,13 @@ switch (post('op')) { case 'add': $idanagrafica = post('idanagrafica'); - $data = $post['data']; + $data = post('data'); // Leggo se l'ordine è cliente o fornitore $rs = $dbo->fetchArray('SELECT id FROM or_tipiordine WHERE dir='.prepare($dir)); $idtipoordine = $rs[0]['id']; - if (isset($post['idanagrafica'])) { + if (post('idanagrafica') !== null) { $numero = get_new_numeroordine($data); if ($dir == 'entrata') { $numero_esterno = get_new_numerosecondarioordine($data); @@ -57,7 +57,7 @@ switch (post('op')) { $numero_esterno = post('numero_esterno'); $numero = post('numero'); - $data = $post['data']; + $data = post('data'); $idanagrafica = post('idanagrafica'); $note = post('note'); $note_aggiuntive = post('note_aggiuntive'); @@ -69,8 +69,8 @@ switch (post('op')) { $totale_imponibile = get_imponibile_ordine($id_record); $totale_ordine = get_totale_ordine($id_record); - $tipo_sconto = $post['tipo_sconto_generico']; - $sconto = $post['sconto_generico']; + $tipo_sconto = post('tipo_sconto_generico'); + $sconto = post('sconto_generico'); if ($dir == 'uscita') { $idrivalsainps = post('idrivalsainps'); @@ -134,7 +134,7 @@ switch (post('op')) { break; case 'addarticolo': - if (isset($post['idarticolo'])) { + if (post('idarticolo') !== null) { $idarticolo = post('idarticolo'); $idiva = post('idiva'); $descrizione = post('descrizione'); @@ -142,8 +142,8 @@ switch (post('op')) { $prezzo_vendita = post('prezzo'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -151,7 +151,7 @@ switch (post('op')) { 'qta' => $qta, ]); - add_articolo_inordine($id_record, $idarticolo, $descrizione, $idiva, $qta, $post['um'], $prezzo_vendita * $qta, $sconto, $sconto_unitario, $tipo_sconto); + add_articolo_inordine($id_record, $idarticolo, $descrizione, $idiva, $qta, post('um'), $prezzo_vendita * $qta, $sconto, $sconto_unitario, $tipo_sconto); App::flash()->info(tr('Articolo aggiunto!')); } @@ -168,8 +168,8 @@ switch (post('op')) { $subtot = $prezzo * $qta; // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -251,7 +251,7 @@ switch (post('op')) { // Modifica riga case 'editriga': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { $idriga = post('idriga'); $descrizione = post('descrizione'); $prezzo = post('prezzo'); @@ -261,8 +261,8 @@ switch (post('op')) { $subtot = $prezzo * $qta; // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -335,10 +335,10 @@ switch (post('op')) { break; case 'add_serial': - $idriga = $post['idriga']; - $idarticolo = $post['idarticolo']; + $idriga = post('idriga'); + $idarticolo = post('idarticolo'); - $serials = (array) $post['serial']; + $serials = (array) post('serial'); foreach ($serials as $key => $value) { if (empty($value)) { unset($serials[$key]); @@ -369,13 +369,13 @@ switch (post('op')) { $idanagrafica = post('idanagrafica'); $idpreventivo = post('idpreventivo'); - $data = $post['data']; + $data = post('data'); // Leggo se l'ordine è cliente o fornitore $rs = $dbo->fetchArray('SELECT id FROM or_tipiordine WHERE dir='.prepare($dir)); $idtipoordine = $rs[0]['id']; - if (isset($post['idanagrafica'])) { + if (post('idanagrafica') !== null) { $numero = get_new_numeroordine($data); if ($dir == 'entrata') { $numero_esterno = get_new_numerosecondarioordine($data); @@ -406,9 +406,9 @@ switch (post('op')) { // Lettura di tutte le righe della tabella in arrivo // Inserisco anche le righe descrittive - foreach ($post['evadere'] as $i => $value) { + foreach (post('evadere') as $i => $value) { // Processo solo le righe da evadere - if ($post['evadere'][$i] == 'on') { + if (post('evadere')[$i] == 'on') { $descrizione = post('descrizione')[$i]; $prezzo = post('subtot')[$i]; $qta = post('qta_da_evadere')[$i]; diff --git a/modules/ordini/crea_documento.php b/modules/ordini/crea_documento.php index a921e32c7..1a8ba9526 100644 --- a/modules/ordini/crea_documento.php +++ b/modules/ordini/crea_documento.php @@ -27,10 +27,10 @@ $row = str_replace('id', 'id_riga_', $id); $module_name = 'Ordini cliente'; -$op = !empty($get['op']) ? $get['op'] : $op; +$op = !empty(get('op')) ? get('op') : $op; $button = tr('Crea ordine'); -$button = !empty($get['op']) ? tr('Aggiungi') : $button; +$button = !empty(get('op')) ? tr('Aggiungi') : $button; // Info documento $rs = $dbo->fetchArray('SELECT * FROM '.$table.' WHERE id='.prepare($id_record)); @@ -52,7 +52,7 @@ if (!empty($rs)) { echo '

'.tr('Seleziona le righe e le relative quantità da inserire nell\'ordine.').'.

- + diff --git a/modules/ordini/row-add.php b/modules/ordini/row-add.php index 345d7db1b..c677a59fb 100644 --- a/modules/ordini/row-add.php +++ b/modules/ordini/row-add.php @@ -46,9 +46,9 @@ if (!empty($rss)) { // Importazione della gestione dedicata $file = 'riga'; -if (isset($get['is_descrizione'])) { +if (get('is_descrizione') !== null) { $file = 'descrizione'; -} elseif (isset($get['is_articolo'])) { +} elseif (get('is_articolo') !== null) { $file = 'articolo'; $options['op'] = 'addarticolo'; diff --git a/modules/ordini/row-edit.php b/modules/ordini/row-edit.php index 922c81e91..f151e2598 100644 --- a/modules/ordini/row-edit.php +++ b/modules/ordini/row-edit.php @@ -22,7 +22,7 @@ $options = [ ]; // Dati della riga -$rsr = $dbo->fetchArray('SELECT * FROM or_righe_ordini WHERE idordine='.prepare($id_record).' AND id='.prepare($get['idriga'])); +$rsr = $dbo->fetchArray('SELECT * FROM or_righe_ordini WHERE idordine='.prepare($id_record).' AND id='.prepare(get('idriga'))); $result = $rsr[0]; $result['prezzo'] = $rsr[0]['subtotale'] / $rsr[0]['qta']; diff --git a/modules/pagamenti/actions.php b/modules/pagamenti/actions.php index 806784fb2..aedae796c 100644 --- a/modules/pagamenti/actions.php +++ b/modules/pagamenti/actions.php @@ -7,32 +7,32 @@ switch (filter('op')) { $descrizione = filter('descrizione'); if (isset($descrizione)) { - foreach ($post['id'] as $key => $id) { + foreach (post('id') as $key => $id) { // Data fatturazione $giorno = 0; // Data fatturazione fine mese - if ($post['scadenza'][$key] == 2) { + if (post('scadenza')[$key] == 2) { $giorno = -1; } // Data fatturazione giorno fisso - if ($post['scadenza'][$key] == 3) { - $giorno = $post['giorno'][$key]; + if (post('scadenza')[$key] == 3) { + $giorno = post('giorno')[$key]; } // Data fatturazione fine mese (giorno fisso) - elseif ($post['scadenza'][$key] == 4) { - $giorno = -$post['giorno'][$key] - 1; + elseif (post('scadenza')[$key] == 4) { + $giorno = -post('giorno')[$key] - 1; } $array = [ - 'num_giorni' => $post['distanza'][$key], + 'num_giorni' => post('distanza')[$key], 'giorno' => $giorno, - 'prc' => $post['percentuale'][$key], + 'prc' => post('percentuale')[$key], 'descrizione' => $descrizione, - 'idconto_vendite' => $post['idconto_vendite'], - 'idconto_acquisti' => $post['idconto_acquisti'], + 'idconto_vendite' => post('idconto_vendite'), + 'idconto_acquisti' => post('idconto_acquisti'), ]; if (!empty($id)) { diff --git a/modules/partitario/actions.php b/modules/partitario/actions.php index d27787dc9..f31513611 100644 --- a/modules/partitario/actions.php +++ b/modules/partitario/actions.php @@ -9,7 +9,7 @@ switch (post('op')) { $numero = post('numero'); $descrizione = post('descrizione'); - if (isset($post['idpianodeiconti2'])) { + if (post('idpianodeiconti2') !== null) { // Controllo che non sia stato usato un numero non valido del conto $query = 'SELECT idpianodeiconti2, numero FROM co_pianodeiconti3 WHERE numero='.prepare($numero).' AND idpianodeiconti2='.prepare($idpianodeiconti2); $rs = $dbo->fetchArray($query); diff --git a/modules/partitario/dettagli_movimento.php b/modules/partitario/dettagli_movimento.php index 1fb711091..d9501a477 100644 --- a/modules/partitario/dettagli_movimento.php +++ b/modules/partitario/dettagli_movimento.php @@ -2,8 +2,8 @@ include_once __DIR__.'/../../core.php'; -$id_movimento = $get['id_movimento']; -$id_conto = $get['id_conto']; +$id_movimento = get('id_movimento'); +$id_conto = get('id_conto'); $query = 'SELECT *, (subtotale-sconto) AS imponibile, (co_movimenti.descrizione) AS desc_fatt, (co_righe_documenti.descrizione) AS desc_riga FROM co_movimenti INNER JOIN co_righe_documenti ON co_movimenti.iddocumento = co_righe_documenti.iddocumento WHERE co_movimenti.id = '.prepare($id_movimento).' AND co_movimenti.idconto = '.prepare($id_conto); $rs = $dbo->fetchArray($query); diff --git a/modules/preventivi/actions.php b/modules/preventivi/actions.php index a9e68ed98..ff17c2b18 100644 --- a/modules/preventivi/actions.php +++ b/modules/preventivi/actions.php @@ -78,16 +78,16 @@ switch (post('op')) { $tempi_consegna = post('tempi_consegna'); $numero = post('numero'); - $tipo_sconto = $post['tipo_sconto_generico']; - $sconto = $post['sconto_generico']; + $tipo_sconto = post('tipo_sconto_generico'); + $sconto = post('sconto_generico'); // $budget = post('budget'); // $budget = str_replace( ",", ".", $budget ); - $data_bozza = $post['data_bozza']; - $data_accettazione = $post['data_accettazione']; - $data_rifiuto = $post['data_rifiuto']; - $data_conclusione = $post['data_conclusione']; + $data_bozza = post('data_bozza'); + $data_accettazione = post('data_accettazione'); + $data_rifiuto = post('data_rifiuto'); + $data_conclusione = post('data_conclusione'); $esclusioni = post('esclusioni'); $descrizione = post('descrizione'); $validita = post('validita'); @@ -136,7 +136,7 @@ switch (post('op')) { break; case 'addintervento': - if (isset($post['idintervento'])) { + if (post('idintervento') !== null) { // Selezione costi da intervento $idintervento = post('idintervento'); $rs = $dbo->fetchArray('SELECT * FROM in_interventi WHERE id='.prepare($idintervento)); @@ -158,7 +158,7 @@ switch (post('op')) { // Scollegamento articolo da ordine case 'unlink_articolo': - if (isset($post['idriga'])) { + if (post('idriga') !== null) { $idriga = post('idriga'); $idarticolo = post('idarticolo'); @@ -178,7 +178,7 @@ switch (post('op')) { // Scollegamento intervento da preventivo case 'unlink': if (isset($_GET['idpreventivo']) && isset($_GET['idintervento'])) { - $idintervento = $get['idintervento']; + $idintervento = get('idintervento'); $query = 'DELETE FROM `co_preventivi_interventi` WHERE idpreventivo='.prepare($id_record).' AND idintervento='.prepare($idintervento); $dbo->query($query); @@ -204,12 +204,12 @@ switch (post('op')) { $idiva = post('idiva'); $descrizione = post('descrizione'); - $qta = $post['qta']; - $prezzo = $post['prezzo']; + $qta = post('qta'); + $prezzo = post('prezzo'); // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, @@ -247,13 +247,13 @@ switch (post('op')) { $rs = $dbo->fetchArray('SELECT * FROM co_righe_preventivi WHERE id='.prepare($idriga)); $is_descrizione = $rs[0]['is_descrizione']; - $qta = $post['qta']; - $prezzo = $post['prezzo']; + $qta = post('qta'); + $prezzo = post('prezzo'); $subtot = $prezzo * $qta; // Calcolo dello sconto - $sconto_unitario = $post['sconto']; - $tipo_sconto = $post['tipo_sconto']; + $sconto_unitario = post('sconto'); + $tipo_sconto = post('tipo_sconto'); $sconto = calcola_sconto([ 'sconto' => $sconto_unitario, 'prezzo' => $prezzo, diff --git a/modules/preventivi/row-add.php b/modules/preventivi/row-add.php index 310cfeca0..79e3f30f6 100644 --- a/modules/preventivi/row-add.php +++ b/modules/preventivi/row-add.php @@ -38,9 +38,9 @@ if (!empty($rss)) { // Importazione della gestione dedicata $file = 'riga'; -if (isset($get['is_descrizione'])) { +if (get('is_descrizione') !== null) { $file = 'descrizione'; -} elseif (isset($get['is_articolo'])) { +} elseif (get('is_articolo') !== null) { $file = 'articolo'; } diff --git a/modules/preventivi/row-edit.php b/modules/preventivi/row-edit.php index f97cb9fa9..6241718f8 100644 --- a/modules/preventivi/row-edit.php +++ b/modules/preventivi/row-edit.php @@ -15,7 +15,7 @@ $options = [ ]; // Dati della riga -$rsr = $dbo->fetchArray('SELECT * FROM co_righe_preventivi WHERE idpreventivo='.prepare($id_record).' AND id='.prepare($get['idriga'])); +$rsr = $dbo->fetchArray('SELECT * FROM co_righe_preventivi WHERE idpreventivo='.prepare($id_record).' AND id='.prepare(get('idriga'))); $result = $rsr[0]; $result['prezzo'] = $rsr[0]['subtotale'] / $rsr[0]['qta']; diff --git a/modules/primanota/actions.php b/modules/primanota/actions.php index 95e656627..0a3356f7b 100644 --- a/modules/primanota/actions.php +++ b/modules/primanota/actions.php @@ -8,7 +8,7 @@ switch (post('op')) { case 'add': $all_ok = true; $iddocumento = post('iddocumento'); - $data = $post['data']; + $data = post('data'); $idmastrino = get_new_idmastrino(); $descrizione = post('descrizione'); @@ -20,10 +20,10 @@ switch (post('op')) { $totale = 0; $totale_pagato = 0; - for ($i = 0; $i < sizeof($post['idconto']); ++$i) { + for ($i = 0; $i < sizeof(post('idconto')); ++$i) { $idconto = post('idconto')[$i]; - $dare = $post['dare'][$i]; - $avere = $post['avere'][$i]; + $dare = post('dare')[$i]; + $avere = post('avere')[$i]; if (!empty($dare) || !empty($avere)) { if (!empty($avere)) { @@ -96,7 +96,7 @@ switch (post('op')) { if (post('crea_modello') == '1') { $idmastrino = get_new_idmastrino('co_movimenti_modelli'); - for ($i = 0; $i < sizeof($post['idconto']); ++$i) { + for ($i = 0; $i < sizeof(post('idconto')); ++$i) { $idconto = post('idconto')[$i]; $query = 'INSERT INTO co_movimenti_modelli(idmastrino, descrizione, idconto) VALUES('.prepare($idmastrino).', '.prepare($descrizione).', '.prepare($idconto).')'; $dbo->query($query); @@ -108,7 +108,7 @@ switch (post('op')) { case 'editriga': $all_ok = true; $iddocumento = post('iddocumento'); - $data = $post['data']; + $data = post('data'); $idmastrino = post('idmastrino'); $descrizione = post('descrizione'); @@ -130,10 +130,10 @@ switch (post('op')) { $ragione_sociale = $rs[0]['ragione_sociale']; $dir = $rs[0]['dir']; - for ($i = 0; $i < sizeof($post['idconto']); ++$i) { + for ($i = 0; $i < sizeof(post('idconto')); ++$i) { $idconto = post('idconto')[$i]; - $dare = $post['dare'][$i]; - $avere = $post['avere'][$i]; + $dare = post('dare')[$i]; + $avere = post('avere')[$i]; if ($dare != '' && $dare != 0) { $totale = $dare; diff --git a/modules/scadenzario/actions.php b/modules/scadenzario/actions.php index 7f080d3c6..e2f5813d4 100644 --- a/modules/scadenzario/actions.php +++ b/modules/scadenzario/actions.php @@ -11,7 +11,7 @@ switch (post('op')) { $totale_utente = 0; // Verifico se il totale sommato è uguale al totale da pagare - foreach ($post['scadenza'] as $idscadenza => $da_pagare) { + foreach (post('scadenza') as $idscadenza => $da_pagare) { $totale_utente += floatval($da_pagare); } @@ -19,7 +19,7 @@ switch (post('op')) { $totale_utente = (string) $totale_utente; if ($totale_utente == $totale_da_pagare) { - foreach ($post['scadenza'] as $idscadenza => $da_pagare) { + foreach (post('scadenza') as $idscadenza => $da_pagare) { $dbo->query('UPDATE co_scadenziario SET da_pagare='.prepare($da_pagare).', pagato='.prepare(post('pagato')[$idscadenza]).', scadenza='.prepare(post('data')[$idscadenza]).' WHERE id='.prepare($idscadenza)); } diff --git a/modules/smtp/actions.php b/modules/smtp/actions.php index 05bfb20e3..4ddf54590 100644 --- a/modules/smtp/actions.php +++ b/modules/smtp/actions.php @@ -5,9 +5,9 @@ include_once __DIR__.'/../../core.php'; switch (post('op')) { case 'add': $dbo->insert('zz_smtps', [ - 'name' => $post['name'], - 'from_name' => $post['from_name'], - 'from_address' => $post['from_address'], + 'name' => post('name'), + 'from_name' => post('from_name'), + 'from_address' => post('from_address'), ]); $id_record = $dbo->lastInsertedID(); @@ -18,27 +18,27 @@ switch (post('op')) { case 'update': $dbo->update('zz_smtps', [ - 'name' => $post['name'], - 'note' => $post['note'], - 'server' => $post['server'], - 'port' => $post['port'], - 'username' => $post['username'], - 'password' => $post['password'], - 'from_name' => $post['from_name'], - 'from_address' => $post['from_address'], - 'encryption' => $post['encryption'], - 'pec' => $post['pec'], - 'main' => $post['main'], + 'name' => post('name'), + 'note' => post('note'), + 'server' => post('server'), + 'port' => post('port'), + 'username' => post('username'), + 'password' => post('password'), + 'from_name' => post('from_name'), + 'from_address' => post('from_address'), + 'encryption' => post('encryption'), + 'pec' => post('pec'), + 'main' => post('main'), ], ['id' => $id_record]); - if (!empty($post['main'])) { + if (!empty(post('main'))) { $dbo->query('UPDATE zz_smtps SET main = 0 WHERE id != '.prepare($id_record)); } App::flash()->info(tr('Informazioni salvate correttamente!')); // Validazione indirizzo email mittente - $check_email = Validate::isValidEmail($post['from_address']); + $check_email = Validate::isValidEmail(post('from_address')); // Se $check_email non è null e la riposta è negativa --> mostro il messaggio di avviso. if (!empty($check_email)) { diff --git a/modules/viste/actions.php b/modules/viste/actions.php index cdcfca160..334b370c6 100644 --- a/modules/viste/actions.php +++ b/modules/viste/actions.php @@ -18,10 +18,10 @@ function check_query($query) switch (filter('op')) { case 'update': - $post['options2'] = htmlspecialchars_decode($post['options2'], ENT_QUOTES); + $options2 = htmlspecialchars_decode(post('options2'), ENT_QUOTES); - if (check_query($post['options2'])) { - $dbo->query('UPDATE `zz_modules` SET `title`='.prepare($post['title']).', `options2`='.prepare($post['options2']).' WHERE `id`='.prepare($id_record)); + if (check_query($options2)) { + $dbo->query('UPDATE `zz_modules` SET `title`='.prepare(post('title')).', `options2`='.prepare($options2).' WHERE `id`='.prepare($id_record)); $rs = true; } else { @@ -39,29 +39,29 @@ switch (filter('op')) { case 'fields': $rs = true; - foreach ((array) $post['query'] as $c => $k) { + foreach ((array) post('query') as $c => $k) { // Fix per la protezone contro XSS, che interpreta la sequenza " $post['name'][$c], - 'query' => $post['query'][$c], - 'visible' => $post['visible'][$c], - 'search' => $post['search'][$c], - 'slow' => $post['slow'][$c], - 'format' => $post['format'][$c], - 'summable' => $post['sum'][$c], - 'search_inside' => $post['search_inside'][$c], - 'order_by' => $post['order_by'][$c], + 'name' => post('name')[$c], + 'query' => post('query')[$c], + 'visible' => post('visible')[$c], + 'search' => post('search')[$c], + 'slow' => post('slow')[$c], + 'format' => post('format')[$c], + 'summable' => post('sum')[$c], + 'search_inside' => post('search_inside')[$c], + 'order_by' => post('order_by')[$c], 'id_module' => $id_record, ]; - if (!empty($post['id'][$c]) && !empty($post['query'][$c])) { - $id = $post['id'][$c]; + if (!empty(post('id')[$c]) && !empty(post('query')[$c])) { + $id = post('id')[$c]; $dbo->update('zz_views', $array, ['id' => $id]); - } elseif (!empty($post['query'][$c])) { + } elseif (!empty(post('query')[$c])) { $array['#order'] = '(SELECT IFNULL(MAX(`order`) + 1, 0) FROM zz_views AS t WHERE id_module='.prepare($id_record).')'; $dbo->insert('zz_views', $array); @@ -70,7 +70,7 @@ switch (filter('op')) { } // Aggiornamento dei permessi relativi - $dbo->sync('zz_group_view', ['id_vista' => $id], ['id_gruppo' => (array) $post['gruppi'][$c]]); + $dbo->sync('zz_group_view', ['id_vista' => $id], ['id_gruppo' => (array) post('gruppi')[$c]]); } else { $rs = false; } @@ -87,24 +87,24 @@ switch (filter('op')) { case 'filters': $rs = true; - foreach ((array) $post['query'] as $c => $k) { + foreach ((array) post('query') as $c => $k) { // Fix per la protezone contro XSS, che interpreta la sequenza " $post['name'][$c], - 'idgruppo' => $post['gruppo'][$c], + 'name' => post('name')[$c], + 'idgruppo' => post('gruppo')[$c], 'idmodule' => $id_record, - 'clause' => $post['query'][$c], - 'position' => !empty($post['position'][$c]) ? 'HVN' : 'WHR', + 'clause' => post('query')[$c], + 'position' => !empty(post('position')[$c]) ? 'HVN' : 'WHR', ]; - if (!empty($post['id'][$c]) && !empty($post['query'][$c])) { - $id = $post['id'][$c]; + if (!empty(post('id')[$c]) && !empty(post('query')[$c])) { + $id = post('id')[$c]; $dbo->update('zz_group_module', $array, ['id' => $id]); - } elseif (!empty($post['query'][$c])) { + } elseif (!empty(post('query')[$c])) { $dbo->insert('zz_group_module', $array); $id = $dbo->lastInsertedID(); diff --git a/plugins/referenti/actions.php b/plugins/referenti/actions.php index 74ab0a282..8387add1f 100644 --- a/plugins/referenti/actions.php +++ b/plugins/referenti/actions.php @@ -8,11 +8,11 @@ switch ($operazione) { case 'addreferente': $dbo->insert('an_referenti', [ 'idanagrafica' => $id_parent, - 'nome' => $post['nome'], - 'mansione' => $post['mansione'], - 'telefono' => $post['telefono'], - 'email' => $post['email'], - 'idsede' => $post['idsede'], + 'nome' => post('nome'), + 'mansione' => post('mansione'), + 'telefono' => post('telefono'), + 'email' => post('email'), + 'idsede' => post('idsede'), ]); $id_record = $dbo->lastInsertedID(); @@ -23,11 +23,11 @@ switch ($operazione) { case 'updatereferente': $dbo->update('an_referenti', [ 'idanagrafica' => $id_parent, - 'nome' => $post['nome'], - 'mansione' => $post['mansione'], - 'telefono' => $post['telefono'], - 'email' => $post['email'], - 'idsede' => $post['idsede'], + 'nome' => post('nome'), + 'mansione' => post('mansione'), + 'telefono' => post('telefono'), + 'email' => post('email'), + 'idsede' => post('idsede'), ], ['id' => $id_record]); App::flash()->info(tr('Salvataggio completato!')); diff --git a/plugins/sedi/actions.php b/plugins/sedi/actions.php index 2b0f64489..2671224e9 100644 --- a/plugins/sedi/actions.php +++ b/plugins/sedi/actions.php @@ -8,16 +8,16 @@ switch ($operazione) { case 'addsede': $dbo->insert('an_sedi', [ 'idanagrafica' => $id_parent, - 'nomesede' => $post['nomesede'], - 'indirizzo' => $post['indirizzo'], - 'indirizzo2' => $post['indirizzo2'], - 'citta' => $post['citta'], - 'cap' => $post['cap'], - 'km' => $post['km'], - 'cellulare' => $post['cellulare'], - 'telefono' => $post['telefono'], - 'email' => $post['email'], - 'idzona' => $post['idzona'], + 'nomesede' => post('nomesede'), + 'indirizzo' => post('indirizzo'), + 'indirizzo2' => post('indirizzo2'), + 'citta' => post('citta'), + 'cap' => post('cap'), + 'km' => post('km'), + 'cellulare' => post('cellulare'), + 'telefono' => post('telefono'), + 'email' => post('email'), + 'idzona' => post('idzona'), ]); $id_record = $dbo->lastInsertedID(); @@ -27,24 +27,24 @@ switch ($operazione) { case 'updatesede': $array = [ - 'nomesede' => $post['nomesede'], - 'indirizzo' => $post['indirizzo'], - 'indirizzo2' => $post['indirizzo2'], - 'piva' => $post['piva'], - 'codice_fiscale' => $post['codice_fiscale'], - 'citta' => $post['citta'], - 'cap' => $post['cap'], - 'provincia' => $post['provincia'], - 'km' => $post['km'], - 'cellulare' => $post['cellulare'], - 'telefono' => $post['telefono'], - 'email' => $post['email'], - 'fax' => $post['fax'], - 'id_nazione' => !empty($post['id_nazione']) ? $post['id_nazione'] : null, - 'idzona' => $post['idzona'], - 'gaddress' => $post['gaddress'], - 'lat' => $post['lat'], - 'lng' => $post['lng'], + 'nomesede' => post('nomesede'), + 'indirizzo' => post('indirizzo'), + 'indirizzo2' => post('indirizzo2'), + 'piva' => post('piva'), + 'codice_fiscale' => post('codice_fiscale'), + 'citta' => post('citta'), + 'cap' => post('cap'), + 'provincia' => post('provincia'), + 'km' => post('km'), + 'cellulare' => post('cellulare'), + 'telefono' => post('telefono'), + 'email' => post('email'), + 'fax' => post('fax'), + 'id_nazione' => !empty(post('id_nazione')) ? post('id_nazione') : null, + 'idzona' => post('idzona'), + 'gaddress' => post('gaddress'), + 'lat' => post('lat'), + 'lng' => post('lng'), ]; $dbo->update('an_sedi', $array, ['id' => $id_record]);