From 81e6146e7274d2c7b7d4500a759f8b404ad6d005 Mon Sep 17 00:00:00 2001 From: Sam Bosley Date: Thu, 14 Jun 2012 20:56:30 -0700 Subject: [PATCH] Ending user reengagement test; always enabled --- .../com/todoroo/astrid/reminders/ReengagementReceiver.java | 4 ---- astrid/src/com/todoroo/astrid/service/abtesting/ABTests.java | 4 ---- 2 files changed, 8 deletions(-) diff --git a/astrid/plugin-src/com/todoroo/astrid/reminders/ReengagementReceiver.java b/astrid/plugin-src/com/todoroo/astrid/reminders/ReengagementReceiver.java index 68a09b261..2205efcfb 100644 --- a/astrid/plugin-src/com/todoroo/astrid/reminders/ReengagementReceiver.java +++ b/astrid/plugin-src/com/todoroo/astrid/reminders/ReengagementReceiver.java @@ -29,8 +29,6 @@ import com.todoroo.astrid.core.SortHelper; import com.todoroo.astrid.data.Task; import com.todoroo.astrid.data.TaskApiDao.TaskCriteria; import com.todoroo.astrid.service.TaskService; -import com.todoroo.astrid.service.abtesting.ABChooser; -import com.todoroo.astrid.service.abtesting.ABTests; import com.todoroo.astrid.utility.Constants; public class ReengagementReceiver extends BroadcastReceiver { @@ -44,8 +42,6 @@ public class ReengagementReceiver extends BroadcastReceiver { @Override public void onReceive(Context context, Intent intent) { DependencyInjectionService.getInstance().inject(this); - if (ABChooser.readChoiceForTest(ABTests.AB_TEST_REENGAGEMENT_ENABLED) == 0) - return; int reengagementReminders = Preferences.getInt(ReengagementService.PREF_REENGAGEMENT_COUNT, 1); Preferences.setInt(ReengagementService.PREF_REENGAGEMENT_COUNT, reengagementReminders + 1); diff --git a/astrid/src/com/todoroo/astrid/service/abtesting/ABTests.java b/astrid/src/com/todoroo/astrid/service/abtesting/ABTests.java index a11a54abf..a852f137d 100644 --- a/astrid/src/com/todoroo/astrid/service/abtesting/ABTests.java +++ b/astrid/src/com/todoroo/astrid/service/abtesting/ABTests.java @@ -120,9 +120,6 @@ public class ABTests { addTest(AB_TEST_PRODUCTEEV_ENABLED, new int[] { 1, 1 }, new int[] { 1, 0 }, new String[] { "producteev-disabled", "producteev-enabled" }); //$NON-NLS-1$ //$NON-NLS-2$ - addTest(AB_TEST_REENGAGEMENT_ENABLED, new int[] { 1, 9 }, - new int[] { 1, 9 }, new String[] { "reengagement-disabled", "reengagement-enabled" }); //$NON-NLS-1$ //$NON-NLS-2$ - boolean allowIdeasTab = Constants.MARKET_STRATEGY.allowIdeasTab(); int[] noIdeasTab = new int[] { 1, 0 }; @@ -132,6 +129,5 @@ public class ABTests { public static final String AB_TEST_IDEAS_TAB = "ideasTab"; //$NON-NLS-1$ public static final String AB_TEST_PRODUCTEEV_ENABLED = "producteev"; //$NON-NLS-1$ - public static final String AB_TEST_REENGAGEMENT_ENABLED = "reengagement"; ////$NON-NLS-1$ }