From 8be6e110094293ceb633d3e116ef153c1b111843 Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Fri, 2 Mar 2018 19:01:30 +0100 Subject: [PATCH] Cleanup del codice --- lib/functions.php | 11 +- modules/anagrafiche/actions.php | 25 +-- .../contratti.pianificazioneinterventi.php | 8 +- modules/interventi/modutil.php | 1 - modules/smtp/actions.php | 20 ++- modules/utenti/edit.php | 3 +- modules/utenti/modutil.php | 11 +- src/App.php | 23 ++- src/Database.php | 13 +- src/Modules.php | 4 +- src/Validate.php | 151 ++++++++---------- 11 files changed, 125 insertions(+), 145 deletions(-) diff --git a/lib/functions.php b/lib/functions.php index 5fa8dab81..62f211e9a 100644 --- a/lib/functions.php +++ b/lib/functions.php @@ -300,13 +300,12 @@ function do_backup($path = null) */ function backup_tables($file) { - global $backup_dir; - global $db_host; - global $db_name; - global $db_username; - global $db_password; + $config = App::getConfig(); + + $dump = new Ifsnop\Mysqldump\Mysqldump('mysql:host='.$config['db_host'].';dbname='.$config['db_name'], $config['db_username'], $config['db_password'], [ + 'add-drop-table' => true, + ]); - $dump = new Ifsnop\Mysqldump\Mysqldump('mysql:host='.$db_host.';dbname='.$db_name, $db_username, $db_password, ['add-drop-table' => true]); $dump->start($file); } diff --git a/modules/anagrafiche/actions.php b/modules/anagrafiche/actions.php index fa70ff36a..01aefaf75 100644 --- a/modules/anagrafiche/actions.php +++ b/modules/anagrafiche/actions.php @@ -9,12 +9,15 @@ $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'])); + // 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' => trim(strtoupper($post['piva'])), - 'codice_fiscale' => trim(strtoupper($post['codice_fiscale'])), + 'piva' => $post['piva'], + 'codice_fiscale' => $post['codice_fiscale'], 'data_nascita' => $post['data_nascita'], 'luogo_nascita' => $post['luogo_nascita'], 'sesso' => $post['sesso'], @@ -68,18 +71,16 @@ switch (post('op')) { $_SESSION['infos'][] = str_replace('_NAME_', '"'.$post['ragione_sociale'].'"', "Informazioni per l'anagrafica _NAME_ salvate correttamente!"); - - //validazione piva. + // Validazione della Partita IVA $check_vat_number = Validate::isValidVatNumber(strtoupper($post['piva'])); - //print_r($check_vat_number); - //exit(); - //se $check_vat_number non è null e la riposta è negativa --> mostro il messaggio di avviso. - if ((!is_null($check_vat_number)) and (!$check_vat_number->valid)){ - - if (!empty($check_vat_number->error->info)){ + // Se $check_vat_number non è null e la riposta è negativa --> mostro il messaggio di avviso. + if ((!is_null($check_vat_number)) && (!$check_vat_number->valid)) { + if (!empty($check_vat_number->error->info)) { $_SESSION['errors'][] = $check_vat_number->error->info; - }else{ - $_SESSION['errors'][] = tr('Attenzione questa partita IVA non sembra essere valida: ').strtoupper($post['piva']); + } else { + $_SESSION['errors'][] = tr('Attenzione: la partita IVA _IVA_ sembra non essere valida', [ + '_IVA_' => $post['piva'], + ]); } } diff --git a/modules/contratti/plugins/contratti.pianificazioneinterventi.php b/modules/contratti/plugins/contratti.pianificazioneinterventi.php index 1e1dfe26b..1659d88f3 100644 --- a/modules/contratti/plugins/contratti.pianificazioneinterventi.php +++ b/modules/contratti/plugins/contratti.pianificazioneinterventi.php @@ -49,7 +49,7 @@ switch (filter('op')) { $intervallo = filter('intervallo'); $parti_da_oggi = post('parti_da_oggi'); - if (!empty($idcontratto_riga) and !empty($intervallo)) { + if (!empty($idcontratto_riga) && !empty($intervallo)) { $qp = 'SELECT *, (SELECT idanagrafica FROM co_contratti WHERE id = '.$id_record.' ) AS idanagrafica, (SELECT data_conclusione FROM co_contratti WHERE id = '.$id_record.' ) AS data_conclusione, (SELECT descrizione FROM in_tipiintervento WHERE idtipointervento=co_righe_contratti.idtipointervento) AS tipointervento FROM co_righe_contratti WHERE id = '.$idcontratto_riga; $rsp = $dbo->fetchArray($qp); @@ -72,14 +72,14 @@ switch (filter('op')) { } //inizio controllo data_conclusione, data valida e maggiore della $min_date - if ((date('Y', strtotime($data_conclusione)) > 1970) and (date('Y-m-d', strtotime($min_date)) < date('Y-m-d', strtotime($data_conclusione)))) { + if ((date('Y', strtotime($data_conclusione)) > 1970) && (date('Y-m-d', strtotime($min_date)) < date('Y-m-d', strtotime($data_conclusione)))) { //Ciclo partendo dalla data_richiesta fino all data conclusione del contratto while (date('Y-m-d', strtotime($data_richiesta)) < date('Y-m-d', strtotime($data_conclusione))) { //calcolo nuova data richiesta $data_richiesta = date('Y-m-d', strtotime($data_richiesta.' + '.intval($intervallo).' days')); //controllo nuova data richiesta --> solo date maggiori o uguali di [oggi o data richiesta iniziale] ma che non superano la data di fine del contratto - if ((date('Y-m-d', strtotime($data_richiesta)) >= $min_date) and (date('Y-m-d', strtotime($data_richiesta)) <= date('Y-m-d', strtotime($data_conclusione)))) { + if ((date('Y-m-d', strtotime($data_richiesta)) >= $min_date) && (date('Y-m-d', strtotime($data_richiesta)) <= date('Y-m-d', strtotime($data_conclusione)))) { //Controllo che non esista già un promemoria idcontratto, idtipointervento e data_richiesta. if (count($dbo->fetchArray("SELECT id FROM co_righe_contratti WHERE data_richiesta = '".$data_richiesta."' AND idtipointervento = '".$idtipointervento."' AND idcontratto = '".$id_record."' ")) == 0) { $query = 'INSERT INTO `co_righe_contratti`(`idcontratto`, `idtipointervento`, `data_richiesta`, `richiesta`, `idsede`) VALUES('.prepare($id_record).', '.prepare($idtipointervento).', '.prepare($data_richiesta).', '.prepare($richiesta).', '.prepare($idsede).')'; @@ -237,7 +237,7 @@ if (count($rsp) != 0) { '; echo ' - '; + '; echo ' '; diff --git a/modules/interventi/modutil.php b/modules/interventi/modutil.php index a3a25ad6d..f1fdfb57d 100644 --- a/modules/interventi/modutil.php +++ b/modules/interventi/modutil.php @@ -31,7 +31,6 @@ function get_ore_intervento($idintervento) */ function link_componente_to_articolo($idintervento, $idimpianto, $idarticolo, $qta) { - global $docroot; $dbo = Database::getConnection(); if (!empty($idimpianto) && !empty($idintervento)) { diff --git a/modules/smtp/actions.php b/modules/smtp/actions.php index a3cd5bf13..54bde95bc 100644 --- a/modules/smtp/actions.php +++ b/modules/smtp/actions.php @@ -36,22 +36,20 @@ switch (post('op')) { } $_SESSION['infos'][] = tr('Informazioni salvate correttamente!'); - - //validazione indirizzo email mittente. - $check_email = Validate::isValidEmail($post['from_address'],1,1); - //print_r($check_email); - //exit(); - //se $check_email non è null e la riposta è negativa --> mostro il messaggio di avviso. - if ((!is_null($check_email)) and (!$check_email->smtp_check)){ - if (!empty($check_email->error->info)){ + // Validazione indirizzo email mittente + $check_email = Validate::isValidEmail($post['from_address'], 1, 1); + // Se $check_email non è null e la riposta è negativa --> mostro il messaggio di avviso. + if ((!is_null($check_email)) && (!$check_email->smtp_check)) { + if (!empty($check_email->error->info)) { $_SESSION['errors'][] = $check_email->error->info; - }else{ - $_SESSION['errors'][] = tr('Attenzione questo indirizzo non sembra essere valido: ').$post['from_address']; + } else { + $_SESSION['errors'][] = tr("Attenzione: l'indirizzo email _EMAIL_ sembra non essere valido", [ + '_EMAIL_' => $post['from_address'], + ]); } } - break; case 'delete': diff --git a/modules/utenti/edit.php b/modules/utenti/edit.php index c675d498a..0740df96b 100644 --- a/modules/utenti/edit.php +++ b/modules/utenti/edit.php @@ -113,7 +113,8 @@ if ($record['nome'] != 'Amministratori') { for ($m = 0; $m < count($moduli); ++$m) { $perms_values = ['-', 'r', 'rw']; $perms_names = [tr('Nessun permesso'), tr('Sola lettura'), tr('Lettura e scrittura')]; - echo menuSelection($moduli[$m], -1, $perms_values, $perms_names); + + echo menuSelection($moduli[$m], $id_record, -1, $perms_values, $perms_names); } echo ' diff --git a/modules/utenti/modutil.php b/modules/utenti/modutil.php index 4b1a1ff3e..c7ae4fef7 100644 --- a/modules/utenti/modutil.php +++ b/modules/utenti/modutil.php @@ -2,12 +2,8 @@ include_once __DIR__.'/../../core.php'; -function menuSelection($element, $depth, $perms_values, $perms_names) +function menuSelection($element, $group_id, $depth, $perms_values, $perms_names) { - global $rootdir; - global $id_module; - global $id_record; - $dbo = Database::getConnection(); ++$depth; @@ -18,16 +14,17 @@ function menuSelection($element, $depth, $perms_values, $perms_names) if ($submenus != null && count($submenus) != 0) { $temp = ''; foreach ($submenus as $submenu) { - $temp .= menuSelection($submenu, $depth, $perms_values, $perms_names); + $temp .= menuSelection($submenu, $group_id, $depth, $perms_values, $perms_names); } } + $result .= ' '.str_repeat('     ', $depth).$name.'