Fixed an off-by-one error in the gcal default calendar selection

pull/14/head
Sam Bosley 13 years ago
parent c9cc27eeb6
commit 06381f4fb5

@ -143,7 +143,7 @@ public class Calendars {
row++; row++;
} }
if (result.defaultIndex == -1 || result.defaultIndex >= calendarCount) { if (result.defaultIndex >= calendarCount) {
result.defaultIndex = 0; result.defaultIndex = 0;
} }

@ -76,7 +76,7 @@ public class GCalControlSet extends PopupControlSet {
adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item); adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
calendarSelector.setPromptId(title); calendarSelector.setPromptId(title);
calendarSelector.setAdapter(adapter); calendarSelector.setAdapter(adapter);
calendarSelector.setSelection(calendars.defaultIndex); calendarSelector.setSelection(calendars.defaultIndex + 1); // plus 1 for the no selection item
calendarSelector.setOnItemSelectedListener(new OnItemSelectedListener() { calendarSelector.setOnItemSelectedListener(new OnItemSelectedListener() {
@Override @Override
public void onItemSelected(AdapterView<?> arg0, View arg1, public void onItemSelected(AdapterView<?> arg0, View arg1,
@ -207,7 +207,9 @@ public class GCalControlSet extends PopupControlSet {
@Override @Override
protected void refreshDisplayView() { protected void refreshDisplayView() {
TextView calendar = (TextView) getDisplayView().findViewById(R.id.calendar_display_which); TextView calendar = (TextView) getDisplayView().findViewById(R.id.calendar_display_which);
if (calendarSelector.getSelectedItemPosition() != 0) { if (hasEvent) {
calendar.setText(R.string.gcal_TEA_has_event);
} else if (calendarSelector.getSelectedItemPosition() != 0) {
calendar.setText((String)calendarSelector.getSelectedItem()); calendar.setText((String)calendarSelector.getSelectedItem());
} else { } else {
calendar.setText(R.string.gcal_TEA_none_selected); calendar.setText(R.string.gcal_TEA_none_selected);

@ -29,6 +29,8 @@
<string name="gcal_TEA_none_selected">Add to cal...</string> <string name="gcal_TEA_none_selected">Add to cal...</string>
<string name="gcal_TEA_has_event">Cal event</string>
<!-- ======================================================== Calendars == --> <!-- ======================================================== Calendars == -->
<!-- Calendar event name when task is completed (%s => task title) --> <!-- Calendar event name when task is completed (%s => task title) -->

Loading…
Cancel
Save