Monotone-Parent: 5352547fba6d82f422f3bb300b39427b9ebf5653

Monotone-Revision: 07ab3efb7fca7b90bb3250e66fef539e6ccac57c

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2009-01-21T17:01:06
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -1,5 +1,15 @@
 2009-01-21  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
+	* SoObjects/SOGo/SOGoGCSFolder.m ([SOGoGCSFolder
+	+folderWithSubscriptionReference:referenceinContainer:aContainer]):
+	same as below.
+
+	* SoObjects/SOGo/SOGoFolder.m ([SOGoFolder -realNameInContainer]):
+	escape the owner name as a potential CSS identifier.
+
+	* SoObjects/SOGo/NSString+Utilities.m ([NSString
+	-asCSSIdentifier]): escape the "_" character first.
+
 	* SoObjects/SOGo/SOGoGCSFolder.m ([SOGoGCSFolder
 	-subscribe:reallyDoinTheNamesOf:delegatedUsersfromMailInvitation:isMailInvitationinContext:localContext]):
 	explicitly set the content-type of the response to text/plain with

SoObjects/SOGo/NSString+Utilities.m

@@ -310,6 +310,7 @@ static NSMutableCharacterSet *urlStartChars = nil;
   NSMutableString *cssIdentifier;
 
   cssIdentifier = [NSMutableString stringWithString: self];
+  [cssIdentifier replaceString: @"_" withString: @"_U_"];
   [cssIdentifier replaceString: @"." withString: @"_D_"];
   [cssIdentifier replaceString: @"#" withString: @"_H_"];
   [cssIdentifier replaceString: @"@" withString: @"_A_"];

SoObjects/SOGo/SOGoFolder.m

@@ -107,7 +107,7 @@
 
   if (isSubscription)
     {
-      ownerName = [self ownerInContext: context];
+      ownerName = [[self ownerInContext: context] asCSSIdentifier];
       realNameInContainer
 	= [nameInContainer substringFromIndex: [ownerName length] + 1];
     }

SoObjects/SOGo/SOGoGCSFolder.m

@@ -43,10 +43,10 @@
 #import <DOM/DOMProtocols.h>
 #import <EOControl/EOQualifier.h>
 #import <GDLAccess/EOAdaptorChannel.h>
+#import <GDLAccess/EOAdaptorContext.h>
 #import <GDLContentStore/GCSChannelManager.h>
 #import <GDLContentStore/GCSFolderManager.h>
 #import <GDLContentStore/GCSFolder.h>
-#import <GDLContentStore/GCSFolderType.h>
 #import <GDLContentStore/NSURL+GCS.h>
 #import <SaxObjC/XMLNamespaces.h>
 #import <UI/SOGoUI/SOGoFolderAdvisory.h>
@@ -164,7 +164,8 @@ static NSArray *childRecordFields = nil;
 
   path = [NSString stringWithFormat: @"/Users/%@/%@/%@",
 		   login, [pathElements objectAtIndex: 0], ocsName];
-  folderName = [NSString stringWithFormat: @"%@_%@", login, ocsName];
+  folderName = [NSString stringWithFormat: @"%@_%@",
+			 [login asCSSIdentifier], ocsName];
   newFolder = [self objectWithName: folderName inContainer: aContainer];
   [newFolder setOCSPath: path];
   [newFolder setOwner: login];

UI/WebServerResources/JavascriptAPIExtensions.js

@@ -59,7 +59,8 @@ String.prototype.asDate = function () {
 };
 
 String.prototype.asCSSIdentifier = function () {
-	var substitutions = { '.': '_D_',
+	var substitutions = { '_': '_U_',
+												'.': '_D_',
 												'#': '_H_',
 												'@': '_A_',
 												'*': '_S_',

UI/WebServerResources/SchedulerUI.js

@@ -1739,14 +1739,16 @@ function updateCalendarProperties(calendarID, calendarName, calendarColor) {
   var idParts = calendarID.split(":");
   var folderName = idParts[1].split("/")[1];
   var nodeID;
+
   if (idParts[0] != UserLogin)
-    nodeID = "/" + idParts[0] + "_" + folderName;
-  else
+    nodeID = "/" + idParts[0].asCSSIdentifier() + "_" + folderName;
+  else {
     nodeID = "/" + folderName;
 	//   log("nodeID: " + nodeID);
-  var calendarNode = $(nodeID);
-  var childNodes = calendarNode.childNodes;
-  childNodes[childNodes.length-1].nodeValue = calendarName;
+		var calendarNode = $(nodeID);
+		var childNodes = calendarNode.childNodes;
+		childNodes[childNodes.length-1].nodeValue = calendarName;
+	}
 
   appendStyleElement(nodeID, calendarColor);
 }

UI/WebServerResources/generic.js

@@ -1135,7 +1135,7 @@ function accessToSubscribedFolder(serverFolder) {
 	var parts = serverFolder.split(":");
 	if (parts.length > 1) {
 		var paths = parts[1].split("/");
-		folder = "/" + parts[0] + "_" + paths[2];
+		folder = "/" + parts[0].asCSSIdentifier() + "_" + paths[2];
 	}
 	else
 		folder = serverFolder;