From 9245982ea6f725105412c2b9c4e83f831a9c5149 Mon Sep 17 00:00:00 2001 From: Barry Attwater Date: Thu, 12 Jun 2025 11:56:49 +0100 Subject: [PATCH] Fix issue with array _clickCaptureElements being referenced by value rather than by index. --- .../src/handlers/AutoCaptureHandler.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/extensions/applicationinsights-clickanalytics-js/src/handlers/AutoCaptureHandler.ts b/extensions/applicationinsights-clickanalytics-js/src/handlers/AutoCaptureHandler.ts index e1e14202f..0fd543aca 100644 --- a/extensions/applicationinsights-clickanalytics-js/src/handlers/AutoCaptureHandler.ts +++ b/extensions/applicationinsights-clickanalytics-js/src/handlers/AutoCaptureHandler.ts @@ -7,7 +7,7 @@ import { IDiagnosticLogger, IProcessTelemetryUnloadContext, ITelemetryUnloadState, IUnloadHook, createUniqueNamespace, eventOff, eventOn, getDocument, getWindow, isNullOrUndefined, mergeEvtNamespace, onConfigChange } from "@microsoft/applicationinsights-core-js"; -import { arrMap, strTrim } from "@nevware21/ts-utils"; +import { arrIncludes, arrMap, strTrim } from "@nevware21/ts-utils"; import { ClickAnalyticsPlugin } from "../ClickAnalyticsPlugin"; import { ActionType } from "../Enums"; import { IAutoCaptureHandler, IClickAnalyticsConfiguration, IPageActionOverrideValues } from "../Interfaces/Datamodel"; @@ -59,7 +59,7 @@ export class AutoCaptureHandler implements IAutoCaptureHandler { _self._pageAction.capturePageAction(element, overrideValues, customProperties, isRightClick); } } - + // Process click event function _processClick(clickEvent: any) { let win = getWindow(); @@ -68,7 +68,7 @@ export class AutoCaptureHandler implements IAutoCaptureHandler { } if (clickEvent) { let element = clickEvent.srcElement || clickEvent.target; - + // populate overrideValues var overrideValues: IPageActionOverrideValues = { clickCoordinateX: clickEvent.pageX, @@ -88,15 +88,15 @@ export class AutoCaptureHandler implements IAutoCaptureHandler { } else { return; } - + while (element && element.tagName) { // control property will be available for