Merge pull request #5 from manghidev/fixes

feat: add safe translation retrieval and fallback for language settings

Author: manghidev

js/app.js

@@ -55,6 +55,15 @@ let currentLanguage = 'en-US';
 let currentSerial = '';
 let currentTab = 'generator';
 
+// Función auxiliar para obtener traducciones de forma segura
+function getTranslations() {
+    if (!translations[currentLanguage]) {
+        console.error('Idioma no válido:', currentLanguage, 'usando en-US como respaldo');
+        currentLanguage = 'en-US';
+    }
+    return translations[currentLanguage];
+}
+
 // Función para cambiar de pestaña
 function switchTab(tabName) {
     currentTab = tabName;
@@ -268,7 +277,7 @@ function formatDecodedDate(date, language = 'es') {
 
 // Función para manejar la decodificación de seriales
 function handleDecodeSerial() {
-    const t = translations[currentLanguage];
+    const t = getTranslations();
     const serialInput = document.getElementById('serialInput');
     const dateOutput = document.getElementById('dateOutput');
 
@@ -294,8 +303,14 @@ function handleDecodeSerial() {
 
 // Función para cambiar el idioma
 function changeLanguage(lang) {
+    // Verificar que el idioma solicitado existe
+    if (!translations[lang]) {
+        console.error('Idioma no válido:', lang);
+        lang = 'en-US'; // Fallback al inglés
+    }
+    
     currentLanguage = lang;
-    const t = translations[currentLanguage];
+    const t = getTranslations();
 
     // Actualizar título de la página
     document.title = t.pageTitle;
@@ -317,7 +332,7 @@ function changeLanguage(lang) {
 
 // Función para actualizar la interfaz de usuario
 function updateUI() {
-    const t = translations[currentLanguage];
+    const t = getTranslations();
 
     // Actualizar textos principales
     document.getElementById('title').textContent = t.title;
@@ -364,8 +379,9 @@ function updateUI() {
 // Función para manejar la generación del serial
 async function handleGenerateSerial() {
     console.log('handleGenerateSerial called'); // Debug
+    console.log('Current language:', currentLanguage); // Debug
 
-    const t = translations[currentLanguage];
+    const t = getTranslations();
     const serialOutput = document.getElementById('serialOutput');
     const copyButton = document.getElementById('copyButton');
 
@@ -430,7 +446,7 @@ async function handleGenerateSerial() {
 
 // Función para copiar al portapapeles
 function copyToClipboard() {
-    const t = translations[currentLanguage];
+    const t = getTranslations();
     const copyButton = document.getElementById('copyButton');
 
     if (currentSerial && currentSerial !== t.initialMessage && !currentSerial.startsWith(t.errorPrefix)) {
@@ -465,9 +481,19 @@ document.addEventListener('DOMContentLoaded', () => {
     // Cargar preferencias del localStorage
     const savedLanguage = localStorage.getItem('preferredLanguage') || 'en-US';
     const savedTab = localStorage.getItem('preferredTab') || 'generator';
-    currentLanguage = savedLanguage;
+    
+    // Verificar que el idioma guardado existe en las traducciones
+    if (translations[savedLanguage]) {
+        currentLanguage = savedLanguage;
+    } else {
+        console.warn('Idioma guardado no válido:', savedLanguage, 'usando en-US');
+        currentLanguage = 'en-US';
+    }
+    
     currentTab = savedTab;
 
+    console.log('Idioma inicializado:', currentLanguage); // Debug
+    
     // Configurar event listeners principales
     document.getElementById('generateButton').addEventListener('click', handleGenerateSerial);
     document.getElementById('copyButton').addEventListener('click', copyToClipboard);