From b732cbd443b262e7c500c5418145ffc6ee82022f Mon Sep 17 00:00:00 2001 From: alexzatsepin Date: Mon, 19 Dec 2016 15:19:20 +0300 Subject: [PATCH] [android] Added search tab selected statistic event --- android/src/com/mapswithme/maps/search/SearchFragment.java | 4 +++- android/src/com/mapswithme/maps/search/TabAdapter.java | 3 ++- android/src/com/mapswithme/util/statistics/Statistics.java | 7 +++++++ 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/android/src/com/mapswithme/maps/search/SearchFragment.java b/android/src/com/mapswithme/maps/search/SearchFragment.java index 7ca6005161..0a655d7a1b 100644 --- a/android/src/com/mapswithme/maps/search/SearchFragment.java +++ b/android/src/com/mapswithme/maps/search/SearchFragment.java @@ -4,6 +4,7 @@ import android.content.Intent; import android.location.Location; import android.os.Build; import android.os.Bundle; +import android.support.annotation.NonNull; import android.support.design.widget.TabLayout; import android.support.v4.app.Fragment; import android.support.v4.app.FragmentManager; @@ -267,8 +268,9 @@ public class SearchFragment extends BaseMwmFragment tabAdapter.setTabSelectedListener(new TabAdapter.OnTabSelectedListener() { @Override - public void onTabSelected(TabAdapter.Tab tab) + public void onTabSelected(@NonNull TabAdapter.Tab tab) { + Statistics.INSTANCE.trackSearchTabSelected(tab.name()); mToolbarController.deactivate(); } }); diff --git a/android/src/com/mapswithme/maps/search/TabAdapter.java b/android/src/com/mapswithme/maps/search/TabAdapter.java index 336e65b33b..7926f44581 100644 --- a/android/src/com/mapswithme/maps/search/TabAdapter.java +++ b/android/src/com/mapswithme/maps/search/TabAdapter.java @@ -3,6 +3,7 @@ package com.mapswithme.maps.search; import android.content.Context; import android.content.res.ColorStateList; import android.content.res.Configuration; +import android.support.annotation.NonNull; import android.support.design.widget.TabLayout; import android.support.v4.app.Fragment; import android.support.v4.app.FragmentManager; @@ -73,7 +74,7 @@ class TabAdapter extends FragmentPagerAdapter interface OnTabSelectedListener { - void onTabSelected(Tab tab); + void onTabSelected(@NonNull Tab tab); } // Workaround for https://code.google.com/p/android/issues/detail?id=180454 diff --git a/android/src/com/mapswithme/util/statistics/Statistics.java b/android/src/com/mapswithme/util/statistics/Statistics.java index e350d89f03..f46b6f7aed 100644 --- a/android/src/com/mapswithme/util/statistics/Statistics.java +++ b/android/src/com/mapswithme/util/statistics/Statistics.java @@ -64,6 +64,7 @@ public enum Statistics public static final String SEARCH_ITEM_CLICKED = "Search. Key clicked"; public static final String SEARCH_ON_MAP_CLICKED = "Search. View on map clicked."; public static final String SEARCH_CANCEL = "Search. Cancel."; + public static final String SEARCH_TAB_SELECTED = "Search_Tab_selected"; // place page public static final String PP_OPEN = "PP. Open"; @@ -183,6 +184,7 @@ public enum Statistics public static final String FROM = "from"; public static final String TO = "to"; public static final String CATEGORY = "category"; + public static final String TAB = "tab"; public static final String COUNT = "Count"; public static final String CHANNEL = "Channel"; public static final String CALLER_ID = "Caller ID"; @@ -330,6 +332,11 @@ public enum Statistics params().add(EventParam.ENABLED, String.valueOf(isEnabled))); } + public void trackSearchTabSelected(@NonNull String tab) + { + trackEvent(EventName.SEARCH_TAB_SELECTED, params().add(EventParam.TAB, tab)); + } + public void trackSearchCategoryClicked(String category) { trackEvent(EventName.SEARCH_CAT_CLICKED, params().add(EventParam.CATEGORY, category));