Sfoglia il codice sorgente

Adds optional label to analytics handler.

dev1
damencho 9 anni fa
parent
commit
a94fab12f1
2 ha cambiato i file con 8 aggiunte e 5 eliminazioni
  1. 2
    1
      JitsiConnection.js
  2. 6
    4
      modules/statistics/AnalyticsAdapter.js

+ 2
- 1
JitsiConnection.js Vedi File

21
 
21
 
22
     this.addEventListener(JitsiConnectionEvents.CONNECTION_FAILED,
22
     this.addEventListener(JitsiConnectionEvents.CONNECTION_FAILED,
23
         function (errType, msg) {
23
         function (errType, msg) {
24
-            Statistics.analytics.sendEvent('connection.failed.' + errType);
24
+            // sends analytics and callstats event
25
+            Statistics.sendEventToAll('connection.failed.' + errType, msg);
25
         }.bind(this));
26
         }.bind(this));
26
 
27
 
27
     this.addEventListener(JitsiConnectionEvents.CONNECTION_DISCONNECTED,
28
     this.addEventListener(JitsiConnectionEvents.CONNECTION_DISCONNECTED,

+ 6
- 4
modules/statistics/AnalyticsAdapter.js Vedi File

17
 // implementation we will use here and we have to postpone it i.e. we will make
17
 // implementation we will use here and we have to postpone it i.e. we will make
18
 // a lazy decision, will wait for loaded or dispose methods to be called.
18
 // a lazy decision, will wait for loaded or dispose methods to be called.
19
 // in the meantime we accumulate any events received
19
 // in the meantime we accumulate any events received
20
-AnalyticsAdapter.prototype.sendEvent = function (action, data) {
20
+AnalyticsAdapter.prototype.sendEvent = function (action, data, label) {
21
     if (this.analytics === null || typeof this.analytics === 'undefined') {
21
     if (this.analytics === null || typeof this.analytics === 'undefined') {
22
         // missing this.analytics but have window implementation, let's use it
22
         // missing this.analytics but have window implementation, let's use it
23
         if (window.Analytics) {
23
         if (window.Analytics) {
26
         else {
26
         else {
27
             AnalyticsAdapter.eventsQueue.push({
27
             AnalyticsAdapter.eventsQueue.push({
28
                 action: action,
28
                 action: action,
29
-                data: data
29
+                data: data,
30
+                label: label
30
             });
31
             });
31
             // stored, lets break here
32
             // stored, lets break here
32
             return;
33
             return;
33
         }
34
         }
34
     }
35
     }
35
     try {
36
     try {
36
-        this.analytics.sendEvent(action + this.browserActionSuffix, data);
37
+        this.analytics.sendEvent(
38
+            action + this.browserActionSuffix, data, label);
37
     } catch (ignored) {}
39
     } catch (ignored) {}
38
 };
40
 };
39
 
41
 
57
     // new analytics lets send all events if any
59
     // new analytics lets send all events if any
58
     if (AnalyticsAdapter.eventsQueue.length) {
60
     if (AnalyticsAdapter.eventsQueue.length) {
59
         AnalyticsAdapter.eventsQueue.forEach(function (event) {
61
         AnalyticsAdapter.eventsQueue.forEach(function (event) {
60
-            this.sendEvent(event.action, event.data);
62
+            this.sendEvent(event.action, event.data. event.label);
61
         }.bind(this));
63
         }.bind(this));
62
         AnalyticsAdapter.eventsQueue.length = 0;
64
         AnalyticsAdapter.eventsQueue.length = 0;
63
     }
65
     }

Loading…
Annulla
Salva