merge of '819ae78a7b35f2cf4b307d16a3b0da9a2d5b767c'

and 'a79ba19dc901642223410e1cd56c639ec05a6098'

Monotone-Parent: 819ae78a7b35f2cf4b307d16a3b0da9a2d5b767c
Monotone-Parent: a79ba19dc901642223410e1cd56c639ec05a6098
Monotone-Revision: a8d7706d1697aa0016456102e8d0198a8f7ef74d

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2010-11-15T19:51:13
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -1,3 +1,11 @@
+2010-11-15  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
+
+	* UI/Scheduler/UIxAppointmentEditor.m (-saveAction): check the
+	equality of componentCalendar and previousCalendar by comparing
+	their ocsPath, since the objects themselves might be different
+	instances when the current user has logged in with a different
+	identifier than his/her uid.
+
 2010-11-12  Francis Lachapelle  <flachapelle@inverse.ca>
 
 	* UI/WebServerResources/MailerUI.js (messageFlagCallback): now

SoObjects/SOGo/SOGoUserManager.m

@@ -443,8 +443,8 @@
         setUserAttributes: [currentUser jsonRepresentation]
                  forLogin: _login];
     }
-    else
-      checkOK = NO;
+  else
+    checkOK = NO;
 
   return checkOK;
 }

UI/Scheduler/UIxAppointmentEditor.m

@@ -375,7 +375,9 @@
 
   if ([co isNew])
     {
-      if (componentCalendar && componentCalendar != previousCalendar)
+      if (componentCalendar
+          && ![[componentCalendar ocsPath]
+                isEqualToString: [previousCalendar ocsPath]])
 	{
 	  // New event in a different calendar -- make sure the user can
 	  // write to the selected calendar since the rights were verified
@@ -397,7 +399,9 @@
       // The event was modified -- save it.
       [co saveComponent: event];
 
-      if (componentCalendar && componentCalendar != previousCalendar)
+      if (componentCalendar
+          && ![[componentCalendar ocsPath]
+                isEqualToString: [previousCalendar ocsPath]])
 	{
 	  // The event was moved to a different calendar.
 	  if (![sm validatePermission: SoPerm_DeleteObjects