Minor cleanup and copy update

astrid/plugin-src/com/todoroo/astrid/core/LabsPreferences.java

@@ -56,7 +56,7 @@ public class LabsPreferences extends TodorooPreferenceActivity {
             setEnabledSummary(preference, value,
                     R.string.EPr_third_party_addons_desc_enabled, R.string.EPr_third_party_addons_desc_disabled);
         } else if (r.getString(R.string.p_end_at_deadline).equals(key)){
-            setEnabledSummary(preference, value, R.string.Cal_end_at_due_time, R.string.Cal_start_at_due_time);
+            setEnabledSummary(preference, value, R.string.EPr_cal_start_at_due_time, R.string.EPr_cal_end_at_due_time);
         } else if (r.getString(R.string.p_ideas_tab_enabled).equals(key)) {
             if (!Constants.MARKET_STRATEGY.allowIdeasTab()) {
                 PreferenceScreen screen = getPreferenceScreen();

astrid/plugin-src/com/todoroo/astrid/gcal/GCalHelper.java

@@ -181,12 +181,12 @@ public class GCalHelper {
                 long estimatedTime = task.getValue(Task.ESTIMATED_SECONDS)  * 1000;
                 if(estimatedTime <= 0)
                     estimatedTime = DEFAULT_CAL_TIME;
-                if (Preferences.getBoolean(R.string.p_end_at_deadline, true)){
-                    values.put("dtstart", dueDate - estimatedTime);
-                    values.put("dtend", dueDate);
-                }else{
+                if (Preferences.getBoolean(R.string.p_end_at_deadline, true)) {
                     values.put("dtstart", dueDate);
                     values.put("dtend", dueDate + estimatedTime);
+                }else{
+                    values.put("dtstart", dueDate - estimatedTime);
+                    values.put("dtend", dueDate);
                 }
                 // setting a duetime to a previously timeless event requires explicitly setting allDay=0
                 values.put("allDay", "0");

astrid/res/values/strings-core.xml

@@ -647,6 +647,12 @@
   <string name="EPr_ideas_tab_enabled">Task Ideas</string>
   <string name="EPr_ideas_tab_description">Get ideas to help you complete tasks</string>
   
+  <!-- Preferences: calendar event start time -->
+  <string name = "EPr_cal_end_or_start_at_due_time">Calendar event time</string>
+  <string name = "EPr_cal_end_at_due_time">End calendar events at due time</string>
+  <string name = "EPr_cal_start_at_due_time">Start calendar events at due time</string>
+  
+  
   <string name="EPr_swipe_lists_restart_alert">You will need to restart Astrid for this change to take effect</string>
   
   <string-array name="EPr_swipe_lists_performance_mode">
@@ -656,11 +662,6 @@
       <item>High Performance</item>
   </string-array>
   
-  <string name = "Cal_end_or_start_at_due_time">Calendar Events</string>
-  <string name = "Cal_end_at_due_time">End Calendar Events at Due Time</string>
-  <string name = "Cal_start_at_due_time">Starts Calendar Events at Due Time</string>
-  
-  
   
   <string-array name="EPr_swipe_lists_performance_desc">
   <!-- slide 49a -->

astrid/res/xml/preferences_labs.xml

@@ -23,7 +23,7 @@
         android:title="@string/EPr_ideas_tab_enabled" 
         android:summary="@string/EPr_ideas_tab_description" />
     <CheckBoxPreference
-        android:title="@string/Cal_end_or_start_at_due_time"
+        android:title="@string/EPr_cal_end_or_start_at_due_time"
         android:key = "@string/p_end_at_deadline" />
     
 </PreferenceScreen>

astrid/src/com/todoroo/astrid/service/abtesting/ABTests.java

@@ -116,17 +116,8 @@ public class ABTests {
 
     public static final String AB_TEST_CAL_KEY = "cal_ab_test"; //$NON-NLS-1$
 
-    private static final int[] calTestNewUserProbs = new int[] {1, 1};
-
-    private static final int[] calTestExistingUserProbs = new int[] {1, 1};
-
-    private static final String[] calTestDescriptions = new String[] {"Start at due time", "End at due time"};  //$NON-NLS-1$//$NON-NLS-2$
-
-
-
-
     private void initialize() {
-        addTest(AB_TEST_CAL_KEY, calTestNewUserProbs, calTestExistingUserProbs, calTestDescriptions);
-
+        addTest(AB_TEST_CAL_KEY, new int[] {1, 1}, new int[] {1, 1},
+                new String[] {"end-at-due-time", "start-at-due-time"}); //$NON-NLS-1$ //$NON-NLS-2$
     }
 }

astrid/src/com/todoroo/astrid/utility/AstridPreferences.java

@@ -60,7 +60,7 @@ public class AstridPreferences {
         Preferences.setIfUnset(prefs, editor, r, R.string.p_third_party_addons, false);
         Preferences.setIfUnset(prefs, editor, r, R.string.p_ideas_tab_enabled, true);
         Preferences.setIfUnset(prefs, editor, r, R.string.p_end_at_deadline,
-                ABChooser.readChoiceForTest(ABTests.AB_TEST_CAL_KEY) == 0);
+                ABChooser.readChoiceForTest(ABTests.AB_TEST_CAL_KEY) != 0);
 
         if ("white-blue".equals(Preferences.getStringValue(R.string.p_theme))) { //$NON-NLS-1$ migrate from when white-blue wasn't the default
             Preferences.setString(R.string.p_theme, ThemeService.THEME_WHITE);