Monotone-Parent: 9571c9ae651f3733c9c79a5898429b428d520a75

Monotone-Revision: eef38dc2fbafb16d2c16acc4dc79d0412e927b3b

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2011-11-30T14:47:11

ChangeLog

@@ -1,5 +1,10 @@
 2011-11-30  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
+	* SoObjects/Appointments/SOGoUserFolder+Appointments.m
+	(-davCalendarUserAddressSet): make sure each email address is
+	reported only once, we also return the user principal in the list
+	(for iCal).
+
 	* SoObjects/SOGo/SOGoObject.m (-davComplianceClassesInContext:):
 	declare "calendar-query-extended", "extended-mkcol",
 	"calendarserver-principal-property-search" as additional calendar

SoObjects/Appointments/SOGoUserFolder+Appointments.m

@@ -59,21 +59,32 @@
   NSArray *tag;
   NSMutableArray *addresses;
   NSEnumerator *emails;
+  NSMutableDictionary *doneEmails;
   NSString *currentEmail;
   SOGoUser *ownerUser;
 
   addresses = [NSMutableArray array];
 
+  doneEmails = [NSMutableDictionary dictionary];
   ownerUser = [SOGoUser userWithLogin: owner];
   emails = [[ownerUser allEmails] objectEnumerator];
   while ((currentEmail = [emails nextObject]))
     {
-      tag = [NSArray arrayWithObjects: @"href", XMLNS_WEBDAV, @"D",
-		     [NSString stringWithFormat: @"mailto:%@", currentEmail],
-		     nil];
-      [addresses addObject: tag];
+      if (![doneEmails objectForKey: currentEmail])
+        {
+          tag = [NSArray arrayWithObjects: @"href", XMLNS_WEBDAV, @"D",
+                         [NSString stringWithFormat: @"mailto:%@", currentEmail],
+                         nil];
+          [addresses addObject: tag];
+          [doneEmails setObject: [NSNull null] forKey: currentEmail];
+        }
     }
 
+  tag = [NSArray arrayWithObjects: @"href", XMLNS_WEBDAV, @"D",
+                 [NSString stringWithFormat: @"/SOGo/dav/%@/", nameInContainer],
+                 nil];
+  [addresses addObjectUniquely: tag];
+
   return addresses;
 }