From a41f50d7c59fe77b2fe8698516bf05e19eb50f60 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Sat, 16 Dec 2017 15:49:29 +0100
Subject: [PATCH] Fix del bug #76

Risoluzione del bug @76, generato dall'utilizzo della funzione JS sbagliata per impostare in automatico il campo IVA.
---
 lib/functions.js                 | 10 ++++++----
 modules/ddt/add_articolo.php     |  2 +-
 modules/fatture/add_articolo.php |  2 +-
 modules/ordini/add_articolo.php  |  2 +-
 modules/preventivi/edit_riga.php |  2 +-
 5 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/lib/functions.js b/lib/functions.js
index f7205667a..fc9da74b4 100644
--- a/lib/functions.js
+++ b/lib/functions.js
@@ -414,10 +414,10 @@ function start_widgets($widgets) {
 
 // Modal
 function launch_modal(title, href, init_modal, id) {
-	
+
 	//Fix - Select2 does not function properly when I use it inside a Bootstrap modal.
 	$.fn.modal.Constructor.prototype.enforceFocus = function() {};
-	
+
     if (id == null) {
         id = '#bs-popup';
     }
@@ -924,7 +924,8 @@ jQuery.fn.selectReset = function () {
 };
 
 /**
- * Aggiorna un <select> creato con select2 impostando un valore di default
+ * Aggiorna un <select> creato con select2 impostando un valore di default.
+ * Da utilizzare per l'impostazione dei select basati su richieste AJAX.
  */
 jQuery.fn.selectSetNew = function (value, label) {
     this.selectReset();
@@ -940,7 +941,8 @@ jQuery.fn.selectSetNew = function (value, label) {
 };
 
 /**
- * Aggiorna un <select> creato con select2 impostando un valore di default
+ * Aggiorna un <select> creato con select2 impostando un valore di default.
+ * Da utilizzare per l'impostazione dei select statici.
  */
 jQuery.fn.selectSet = function (value) {
     this.val(value).trigger("change");
diff --git a/modules/ddt/add_articolo.php b/modules/ddt/add_articolo.php
index a32d17d88..889e04ee0 100644
--- a/modules/ddt/add_articolo.php
+++ b/modules/ddt/add_articolo.php
@@ -119,7 +119,7 @@ echo '
                 $data = $(this).selectData();
                 $("#prezzo").val($data.prezzo_'.($dir == 'entrata' ? 'vendita' : 'acquisto').');
                 $("#descrizione").val($data.descrizione);
-                $("#idiva").selectSetNew($data.idiva_vendita, $data.iva_vendita);
+                $("#idiva").selectSet($data.idiva_vendita, $data.iva_vendita);
                 $("#um").selectSetNew($data.um, $data.um);
             }else{
                 $("#prezzi_articolo button").addClass("disabled");
diff --git a/modules/fatture/add_articolo.php b/modules/fatture/add_articolo.php
index 2605df1a8..dced8800a 100644
--- a/modules/fatture/add_articolo.php
+++ b/modules/fatture/add_articolo.php
@@ -140,7 +140,7 @@ echo '
                 $data = $(this).selectData();
                 $("#prezzo").val($data.prezzo_'.($dir == 'entrata' ? 'vendita' : 'acquisto').');
                 $("#descrizione").val($data.descrizione);
-                $("#idiva").selectSetNew($data.idiva_vendita, $data.iva_vendita);
+                $("#idiva").selectSet($data.idiva_vendita, $data.iva_vendita);
                 $("#um").selectSetNew($data.um, $data.um);
             }else{
                 $("#prezzi_articolo button").addClass("disabled");
diff --git a/modules/ordini/add_articolo.php b/modules/ordini/add_articolo.php
index c24fe9a52..736bbb79b 100644
--- a/modules/ordini/add_articolo.php
+++ b/modules/ordini/add_articolo.php
@@ -128,7 +128,7 @@ echo '
                 $data = $(this).selectData();
                 $("#prezzo").val($data.prezzo_'.($dir == 'entrata' ? 'vendita' : 'acquisto').');
                 $("#descrizione").val($data.descrizione);
-                $("#idiva").selectSetNew($data.idiva_vendita, $data.iva_vendita);
+                $("#idiva").selectSet($data.idiva_vendita, $data.iva_vendita);
                 $("#um").selectSetNew($data.um, $data.um);
             }else{
                 $("#prezzi_articolo button").addClass("disabled");
diff --git a/modules/preventivi/edit_riga.php b/modules/preventivi/edit_riga.php
index 1b6d681f1..38399888c 100644
--- a/modules/preventivi/edit_riga.php
+++ b/modules/preventivi/edit_riga.php
@@ -65,7 +65,7 @@ echo '
 echo '
     <div class="row">
         <div class="col-md-12">
-            {[ "type": "select", "label": "'.tr('Articolo').'", "name": "idarticolo", "value": "'.$idarticolo.'", "ajax-source": "articoli", "extra": "onchange=\"session_set(\'superselect,idarticolo\', $(this).val(), 0); $data = $(this).selectData(); $(\'#prezzo\').val($data.prezzo_vendita); $(\'#desc\').val($data.descrizione); $(\'#um\').selectSetNew($data.um, $data.um); $(\'#idiva\').selectSetNew($data.iva_vendita, $data.iva_vendita);\"" ]}
+            {[ "type": "select", "label": "'.tr('Articolo').'", "name": "idarticolo", "value": "'.$idarticolo.'", "ajax-source": "articoli", "extra": "onchange=\"session_set(\'superselect,idarticolo\', $(this).val(), 0); $data = $(this).selectData(); $(\'#prezzo\').val($data.prezzo_vendita); $(\'#desc\').val($data.descrizione); $(\'#um\').selectSetNew($data.um, $data.um); $(\'#idiva\').selectSet($data.iva_vendita, $data.iva_vendita);\"" ]}
         </div>
     </div>';