Merge pull request #9 from antistatique/1-0-x/prevent-js-error-langua…

…ge-switcher-not-found
antistatique · Sep 12, 2022 · 8836611 · 8836611
2 parents 979b6cf + 9249165
commit 8836611
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,4 +6,5 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 - init module with Cookies & Browser redirection on Homepage only
+- handling of missing common JS library
 - remove dependency on JQuery
diff --git a/README.md b/README.md
@@ -43,7 +43,13 @@ Drupal.homeRedirectLang.setPreferredLanguage('fr');
 ```
 
 ```javascript
+// Prevent attempting to attach cookie if the common library is not available.
+if (typeof Drupal.homeRedirectLang == "undefined") {
+  return;
+}
+
 let links = document.querySelectorAll('.language-link');
+
 links.forEach(link => {
   link.addEventListener('click', function (event) {
     var hreflang = event.target.getAttribute('hreflang');

diff --git a/js/home_redirect_lang.language_switcher.js b/js/home_redirect_lang.language_switcher.js
@@ -15,7 +15,14 @@
    */
   Drupal.behaviors.homeRedirectLangSwitcher = {
     attach: function (context) {
+
+      // Prevent attempting to attach cookie if the common library is not available.
+      if (typeof Drupal.homeRedirectLang == "undefined") {
+        return;
+      }
+
       let links = document.querySelectorAll('.language-switcher-language-url .language-link');
+
       links.forEach(link => {
         link.addEventListener('click', function (event) {
           var hreflang = event.target.getAttribute('hreflang');