diff --git a/SoObjects/SOGo/SOGoGCSFolder.m b/SoObjects/SOGo/SOGoGCSFolder.m index bd60c49b2..4770c864f 100644 --- a/SoObjects/SOGo/SOGoGCSFolder.m +++ b/SoObjects/SOGo/SOGoGCSFolder.m @@ -282,6 +282,11 @@ static NSArray *childRecordFields = nil; else name = primaryDN; } + else + { + [self errorWithFormat: @"(%s): No c_foldername defined (%@)", + __PRETTY_FUNCTION__, [container defaultFolderName]]; + } return name; } @@ -342,7 +347,12 @@ static NSArray *childRecordFields = nil; [fc cancelFetch]; [cm releaseChannel: fc]; } - NS_HANDLER; + NS_HANDLER + { + [self errorWithFormat: @"(%s): Could not fetch c_foldername from %@ for c_path %@", + __PRETTY_FUNCTION__, [folderLocation gcsTableName], ocsPath]; + [self errorWithFormat: @"%@", localException]; + } NS_ENDHANDLER; } @@ -439,7 +449,11 @@ static NSArray *childRecordFields = nil; record = [[self folderManager] recordAtPath: _path]; if (!record) - return nil; + { + [self errorWithFormat: @"(%s): Can't find record at path %@", + __PRETTY_FUNCTION__, _path]; + return nil; + } [cache setValue: [record jsonRepresentation] forKey: _path]; } diff --git a/SoObjects/SOGo/SOGoParentFolder.m b/SoObjects/SOGo/SOGoParentFolder.m index fe6ef6077..ae523f935 100644 --- a/SoObjects/SOGo/SOGoParentFolder.m +++ b/SoObjects/SOGo/SOGoParentFolder.m @@ -289,7 +289,7 @@ static SoSecurityManager *sm = nil; // We check with -ocsFolderForPath if the folder also exists in the database. // This is important because user A could delete folder X, and user B has subscribed to it. - // If the "default roles" are enabled for calendars/address books, -validatePersmission:.. will + // If the "default roles" are enabled for calendars/address books, -validatePermission:.. will // work (grabbing the default role) and the deleted resource will be incorrectly returned. if (subscribedFolder && [subscribedFolder ocsFolderForPath: [subscribedFolder ocsPath]]