Fix rotation issue with deadline control set

pull/437/head
Alex Baker 10 years ago
parent 84dfa3ff5c
commit cdb237afe8

@ -18,8 +18,7 @@ import javax.inject.Inject;
import static org.tasks.time.DateTimeUtils.currentTimeMillis; import static org.tasks.time.DateTimeUtils.currentTimeMillis;
public class DatePickerActivity extends InjectingAppCompatActivity public class DatePickerActivity extends InjectingAppCompatActivity implements DatePickerDialog.OnDateSetListener {
implements DatePickerDialog.OnDateSetListener, DialogInterface.OnDismissListener {
private static final String FRAG_TAG_DATE_PICKER = "frag_tag_date_picker"; private static final String FRAG_TAG_DATE_PICKER = "frag_tag_date_picker";
@ -44,7 +43,12 @@ public class DatePickerActivity extends InjectingAppCompatActivity
dialog.setAccentColor(themeAccent.getAccentColor()); dialog.setAccentColor(themeAccent.getAccentColor());
dialog.show(fragmentManager, FRAG_TAG_DATE_PICKER); dialog.show(fragmentManager, FRAG_TAG_DATE_PICKER);
} }
dialog.setOnDismissListener(this); dialog.setOnCancelListener(new DialogInterface.OnCancelListener() {
@Override
public void onCancel(DialogInterface dialogInterface) {
finish();
}
});
dialog.setOnDateSetListener(this); dialog.setOnDateSetListener(this);
} }
@ -58,10 +62,6 @@ public class DatePickerActivity extends InjectingAppCompatActivity
setResult(RESULT_OK, new Intent() {{ setResult(RESULT_OK, new Intent() {{
putExtra(EXTRA_TIMESTAMP, new DateTime(year, monthOfYear + 1, dayOfMonth).getMillis()); putExtra(EXTRA_TIMESTAMP, new DateTime(year, monthOfYear + 1, dayOfMonth).getMillis());
}}); }});
}
@Override
public void onDismiss(DialogInterface dialog) {
finish(); finish();
} }
} }

@ -21,7 +21,7 @@ import javax.inject.Inject;
import static org.tasks.time.DateTimeUtils.currentTimeMillis; import static org.tasks.time.DateTimeUtils.currentTimeMillis;
public class TimePickerActivity extends InjectingAppCompatActivity implements TimePickerDialog.OnTimeSetListener, DialogInterface.OnDismissListener { public class TimePickerActivity extends InjectingAppCompatActivity implements TimePickerDialog.OnTimeSetListener {
private static final String FRAG_TAG_TIME_PICKER = "frag_tag_time_picker"; private static final String FRAG_TAG_TIME_PICKER = "frag_tag_time_picker";
@ -31,7 +31,6 @@ public class TimePickerActivity extends InjectingAppCompatActivity implements Ti
@Inject ThemeAccent themeAccent; @Inject ThemeAccent themeAccent;
private DateTime initial; private DateTime initial;
private boolean isChangingConfigurations;
@Override @Override
protected void onCreate(Bundle savedInstanceState) { protected void onCreate(Bundle savedInstanceState) {
@ -48,7 +47,12 @@ public class TimePickerActivity extends InjectingAppCompatActivity implements Ti
dialog.setAccentColor(themeAccent.getAccentColor()); dialog.setAccentColor(themeAccent.getAccentColor());
dialog.show(fragmentManager, FRAG_TAG_TIME_PICKER); dialog.show(fragmentManager, FRAG_TAG_TIME_PICKER);
} }
dialog.setOnDismissListener(this); dialog.setOnCancelListener(new DialogInterface.OnCancelListener() {
@Override
public void onCancel(DialogInterface dialogInterface) {
finish();
}
});
dialog.setOnTimeSetListener(this); dialog.setOnTimeSetListener(this);
} }
@ -66,20 +70,6 @@ public class TimePickerActivity extends InjectingAppCompatActivity implements Ti
.withMinuteOfHour(minutes) .withMinuteOfHour(minutes)
.getMillis()); .getMillis());
}}); }});
}
@Override
protected void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
isChangingConfigurations = true;
}
@Override
public void onDismiss(DialogInterface dialog) {
if (isChangingConfigurations) {
isChangingConfigurations = false;
return;
}
finish(); finish();
} }
} }

@ -210,6 +210,7 @@ public class DeadlineControlSet extends TaskEditControlFragment {
startActivityForResult(new Intent(context, DatePickerActivity.class) {{ startActivityForResult(new Intent(context, DatePickerActivity.class) {{
putExtra(DatePickerActivity.EXTRA_TIMESTAMP, date); putExtra(DatePickerActivity.EXTRA_TIMESTAMP, date);
}}, REQUEST_DATE); }}, REQUEST_DATE);
updateDueDateOptions();
break; break;
} }
} }
@ -238,6 +239,7 @@ public class DeadlineControlSet extends TaskEditControlFragment {
startActivityForResult(new Intent(context, TimePickerActivity.class) {{ startActivityForResult(new Intent(context, TimePickerActivity.class) {{
putExtra(TimePickerActivity.EXTRA_TIMESTAMP, getDueDateTime()); putExtra(TimePickerActivity.EXTRA_TIMESTAMP, getDueDateTime());
}}, REQUEST_TIME); }}, REQUEST_TIME);
updateDueTimeOptions();
break; break;
} }
} }

Loading…
Cancel
Save