Monotone-Parent: 935cf293ec636367ef64469ec9130956087caa1e

Monotone-Revision: 76a822505adea00b3bf98056aa500f08ba7e35b7

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2008-06-02T20:05:38
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -1,3 +1,16 @@
+2008-06-02  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
+
+	* UI/MailPartViewers/UIxMailPartICalViewer.m
+	([UIxMailPartICalViewer -canOriginalEventBeUpdated]): new template
+	accessor that return whether the use status has changed in the
+	original reply and if its sequence number concurs.
+
+	* UI/MailPartViewers/UIxMailPartICalActions.m
+	([UIxMailPartICalActions -updateUserStatusAction]): we accept the
+	modification if the sequence number from the reply is equal or
+	SUPERIOR (for Lightning bugs), rather than inferior, where the
+	reply is obsolete anyway.
+
 2008-05-31  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
 	* SoObjects/Mailer/SOGoMailFolder.m ([SOGoMailFolder

UI/MailPartViewers/UIxMailPartICalActions.m

@@ -319,7 +319,7 @@
       eventObject = [self _eventObjectWithUID: [emailEvent uid]];
       calendarEvent = [eventObject component: NO secure: NO];
       if (([[emailEvent sequence] compare: [calendarEvent sequence]]
-	  != NSOrderedDescending)
+	  != NSOrderedAscending)
 	  && ([self _updateParticipantStatusInEvent: calendarEvent
 		    fromEvent: emailEvent
 		    inObject: eventObject]))

UI/MailPartViewers/UIxMailPartICalViewer.m

@@ -416,4 +416,11 @@
 	  != NSOrderedSame);
 }
 
+- (BOOL) canOriginalEventBeUpdated
+{
+  return ([self hasSenderStatusChanged]
+	  && ([[inEvent sequence] compare: [storedEvent sequence]]
+	      != NSOrderedAscending));
+}
+
 @end /* UIxMailPartICalViewer */

UI/Templates/MailPartViewers/UIxMailPartICalViewer.wox

@@ -84,7 +84,7 @@
 	</var:if>
 
 	<var:if condition="isReplySenderAnAttendee">
-	  <var:if condition="hasSenderStatusChanged"
+	  <var:if condition="canOriginalEventBeUpdated"
 	    ><p class="uix_ical_toolbar">
 	      <input id="iCalendarUpdateUserStatus" class="button"
 		type="button" label:value="Update status"/>