diff options
| -rw-r--r-- | indra/llxuixml/lltrans.cpp | 50 | ||||
| -rw-r--r-- | indra/llxuixml/lltrans.h | 4 | ||||
| -rw-r--r-- | indra/newview/lllogininstance.cpp | 1 | ||||
| -rw-r--r-- | indra/newview/llstartup.cpp | 37 | ||||
| -rw-r--r-- | indra/newview/llxmlrpclistener.cpp | 7 | 
5 files changed, 76 insertions, 23 deletions
| diff --git a/indra/llxuixml/lltrans.cpp b/indra/llxuixml/lltrans.cpp index b403b86048..5388069c24 100644 --- a/indra/llxuixml/lltrans.cpp +++ b/indra/llxuixml/lltrans.cpp @@ -30,6 +30,7 @@  #include "llfasttimer.h"	// for call count statistics  #include "llxuiparser.h" +#include "llsd.h"  #include "llxmlnode.h"  #include <map> @@ -155,13 +156,28 @@ std::string LLTrans::getString(const std::string &xml_desc, const LLStringUtil::  	}  	else  	{ -		LLSD args; -		args["STRING_NAME"] = xml_desc;  		LL_WARNS_ONCE("configuration") << "Missing String in strings.xml: [" << xml_desc << "]" << LL_ENDL; +		return "MissingString("+xml_desc+")"; +	} +} -		//LLNotificationsUtil::add("MissingString", args); // *TODO: resurrect -		//return xml_desc; +//static +std::string LLTrans::getString(const std::string &xml_desc, const LLSD& msg_args) +{ +	// Don't care about time as much as call count.  Make sure we're not +	// calling LLTrans::getString() in an inner loop. JC +	LLFastTimer timer(FTM_GET_TRANS); +	template_map_t::iterator iter = sStringTemplates.find(xml_desc); +	if (iter != sStringTemplates.end()) +	{ +		std::string text = iter->second.mText; +		LLStringUtil::format(text, msg_args); +		return text; +	} +	else +	{ +		LL_WARNS_ONCE("configuration") << "Missing String in strings.xml: [" << xml_desc << "]" << LL_ENDL;  		return "MissingString("+xml_desc+")";  	}  } @@ -183,11 +199,27 @@ bool LLTrans::findString(std::string &result, const std::string &xml_desc, const  	}  	else  	{ -		LLSD args; -		args["STRING_NAME"] = xml_desc; -		LL_WARNS_ONCE("configuration") << "Missing String in strings.xml: [" << xml_desc << "]" << LL_ENDL; -		//LLNotificationsUtil::add("MissingString", args); -		 +		LL_WARNS_ONCE("configuration") << "Missing String in strings.xml: [" << xml_desc << "]" << LL_ENDL;	 +		return false; +	} +} + +//static +bool LLTrans::findString(std::string &result, const std::string &xml_desc, const LLSD& msg_args) +{ +	LLFastTimer timer(FTM_GET_TRANS); + +	template_map_t::iterator iter = sStringTemplates.find(xml_desc); +	if (iter != sStringTemplates.end()) +	{ +		std::string text = iter->second.mText; +		LLStringUtil::format(text, msg_args); +		result = text; +		return true; +	} +	else +	{ +		LL_WARNS_ONCE("configuration") << "Missing String in strings.xml: [" << xml_desc << "]" << LL_ENDL;	  		return false;  	}  } diff --git a/indra/llxuixml/lltrans.h b/indra/llxuixml/lltrans.h index b7091f77e8..128b51d383 100644 --- a/indra/llxuixml/lltrans.h +++ b/indra/llxuixml/lltrans.h @@ -34,6 +34,8 @@  class LLXMLNode; +class LLSD; +  /**   * @brief String template loaded from strings.xml   */ @@ -74,7 +76,9 @@ public:  	 * @returns Translated string  	 */  	static std::string getString(const std::string &xml_desc, const LLStringUtil::format_map_t& args); +	static std::string getString(const std::string &xml_desc, const LLSD& args);  	static bool findString(std::string &result, const std::string &xml_desc, const LLStringUtil::format_map_t& args); +	static bool findString(std::string &result, const std::string &xml_desc, const LLSD& args);  	// Returns translated string with [COUNT] replaced with a number, following  	// special per-language logic for plural nouns.  For example, some languages diff --git a/indra/newview/lllogininstance.cpp b/indra/newview/lllogininstance.cpp index 36c5d12897..00de6a86e1 100644 --- a/indra/newview/lllogininstance.cpp +++ b/indra/newview/lllogininstance.cpp @@ -608,6 +608,7 @@ void LLLoginInstance::constructAuthParams(LLPointer<LLCredential> user_credentia  	request_params["channel"] = LLVersionInfo::getChannel();  	request_params["id0"] = mSerialNumber;  	request_params["host_id"] = gSavedSettings.getString("HostID"); +	request_params["extended_errors"] = true; // request message_id and message_args  	mRequestData.clear();  	mRequestData["method"] = "login_to_simulator"; diff --git a/indra/newview/llstartup.cpp b/indra/newview/llstartup.cpp index ca908ef822..2560320257 100644 --- a/indra/newview/llstartup.cpp +++ b/indra/newview/llstartup.cpp @@ -995,6 +995,7 @@ bool idle_startup()  	if(STATE_LOGIN_PROCESS_RESPONSE == LLStartUp::getStartupState())   	{ +		// Generic failure message  		std::ostringstream emsg;  		emsg << LLTrans::getString("LoginFailed") << "\n";  		if(LLLoginInstance::getInstance()->authFailure()) @@ -1003,24 +1004,32 @@ bool idle_startup()  			                      << LLLoginInstance::getInstance()->getResponse() << LL_ENDL;  			LLSD response = LLLoginInstance::getInstance()->getResponse();  			// Still have error conditions that may need some  -			// sort of handling. +			// sort of handling - dig up specific message  			std::string reason_response = response["reason"];  			std::string message_response = response["message"]; -	 -			if(!message_response.empty()) +			std::string message_id = response["message_id"]; +			std::string message; // actual string to show the user + +			if(!message_id.empty() && LLTrans::findString(message, message_id, response["message_args"]))  			{ -				// XUI: fix translation for strings returned during login -				// We need a generic table for translations -				std::string big_reason = LLAgent::sTeleportErrorMessages[ message_response ]; -				if ( big_reason.size() == 0 ) -				{ -					emsg << message_response; -				} -				else -				{ -					emsg << big_reason; -				} +				// message will be filled in with the template and arguments  			} +			else if(!message_response.empty()) +			{ +				// *HACK: "no_inventory_host" sent as the message itself. +				// Remove this clause when server is sending message_id as well. +				message = LLAgent::sTeleportErrorMessages[ message_response ]; +			} + +			if (message.empty()) +			{ +				// Fallback to server-supplied string; necessary since server +				// may add strings that this viewer is not yet aware of +				message = message_response; +			} + +			emsg << message; +  			if(reason_response == "key")  			{ diff --git a/indra/newview/llxmlrpclistener.cpp b/indra/newview/llxmlrpclistener.cpp index 2596f239ca..97a9eb7f5f 100644 --- a/indra/newview/llxmlrpclistener.cpp +++ b/indra/newview/llxmlrpclistener.cpp @@ -499,6 +499,13 @@ private:                  // 'array' as the value of this 'key'.                  responses.insert(key, array);              } +            else if (xmlrpc_type_struct == type) +            { +                LLSD submap = parseValues(status_string, +                                          STRINGIZE(key_pfx << key << ':'), +                                          current); +                responses.insert(key, submap); +            }              else              {                  // whoops - unrecognized type | 
