Bug AOEpeople#4, add support for data-cookie-notice attribute

nfreear · May 30, 2018 · 9422b8a · 9422b8a
1 parent 9c17763
commit 9422b8a
Showing 1 changed file with 18 additions and 3 deletions.
diff --git a/src/cookie.notice.js b/src/cookie.notice.js
@@ -46,7 +46,8 @@
         'buttonTextColor': '#fff',
         'noticeBgColor': '#000',
         'noticeTextColor': '#fff',
-        'linkColor': '#009fdd'
+        'linkColor': '#009fdd',
+        'debug': false
     };
 
     /**
@@ -78,8 +79,22 @@
             return;
         }
 
+        // 'data-' attribute - data-cookie-notice='{ "key": "value", ... }'
+        var elemCfg = document.querySelector('script[ data-cookie-notice ]');
+        var config;
+        try {
+            config = elemCfg ? JSON.parse(elemCfg.getAttribute('data-cookie-notice')) : {};
+        } catch (ex) {
+            console.error('data-cookie-notice JSON error:', elemCfg, ex);
+            config = {};
+        }
+
         // Extend default params
-        var params = extendDefaults(defaults, arguments[0] || {});
+        var params = extendDefaults(defaults, arguments[0] || config || {});
+
+        if (params.debug) {
+            console.warn('cookie-notice:', params);
+        }
 
         // Get current locale for notice text
         var noticeText = getStringForCurrentLocale(params.messageLocales);
@@ -313,4 +328,4 @@
     };
     /* end-test-code */
 
-}());
+}());