From 73062dbfdfc5edd1f657ec7beb52cabac8c19393 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Tue, 12 Nov 2019 17:16:10 +0100
Subject: [PATCH] Correzione #672

---
 add.php                           |  2 +-
 assets/src/js/functions/select.js | 13 +++++++++++--
 modules/articoli/actions.php      | 10 ++++++++--
 3 files changed, 20 insertions(+), 5 deletions(-)

diff --git a/add.php b/add.php
index 1ca81cd72..d1877e275 100644
--- a/add.php
+++ b/add.php
@@ -77,7 +77,7 @@ $(document).ready(function(){
             // Selezione automatica nuovo valore per il select
             select = "#'.get('select').'";
             if ($(select).val() !== undefined) {
-                $(select).selectSetNew(response.id, response.text);
+                $(select).selectSetNew(response.id, response.text, response.data);
             }
 
             $("#bs-popup2").modal("hide");
diff --git a/assets/src/js/functions/select.js b/assets/src/js/functions/select.js
index f123163ef..31f64456f 100644
--- a/assets/src/js/functions/select.js
+++ b/assets/src/js/functions/select.js
@@ -124,12 +124,13 @@ jQuery.fn.selectReset = function (placeholder) {
  * 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) {
+jQuery.fn.selectSetNew = function (value, label, data) {
     this.selectReset();
 
     this.selectAdd([{
         'value': value,
         'text': label,
+        'data': data,
     }]);
 
     this.selectSet(value);
@@ -153,7 +154,15 @@ jQuery.fn.selectSet = function (value) {
 jQuery.fn.selectAdd = function (values) {
     $this = this;
 
-    values.forEach(function (item, index, array) {
+    values.forEach(function (item) {
+        if (item.data) {
+            Object.keys(item.data).forEach(function(element) {
+                item['data-' + element] = item.data[element];
+            });
+        }
+
+        delete item.data;
+
         var option = $('<option/>', item);
 
         $this.append(option);
diff --git a/modules/articoli/actions.php b/modules/articoli/actions.php
index 47f9cf313..32f7d1257 100644
--- a/modules/articoli/actions.php
+++ b/modules/articoli/actions.php
@@ -30,7 +30,13 @@ switch (post('op')) {
         $id_record = $dbo->lastInsertedID();
 
         if (isAjaxRequest()) {
-            echo json_encode(['id' => $id_record, 'text' => post('descrizione')]);
+            echo json_encode([
+                'id' => $id_record,
+                'text' => post('descrizione'),
+                'data' => [
+                    'descrizione' => post('descrizione'),
+                ],
+            ]);
         }
 
         flash()->info(tr('Aggiunto un nuovo articolo'));
@@ -42,7 +48,7 @@ switch (post('op')) {
         $qta = post('qta');
 
         // Inserisco l'articolo e avviso se esiste un altro articolo con stesso codice.
-        if ($n = $dbo->fetchNum('SELECT * FROM mg_articoli WHERE codice='.prepare(post('codice')).' AND id != '.$id_record.'') > 0) {
+        if ($n = $dbo->fetchNum('SELECT * FROM mg_articoli WHERE codice='.prepare(post('codice')).' AND id != '.prepare($id_record)) > 0) {
             flash()->warning(tr('Attenzione: il codice _CODICE_ è già stato utilizzato _N_ volta', [
                 '_CODICE_' => post('codice'),
                 '_N_' => $n,