Changeset 3462432

Timestamp:

02/16/2026 11:20:49 AM (5 days ago)

Author:

gkanters

Message:

Fix language switcher replacement by using stable class selectors instead of translated menu text; include current development version updates in trunk only.

File:

• ai-translate/trunk/ai-translate.php (modified) (2 diffs)

ai-translate/trunk/ai-translate.php

@@ -6 +6 @@
  * Author: NetCare
  * Author URI: https://netcare.nl/
- * Version: 2.2.7
+ * Version: 2.2.8
  * Requires at least: 5.0
  * Tested up to: 6.9
@@ -3047 +3047 @@
         function initLanguageSwitcherReplacement() {
 
-            // Find language switcher menu items (links with href="#" containing "Language Switcher")
-            const languageItems = document.querySelectorAll('a[href="#"]');
-
-            languageItems.forEach(function(link, index) {
-                if (link.textContent.trim() === 'Language Switcher') {
-                    const listItem = link.closest('li');
-                    if (listItem) {
-                        // Replace the entire menu item
-                        listItem.outerHTML = <?php echo json_encode($replacement_html); ?>;
-                    }
+            // Find language switcher items by stable classes/object markers (not translated text).
+            const candidates = document.querySelectorAll(
+                'li.menu-item-language-switcher > a[href="#"], li[class*="menu-item-object-ai_language_switcher"] > a[href="#"], a.ai-menu-language-current[href="#"]'
+            );
+
+            candidates.forEach(function(link) {
+                const listItem = link.closest('li');
+                if (!listItem) {
+                    return;
+                }
+                // Replace only when not already rendered with a submenu.
+                if (!listItem.querySelector('.sub-menu') || !listItem.classList.contains('menu-item-language-switcher')) {
+                    listItem.outerHTML = <?php echo json_encode($replacement_html); ?>;
                 }
             });
 
-            // Also check for items that might have been created differently
-            const allMenuItems = document.querySelectorAll('li.menu-item a[href="#"]');
-
-            allMenuItems.forEach(function(link, index) {
-                if (link.textContent.includes('NL') && link.querySelector('img')) {
-                    // This looks like our language switcher, make sure it has submenu
-                    const listItem = link.closest('li');
-                    if (listItem && !listItem.querySelector('.sub-menu')) {
-                        // Add submenu if missing
-                        const submenu = <?php echo json_encode($submenu_html); ?>;
-                        listItem.insertAdjacentHTML('beforeend', submenu);
-                        listItem.classList.add('menu-item-has-children');
-                    }
+            // Safety net: if a language-switcher container exists without submenu, append it.
+            const switcherItems = document.querySelectorAll('li.menu-item-language-switcher');
+            switcherItems.forEach(function(listItem) {
+                if (!listItem.querySelector('.sub-menu')) {
+                    listItem.insertAdjacentHTML('beforeend', <?php echo json_encode($submenu_html); ?>);
+                    listItem.classList.add('menu-item-has-children');
                 }
             });