use Set and plain for loop

Author: mollfpr

scripts/utils/Translator/Translator.ts

@@ -53,7 +53,7 @@ abstract class Translator {
      */
     public validateTemplateHTML(original: string, translated: string): boolean {
         // Attributes that are allowed to be translated
-        const TRANSLATABLE_ATTRIBUTES = ['alt', 'title', 'placeholder', 'aria-label', 'aria-describedby', 'aria-labelledby', 'value'];
+        const TRANSLATABLE_ATTRIBUTES = new Set(['alt', 'title', 'placeholder', 'aria-label', 'aria-describedby', 'aria-labelledby', 'value']);
 
         const parseHTMLStructure = (s: string) => {
             const doc = parseDocument(s);
@@ -65,12 +65,12 @@ abstract class Translator {
                 // Extract attributes, excluding translatable ones
                 const attributes: string[] = [];
                 if (element.attribs) {
-                    Object.entries(element.attribs).forEach(([attrName, attrValue]) => {
+                    for (const [attrName, attrValue] of Object.entries(element.attribs ?? {})) {
                         const normalizedAttrName = attrName.toLowerCase();
-                        if (!TRANSLATABLE_ATTRIBUTES.includes(normalizedAttrName)) {
+                        if (!TRANSLATABLE_ATTRIBUTES.has(normalizedAttrName)) {
                             attributes.push(`${normalizedAttrName}="${attrValue ?? ''}"`);
                         }
-                    });
+                    }
                 }
 
                 return {