Monotone-Parent: 56159df9717624adf99da2f21377b40d435b1db7

Monotone-Revision: 2ae53e4d24cdb745c3a8c8e246d99d096538c2e3

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2008-12-17T14:32:13
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -1,3 +1,14 @@
+2008-12-17  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
+
+	* UI/Scheduler/UIxAppointmentEditor.m ([UIxAppointmentEditor
+	-dealloc]): we release componentCalendar here to avoid a leak
+	(since if set, it was through a call to ASSIGN()).
+
+	* UI/Scheduler/UIxComponentEditor.m ([UIxComponentEditor
+	-setComponent:newComponent]): since componentCalendar is released
+	at deallocation, we need to retain it, and to release the previous
+	value to avoid a leak...
+
 2008-12-15  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
 	* SoObjects/SOGo/SOGoObject.m ([SOGoObject +webdavAclManager]):

UI/Scheduler/UIxAppointmentEditor.m

@@ -62,6 +62,7 @@
       event = nil;
       isAllDay = NO;
       isTransparent = NO;
+      componentCalendar = nil;
     }
 
   return self;
@@ -73,6 +74,7 @@
   [event release];
   [aptStartDate release];
   [aptEndDate release];
+  [componentCalendar release];
   [super dealloc];
 }
 
@@ -275,7 +277,6 @@
   SOGoAppointmentObject *co;
   SoSecurityManager *sm;
   NSException *ex;
-  NSString *aOwner;
 
   co = [self clientObject];
   if ([co isKindOfClass: [SOGoAppointmentOccurence class]])

UI/Scheduler/UIxComponentEditor.m

@@ -380,9 +380,11 @@ iRANGE(2);
 	  [self _loadAttendees];
 	  [self _loadRRules];
 
+	  [componentCalendar release];
 	  componentCalendar = [co container];
 	  if ([componentCalendar isKindOfClass: [SOGoCalendarComponent class]])
 	    componentCalendar = [componentCalendar container];
+	  [componentCalendar retain];
 	}
     }
 //   /* cycles */