fix: attr__href missing issue

usermaven · Oct 16, 2024 · 841bb22 · 841bb22
1 parent 7da38c3
commit 841bb22
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/packages/javascript-sdk/src/tracking/autocapture.ts b/packages/javascript-sdk/src/tracking/autocapture.ts
@@ -159,14 +159,21 @@ class AutoCapture {
 
     private getElementsJson(targetElementList: Element[], e: Event): any[] {
         const elementsJson: any[] = [];
-        let href;
+        let href: false | string | null = null;
+        let explicitNoCapture = false;
 
         _each(targetElementList, (el) => {
             if (isTag(el, 'a')) {
                 href = el.getAttribute('href');
                 href = shouldCaptureElement(el) && shouldCaptureValue(href) && href;
             }
 
+            // Check for 'ph-no-capture' class
+            const classes = getClassName(el).split(' ');
+            if (_includes(classes, 'ph-no-capture')) {
+                explicitNoCapture = true;
+            }
+
             elementsJson.push(this.getPropertiesFromElement(el));
         });
 
@@ -178,7 +185,7 @@ class AutoCapture {
             elementsJson[0]['attr__href'] = this.sanitizeUrl(href);
         }
 
-        return elementsJson;
+        return explicitNoCapture ? [] : elementsJson;
     }
 
     private getPropertiesFromElement(element: Element): any {