From 6238bc5f9451ab65c1e48675d8d875e8be96ff78 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Sat, 29 Jun 2019 11:12:43 +0200
Subject: [PATCH] Fix #610

---
 plugins/importFE/src/InvoiceHook.php  | 17 ++++++++++++-----
 plugins/receiptFE/src/ReceiptHook.php | 17 ++++++++++++-----
 update/2_4_10.sql                     |  4 ++--
 3 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/plugins/importFE/src/InvoiceHook.php b/plugins/importFE/src/InvoiceHook.php
index 094823961..7649f17d0 100644
--- a/plugins/importFE/src/InvoiceHook.php
+++ b/plugins/importFE/src/InvoiceHook.php
@@ -17,13 +17,20 @@ class InvoiceHook extends HookManager
     public function response($results)
     {
         $count = count($results);
+        $notify = false;
 
         $module = Modules::get('Fatture di acquisto');
-        $plugin = $module->plugins->first(function ($value, $key) {
-            return $value->name == 'Fatturazione Elettronica';
-        });
+        $plugins = $module->plugins;
 
-        $link = ROOTDIR.'/controller.php?id_module='.$module->id.'#tab_'.$plugin->id;
+        if (!empty($plugins)) {
+            $notify = !empty($count);
+
+            $plugin = $plugins->first(function ($value, $key) {
+                return $value->name == 'Fatturazione Elettronica';
+            });
+
+            $link = ROOTDIR.'/controller.php?id_module='.$module->id.'#tab_'.$plugin->id;
+        }
 
         $message = tr('Ci sono _NUM_ fatture passive da importare', [
             '_NUM_' => $count,
@@ -33,7 +40,7 @@ class InvoiceHook extends HookManager
             'icon' => 'fa fa-file-text-o text-yellow',
             'link' => $link,
             'message' => $message,
-            'notify' => !empty($count),
+            'notify' => $notify,
         ];
     }
 }
diff --git a/plugins/receiptFE/src/ReceiptHook.php b/plugins/receiptFE/src/ReceiptHook.php
index e9e34ab7f..e6dcee15d 100644
--- a/plugins/receiptFE/src/ReceiptHook.php
+++ b/plugins/receiptFE/src/ReceiptHook.php
@@ -17,13 +17,20 @@ class ReceiptHook extends HookManager
     public function response($results)
     {
         $count = count($results);
+        $notify = false;
 
         $module = Modules::get('Fatture di vendita');
-        $plugin = $module->plugins->first(function ($value, $key) {
-            return $value->name == 'Ricevute FE';
-        });
+        $plugins = $module->plugins;
 
-        $link = ROOTDIR.'/controller.php?id_module='.$module->id.'#tab_'.$plugin->id;
+        if (!empty($plugins)) {
+            $notify = !empty($count);
+
+            $plugin = $plugins->first(function ($value, $key) {
+                return $value->name == 'Ricevute FE';
+            });
+
+            $link = ROOTDIR.'/controller.php?id_module='.$module->id.'#tab_'.$plugin->id;
+        }
 
         $message = tr('Ci sono _NUM_ ricevute da importare', [
             '_NUM_' => $count,
@@ -33,7 +40,7 @@ class ReceiptHook extends HookManager
             'icon' => 'fa fa-ticket text-yellow',
             'link' => $link,
             'message' => $message,
-            'notify' => !empty($count),
+            'notify' => $notify,
         ];
     }
 }
diff --git a/update/2_4_10.sql b/update/2_4_10.sql
index 33c318beb..108d33d8f 100644
--- a/update/2_4_10.sql
+++ b/update/2_4_10.sql
@@ -110,7 +110,7 @@ UPDATE `zz_views` SET `query` = 'CONCAT_WS(co_movimenti_modelli.nome, co_movimen
 UPDATE `co_movimenti_modelli` SET `nome` = `descrizione` WHERE `nome` = '';
 
 -- Rimuovo le interruzioni di riga per descrizioni vuote 
---UPDATE `in_interventi` SET `descrizione` = REPLACE(`descrizione`, '\n', '') where `descrizione` LIKE '%\n';
+-- UPDATE `in_interventi` SET `descrizione` = REPLACE(`descrizione`, '\n', '') where `descrizione` LIKE '%\n';
 
 -- Aggiunto tabella co_tipi_scadenze
 CREATE TABLE `co_tipi_scadenze` (
@@ -134,4 +134,4 @@ INSERT INTO `zz_views` (`id_module`, `name`, `query`, `order`, `search`, `slow`,
 ((SELECT `id` FROM `zz_modules` WHERE `name` = 'Tipi scadenze'), 'id', 'id', 1, 1, 0, 0, 0);
 
 -- Aggiungo possibilità di vedere la descrizione per le scadenze generiche
-UPDATE `zz_views` SET `query` = 'IF(an_anagrafiche.ragione_sociale IS NULL, co_scadenziario.descrizione, an_anagrafiche.ragione_sociale)' WHERE `zz_views`.`name` = 'Anagrafica' AND `zz_views`.`id_module` = (SELECT `id` FROM `zz_modules` WHERE `name` = 'Scadenzario')   ;
\ No newline at end of file
+UPDATE `zz_views` SET `query` = 'IF(an_anagrafiche.ragione_sociale IS NULL, co_scadenziario.descrizione, an_anagrafiche.ragione_sociale)' WHERE `zz_views`.`name` = 'Anagrafica' AND `zz_views`.`id_module` = (SELECT `id` FROM `zz_modules` WHERE `name` = 'Scadenzario');