From 8202e5a6272e61cc2441392e6f7ce0362689cd4c Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Sat, 16 Dec 2017 19:03:26 +0100
Subject: [PATCH] Fix dell'issue #77
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Il primo problema era causato da una mancanza nella progettazione del select, che ora mostra **categoria (sottocategoria)** con sottocategoria impostata a "-" se non definita.
Il secondo punto era invece causato da un problema di ambiguità nella query per l'individuazione degli articoli filtrati.
[Vedasi relativa issue]
---
 ajax_select.php          | 10 +++++++---
 modules/articoli/add.php |  4 ++--
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/ajax_select.php b/ajax_select.php
index badc7559b..366e76edf 100644
--- a/ajax_select.php
+++ b/ajax_select.php
@@ -237,8 +237,8 @@ switch ($op) {
         }
 
         if (!empty($search)) {
-            $search_fields[] = 'descrizione LIKE '.prepare('%'.$search.'%');
-            $search_fields[] = 'codice LIKE '.prepare('%'.$search.'%');
+            $search_fields[] = 'mg_articoli.descrizione LIKE '.prepare('%'.$search.'%');
+            $search_fields[] = 'mg_articoli.codice LIKE '.prepare('%'.$search.'%');
         }
 
         $wh = '';
@@ -259,8 +259,12 @@ switch ($op) {
         $rs = $dbo->fetchArray($query);
         foreach ($rs as $r) {
             if ($prev != $r['id_sottocategoria']) {
+                $categoria = $dbo->fetchArray('SELECT `nome` FROM `mg_categorie` WHERE `id`='.prepare($r['id_categoria']))[0]['nome'];
+
+                $sottocategoria = $dbo->fetchArray('SELECT `nome` FROM `mg_categorie` WHERE `id`='.prepare($r['id_sottocategoria']))[0]['nome'];
+
                 $prev = $r['id_sottocategoria'];
-                $results[] = ['text' => $dbo->fetchArray('SELECT `nome` FROM `mg_categorie` WHERE `id`='.prepare($r['id_categoria']))[0]['nome'], 'children' => []];
+                $results[] = ['text' => $categoria.' ('.(!empty($r['id_sottocategoria']) ? $sottocategoria : '-').')', 'children' => []];
             }
 
             if (empty($r['idiva_vendita'])) {
diff --git a/modules/articoli/add.php b/modules/articoli/add.php
index 7d89ecb66..e4a6e27da 100644
--- a/modules/articoli/add.php
+++ b/modules/articoli/add.php
@@ -22,7 +22,7 @@ unset($_SESSION['superselect']['id_categoria']);
 		</div>
 
 		<div class="col-md-6">
-			{[ "type": "select", "label": "<?php echo tr('Inserisci la subcategoria:'); ?>", "name": "subcategoria", "value": "", "ajax-source": "sottocategorie", "icon-after": "add|<?php echo Modules::get('Categorie')['id']; ?>||hide" ]}
+			{[ "type": "select", "label": "<?php echo tr('Inserisci la subcategoria:'); ?>", "name": "subcategoria", "id": "subcategoria_add", "value": "", "ajax-source": "sottocategorie", "icon-after": "add|<?php echo Modules::get('Categorie')['id']; ?>||hide" ]}
 		</div>
 	</div>
 
@@ -36,7 +36,7 @@ unset($_SESSION['superselect']['id_categoria']);
 
 <script>
 $(document).ready(function () {
-    var sub = $('#add_form').find('#subcategoria');
+    var sub = $('#add_form').find('#subcategoria_add');
     var original = sub.parent().find(".input-group-addon button").data("href");
 
     $('#add_form').find('#categoria').change( function(){