Bug 55238

Summary:	Timezone for DUE in allday events unhandled in calendar.xml script
Product:	SyncEvolution	Reporter:	peterjan
Component:	SyncEvolution	Assignee:	Patrick Ohly <patrick.ohly>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium	CC:	syncevolution-issues
Version:	1.3
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description peterjan 2012-09-23 12:13:47 UTC

Timezone for DUE in allday events is unhandled in calendar.xml script ( src/syncevo/configs/scripting/11calendar.xml )

> if (ITEMDATATYPE()=="vCalendar10" && ALLDAYCOUNT(DUE,DUE,TRUE,TRUE)>0) {
>     DUE = DATEONLY(DUE);
> }

should probably have:
>     DUE = CONVERTTOUSERZONE(DUE);
before DUE = DATEONLY(DUE);

Current code leads to wrong due date of a todo task (at least for me and GMT+2 on both the phone and the pc):
> [2012-09-23 12:21:22.795] - Field: DUE = 2012-10-02T22:00:00Z (TZ: UTC) (timestamp)
> [2012-09-23 12:21:22.795] - Expression result: 2012-10-02T22:00:00Z (TZ: UTC) (timestamp)
> [2012-09-23 12:21:22.795] - Parameter #1 (by value) = 2012-10-02T22:00:00Z (TZ: UTC) (timestamp)
> [2012-09-23 12:21:22.795] - DATEONLY() function result = 2012-10-02 (floating) (timestamp)
> [2012-09-23 12:21:22.795] - Assigned expression result = 2012-10-02 (floating) (timestamp)

BTW - thank you for your effort on the project.

Comment 1 Patrick Ohly 2012-09-26 13:03:48 UTC

(In reply to comment #0)
> Timezone for DUE in allday events is unhandled in calendar.xml script (
> src/syncevo/configs/scripting/11calendar.xml )
> 
> > if (ITEMDATATYPE()=="vCalendar10" && ALLDAYCOUNT(DUE,DUE,TRUE,TRUE)>0) {
> >     DUE = DATEONLY(DUE);
> > }
> 
> should probably have:
> >     DUE = CONVERTTOUSERZONE(DUE);
> before DUE = DATEONLY(DUE);

Looks reasonable. Does it fix the problem for you?

Comment 2 peterjan 2012-09-26 19:38:24 UTC

Yes it does - my patched version works fine.

Comment 3 Patrick Ohly 2012-10-05 10:29:56 UTC

Fix is included in 1.3.1 (under preparation):

commit e8adfadceeb062ea3d311f1f651332301cd306c8
Author: Patrick Ohly <patrick.ohly@intel.com>
Date:   Fri Oct 5 10:22:23 2012 +0000

    vCalendar 1.0 + tasks: DUE date could be shifted by a day (FDO #55238)
    
    Because of incomplete support for time conversion, the due date
    could get mixed up when phone and PC were set to something other
    than UTC.
    
    Reported and fixed by Peter Jan.

diff --git a/src/syncevo/configs/scripting/11calendar.xml b/src/syncevo/configs/scripting/11calendar.xml
index 613f061..db4720f 100644
--- a/src/syncevo/configs/scripting/11calendar.xml
+++ b/src/syncevo/configs/scripting/11calendar.xml
@@ -117,6 +117,7 @@
         if (SUMMARY==EMPTY) SUMMARY="unnamed"; // set dummy summary if we still don't have one
         // due shaping for non-iCalendar 2.0
         if (ITEMDATATYPE()=="vCalendar10" && ALLDAYCOUNT(DUE,DUE,TRUE,TRUE)>0) {
+          DUE = CONVERTTOUSERZONE(DUE);
           DUE = DATEONLY(DUE);
         }
         if (ITEMDATATYPE()=="vCalendar10") {

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.