mobile/davx5-ose
mobile/davx5-ose
1
0
Fork 0
mirror of https://github.com/bitfireAT/davx5-ose synced 2024-10-15 15:59:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Better reminder (VALARM) handling

Browse source 
* handle WEEKS in duration correctly (fixes #398)
* handle positive and negative TRIGGER duration values correctly
This commit is contained in:
Ricki Hirner 2015-05-03 22:36:16 +02:00
parent 02cf7d8603
commit cea35a9162
1 changed files with 13 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
app/src/main/java/at/bitfire/davdroid/resource/LocalCalendar.java
View file

@ -511,7 +511,7 @@ public class LocalCalendar extends LocalCollection<Event> {
}
void populateReminder(Event event, ContentValues row) throws RemoteException {
VAlarm alarm = new VAlarm(new Dur(0, 0, row.getAsInteger(Reminders.MINUTES), 0));
VAlarm alarm = new VAlarm(new Dur(0, 0, -row.getAsInteger(Reminders.MINUTES), 0));
PropertyList props = alarm.getProperties();
props.add(Action.DISPLAY);
@ -722,12 +722,18 @@ public class LocalCalendar extends LocalCollection<Event> {
protected Builder buildReminder(Builder builder, VAlarm alarm) {
int minutes = 0;
Dur duration;
if (alarm.getTrigger() != null && (duration = alarm.getTrigger().getDuration()) != null)
minutes = duration.getDays() * 24*60 + duration.getHours()*60 + duration.getMinutes();
Log.d(TAG, "Adding alarm " + minutes + " min before");
if (alarm.getTrigger() != null) {
Dur duration = alarm.getTrigger().getDuration();
if (duration != null) {
// negative value in TRIGGER means positive value in Reminders.MINUTES and vice versa
minutes = -(((duration.getWeeks() * 7 + duration.getDays()) * 24 + duration.getHours()) * 60 + duration.getMinutes());
if (duration.isNegative())
minutes *= -1;
}
}
Log.d(TAG, "Adding alarm " + minutes + " minutes before");
return builder
.withValue(Reminders.METHOD, Reminders.METHOD_ALERT)
.withValue(Reminders.MINUTES, minutes);