[CalendarServer-changes] [3641] CalendarServer/trunk

[source_changes at macosforge.org]

Revision: 3641
          http://trac.macosforge.org/projects/calendarserver/changeset/3641
Author:   cdaboo at apple.com
Date:     2009-02-05 07:18:19 -0800 (Thu, 05 Feb 2009)
Log Message:
-----------
Handle property comparisons properly since some are multi-valued.

Modified Paths:
--------------
    CalendarServer/trunk/run
    CalendarServer/trunk/twistedcaldav/scheduling/icaldiff.py

Modified: CalendarServer/trunk/run
===================================================================
--- CalendarServer/trunk/run	2009-02-05 15:16:34 UTC (rev 3640)
+++ CalendarServer/trunk/run	2009-02-05 15:18:19 UTC (rev 3641)
@@ -1,7 +1,7 @@
 #!/bin/bash
 
 ##
-# Copyright (c) 2005-2007 Apple Inc. All rights reserved.
+# Copyright (c) 2005-2009 Apple Inc. All rights reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -692,7 +692,7 @@
 
 caldavtester="${top}/CalDAVTester";
 
-svn_get "CalDAVTester" "${caldavtester}" "${svn_uri_base}/CalDAVTester/trunk" 3626;
+svn_get "CalDAVTester" "${caldavtester}" "${svn_uri_base}/CalDAVTester/trunk" 3640;
 
 #
 # Calendar Server

Modified: CalendarServer/trunk/twistedcaldav/scheduling/icaldiff.py
===================================================================
--- CalendarServer/trunk/twistedcaldav/scheduling/icaldiff.py	2009-02-05 15:16:34 UTC (rev 3640)
+++ CalendarServer/trunk/twistedcaldav/scheduling/icaldiff.py	2009-02-05 15:18:19 UTC (rev 3641)
@@ -1,5 +1,5 @@
 ##
-# Copyright (c) 2005-2007 Apple Inc. All rights reserved.
+# Copyright (c) 2005-2009 Apple Inc. All rights reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -167,8 +167,9 @@
         
         for prop in props_to_test:
             # Change => no merge
-            if old_comp.getProperty(prop) != new_comp.getProperty(prop):
-                # Always overwrite as we have a big change going on
+            old_props = set(old_comp.properties(prop))
+            new_props = set(new_comp.properties(prop))
+            if old_props.difference(new_props):
                 return True
 
         return False
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20090205/df4b5cde/attachment.html>