@splitsoftware/splitio-commons 1.6.0 → 1.6.1

package/CHANGES.txt

@@ -1,3 +1,6 @@
+1.6.1 (July 22, 2022)
+- Updated GoogleAnalyticsToSplit integration to validate `autoRequire` config parameter and avoid some wrong warning logs when mapping GA hit fields to Split event properties.
+
 1.6.0 (July 21, 2022)
 - Added `autoRequire` configuration option to the Google Analytics to Split integration, which takes care of requiring the splitTracker plugin on trackers dynamically created by Google tag managers (See https://help.split.io/hc/en-us/articles/360040838752#set-up-with-gtm-and-gtag.js).
 - Updated browser listener to push remaining impressions and events on 'visibilitychange' and 'pagehide' DOM events, instead of 'unload', which is not reliable in modern mobile and desktop Web browsers.

package/cjs/integrations/ga/GaToSplit.js

@@ -64,7 +64,9 @@ function mapperBuilder(mapping) {
         var fields = mapping.eventProperties[hitType];
         if (fields) {
             for (var i = 0; i < fields.length; i++) {
-                properties[fields[i]] = model.get(fields[i]);
+                var fieldValue = model.get(fields[i]);
+                if (fieldValue !== undefined)
+                    properties[fields[i]] = fieldValue;
             }
         }
         return {
@@ -247,6 +249,6 @@ function GaToSplit(sdkOptions, params) {
         return SplitTracker;
     }());
     // Register the plugin, even if config is invalid, since, if not provided, it will block `ga` command queue.
-    providePlugin(window, 'splitTracker', SplitTracker, log, sdkOptions.autoRequire);
+    providePlugin(window, 'splitTracker', SplitTracker, log, sdkOptions.autoRequire === true);
 }
 exports.GaToSplit = GaToSplit;

package/cjs/integrations/ga/GoogleAnalyticsToSplit.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.GoogleAnalyticsToSplit = void 0;
 var GaToSplit_1 = require("./GaToSplit");
 function GoogleAnalyticsToSplit(options) {
+    if (options === void 0) { options = {}; }
     // GaToSplit integration factory
     function GoogleAnalyticsToSplitFactory(params) {
         return (0, GaToSplit_1.GaToSplit)(options, params);

package/esm/integrations/ga/GaToSplit.js

@@ -61,7 +61,9 @@ function mapperBuilder(mapping) {
         var fields = mapping.eventProperties[hitType];
         if (fields) {
             for (var i = 0; i < fields.length; i++) {
-                properties[fields[i]] = model.get(fields[i]);
+                var fieldValue = model.get(fields[i]);
+                if (fieldValue !== undefined)
+                    properties[fields[i]] = fieldValue;
             }
         }
         return {
@@ -241,5 +243,5 @@ export function GaToSplit(sdkOptions, params) {
         return SplitTracker;
     }());
     // Register the plugin, even if config is invalid, since, if not provided, it will block `ga` command queue.
-    providePlugin(window, 'splitTracker', SplitTracker, log, sdkOptions.autoRequire);
+    providePlugin(window, 'splitTracker', SplitTracker, log, sdkOptions.autoRequire === true);
 }

package/esm/integrations/ga/GoogleAnalyticsToSplit.js

@@ -1,5 +1,6 @@
 import { GaToSplit } from './GaToSplit';
 export function GoogleAnalyticsToSplit(options) {
+    if (options === void 0) { options = {}; }
     // GaToSplit integration factory
     function GoogleAnalyticsToSplitFactory(params) {
         return GaToSplit(options, params);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio-commons",
-  "version": "1.6.0",
+  "version": "1.6.1",
   "description": "Split Javascript SDK common components",
   "main": "cjs/index.js",
   "module": "esm/index.js",

package/src/integrations/ga/GaToSplit.ts

@@ -78,7 +78,8 @@ function mapperBuilder(mapping: typeof defaultMapping) {
     const fields: string[] = mapping.eventProperties[hitType];
     if (fields) {
       for (let i = 0; i < fields.length; i++) {
-        properties[fields[i]] = model.get(fields[i]);
+        const fieldValue = model.get(fields[i]);
+        if (fieldValue !== undefined) properties[fields[i]] = fieldValue;
       }
     }
 
@@ -290,5 +291,5 @@ export function GaToSplit(sdkOptions: GoogleAnalyticsToSplitOptions, params: IIn
   }
 
   // Register the plugin, even if config is invalid, since, if not provided, it will block `ga` command queue.
-  providePlugin(window, 'splitTracker', SplitTracker, log, sdkOptions.autoRequire);
+  providePlugin(window, 'splitTracker', SplitTracker, log, sdkOptions.autoRequire === true);
 }

package/src/integrations/ga/GoogleAnalyticsToSplit.ts

@@ -2,7 +2,7 @@ import { IIntegrationFactoryParams, IntegrationFactory } from '../types';
 import { GaToSplit } from './GaToSplit';
 import { GoogleAnalyticsToSplitOptions } from './types';
 
-export function GoogleAnalyticsToSplit(options: GoogleAnalyticsToSplitOptions): IntegrationFactory {
+export function GoogleAnalyticsToSplit(options: GoogleAnalyticsToSplitOptions = {}): IntegrationFactory {
 
   // GaToSplit integration factory
   function GoogleAnalyticsToSplitFactory(params: IIntegrationFactoryParams) {

package/types/integrations/ga/GoogleAnalyticsToSplit.d.ts

@@ -1,3 +1,3 @@
 import { IntegrationFactory } from '../types';
 import { GoogleAnalyticsToSplitOptions } from './types';
-export declare function GoogleAnalyticsToSplit(options: GoogleAnalyticsToSplitOptions): IntegrationFactory;
+export declare function GoogleAnalyticsToSplit(options?: GoogleAnalyticsToSplitOptions): IntegrationFactory;