From 4d9e2c78e6fe7a53ad748c94df73b781197f556f Mon Sep 17 00:00:00 2001
From: fcarraUniSa <fcarra@unisa.it>
Date: Tue, 23 Dec 2025 12:19:20 +0100
Subject: [PATCH] Update storage.ts

---
 services/storage.ts | 95 +++++++++++++++++++++++++--------------------
 1 file changed, 52 insertions(+), 43 deletions(-)

diff --git a/services/storage.ts b/services/storage.ts
index f6b200f..aa22123 100644
--- a/services/storage.ts
+++ b/services/storage.ts
@@ -1,6 +1,22 @@
+
 import { EmailTemplate } from '../types';
 
-// Helper functions remain synchronous as they are utility functions
+// Fallback for crypto.randomUUID in non-secure (HTTP) contexts
+export const generateUUID = () => {
+  if (typeof crypto !== 'undefined' && crypto.randomUUID) {
+    try {
+      return crypto.randomUUID();
+    } catch (e) {
+      // Fallback if it exists but fails
+    }
+  }
+  return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, function(c) {
+    const r = Math.random() * 16 | 0;
+    const v = c === 'x' ? r : (r & 0x3 | 0x8);
+    return v.toString(16);
+  });
+};
+
 export const generateTemplateKey = (name: string): string => {
   return name.trim().toLowerCase().replace(/\s+/g, '_').replace(/[^a-z0-9_]/g, '');
 };
@@ -13,10 +29,15 @@ export const generateSQL = (template: EmailTemplate): string => {
   const subject = template.subject.replace(/'/g, "''");
   const vars = JSON.stringify(template.variables).replace(/'/g, "''");
   const key = generateTemplateKey(template.name);
+  const name = template.name.replace(/'/g, "''");
+  const desc = (template.description || '').replace(/'/g, "''");
 
   return `INSERT INTO email_templates (id, template_key, name, description, subject, header_html, body_html, footer_html, full_html, required_variables)
-VALUES ('${template.id}', '${key}', '${template.name.replace(/'/g, "''")}', '${template.description?.replace(/'/g, "''") || ''}', '${subject}', '${header}', '${body}', '${footer}', '${fullHtml}', '${vars}')
+VALUES ('${template.id}', '${key}', '${name}', '${desc}', '${subject}', '${header}', '${body}', '${footer}', '${fullHtml}', '${vars}')
 ON DUPLICATE KEY UPDATE 
+    template_key = VALUES(template_key),
+    name = VALUES(name),
+    description = VALUES(description),
     subject = VALUES(subject),
     header_html = VALUES(header_html),
     body_html = VALUES(body_html),
@@ -35,24 +56,18 @@ export const generateN8nCode = (template: EmailTemplate): string => {
   const hasVars = template.variables.length > 0;
 
   return `// Nodo Code n8n - Popolatore Template
-// 1. Assicurati che il nodo precedente (SQL) restituisca 'full_html' e 'subject'.
-// 2. Aggiusta il percorso (item.json.full_html) se l'output del tuo nodo SQL è diverso.
-
 for (const item of items) {
   const templateHtml = item.json.full_html;
   const templateSubject = item.json.subject;
 
-  // Definisci qui i tuoi dati dinamici
   const replacements = {
-${hasVars ? varsMap : '  // Nessuna variabile rilevata in questo template'}
+${hasVars ? varsMap : '  // Nessuna variabile rilevata'}
   };
 
   let finalHtml = templateHtml;
   let finalSubject = templateSubject;
   
-  // Esegui sostituzione
   for (const [key, value] of Object.entries(replacements)) {
-    // Sostituisce {{key}} globalmente nell'HTML e nell'Oggetto
     const regex = new RegExp('{{' + key + '}}', 'g');
     finalHtml = finalHtml.replace(regex, value);
     if (finalSubject) {
@@ -60,54 +75,48 @@ ${hasVars ? varsMap : '  // Nessuna variabile rilevata in questo template'}
     }
   }
 
-  // Output del contenuto processato
   item.json.processed_html = finalHtml;
   item.json.processed_subject = finalSubject;
 }
-
 return items;`;
 };
 
-// Async API calls to replace synchronous localStorage
 export const getTemplates = async (): Promise<EmailTemplate[]> => {
-  try {
-    const response = await fetch('/api/templates');
-    if (!response.ok) throw new Error('Fallito il recupero');
-    return await response.json();
-  } catch (e) {
-    console.error("Fallito il caricamento dei template", e);
-    return [];
+  console.log("Fetching templates from /api/templates...");
+  const response = await fetch('/api/templates');
+  if (!response.ok) {
+    const errorBody = await response.text();
+    throw new Error(`Errore API (${response.status}): ${errorBody || response.statusText}`);
   }
+  return await response.json();
 };
 
 export const saveTemplate = async (template: EmailTemplate): Promise<void> => {
-  try {
-    const response = await fetch('/api/templates', {
-      method: 'POST',
-      headers: {
-        'Content-Type': 'application/json',
-      },
-      body: JSON.stringify(template),
-    });
-    
-    if (!response.ok) {
-      const error = await response.json();
-      throw new Error(error.message || 'Salvataggio fallito');
+  console.log("Saving template to /api/templates...", template.name);
+  const response = await fetch('/api/templates', {
+    method: 'POST',
+    headers: {
+      'Content-Type': 'application/json',
+    },
+    body: JSON.stringify(template),
+  });
+  
+  if (!response.ok) {
+    let errorMessage = `Errore HTTP ${response.status}`;
+    try {
+        const errorData = await response.json();
+        errorMessage = errorData.details || errorData.error || errorMessage;
+    } catch (e) {
+        const textError = await response.text();
+        if (textError) errorMessage = textError;
     }
-  } catch (e) {
-    console.error("Fallito il salvataggio del template", e);
-    throw e;
+    throw new Error(errorMessage);
   }
 };
 
 export const deleteTemplate = async (id: string): Promise<void> => {
-  try {
-    const response = await fetch(`/api/templates/${id}`, {
-      method: 'DELETE',
-    });
-    if (!response.ok) throw new Error('Eliminazione fallita');
-  } catch (e) {
-    console.error("Fallita l'eliminazione del template", e);
-    throw e;
-  }
+  const response = await fetch(`/api/templates/${id}`, {
+    method: 'DELETE',
+  });
+  if (!response.ok) throw new Error('Eliminazione fallita');
 };