From c7e6ef054d1e3c1c21994e4e8490d645a8fe5419 Mon Sep 17 00:00:00 2001
From: "Samuele E. Locatelli" <samuele@steamware.net>
Date: Tue, 4 Feb 2020 13:27:03 +0100
Subject: [PATCH] FIX IO: migliore log errore trans.stato + fix gestione veto
 forceODL

---
 Jenkinsfile         | 2 +-
 MapoDb/DataLayer.cs | 4 ++--
 MapoDb/MapoDb.cs    | 5 +++--
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index 02edbab3..318049ec 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -12,7 +12,7 @@ pipeline {
       steps {
       /* calcolo numero versione... diverso x branch MASTER/DEVELOP */
       script {
-        withEnv(['NEXT_BUILD_NUMBER=1283']) {
+        withEnv(['NEXT_BUILD_NUMBER=1284']) {
           // env.versionNumber = VersionNumber(versionNumberString : '6.9.${BUILD_DATE_FORMATTED, "yyMM"}.${BUILDS_ALL_TIME}', projectStartDate : '2006-01-01', skipFailedBuilds: true)
           env.versionNumber = VersionNumber(versionNumberString : '6.9.${BUILD_DATE_FORMATTED, "yyMM"}.${BUILDS_ALL_TIME}', projectStartDate : '2006-01-01', skipFailedBuilds: true, overrideBuildsAllTime: '${NEXT_BUILD_NUMBER}')
           env.APP_NAME = 'MAPO'
diff --git a/MapoDb/DataLayer.cs b/MapoDb/DataLayer.cs
index 5fa5b9db..9967a7f9 100644
--- a/MapoDb/DataLayer.cs
+++ b/MapoDb/DataLayer.cs
@@ -1601,7 +1601,7 @@ namespace MapoDb
       // verifico NON CI SIA un veto a NUOVI split... 2 min di default...
       string redKey = vetoSplitOdlMaccHash(idxMacchina);
       string rawData = memLayer.ML.getRSV(redKey);
-      if (rawData == "")
+      if (string.IsNullOrEmpty(rawData))
       {
         // registro VETO x altri split... 2 minuti
         memLayer.ML.setRSV(redKey, $"Inizio SPLIT-ODL {DateTime.Now}", 120);
@@ -2048,7 +2048,7 @@ namespace MapoDb
       {
         logger.lg.scriviLog(string.Format("[pzCounterTC] Impossibile recuperare dati ODL x idxMacchina {0}", idxMacchina), tipoLog.WARNING);
         // sleep...
-        System.Threading.Thread.Sleep(taSP_ms_ant*2);
+        System.Threading.Thread.Sleep(taSP_ms_ant * 2);
         // riprovo lettura...             
         okDatiProd = getStatoProd(idxMacchina, ref datiProdAct, dataRif);
         maxTry--;
diff --git a/MapoDb/MapoDb.cs b/MapoDb/MapoDb.cs
index 0d744945..cc59011a 100644
--- a/MapoDb/MapoDb.cs
+++ b/MapoDb/MapoDb.cs
@@ -199,7 +199,7 @@ namespace MapoDb
           catch (Exception exc)
           {
             // non dovrebbe succedere... input utente da barcode dovrebbero TUTTI essere inseriti in tab transizione con famiglia 1...
-            logger.lg.scriviLog(string.Format("Errore controllo transizione stato x evento barcode:{0}{1}", Environment.NewLine, exc), tipoLog.EXCEPTION);
+            logger.lg.scriviLog($"Errore controllo transizione stato x evento barcode: BARCODE | IdxMacchina: {IdxMacchina} | InizioStato: {InizioStato} | IdxTipo: {IdxTipo} | CodArt: {CodArt} | Value: {Value} | MatrOpr: {MatrOpr} | pallet: {pallet}{Environment.NewLine}{exc}", tipoLog.EXCEPTION);
           }
           break;
         case tipoInputEvento.hw:
@@ -216,9 +216,10 @@ namespace MapoDb
               }
             }
           }
-          catch
+          catch (Exception exc)
           {
             // non trovo riga [0]... NON scrivo!
+            logger.lg.scriviLog($"Errore controllo transizione stato x evento barcode: HW | IdxMacchina: {IdxMacchina} | InizioStato: {InizioStato} | IdxTipo: {IdxTipo} | CodArt: {CodArt} | Value: {Value} | MatrOpr: {MatrOpr} | pallet: {pallet}{Environment.NewLine}{exc}", tipoLog.EXCEPTION);
           }
           break;
       }