[PATCH] three tiny patches for moto-sync

[floe]

Hi, I'm using 0.36 with moto-sync from SVN and had two issues. First, there's still a reference to hashtable.reset_reports(), which doesn't exist. Second, there's some confusion WRT to timezone-aware and timezone-agnostic date values, leading to a "TypeError?: can't compare offset-naive and offset-aware datetimes".

Yours, Florian

[floe]

Hm, and here's another one:

@@ -611,7 +611,8 @@

elif repeat_type == MOTO_REPEAT_WEEKLY:

appendXMLTag(doc, e, 'Frequency', 'WEEKLY') if repeat_day:

• repeat_days = moto_repeat_day_to_weekdays(repeat_day, eventdt.weekday()).sort()

+ repeat_days = moto_repeat_day_to_weekdays(repeat_day, eventdt.weekday()) + repeat_days.sort()

appendXMLTag(doc, e, 'ByDay?', ','.join([VCAL_DAYS[n] for n in repeat_days]))

else:

repeat_days = []

Calling .sort() on a list does an in-place sort and returns None, so this is the correct way.

[dgollub]

The hashtable patch-hunk is valid i think. Not quite sure about the timestamp hunk.

floe, could you give some information about your device - so we can try to reproduce the issue? andrewb, could you review the RRULE and timestamp related changes? are those good to go?

[floe]

I have a Razr V3xx. But I believe all three bugs are independent of the device..

[felixmoeller]

this is still valid for current SVN. It is about source:/plugins/moto-sync/motosync.py

The third patch is still valid too:

Index: motosync.py
===================================================================
--- motosync.py (revision 3767)
+++ motosync.py (working copy)
@@ -611,7 +611,8 @@
     elif repeat_type == MOTO_REPEAT_WEEKLY:
         appendXMLTag(doc, e, 'Frequency', 'WEEKLY')
         if repeat_day:
-            repeat_days = moto_repeat_day_to_weekdays(repeat_day, eventdt.weekday()).sort()
+            repeat_days = moto_repeat_day_to_weekdays(repeat_day, eventdt.weekday())
+            repeat_days.sort()
             appendXMLTag(doc, e, 'ByDay', ','.join([VCAL_DAYS[n] for n in repeat_days]))
         else:
             repeat_days = []