[evolution-data-server/gnome-3-0] Revert "Bug #645783 - Return safe paths from CamelService::get_path()"

[David Woodhouse <dwmw2 src gnome org>]

commit 3e77c319928add37b861548c2f47764b1f658f16
Author: David Woodhouse <David Woodhouse intel com>
Date:   Tue Apr 19 21:45:21 2011 +0100

    Revert "Bug #645783 - Return safe paths from CamelService::get_path()"
    
    This reverts commit 0db09cf5193608cc79f04f38db9b848a0ba03fe0, which was
    unmerged, didn't build, and didn't correctly reference the upstream commit
    that it was based on.
    
    Bad Fridrich; no biscuit

 camel/camel-service.c |   69 ++++++++----------------------------------------
 1 files changed, 12 insertions(+), 57 deletions(-)
---
diff --git a/camel/camel-service.c b/camel/camel-service.c
index b922e47..888ec49 100644
--- a/camel/camel-service.c
+++ b/camel/camel-service.c
@@ -34,8 +34,6 @@
 
 #include <glib/gi18n-lib.h>
 
-#include <libedataserver/e-data-server-util.h>
-
 #include "camel-debug.h"
 #include "camel-operation.h"
 #include "camel-service.h"
@@ -131,8 +129,8 @@ service_get_path (CamelService *service)
 {
 	CamelProvider *prov = service->provider;
 	CamelURL *url = service->url;
-	GString *use_path1 = NULL, *use_path2 = NULL;
-	gchar *ret_path = NULL;
+	GString *gpath;
+	gchar *path;
 
 	/* A sort of ad-hoc default implementation that works for our
 	 * current set of services.
@@ -140,76 +138,33 @@ service_get_path (CamelService *service)
 
 	gpath = g_string_new (service->provider->protocol);
 	if (CAMEL_PROVIDER_ALLOWS (prov, CAMEL_URL_PART_USER)) {
-		use_path1 = g_string_new ("");
-
 		if (CAMEL_PROVIDER_ALLOWS (prov, CAMEL_URL_PART_HOST)) {
-			g_string_append_printf (use_path1, "%s %s",
+			g_string_append_printf (gpath, "/%s %s",
 						url->user ? url->user : "",
 						url->host ? url->host : "");
 
 			if (url->port)
-				g_string_append_printf (use_path1, ":%d", url->port);
+				g_string_append_printf (gpath, ":%d", url->port);
 		} else {
-			g_string_append_printf (use_path1, "%s%s", url->user ? url->user : "",
+			g_string_append_printf (gpath, "/%s%s", url->user ? url->user : "",
 						CAMEL_PROVIDER_NEEDS (prov, CAMEL_URL_PART_USER) ? "" : "@");
 		}
-
-		e_filename_make_safe (use_path1->str);
 	} else if (CAMEL_PROVIDER_ALLOWS (prov, CAMEL_URL_PART_HOST)) {
-		use_path1 = g_string_new ("");
-
-		g_string_append_printf (use_path1, "%s%s",
+		g_string_append_printf (gpath, "/%s%s",
 					CAMEL_PROVIDER_NEEDS (prov, CAMEL_URL_PART_HOST) ? "" : "@",
 					url->host ? url->host : "");
 
 		if (url->port)
-			g_string_append_printf (use_path1, ":%d", url->port);
-
-		e_filename_make_safe (use_path1->str);
-	}
-
-	if (CAMEL_PROVIDER_NEEDS (prov, CAMEL_URL_PART_PATH) && url->path && *url->path) {
-		use_path2 = g_string_new (*url->path == '/' ? url->path + 1 : url->path);
-
-		/* fix directory separators, if needed */
-		if (G_DIR_SEPARATOR != '/') {
-			gchar **elems = g_strsplit (use_path2->str, "/", -1);
-
-			if (elems) {
-				gint ii;
-
-				g_string_truncate (use_path2, 0);
-
-				for (ii = 0; elems[ii]; ii++) {
-					gchar *elem = elems[ii];
-
-					if (*elem) {
-						e_filename_make_safe (elem);
-
-						if (use_path2->len)
-							g_string_append_c (use_path2, G_DIR_SEPARATOR);
-						g_string_append (use_path2, elem);
-					}
-				}
-
-				g_strfreev (elems);
-			}
-		}
-	}
-
-	if (!use_path1 && use_path2) {
-		use_path1 = use_path2;
-		use_path2 = NULL;
+			g_string_append_printf (gpath, ":%d", url->port);
 	}
 
-	ret_path = g_build_filename (service->priv->provider->protocol, use_path1 ? use_path1->str : NULL, use_path2 ? use_path2->str : NULL, NULL);
+	if (CAMEL_PROVIDER_NEEDS (prov, CAMEL_URL_PART_PATH))
+		g_string_append_printf (gpath, "%s%s", *url->path == '/' ? "" : "/", url->path);
 
-	if (use_path1)
-		g_string_free (use_path1, TRUE);
-	if (use_path2)
-		g_string_free (use_path2, TRUE);
+	path = gpath->str;
+	g_string_free (gpath, FALSE);
 
-	return ret_path;
+	return path;
 }
 
 static void