applied comments

SoObjects/Appointments/SOGoAppointmentObject.m

@@ -480,10 +480,8 @@
     [unavailableAttendees release];
     return [NSException exceptionWithHTTPStatus:403 reason: reason];
   }
-  else {
-    [unavailableAttendees release];
-    return nil;
-  }
+  [unavailableAttendees release];
+  return nil;
 }
 
 //

UI/PreferencesUI/UIxPreferences.m

@@ -640,7 +640,7 @@ static NSArray *reminderValues = nil;
   return [userDefaults busyOffHours];
 }
 
-- (NSArray *) whiteListValue
+- (NSArray *) whiteList
 {
   SOGoUserSettings *us;
   NSMutableDictionary *moduleSettings;
@@ -652,7 +652,7 @@ static NSArray *reminderValues = nil;
   return whiteList;
 }
 
-- (void) setWhiteListValue: (NSArray *) whiteList
+- (void) setWhiteList: (NSArray *) whiteList
 {
   SOGoUserSettings *us;
   NSMutableDictionary *moduleSettings;

UI/Templates/PreferencesUI/UIxPreferences.wox

@@ -248,8 +248,8 @@
               <a const:id="appointmentsWhiteListDelete" href="#">
                 <img rsrc:src="remove-icon.png" label:title="Delete" id="WhiteListDelete" /></a>
             </div>
-            <input type="hidden" const:id="whiteListValue"
-            const:name="whiteListValue" var:value="whiteListValue"/>
+            <input type="hidden" const:id="whiteList"
+            const:name="whiteList" var:value="whiteList"/>
             
             <label id="calendarCategoriesLabel"><var:string label:value="Categories"/></label>
             <div id="calendarCategoriesListWrapper" class="listWrapper">

UI/WebServerResources/UIxPreferences.js

@@ -229,7 +229,7 @@ function initPreferences() {
   // Calendar whiteList
   var whiteList = $("appointmentsWhiteListWrapper");
   if(whiteList) {
-    var whiteListValue = $("whiteListValue").getValue();
+    var whiteListValue = $("whiteList").getValue();
     if (whiteListValue != "") {
       whiteListValue = whiteListValue.split(",");
       var tablebody = $("appointmentsWhiteListWrapper").childNodesWithTag("table")[0].tBodies[0];
@@ -1133,8 +1133,8 @@ function serializeAppointmentsWhiteList() {
     var user = uid + "=" + value;
     if (uid != null)
       values.push(user);
-  
-  $("whiteListValue").value = values;
+  }
+  $("whiteList").value = values;
 }
 
 function onCalendarCategoryAdd(e) {