Monotone-Parent: 9b5ebc598474f9893ae9cd49b70769d7dbe2996f

Monotone-Revision: 555ffbaeafab3a0a23649acb1714e47a78d35d81

Monotone-Author: flachapelle@inverse.ca
Monotone-Date: 2007-11-21T16:05:07
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -1,3 +1,13 @@
+2007-11-21  Francis Lachapelle  <flachapelle@inverse.ca>
+
+	* UI/Scheduler/UIxComponentEditor.m ([UIxComponentEditor
+	-calendarList]): also returns the calendar of the current
+	event. This is required	so the proper calender is selected when
+	you view an event to which you don't have write access.
+
+	* UI/Common/UIxAclEditor.m ([UIxAclEditor -currentUserIsOwner]): 
+	also returns true for super users.
+
 2007-11-21  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
 	* SoObjects/Mailer/SOGoMailObject.m ([SOGoMailObject

UI/Common/UIxAclEditor.m

@@ -30,6 +30,7 @@
 #import <SoObjects/SOGo/SOGoContentObject.h>
 #import <SoObjects/SOGo/SOGoPermissions.h>
 #import <SoObjects/SOGo/NSArray+Utilities.h>
+#import <SoObjects/SOGo/SOGoUser.h>
 
 #import "UIxAclEditor.h"
 
@@ -149,17 +150,6 @@
   return [self _displayNameForUID: currentUser];
 }
 
-- (NSString *) toolbar
-{
-  NSString *currentLogin, *ownerLogin;
-
-  currentLogin = [[context activeUser] login];
-  ownerLogin = [[self clientObject] ownerInContext: context];
-
-  return (([ownerLogin isEqualToString: currentLogin])
-          ? @"SOGoAclOwner.toolbar" : @"SOGoAclAssistant.toolbar");
-}
-
 - (void) setUserUIDS: (NSString *) retainedUsers
 {
   if ([retainedUsers length] > 0)
@@ -199,13 +189,17 @@
 - (BOOL) currentUserIsOwner
 {
   SOGoObject *clientObject;
+  SOGoUser *currentUser;
   NSString *currentUserLogin, *ownerLogin;
 
   clientObject = [self clientObject];
   ownerLogin = [clientObject ownerInContext: context];
-  currentUserLogin = [[context activeUser] login];
-
-  return [ownerLogin isEqualToString: currentUserLogin];
+  currentUser = [context activeUser];
+  currentUserLogin = [currentUser login];
+  
+  return ([ownerLogin isEqualToString: currentUserLogin]
+	  || ([currentUser respondsToSelector: @selector (isSuperUser)]
+	      && [currentUser isSuperUser]));
 }
 
 // - (id <WOActionResults>) addUserInAcls

UI/Scheduler/UIxComponentEditor.h

@@ -74,6 +74,8 @@
 - (void) setItem: (id) _item;
 - (id) item;
 
+- (SOGoAppointmentFolder *) componentCalendar;
+
 - (NSArray *) categoryList;
 - (void) setCategories: (NSArray *) _categories;
 - (NSArray *) categories;

UI/Scheduler/UIxComponentEditor.m

@@ -453,7 +453,7 @@
 
 - (NSArray *) calendarList
 {
-  SOGoAppointmentFolder *currentCalendar;
+  SOGoAppointmentFolder *calendar, *currentCalendar;
   SOGoAppointmentFolders *calendarParent;
   NSEnumerator *allCalendars;
   SoSecurityManager *sm;
@@ -467,9 +467,11 @@
       calendarParent
 	= [[context activeUser] calendarsFolderInContext: context];
       sm = [SoSecurityManager sharedSecurityManager];
+      calendar = [self componentCalendar];
       allCalendars = [[calendarParent subFolders] objectEnumerator];
       while ((currentCalendar = [allCalendars nextObject]))
-	if (![sm validatePermission: perm
+	if ([calendar isEqual: currentCalendar] ||
+	    ![sm validatePermission: perm
 		 onObject: currentCalendar
 		 inContext: context])
 	  [calendarList addObject: currentCalendar];

UI/Templates/SchedulerUI/UIxComponentEditor.wox

@@ -48,7 +48,7 @@
           <var:popup const:id="calendarList"
 	    list="calendarList" item="item"
             string="calendarDisplayName"
-	    selection="componentCalendar"
+	    var:selection="componentCalendar"
 	    /></span></span>
       <span class="checkBoxList"><var:string label:value="Priority:" />
          <span class="content"><var:popup list="priorities" item="item"

UI/Templates/UIxAclEditor.wox

@@ -8,7 +8,7 @@
   xmlns:rsrc="OGo:url"
   className="UIxPageFrame"
   title="title"
-  var:toolbar="toolbar"
+  const:toolbar="SOGoAclOwner.toolbar"
   const:popup="YES">
 
   <form id="aclForm" const:href="saveAcls">