svn merge -r 61099:61168 svn+ssh://svn/svn/linden/branches/release-candidate into release

2 files changed, 25 insertions, 18 deletions

diff --git a/indra/llcommon/lluri.cpp b/indra/llcommon/lluri.cpp
index 52270376f8..ee14028ec9 100644
--- a/indra/llcommon/lluri.cpp
+++ b/indra/llcommon/lluri.cpp
@@ -218,24 +218,11 @@ LLURI LLURI::buildHTTP(const std::string& prefix,
 					   const LLSD& path,
 					   const LLSD& query)
 {
-	LLURI result = buildHTTP(prefix, path);
+	LLURI uri = buildHTTP(prefix, path);
+	uri.mEscapedQuery = mapToQueryString(query);
 	// break out and escape each query component
-	if (query.isMap())
-	{
-		for (LLSD::map_const_iterator it = query.beginMap();
-			 it != query.endMap();
-			 it++)
-		{
-			result.mEscapedQuery += escapeQueryVariable(it->first) +
-				(it->second.isUndefined() ? "" : "=" + escapeQueryValue(it->second.asString())) +
-				"&";
-		}
-		if (query.size() > 0)
-		{
-			result.mEscapedOpaque += "?" + result.mEscapedQuery;
-		}
-	}
-	return result;
+	uri.mEscapedOpaque += "?" + uri.mEscapedQuery ;
+	return uri;
 }
 
 // static
@@ -255,7 +242,6 @@ LLURI LLURI::buildHTTP(const std::string& host,
 	return LLURI::buildHTTP(llformat("%s:%u", host.c_str(), port), path, query);
 }
 
-
 namespace {
 	LLURI buildBackboneURL(LLApp* app,
 				const std::string& p1 = "",
@@ -507,3 +493,23 @@ LLSD LLURI::queryMap(std::string escaped_query_string)
 	return result;
 }
 
+std::string LLURI::mapToQueryString(const LLSD& queryMap)
+{
+	std::string query_string;
+
+	if (queryMap.isMap())
+	{
+		for (LLSD::map_const_iterator iter = queryMap.beginMap();
+			 iter != queryMap.endMap();
+			 iter++)
+		{
+			query_string += escapeQueryVariable(iter->first) +
+				(iter->second.isUndefined() ? "" : "=" + escapeQueryValue(iter->second.asString())) + "&" ;
+		}
+		//if (queryMap.size() > 0)
+		//{
+		//	query_string += "?" + query_string ;
+		//}
+	}
+	return query_string;
+}
diff --git a/indra/llcommon/lluri.h b/indra/llcommon/lluri.h
index f42cc102e6..78ffcf0a87 100644
--- a/indra/llcommon/lluri.h
+++ b/indra/llcommon/lluri.h
@@ -70,6 +70,7 @@ public:
   std::string query() const;		// ex.: "x=34", section after "?"
   LLSD queryMap() const;			// above decoded into a map
   static LLSD queryMap(std::string escaped_query_string);
+  static std::string mapToQueryString(const LLSD& queryMap);
 
   // Escaping Utilities
   // Escape a string by urlencoding all the characters that aren't in the allowed string.