From 21005e4e8829a7c5509bee2bf22656f1da5e4eb3 Mon Sep 17 00:00:00 2001
From: Dasc3er <Dasc3er@users.noreply.github.com>
Date: Fri, 30 Oct 2020 14:19:02 +0100
Subject: [PATCH] Miglioramenti JS minori per inserimento articolo

---
 include/common/articolo.php           | 104 ++++++++++----------------
 modules/articoli/ajax/complete.php    |   5 +-
 plugins/dettagli_articolo/actions.php |   4 +-
 plugins/dettagli_articolo/edit.php    |   6 +-
 update/2_4_18.sql                     |   2 +-
 5 files changed, 50 insertions(+), 71 deletions(-)

diff --git a/include/common/articolo.php b/include/common/articolo.php
index 3dd9275d4..43457ce12 100755
--- a/include/common/articolo.php
+++ b/include/common/articolo.php
@@ -49,11 +49,8 @@ if (empty($result['idarticolo'])) {
 
     <script>
         $(document).ready(function (){
-            ottieniPrezziArticolo("'.$articolo['id'].'").then(function (){
+            ottieniDettagliArticolo("'.$articolo['id'].'").then(function (){
                 verificaPrezzoArticolo();
-            });
-
-            ottieniScontiArticolo("'.$articolo['id'].'").then(function (){
                 verificaScontoArticolo();
             });
         });
@@ -108,10 +105,13 @@ if ($module['name'] != 'Contratti' && $module['name'] != 'Preventivi') {
 echo '
 <script>
 var direzione = "'.$options['dir'].'";
+globals.aggiunta_articolo = {
+};
+
 $(document).ready(function () {
     if (direzione === "uscita") {
-        aggiorna_qta_minima();
-        $("#qta").keyup(aggiorna_qta_minima);
+        aggiornaQtaMinima();
+        $("#qta").keyup(aggiornaQtaMinima);
     }
 });
 
@@ -149,15 +149,13 @@ $("#idarticolo").on("change", function() {
 
     // Autoimpostazione dei campi relativi all\'articolo
     let $data = $(this).selectData();
-    ottieniPrezziArticolo($data.id).then(function() {
+    ottieniDettagliArticolo($data.id).then(function() {
         if ($("#prezzo_unitario").val().toEnglish() === 0){
-            aggiornaPrezzoArticolo()
+            aggiornaPrezzoArticolo();
         } else {
             verificaPrezzoArticolo();
         }
-    });
 
-    ottieniScontiArticolo($data.id).then(function() {
         if ($("#sconto").val().toEnglish() === 0){
             aggiornaScontoArticolo();
         } else {
@@ -177,7 +175,7 @@ $("#idarticolo").on("change", function() {
     else {
         $("#id_dettaglio_fornitore").val($data.id_dettaglio_fornitore);
         $("#qta_minima").val($data.qta_minima);
-        aggiorna_qta_minima();
+        aggiornaQtaMinima();
     }
 
     let id_conto = $data.idconto_'.($options['dir'] == 'entrata' ? 'vendita' : 'acquisto').';
@@ -199,80 +197,58 @@ $(document).on("change", "input[name^=qta], input[name^=prezzo_unitario], input[
 });
 
 /**
-* Restituisce il prezzo registrato per una specifica quantità dell\'articolo.
+* Restituisce il dettaglio registrato per una specifica quantità dell\'articolo.
 */
-function getPrezzoPerQuantita(qta) {
-    const data = $("#prezzo_unitario").data("prezzi");
-    if (!data) return 0;
+function getDettaglioPerQuantita(qta) {
+    const data = globals.aggiunta_articolo.dettagli;
+    if (!data) return null;
 
-    let prezzo_predefinito = null;
-    let prezzo_selezionato = null;
-    for (const prezzo of data) {
-        if (prezzo.minimo == null && prezzo.massimo == null) {
-            prezzo_predefinito = prezzo.prezzo_unitario;
+    let dettaglio_predefinito = null;
+    let dettaglio_selezionato = null;
+    for (const dettaglio of data) {
+        if (dettaglio.minimo == null && dettaglio.massimo == null) {
+            dettaglio_predefinito = dettaglio;
             continue;
         }
 
-        if (qta >= prezzo.minimo && qta <= prezzo.massimo) {
-            prezzo_selezionato = prezzo.prezzo_unitario;
+        if (qta >= dettaglio.minimo && qta <= dettaglio.massimo) {
+            dettaglio_selezionato = dettaglio;
         }
     }
 
-    if (prezzo_selezionato == null) {
-        prezzo_selezionato = prezzo_predefinito;
+    if (dettaglio_selezionato == null) {
+        dettaglio_selezionato = dettaglio_predefinito;
     }
 
-    return parseFloat(prezzo_selezionato);
+    return dettaglio_selezionato;
+}
+
+/**
+* Restituisce il prezzo registrato per una specifica quantità dell\'articolo.
+*/
+function getPrezzoPerQuantita(qta) {
+    const dettaglio = getDettaglioPerQuantita(qta);
+
+    return dettaglio ? parseFloat(dettaglio.prezzo_unitario) : 0;
 }
 
 /**
 * Restituisce lo sconto registrato per una specifica quantità dell\'articolo.
 */
 function getScontoPerQuantita(qta) {
-    const data = $("#sconto").data("sconti");
-    if (!data) return 0;
+    const dettaglio = getDettaglioPerQuantita(qta);
 
-    let sconto_predefinito = null;
-    let sconto_selezionato = null;
-    for (const prezzo of data) {
-        if (prezzo.minimo == null && prezzo.massimo == null) {
-            sconto_predefinito = prezzo.sconto;
-            continue;
-        }
-
-        if (qta >= prezzo.minimo && qta <= prezzo.massimo) {
-            sconto_selezionato = prezzo.sconto;
-        }
-    }
-
-    if (sconto_selezionato == null) {
-        sconto_selezionato = sconto_predefinito;
-    }
-
-    return parseFloat(sconto_selezionato);
-}
-
-
-/**
-* Funzione per registrare localmente i prezzi definiti per l\'articolo in relazione ad una specifica anagrafica.
-*/
-function ottieniPrezziArticolo(id_articolo) {
-    return $.get(globals.rootdir + "/ajax_complete.php?module=Articoli&op=prezzi_articolo&id_anagrafica='.$options['idanagrafica'].'&id_articolo=" + id_articolo + "&dir=" + direzione, function(response) {
-        const data = JSON.parse(response);
-
-        $("#prezzo_unitario").data("prezzi", data);
-    });
+    return dettaglio ? parseFloat(dettaglio.sconto_percentuale) : 0;
 }
 
 /**
-* Funzione per registrare localmente gli sconti definiti per l\'articolo in relazione ad una specifica anagrafica.
+* Funzione per registrare localmente i dettagli definiti per l\'articolo in relazione ad una specifica anagrafica.
 */
-
-function ottieniScontiArticolo(id_articolo) {
-    return $.get(globals.rootdir + "/ajax_complete.php?module=Articoli&op=prezzi_articolo&id_anagrafica='.$options['idanagrafica'].'&id_articolo=" + id_articolo + "&dir=" + direzione, function(response) {
+function ottieniDettagliArticolo(id_articolo) {
+    return $.get(globals.rootdir + "/ajax_complete.php?module=Articoli&op=dettagli_articolo&id_anagrafica='.$options['idanagrafica'].'&id_articolo=" + id_articolo + "&dir=" + direzione, function(response) {
         const data = JSON.parse(response);
 
-        $("#sconto").data("sconti", data);
+        globals.aggiunta_articolo.dettagli = data;
     });
 }
 
@@ -305,7 +281,7 @@ function verificaPrezzoArticolo() {
 function verificaScontoArticolo() {
     let qta = $("#qta").val().toEnglish();
     let sconto_previsto = getScontoPerQuantita(qta);
-    console.log(sconto_previsto);
+
     let sconto_input = $("#sconto");
     let sconto = sconto_input.val().toEnglish();
 
@@ -344,7 +320,7 @@ function aggiornaScontoArticolo() {
 /**
 * Funzione per l\'aggiornamento dinamico della quantità minima per l\'articolo.
 */
-function aggiorna_qta_minima() {
+function aggiornaQtaMinima() {
     let qta_minima = parseFloat($("#qta_minima").val());
     let qta = $("#qta").val().toEnglish();
 
diff --git a/modules/articoli/ajax/complete.php b/modules/articoli/ajax/complete.php
index 1611bcd00..fb1a2de28 100755
--- a/modules/articoli/ajax/complete.php
+++ b/modules/articoli/ajax/complete.php
@@ -116,7 +116,7 @@ switch ($resource) {
      * - id_articolo
      * - id_anagrafica
      */
-    case 'prezzi_articolo':
+    case 'dettagli_articolo':
         $id_articolo = get('id_articolo');
         $id_anagrafica = get('id_anagrafica');
         $direzione = get('dir') == 'uscita' ? 'uscita' : 'entrata';
@@ -127,7 +127,8 @@ switch ($resource) {
 
         $prezzi_ivati = setting('Utilizza prezzi di vendita comprensivi di IVA');
 
-        $query = 'SELECT minimo, massimo, sconto,
+        $query = 'SELECT minimo, massimo,
+            sconto_percentuale,
             '.($prezzi_ivati ? 'prezzo_unitario_ivato' : 'prezzo_unitario').' AS prezzo_unitario
         FROM mg_prezzi_articoli
         WHERE id_articolo = '.prepare($id_articolo).' AND dir = '.prepare($direzione).' |where|
diff --git a/plugins/dettagli_articolo/actions.php b/plugins/dettagli_articolo/actions.php
index 27bb05c0c..6200c79c4 100755
--- a/plugins/dettagli_articolo/actions.php
+++ b/plugins/dettagli_articolo/actions.php
@@ -75,7 +75,7 @@ switch (filter('op')) {
         if (empty($dettaglio_predefinito)) {
             $dettaglio_predefinito = DettaglioPrezzo::build($articolo, $anagrafica, $direzione);
         }
-        $dettaglio_predefinito->sconto = $sconto;
+        $dettaglio_predefinito->sconto_percentuale = $sconto;
         $dettaglio_predefinito->setPrezzoUnitario($prezzo_unitario);
         $dettaglio_predefinito->save();
 
@@ -102,7 +102,7 @@ switch (filter('op')) {
 
                 $dettaglio->minimo = $minimi[$key];
                 $dettaglio->massimo = $massimi[$key];
-                $dettaglio->sconto = $sconti[$key];
+                $dettaglio->sconto_percentuale = $sconti[$key];
                 $dettaglio->setPrezzoUnitario($prezzo_unitario);
                 $dettaglio->save();
             }
diff --git a/plugins/dettagli_articolo/edit.php b/plugins/dettagli_articolo/edit.php
index 7f523c797..f31ba4ed5 100755
--- a/plugins/dettagli_articolo/edit.php
+++ b/plugins/dettagli_articolo/edit.php
@@ -123,7 +123,7 @@ if (!$clienti->isEmpty()) {
                         </td>
 
                         <td class="text-right">
-                            '.numberFormat($dettaglio->sconto).'%
+                            '.numberFormat($dettaglio->sconto_percentuale).'%
                         </td>
 
                         <td>';
@@ -309,9 +309,11 @@ if (!$fornitori_disponibili->isEmpty()) {
                         <td class="text-right">
                             '.moneyFormat($dettaglio->prezzo_unitario).'
                         </td>
+
                         <td class="text-right">
-                            '.numberFormat($dettaglio->sconto).'%
+                            '.numberFormat($dettaglio->sconto_percentuale).'%
                         </td>
+
                         <td>';
 
                 if (!isset($dettaglio->minimo) && !isset($dettaglio->massimo)) {
diff --git a/update/2_4_18.sql b/update/2_4_18.sql
index b0e2d82f5..09a556d1c 100644
--- a/update/2_4_18.sql
+++ b/update/2_4_18.sql
@@ -329,7 +329,7 @@ UPDATE `zz_modules` SET `options` = 'SELECT |select| FROM `co_scadenziario`\r\n
 DELETE FROM `zz_tokens` WHERE `zz_tokens`.`enabled` = 0;
 
 -- Aggiunto colonna sconto per le coppie anagrafica articolo
-ALTER TABLE `mg_prezzi_articoli` ADD `sconto` DECIMAL(15,6) NOT NULL AFTER `massimo`;
+ALTER TABLE `mg_prezzi_articoli` ADD `sconto_percentuale` DECIMAL(15,6) NOT NULL AFTER `massimo`;
 
 -- Aggiunta impostazione per mostrare o nascondere barra plugin
 INSERT INTO `zz_settings` (`id`, `nome`, `valore`, `tipo`, `editable`, `sezione`, `order`, `help`) VALUES (NULL, 'Nascondere la barra dei plugin di default', '0', 'boolean', '1', 'Generali',  '2', NULL);