split analytics into global config and tracker-specific fields, fix u…

…p install tracking
konklone · Apr 24, 2014 · c5b9b05 · c5b9b05
1 parent 5f27424
commit c5b9b05
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 15 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,7 +2,7 @@
 /bin
 /gen
 /res/values/keys.xml
-/res/*/analytics*
+/res/*/tracker_*
 /.idea
 
 # gradle/android studio stuff

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
@@ -155,13 +155,20 @@
         </receiver>
 
 		<service android:name=".notifications.NotificationService" />
-
-		<!-- Used for install referrer tracking -->
-		<receiver android:name="com.google.android.apps.analytics.AnalyticsReceiver" android:exported="true">
-		  <intent-filter>
-		    <action android:name="com.android.vending.INSTALL_REFERRER" />
-		  </intent-filter>
-		</receiver>
+
+        <!-- Used for install referrer tracking-->
+        <service android:name="com.google.android.gms.analytics.CampaignTrackingService"/>
+        <receiver
+            android:name="com.google.android.gms.analytics.CampaignTrackingReceiver"
+            android:exported="true">
+            <intent-filter>
+                <action android:name="com.android.vending.INSTALL_REFERRER" />
+            </intent-filter>
+        </receiver>
+
+        <meta-data android:name="com.google.android.gms.analytics.globalConfigResource"
+            android:resource="@xml/analytics"
+            />
 
     </application>
 </manifest>
diff --git a/res/xml/analytics.xml b/res/xml/analytics.xml
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<resources xmlns:android="http://schemas.android.com/apk/res/android">
+    <bool name="ga_anonymizeIp">true</bool>
+    <string name="ga_logLevel">warn</string>
+    <integer name="ga_dispatchPeriod">1</integer>
+</resources>
diff --git a/src/com/sunlightlabs/android/congress/CongressApp.java b/src/com/sunlightlabs/android/congress/CongressApp.java
@@ -22,13 +22,13 @@ public synchronized Tracker appTracker() {
         if (local == null) {
 
             try {
-                getResources().getXml(R.xml.analytics);
+                getResources().getXml(R.xml.tracker_app);
             } catch(Resources.NotFoundException e) {
                 hasLocal = false;
                 return null;
             }
 
-            local = GoogleAnalytics.getInstance(this).newTracker(R.xml.analytics);
+            local = GoogleAnalytics.getInstance(this).newTracker(R.xml.tracker_app);
         }
 
         return local;
@@ -39,13 +39,13 @@ public synchronized Tracker globalTracker() {
 
         if (global == null) {
             try {
-                getResources().getXml(R.xml.analytics_global);
+                getResources().getXml(R.xml.tracker_global);
             } catch(Resources.NotFoundException e) {
                 hasGlobal = false;
                 return null;
             }
 
-            global = GoogleAnalytics.getInstance(this).newTracker(R.xml.analytics_global);
+            global = GoogleAnalytics.getInstance(this).newTracker(R.xml.tracker_global);
         }
 
         return global;

diff --git a/analytics.xml.example → tracker.xml.example b/analytics.xml.example → tracker.xml.example
@@ -4,7 +4,4 @@ tools:ignore="TypographyDashes">
   <string name="ga_trackingId">UA-XXX-YY</string>
   <bool name="ga_autoActivityTracking">true</bool>
   <bool name="ga_reportUncaughtExceptions">true</bool>
-  <bool name="ga_anonymizeIp">true</bool>
-  <string name="ga_logLevel">warning</string>
-  <integer name="ga_dispatchPeriod">1</integer>
 </resources>