From 2690f94d4d05d915e5140446fc592f682ec5d436 Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Tue, 31 Mar 2020 20:31:03 +0200 Subject: [PATCH 1/2] Fix selezione stampe per email --- modules/emails/src/Mail.php | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/modules/emails/src/Mail.php b/modules/emails/src/Mail.php index fe5bb7e3d..83f0ce9bc 100755 --- a/modules/emails/src/Mail.php +++ b/modules/emails/src/Mail.php @@ -65,11 +65,7 @@ class Mail extends Model public function resetPrints() { - $prints = $this->prints; - - foreach ($prints as $print) { - $this->prints()->detach($print->id, ['id_email' => $this->id]); - } + $this->prints()->detach(); } /** From 071f3b827f4f7fb320bfba2d45965bff35d0d69b Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Tue, 31 Mar 2020 20:34:31 +0200 Subject: [PATCH 2/2] Stile del codice --- actions.php | 1 + editor.php | 2 - modules/articoli/ajax/complete.php | 6 +- modules/articoli/validation.php | 2 +- modules/categorie_articoli/edit.php | 16 +- modules/categorie_articoli/row-list.php | 8 +- modules/fatture/edit.php | 2 +- modules/interventi/src/API/v1/Impianti.php | 6 +- modules/interventi/src/API/v1/Rapportino.php | 7 +- modules/my_impianti/src/API/v1/Impianti.php | 5 - modules/preventivi/buttons.php | 2 +- modules/stampe/edit.php | 2 +- plugins/exportFE/src/Validator.php | 3 +- .../add_pianificazione.php | 2 +- plugins/pianificazione_fatturazione/edit.php | 70 ++-- plugins/pianificazione_interventi/edit.php | 344 +++++++++--------- 16 files changed, 230 insertions(+), 248 deletions(-) diff --git a/actions.php b/actions.php index 0c9ef414b..fd99f1fe0 100755 --- a/actions.php +++ b/actions.php @@ -241,6 +241,7 @@ elseif (post('op') == 'send-email') { $mail->addUpload($file); } + // Salvataggio email nella coda di invio $mail->save(); // Invio mail istantaneo diff --git a/editor.php b/editor.php index cfa6f37d9..e253c2bbf 100755 --- a/editor.php +++ b/editor.php @@ -134,13 +134,11 @@ if (empty($record) || !$has_access) { if (!empty($record['deleted_at'])) { $operation = $dbo->fetchOne("SELECT zz_operations.created_at, username FROM zz_operations INNER JOIN zz_users ON zz_operations.id_utente = zz_users.id WHERE op='delete' AND id_module=".prepare($id_module).' AND id_record='.prepare($id_record).' ORDER BY zz_operations.created_at DESC'); - $info = tr('Il record è stato eliminato il _DATE_ da _USER_', [ '_DATE_' => (($operation['created_at']) ? Translator::timestampToLocale($operation['created_at']) : Translator::timestampToLocale($record['deleted_at'])), '_USER_' => ((!empty($operation['username'])) ? $operation['username'] : 'N.D.'), ]).'. '; - echo '
diff --git a/modules/articoli/ajax/complete.php b/modules/articoli/ajax/complete.php index 4dd90567b..9b06fb8b5 100755 --- a/modules/articoli/ajax/complete.php +++ b/modules/articoli/ajax/complete.php @@ -24,7 +24,7 @@ switch ($resource) { ($fatture[$i]['n2_fattura'] != '') ? $n_fattura = $fatture[$i]['n2_fattura'] : $n_fattura = $fatture[$i]['n_fattura']; $link_id = Modules::get('Fatture di vendita')['id']; - echo "Fatt. n. ".$n_fattura." del ".Translator::dateToLocale($fatture[$i]['data_fattura'])." \n"; + echo "Fatt. n. ".$n_fattura.' del '.Translator::dateToLocale($fatture[$i]['data_fattura'])." \n"; echo "".moneyFormat($fatture[$i]['costo_unitario'])."\n"; array_push($ids, '"'.$fatture[$i]['iddocumento'].'"'); } @@ -51,7 +51,7 @@ switch ($resource) { ($fatture[$i]['n2_fattura'] != '') ? $n_fattura = $fatture[$i]['n2_fattura'] : $n_fattura = $fatture[$i]['n_fattura']; $link_id = Modules::get('Fatture di vendita')['id']; - echo "Fatt. n. ".$n_fattura." del ".Translator::dateToLocale($fatture[$i]['data_fattura'])."\n"; + echo "Fatt. n. ".$n_fattura.' del '.Translator::dateToLocale($fatture[$i]['data_fattura'])."\n"; echo "".moneyFormat($fatture[$i]['costo_unitario'])."\n"; } echo "\n"; @@ -78,7 +78,7 @@ switch ($resource) { ($fatture[$i]['n2_fattura'] != '') ? $n_fattura = $fatture[$i]['n2_fattura'] : $n_fattura = $fatture[$i]['n_fattura']; $link_id = Modules::get('Fatture di acquisto')['id']; - echo "Fatt. n. ".$n_fattura." del ".Translator::dateToLocale($fatture[$i]['data_fattura'])."\n"; + echo "Fatt. n. ".$n_fattura.' del '.Translator::dateToLocale($fatture[$i]['data_fattura'])."\n"; echo "".moneyFormat($fatture[$i]['costo_unitario'])."\n"; } echo "\n"; diff --git a/modules/articoli/validation.php b/modules/articoli/validation.php index 2f9aa0d2b..fc1d7efe9 100755 --- a/modules/articoli/validation.php +++ b/modules/articoli/validation.php @@ -22,7 +22,7 @@ switch ($name) { ]; break; - + case 'barcode': $disponibile = Articolo::where([ ['barcode', $value], diff --git a/modules/categorie_articoli/edit.php b/modules/categorie_articoli/edit.php index 2386c82ed..c2e5cc3b6 100755 --- a/modules/categorie_articoli/edit.php +++ b/modules/categorie_articoli/edit.php @@ -94,21 +94,21 @@ if (!empty($elementi)) { $descrizione = tr('Articolo _CODICE_ _DELETED_AT_', [ '_CODICE_' => !empty($elemento['codice']) ? $elemento['codice'] : $elemento['barcode'], '_DELETED_AT_' => (!empty($elemento['deleted_at']) ? tr('Eliminato il:').' '.Translator::dateToLocale($elemento['deleted_at']) : ''), - ]); - $modulo = 'Articoli'; - $id = $elemento['id']; + ]); + $modulo = 'Articoli'; + $id = $elemento['id']; - echo ' + echo '
  • '.Modules::link($modulo, $id, $descrizione).'
    • '; -} + } -echo ' + echo '
    '; -}else { +} else { echo ' '.tr('Elimina').' '; -} \ No newline at end of file +} diff --git a/modules/categorie_articoli/row-list.php b/modules/categorie_articoli/row-list.php index 9c949f353..cc846c639 100755 --- a/modules/categorie_articoli/row-list.php +++ b/modules/categorie_articoli/row-list.php @@ -4,12 +4,8 @@ include_once __DIR__.'/../../core.php'; $subcategorie = $dbo->fetchArray('SELECT * FROM `mg_categorie` WHERE `parent`='.prepare($id_record).' ORDER BY nome ASC '); - - - - foreach ($subcategorie as $sub) { - $n_articoli = $dbo->fetchNum('SELECT * FROM `mg_articoli` WHERE `id_sottocategoria`='.prepare($sub['id'])); + $n_articoli = $dbo->fetchNum('SELECT * FROM `mg_articoli` WHERE `id_sottocategoria`='.prepare($sub['id'])); echo ' '.$sub['nome'].' @@ -17,7 +13,7 @@ foreach ($subcategorie as $sub) { '.$sub['nota'].' - + diff --git a/modules/fatture/edit.php b/modules/fatture/edit.php index 3e431569d..5615a9ac4 100755 --- a/modules/fatture/edit.php +++ b/modules/fatture/edit.php @@ -615,7 +615,7 @@ if (!$block_edit) { $art_query = 'SELECT id FROM mg_articoli WHERE attivo = 1 AND deleted_at IS NULL'; if ($dir == 'entrata') { $art_query .= ' AND (qta > 0 OR servizio = 1)'; - }else{ + } else { //Gli articoli possono essere creati al volo direttamente dal modale di aggiunta articolo $art_query .= ' OR 1=1'; } diff --git a/modules/interventi/src/API/v1/Impianti.php b/modules/interventi/src/API/v1/Impianti.php index c7ed69140..b5830dac8 100755 --- a/modules/interventi/src/API/v1/Impianti.php +++ b/modules/interventi/src/API/v1/Impianti.php @@ -3,12 +3,8 @@ namespace Modules\Interventi\API\v1; use API\Interfaces\CreateInterface; -use API\Interfaces\DeleteInterface; use API\Interfaces\RetrieveInterface; -use API\Interfaces\UpdateInterface; use API\Resource; -use Modules; -use Modules\Anagrafiche\Anagrafica; class Impianti extends Resource implements RetrieveInterface, CreateInterface { @@ -32,7 +28,7 @@ class Impianti extends Resource implements RetrieveInterface, CreateInterface $id_record = $data['id_intervento']; $database = database(); - $database->query('DELETE FROM my_impianti_interventi WHERE `idintervento` = :id_intervento', [ + $database->query('DELETE FROM my_impianti_interventi WHERE `idintervento` = :id_intervento', [ ':id_intervento' => $id_record, ]); diff --git a/modules/interventi/src/API/v1/Rapportino.php b/modules/interventi/src/API/v1/Rapportino.php index 2f5683933..93fe8520d 100755 --- a/modules/interventi/src/API/v1/Rapportino.php +++ b/modules/interventi/src/API/v1/Rapportino.php @@ -3,14 +3,10 @@ namespace Modules\Interventi\API\v1; use API\Interfaces\CreateInterface; -use API\Interfaces\DeleteInterface; use API\Interfaces\RetrieveInterface; -use API\Interfaces\UpdateInterface; use API\Resource; -use Modules; -use Modules\Anagrafiche\Anagrafica; -use Modules\Emails\Template; use Modules\Emails\Mail; +use Modules\Emails\Template; class Rapportino extends Resource implements RetrieveInterface, CreateInterface { @@ -65,7 +61,6 @@ class Rapportino extends Resource implements RetrieveInterface, CreateInterface $mail->save(); return [ - ]; } } diff --git a/modules/my_impianti/src/API/v1/Impianti.php b/modules/my_impianti/src/API/v1/Impianti.php index bf7bd3d41..7a86f7980 100755 --- a/modules/my_impianti/src/API/v1/Impianti.php +++ b/modules/my_impianti/src/API/v1/Impianti.php @@ -2,13 +2,8 @@ namespace Modules\Impianti\API\v1; -use API\Interfaces\CreateInterface; -use API\Interfaces\DeleteInterface; use API\Interfaces\RetrieveInterface; -use API\Interfaces\UpdateInterface; use API\Resource; -use Modules; -use Modules\Anagrafiche\Anagrafica; class Impianti extends Resource implements RetrieveInterface { diff --git a/modules/preventivi/buttons.php b/modules/preventivi/buttons.php index 24002e448..1f50d0b04 100755 --- a/modules/preventivi/buttons.php +++ b/modules/preventivi/buttons.php @@ -6,7 +6,7 @@ echo' '; // Crea revisione -if (!$record['is_completato'] || ($record['is_completato'] && $record['stato']=='Rifiutato')) { +if (!$record['is_completato'] || ($record['is_completato'] && $record['stato'] == 'Rifiutato')) { echo ' '; } diff --git a/modules/stampe/edit.php b/modules/stampe/edit.php index 83f73333f..0d06c636d 100755 --- a/modules/stampe/edit.php +++ b/modules/stampe/edit.php @@ -50,7 +50,7 @@ echo ' if (!empty($variables)) { echo ' -

    '.tr("Puoi utilizzare le seguenti variabili per generare il nome del file").':

    +

    '.tr('Puoi utilizzare le seguenti variabili per generare il nome del file').':