From 5fcdb7f918a39dd24333c272282369b66a1cc46b Mon Sep 17 00:00:00 2001 From: Thomas Zilio Date: Mon, 22 Jul 2019 15:04:39 +0200 Subject: [PATCH 1/5] Fix ripristino qta_evasa --- CHANGELOG.md | 3 +++ modules/fatture/actions.php | 15 ++++++------ modules/fatture/modutil.php | 48 +++++-------------------------------- 3 files changed, 16 insertions(+), 50 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 1e34e2bc1..57d041d85 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -48,6 +48,9 @@ Il formato utilizzato è basato sulle linee guida di [Keep a Changelog](http://k - Miglioramento del sistema di importazione delle ricevute delle Fatture Elettroniche, per permetterne il caricamento manuale - Standardizzazione dei nomi predefiniti delle stampa e dei relativi file generati +### Rimosso (Removed) + - Supporto ai raggruppamenti di **Contratti** e **Preventivi** nelle **Fatture** + ### Fixed - Fix export delle tabelle principali in Excel diff --git a/modules/fatture/actions.php b/modules/fatture/actions.php index d31a481ff..5d5f98bac 100644 --- a/modules/fatture/actions.php +++ b/modules/fatture/actions.php @@ -151,10 +151,10 @@ switch (post('op')) { // eliminazione documento case 'delete': - $rs = $dbo->fetchArray('SELECT id FROM co_righe_documenti WHERE iddocumento='.prepare($id_record)); + $righe = $dbo->fetchArray('SELECT id FROM co_righe_documenti WHERE iddocumento='.prepare($id_record)); // Controllo sui seriali - foreach ($rs as $r) { + foreach ($righe as $r) { $non_rimovibili = seriali_non_rimuovibili('id_riga_documento', $r['id'], $dir); if (!empty($non_rimovibili)) { flash()->error(tr('Alcuni serial number sono già stati utilizzati!')); @@ -163,16 +163,10 @@ switch (post('op')) { } } - // Rimozione righe - foreach ($rs as $r) { - rimuovi_riga_fattura($id_record, $r['id'], $dir); - } - // Se ci sono dei preventivi collegati li rimetto nello stato "In attesa di pagamento" $rs = $dbo->fetchArray('SELECT idpreventivo FROM co_righe_documenti WHERE iddocumento='.prepare($id_record).' AND idpreventivo IS NOT NULL'); for ($i = 0; $i < sizeof($rs); ++$i) { $dbo->query("UPDATE co_preventivi SET idstato=(SELECT id FROM co_statipreventivi WHERE descrizione='In lavorazione') WHERE id=".prepare($rs[$i]['idpreventivo'])); - $dbo->query('UPDATE co_righe_preventivi SET qta_evasa=0 WHERE idpreventivo='.prepare($rs[$i]['idpreventivo'])); } // Se ci sono degli interventi collegati li rimetto nello stato "Completato" @@ -181,6 +175,11 @@ switch (post('op')) { $dbo->query("UPDATE in_interventi SET idstatointervento = (SELECT idstatointervento FROM in_statiintervento WHERE descrizione = 'Completato') WHERE id=".prepare($rs[$i]['idintervento'])); } + // Rimozione righe + foreach ($righe as $r) { + rimuovi_riga_fattura($id_record, $r['id'], $dir); + } + elimina_scadenza($id_record); elimina_movimento($id_record); diff --git a/modules/fatture/modutil.php b/modules/fatture/modutil.php index e521e0ff1..a7397fe76 100755 --- a/modules/fatture/modutil.php +++ b/modules/fatture/modutil.php @@ -696,51 +696,15 @@ function rimuovi_riga_fattura($id_documento, $id_riga, $dir) } // Rimozione articoli collegati ad un preventivo importato con riga unica - if (empty($riga['idarticolo']) && $riga['idpreventivo']) { - //rimetto a magazzino gli articoli collegati al preventivo - $rsa = $dbo->fetchArray('SELECT id, idarticolo, qta FROM co_righe_preventivi WHERE idpreventivo = '.prepare($riga['idpreventivo'])); - for ($i = 0; $i < sizeof($rsa); ++$i) { - if ($riga['is_preventivo']) { - if (!empty($rsa[$i]['idarticolo'])) { - add_movimento_magazzino($rsa[$i]['idarticolo'], $rsa[$i]['qta'], ['iddocumento' => $id_documento]); - } - } else { - $qta_evasa = $rsa[$i]['qta_evasa'] + $riga['qta']; - // Ripristino le quantità da evadere nel preventivo - $dbo->update('co_righe_preventivi', - [ - 'qta_evasa' => $qta_evasa, - ], - [ - 'id' => $rsa[$i]['id'], - ] - ); - } - } + if ($riga['idpreventivo']) { + $query = 'UPDATE co_righe_preventivi SET qta_evasa = qta_evasa - '.$riga['qta'].' WHERE idarticolo='.prepare($riga['idarticolo']).' AND descrizione = '.prepare($riga['descrizione']).' AND idpreventivo = '.prepare($riga['idpreventivo']); + $dbo->query($query); } // Rimozione articoli collegati ad un contratto importato con riga unica - if (empty($riga['idarticolo']) && $riga['idcontratto']) { - //rimetto a magazzino gli articoli collegati al contratto - $rsa = $dbo->fetchArray('SELECT id, idarticolo, qta FROM co_righe_contratti WHERE idcontratto = '.prepare($riga['idcontratto'])); - for ($i = 0; $i < sizeof($rsa); ++$i) { - if ($riga['is_contratto']) { - if (!empty($rsa[$i]['idarticolo'])) { - add_movimento_magazzino($rsa[$i]['idarticolo'], $rsa[$i]['qta'], ['iddocumento' => $id_documento]); - } - } else { - $qta_evasa = $rsa[$i]['qta_evasa'] + $riga['qta']; - // Ripristino le quantità da evadere nel contratto - $dbo->update('co_righe_contratti', - [ - 'qta_evasa' => $qta_evasa, - ], - [ - 'id' => $rsa[$i]['id'], - ] - ); - } - } + if ($riga['idcontratto']) { + $query = 'UPDATE co_righe_contratti SET qta_evasa = qta_evasa - '.$riga['qta'].' WHERE idarticolo='.prepare($riga['idarticolo']).' AND descrizione = '.prepare($riga['descrizione']).' AND idcontratto = '.prepare($riga['idcontratto']); + $dbo->query($query); } //Rimozione righe generiche From f107eadcc93de4d894035088de72683422a6b487 Mon Sep 17 00:00:00 2001 From: Luca Date: Mon, 22 Jul 2019 18:54:32 +0200 Subject: [PATCH 2/5] Fix minori --- include/init/init.php | 20 +++++++++++--------- modules/contratti/buttons.php | 2 +- modules/ddt/edit.php | 6 +++--- modules/ordini/buttons.php | 4 ++-- modules/preventivi/buttons.php | 2 +- 5 files changed, 18 insertions(+), 16 deletions(-) diff --git a/include/init/init.php b/include/init/init.php index 8f24b4f55..8945f85c4 100644 --- a/include/init/init.php +++ b/include/init/init.php @@ -193,18 +193,20 @@ if (!$has_settings) {

'.tr('Impostazioni di base').'

-
'; +
+
'; + - foreach ($settings as $setting => $required) { - if (empty(setting($setting))) { - echo ' -
- '.Settings::input($setting, $required).' -
'; + foreach ($settings as $setting => $required) { + if (empty(setting($setting))) { + echo ' +
+ '.Settings::input($setting, $required).' +
'; + } } - } - echo ' + echo '
'; } diff --git a/modules/contratti/buttons.php b/modules/contratti/buttons.php index 1b9b0926d..403407bf5 100644 --- a/modules/contratti/buttons.php +++ b/modules/contratti/buttons.php @@ -9,7 +9,7 @@ $rs_documento = $dbo->fetchArray('SELECT * FROM co_righe_contratti WHERE idcontr /* permetto di fatturare il contratto solo se contiene righe e si trova in uno stato fatturabile */ echo ' -'; diff --git a/modules/ddt/edit.php b/modules/ddt/edit.php index f0f30d379..0053b64bd 100644 --- a/modules/ddt/edit.php +++ b/modules/ddt/edit.php @@ -134,11 +134,11 @@ $_SESSION['superselect']['idsede_destinazione'] = $record['idsede_destinazione']
- {[ "type": "select", "label": "", "name": "idaspettobeni", "value": "$idaspettobeni$", "ajax-source": "aspetto-beni", "readonly": "", "icon-after": "add|" ]} + {[ "type": "select", "label": "", "name": "idaspettobeni", "value": "$idaspettobeni$", "ajax-source": "aspetto-beni", "readonly": "", "icon-after": "add||||" ]}
- {[ "type": "select", "label": "", "name": "idcausalet", "value": "$idcausalet$", "ajax-source": "causali", "readonly": "", "icon-after": "add|" ]} + {[ "type": "select", "label": "", "name": "idcausalet", "value": "$idcausalet$", "ajax-source": "causali", "readonly": "", "icon-after": "add||||" ]}
@@ -160,7 +160,7 @@ $_SESSION['superselect']['idsede_destinazione'] = $record['idsede_destinazione']
- {[ "type": "select", "label": "", "name": "idvettore", "ajax-source": "vettori", "value": "$idvettore$", "readonly": "", "disabled": , "required": , "icon-after": "add||tipoanagrafica=Vettore|" ]} + {[ "type": "select", "label": "", "name": "idvettore", "ajax-source": "vettori", "value": "$idvettore$", "readonly": "", "disabled": , "required": , "icon-after": "add||tipoanagrafica=Vettore||" ]}
diff --git a/modules/ordini/buttons.php b/modules/ordini/buttons.php index 955a6c975..c6350a361 100644 --- a/modules/ordini/buttons.php +++ b/modules/ordini/buttons.php @@ -12,13 +12,13 @@ echo ' echo '
  • -  '.tr('ddt').' +  '.tr('Ddt').'
    • '; echo '
  • -  '.tr('fattura').' +  '.tr('Fattura').'
    • '; diff --git a/modules/preventivi/buttons.php b/modules/preventivi/buttons.php index 2df24a2b0..dba54cc3a 100644 --- a/modules/preventivi/buttons.php +++ b/modules/preventivi/buttons.php @@ -21,7 +21,7 @@ echo '
    • -  '.tr('Ordine').' +  '.tr('Ordine').'
      • From c6473415ed1bc499dac48da18a72b957d9394af0 Mon Sep 17 00:00:00 2001 From: Luca Date: Tue, 23 Jul 2019 12:13:58 +0200 Subject: [PATCH 3/5] Fix minori --- editor.php | 26 ++++++++++++++++++++------ include/top.php | 2 +- modules/anagrafiche/edit.php | 17 +++++++++-------- modules/my_impianti/edit.php | 8 ++++---- 4 files changed, 34 insertions(+), 19 deletions(-) diff --git a/editor.php b/editor.php index 2cf47276b..fa42e97c9 100755 --- a/editor.php +++ b/editor.php @@ -107,13 +107,27 @@ 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_module)." ORDER BY zz_operations.created_at DESC"); + + $info = tr('Il record è stato eliminato il _DATE_ da _USER_ ,', [ + '_DATE_' => Translator::dateToLocale($operation['created_at']), + '_USER_' => $operation['username'], + ]); + echo ' -
      -

      '.tr('Ripristinare il record?').'

      - - - '.tr('Ripristina').' - +
      +
      + +
      '.$info.' '.tr('Ripristinare il record?').'
      + +
      + + '.tr('Salva e ripristina').' + +
      + +
      '; -?> diff --git a/modules/fatture/init.php b/modules/fatture/init.php index a1cc7d11c..9c40c6eeb 100644 --- a/modules/fatture/init.php +++ b/modules/fatture/init.php @@ -11,7 +11,7 @@ if ($module['name'] == 'Fatture di vendita') { if (isset($id_record)) { $fattura = Modules\Fatture\Fattura::with('tipo', 'stato')->find($id_record); - $record = $dbo->fetchOne('SELECT *, + $record = $dbo->fetchOne('SELECT co_documenti.*, co_tipidocumento.reversed AS is_reversed, co_documenti.idagente AS idagente_fattura, co_documenti.note, diff --git a/modules/preventivi/edit.php b/modules/preventivi/edit.php index 46edb2fc3..3a7cbfa48 100644 --- a/modules/preventivi/edit.php +++ b/modules/preventivi/edit.php @@ -43,7 +43,9 @@ $_SESSION['superselect']['idanagrafica'] = $record['idanagrafica'];
      {[ "type": "select", "label": "", "name": "idreferente", "value": "$idreferente$", "ajax-source": "referenti" ]} diff --git a/plugins/exportFE/edit.php b/plugins/exportFE/edit.php index eabdcee13..53abc5971 100644 --- a/plugins/exportFE/edit.php +++ b/plugins/exportFE/edit.php @@ -257,25 +257,43 @@ echo '
      '; -if ($generated) { - echo ' +echo ' '; -} diff --git a/plugins/referenti/actions.php b/plugins/referenti/actions.php index d4609e22e..f185445cd 100644 --- a/plugins/referenti/actions.php +++ b/plugins/referenti/actions.php @@ -44,6 +44,7 @@ switch ($operazione) { case 'deletereferente': $dbo->query('DELETE FROM `an_referenti` WHERE `id`='.prepare($id_record)); + $dbo->query('UPDATE co_preventivi SET idreferente = 0 WHERE `idreferente` = '.prepare($id_record)); flash()->info(tr('Referente eliminato!')); diff --git a/plugins/statistiche_anagrafiche/info.php b/plugins/statistiche_anagrafiche/info.php index 2c8568050..b230e9397 100644 --- a/plugins/statistiche_anagrafiche/info.php +++ b/plugins/statistiche_anagrafiche/info.php @@ -47,7 +47,7 @@ elseif ($anagrafica->isTipo('Tecnico')) { } $totale_interventi = 0; -foreach ($interventi as $intervento){ +foreach ($interventi as $intervento) { $costi = get_costi_intervento($intervento['id']); $totale_interventi += $costi['totale_scontato'];