support tracker name provided on 1st argument

EmilianoSanchez · EmilianoSanchez · commit d8a728c2f153 · 2022-06-21T11:52:06.000-03:00
diff --git a/src/integrations/ga/__tests__/GaToSplit.spec.ts b/src/integrations/ga/__tests__/GaToSplit.spec.ts
@@ -272,13 +272,15 @@ test('GaToSplit: `autoRequire` script and flag param', () => {
   window.ga('create', 'UA-ID-1', 'auto', 't1');
   window.ga('create', 'UA-ID-2', { name: 't2' });
   window.ga('create', 'UA-ID-3', 'auto', { name: 't3' });
+  window.ga('create', { trackingId: 'UA-ID-4', name: 't4' });
 
   expect(window.ga.q.map(args => args[0])).toEqual([
     'provide', 'provide',
     'create', 'require',
     'create', 't1.require',
     'create', 't2.require',
     'create', 't3.require',
+    'create', 't4.require',
   ]);
 
   // test teardown
diff --git a/src/integrations/ga/autoRequire.js b/src/integrations/ga/autoRequire.js
@@ -8,18 +8,14 @@
   i[r].q = i[r].q || [];
 
   var ts = {}; // Tracker names
+  function name(arg) { return typeof arg === 'object' && typeof arg.name === 'string' && arg.name; }
+
   var o = i[r].q.push;
   i[r].q.push = function (v) {
     var result = o.apply(this, arguments);
 
     if (v && v[0] === 'create') {
-      var t = typeof v[2] === 'object' && typeof v[2].name === 'string' ?
-        v[2].name : // `ga('create', 'UA-ID', { name: 'trackerName', ... })`
-        typeof v[3] === 'object' && typeof v[3].name === 'string' ?
-          v[3].name : // `ga('create', 'UA-ID', 'auto', { name: 'trackerName', ... })`
-          typeof v[3] === 'string' ?
-            v[3] : // `ga('create', 'UA-ID', 'auto', 'trackerName')`
-            undefined; // Default name, e.g.: `ga('create', 'UA-ID', 'auto')`
+      var t = name(v[1]) || name(v[2]) || name(v[3]) || (typeof v[3] === 'string' ? v[3] : undefined); // Get tracker name
 
       if (!ts[t]) {
         ts[t] = true;
