fix: correctly identify empty cookie value | fix #12

porscheofficial · Mar 15, 2023 · c3c3a66 · c3c3a66
1 parent db7da86
commit c3c3a66
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/...ages/cookie-consent-banner/src/components/cookie-consent-banner/cookie-consent-banner.tsx b/...ages/cookie-consent-banner/src/components/cookie-consent-banner/cookie-consent-banner.tsx
@@ -125,11 +125,14 @@ export class CookieConsentBanner {
       .filter((category) => category.isMandatory)
       .map((category) => category.key);
 
-    const cookieValueString =
-      `; ${document.cookie}`.split(`; ${this.cookieName}=`).pop() ??
-      "".split(";").shift();
+    let cookieValues: string[] = [];
 
-    const cookieValues = cookieValueString ? cookieValueString.split(",") : [];
+    if (document.cookie) {
+      const cookieValueString =
+        `; ${document.cookie}`.split(`; ${this.cookieName}=`).pop() ??
+        "".split(";").shift();
+      cookieValues = cookieValueString ? cookieValueString.split(",") : [];
+    }
 
     if (cookieValues.length === 0) {
       this.isShown = true;