refactor: su marche, parent = 0 diventa NULL

Author: loviuz

modules/marche/actions.php

@@ -137,16 +137,25 @@
         }
 
         // Verifica se esiste già una marca con lo stesso nome
-        $marca_new = Marca::where('name', $nome)->where('id', '!=', $id_record)->first();
+        $marca_new = Marca::where('name', $nome)->where('id', '!=', $id_record)->where('parent', '=', $id_original)->first();
 
         if (!empty($marca_new)) {
             // Mostra un messaggio di errore con link alla marca esistente
-            $message = tr('Esiste già una marca con il nome _NOME_', [
+            $message = tr('Esiste già _TYPE_ con il nome _NOME_', [
                 '_NOME_' => '"'.$nome.'"',
+                '_TYPE_' => $id_original ? tr('un modello') : tr('una marca'),
             ]);
 
             $link = Modules::link('Marche', $marca_new->id, $marca_new->name);
             flash()->error($message.': '.$link);
+
+            if (empty($id_original)) {
+                redirect(base_path().'/controller.php?id_module='.$id_module);
+                exit();
+            }
+
+            redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
+            exit();
         } else {
             $marca = Marca::build($nome);
             $marca->parent = $id_original;
@@ -167,7 +176,7 @@
             // Redirect alla marca se si sta aggiungendo una modello
             $database->commitTransaction();
             redirect(base_path().'/editor.php?id_module='.$id_module.'&id_record='.($id_original ?: $id_record));
-            exit;
+            exit();
         }
 
         break;

update/2_9_3.sql

@@ -200,3 +200,21 @@ ORDER BY
 UPDATE `zz_views` INNER JOIN `zz_modules` ON `zz_views`.`id_module`=`zz_modules`.`id`
 SET `zz_views`.`query` = 'CONCAT_WS('' - '', an_zone.nome, an_zone.descrizione)'
 WHERE `zz_modules`.`name` = 'Anagrafiche' AND `zz_views`.`name` = 'Zone';
+
+-- Marche: parent = 0 diventa NULL
+ALTER TABLE `zz_marche` CHANGE `parent` `parent` INT NULL DEFAULT NULL;
+UPDATE `zz_marche` SET `parent` = NULL WHERE `parent` = 0;
+UPDATE `mg_articoli` SET `id_marca` = NULL WHERE `id_marca` = 0;
+
+UPDATE `zz_modules` SET `options` = 'SELECT
+    |select|
+FROM
+    `zz_marche`
+WHERE
+    1=1
+    AND
+    `parent` IS NULL
+HAVING
+    2=2
+ORDER BY
+    `name`' WHERE `zz_modules`.`name` = 'Marche';