diff options
| author | Fabrice Di Meglio <fdimeglio@google.com> | 2010-09-24 09:18:25 -0700 |
|---|---|---|
| committer | Fabrice Di Meglio <fdimeglio@google.com> | 2010-09-24 09:18:25 -0700 |
| commit | 8f57caf8eda5d0e5d3c0892da49aa47c30dc1c9a (patch) | |
| tree | 754d4d1538cccb6b558ae40291792e6fe8348d06 | |
| parent | 0b7d6de1559a4a78af76ab501e0a15afc396c2b9 (diff) | |
| download | frameworks_base-8f57caf8eda5d0e5d3c0892da49aa47c30dc1c9a.zip frameworks_base-8f57caf8eda5d0e5d3c0892da49aa47c30dc1c9a.tar.gz frameworks_base-8f57caf8eda5d0e5d3c0892da49aa47c30dc1c9a.tar.bz2 | |
Fix bug #2585298 (google calendar sync clears eventTimezone field for recurring allDay events)
- force UTC timezone if recurrence is corresponding to an "allday" event
Change-Id: I3603438c0ad8019051ed4a8bd39d4aa7c0401eb1
| -rw-r--r-- | core/java/android/pim/RecurrenceSet.java | 4 | ||||
| -rw-r--r-- | core/tests/coretests/src/android/pim/RecurrenceSetTest.java | 4 |
2 files changed, 5 insertions, 3 deletions
diff --git a/core/java/android/pim/RecurrenceSet.java b/core/java/android/pim/RecurrenceSet.java index 635323e..f5e1bac 100644 --- a/core/java/android/pim/RecurrenceSet.java +++ b/core/java/android/pim/RecurrenceSet.java @@ -181,7 +181,9 @@ public class RecurrenceSet { boolean inUtc = start.parse(dtstart); boolean allDay = start.allDay; - if (inUtc) { + // We force TimeZone to UTC for "all day recurring events" as the server is sending no + // TimeZone in DTSTART for them + if (inUtc || allDay) { tzid = Time.TIMEZONE_UTC; } diff --git a/core/tests/coretests/src/android/pim/RecurrenceSetTest.java b/core/tests/coretests/src/android/pim/RecurrenceSetTest.java index 64cd6c4..5d01ba0 100644 --- a/core/tests/coretests/src/android/pim/RecurrenceSetTest.java +++ b/core/tests/coretests/src/android/pim/RecurrenceSetTest.java @@ -46,7 +46,7 @@ public class RecurrenceSetTest extends TestCase { String recurrence = "DTSTART;VALUE=DATE:20090821\nDTEND;VALUE=DATE:20090822\n" + "RRULE:FREQ=YEARLY;WKST=SU"; verifyPopulateContentValues(recurrence, "FREQ=YEARLY;WKST=SU", null, - null, null, 1250812800000L, null, "P1D", 1); + null, null, 1250812800000L, "UTC", "P1D", 1); } // Test 2 day all-day event @@ -55,7 +55,7 @@ public class RecurrenceSetTest extends TestCase { String recurrence = "DTSTART;VALUE=DATE:20090821\nDTEND;VALUE=DATE:20090823\n" + "RRULE:FREQ=YEARLY;WKST=SU"; verifyPopulateContentValues(recurrence, "FREQ=YEARLY;WKST=SU", null, - null, null, 1250812800000L, null, "P2D", 1); + null, null, 1250812800000L, "UTC", "P2D", 1); } // run populateContentValues and verify the results |