From d350e21a4ac8da43408a28c08d63da187a90fbb3 Mon Sep 17 00:00:00 2001
From: MatteoPistorello <pistorellomatteo8@gmail.com>
Date: Mon, 13 Dec 2021 09:35:13 +0100
Subject: [PATCH] Aggiunto pulsante di modifica per le varianti

---
 CHANGELOG.md                                  |  7 +++-
 modules/combinazioni_articoli/actions.php     | 19 ++++++++-
 ...gestione-variante.php => add-variante.php} |  2 +-
 .../combinazioni_articoli/edit-variante.php   | 40 +++++++++++++++++++
 modules/combinazioni_articoli/edit.php        | 13 +++++-
 5 files changed, 76 insertions(+), 5 deletions(-)
 rename modules/combinazioni_articoli/{gestione-variante.php => add-variante.php} (98%)
 create mode 100644 modules/combinazioni_articoli/edit-variante.php

diff --git a/CHANGELOG.md b/CHANGELOG.md
index dfa2bc515..029eff0d1 100755
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,7 +4,7 @@ Tutti i maggiori cambiamenti di questo progetto saranno documentati in questo fi
 
 Il formato utilizzato è basato sulle linee guida di [Keep a Changelog](http://keepachangelog.com/), e il progetto segue il [Semantic Versioning](http://semver.org/) per definire le versioni delle release.
 
-- [2.4.28 (2021-12-07)](#2428-2021-12-07)
+- [2.4.28 (2021-12-13)](#2428-2021-12-13)
 - [2.4.27 (2021-10-25)](#2427-2021-10-26)
 - [2.4.26 (2021-09-24)](#2426-2021-09-24)
 - [2.4.25 (2021-08-25)](#2425-2021-08-25)
@@ -39,7 +39,7 @@ Il formato utilizzato è basato sulle linee guida di [Keep a Changelog](http://k
 - [2.2 (2016-11-10)](#22-2016-11-10)
 - [2.1 (2015-04-02)](#21-2015-04-02)
 
-## 2.4.28 (2021-12-07)
+## 2.4.28 (2021-12-13)
 
 ### Aggiunto (Added)
  - Aggiunta colonna documento di acquisto e prezzo nel plugin **seriali**
@@ -65,6 +65,8 @@ Il formato utilizzato è basato sulle linee guida di [Keep a Changelog](http://k
  - Aggiunte note interne in template mail
  - Aggiunta duplicazione **DDT**
  - Aggiunto codice distinta nello scadenzario
+ - Aggiunta azione di gruppo Aggiorna banca da Scadenzario e Fatture
+ - Aggiunto pulsante di modifica per le varianti di una combinazione
  
 ### Modificato (Changed)
  - Compilazione automatica tipo documento in fase di import FE solo se il campo non è impostato 
@@ -90,6 +92,7 @@ Il formato utilizzato è basato sulle linee guida di [Keep a Changelog](http://k
  - Fix stampa documento tramite comando da tastiera
  - Fix campi abi cab nel modulo **banche**
  - Fix stampa fatturato
+ - Fix salvataggio categorie durante import articoli da CSV
 
 ## 2.4.27 (2021-10-26)
 
diff --git a/modules/combinazioni_articoli/actions.php b/modules/combinazioni_articoli/actions.php
index c0367c03b..b56fb7f61 100644
--- a/modules/combinazioni_articoli/actions.php
+++ b/modules/combinazioni_articoli/actions.php
@@ -43,7 +43,24 @@ switch (filter('op')) {
 
         break;
 
-    case 'gestione-variante':
+    case 'edit-variante':
+        $valori = (array) filter('attributo');
+        $id_articolo = filter('id_articolo');
+
+        $database->delete('mg_articolo_attributo', ['id_articolo' => $id_articolo]);
+       
+        foreach ($valori as $valore){
+            $database->insert('mg_articolo_attributo', [
+                'id_articolo' => $id_articolo,
+                'id_valore' => $valore,
+            ]);
+        }
+
+        flash()->info(tr('Variante modificata correttamente!'));
+
+        break;
+
+    case 'add-variante':
         $combinazione->generaVariante((array) filter('attributo'), filter('id_articolo'));
 
         flash()->info(tr('Variante aggiunta correttamente!'));
diff --git a/modules/combinazioni_articoli/gestione-variante.php b/modules/combinazioni_articoli/add-variante.php
similarity index 98%
rename from modules/combinazioni_articoli/gestione-variante.php
rename to modules/combinazioni_articoli/add-variante.php
index d113f2604..e173c165c 100644
--- a/modules/combinazioni_articoli/gestione-variante.php
+++ b/modules/combinazioni_articoli/add-variante.php
@@ -19,7 +19,7 @@ $varianti_esistenti = array_column($varianti_esistenti, 'variante');
 
 echo '
 <form action="" method="post" id="form-variante">
-    <input type="hidden" name="op" value="gestione-variante">
+    <input type="hidden" name="op" value="add-variante">
     <input type="hidden" name="backto" value="record-edit">
 
     <p>'.tr('Completa le informazioni dei diversi Attributi per generare una variante della Combinazione').'.</p>
diff --git a/modules/combinazioni_articoli/edit-variante.php b/modules/combinazioni_articoli/edit-variante.php
new file mode 100644
index 000000000..cfd80357e
--- /dev/null
+++ b/modules/combinazioni_articoli/edit-variante.php
@@ -0,0 +1,40 @@
+<?php
+
+include_once __DIR__.'/init.php';
+
+echo '
+<form action="" method="post" id="form-variante">
+    <input type="hidden" name="op" value="edit-variante">
+    <input type="hidden" name="backto" value="record-edit">
+    <input type="hidden" name="id_articolo" value="'.get('id_articolo').'">
+
+    <div class="row">';
+
+$attributi = $combinazione->attributi;
+foreach ($attributi as $key => $attributo) {
+    $value = $dbo->fetchOne('SELECT mg_valori_attributi.id AS valore FROM mg_valori_attributi LEFT JOIN mg_articolo_attributo ON mg_articolo_attributo.id_valore=mg_valori_attributi.id WHERE id_articolo='.prepare(get('id_articolo')).' AND id_attributo='.prepare($attributo->id).'  AND deleted_at IS NULL')['valore'];
+    echo '
+        <div class="col-md-4">
+            {[ "type": "select", "label": "'.$attributo->nome.'", "name": "attributo['.$key.']", "values": "query=SELECT id, nome AS descrizione FROM mg_valori_attributi WHERE id_attributo = '.prepare($attributo->id).' AND deleted_at IS NULL", "value": "'.$value.'", "required": 1 ]}
+		</div>';
+}
+echo '
+    </div>
+
+    <div class="alert alert-info hidden" id="variante-esistente">
+        <i class="fa fa-info-circle"></i> '.tr('La variante indicata è già presente per la combinazione corrente').'.
+    </div>
+
+    <!-- PULSANTI -->
+    <div class="row">
+        <div class="col-md-12 text-right">
+            <button type="submit" class="btn btn-success">
+                <i class="fa fa-check"></i> '.tr('Salva').'
+            </button>
+        </div>
+    </div>
+</form>
+
+<script>
+    $(document).ready(init);
+</script>';
diff --git a/modules/combinazioni_articoli/edit.php b/modules/combinazioni_articoli/edit.php
index eb104e7f6..190809cbd 100644
--- a/modules/combinazioni_articoli/edit.php
+++ b/modules/combinazioni_articoli/edit.php
@@ -67,6 +67,7 @@ echo '
                     <th width="10%">'.tr('Foto').'</th>
                     <th>'.tr('Variante').'</th>
                     <th>'.tr('Articolo').'</th>
+                    <th class="text-center">'.tr('#').'</th>
                 </tr>
             </thead>
 
@@ -79,6 +80,11 @@ foreach ($articoli as $articolo) {
                     <td><img class="img-thumbnail img-responsive" src="'.$articolo->image.'"></td>
                     <td>'.$articolo->nome_variante.'</td>
                     <td>'.Modules::link('Articoli', $articolo->id, $articolo->codice.' - '.$articolo->descrizione).'</td>
+                    <td class="text-center">
+                        <button type="button" class="btn btn-warning btn-xs" onclick="modificaVariante('.$articolo->id.')">
+                            <i class="fa fa-edit"></i>
+                        </button>
+                    </td>
                 </tr>';
 }
 
@@ -95,7 +101,12 @@ echo '
 <script>
 function aggiungiVariante(button) {
     // Apertura modal
-    openModal("'.tr('Aggiungi variante').'", "'.$module->fileurl('gestione-variante.php').'?id_module=" + globals.id_module + "&id_record=" + globals.id_record);
+    openModal("'.tr('Aggiungi variante').'", "'.$module->fileurl('add-variante.php').'?id_module=" + globals.id_module + "&id_record=" + globals.id_record);
+}
+
+function modificaVariante(id) {
+    // Modifica modal
+    openModal("'.tr('Modifica variante').'", "'.$module->fileurl('edit-variante.php').'?id_module=" + globals.id_module + "&id_record=" + globals.id_record + "&id_articolo=" + id);
 }
 
 function generaVarianti(button) {