From 938be601e204351307f1ff58b78cb6b59bf431dc Mon Sep 17 00:00:00 2001 From: Ray Greenwell Date: Sun, 24 Feb 2008 04:13:11 +0000 Subject: [PATCH] Screw it, let's just do the right thing. We'll forget later. - Use the chaining method to get a resource from ResourceManager. - Fixed getAll so that it will not overwrite a found property with one in the parent. Made it also understand locales and do the chaining. This change'll cause an extra object hash lookup or two for every translation key, but it's more Right. git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@4947 542714f4-19e9-0310-aa3c-eee0fc999fb1 --- src/as/com/threerings/util/MessageBundle.as | 85 ++++++++++++-------- src/as/com/threerings/util/MessageManager.as | 56 +++---------- 2 files changed, 62 insertions(+), 79 deletions(-) diff --git a/src/as/com/threerings/util/MessageBundle.as b/src/as/com/threerings/util/MessageBundle.as index f496f7b40..a6acfc917 100644 --- a/src/as/com/threerings/util/MessageBundle.as +++ b/src/as/com/threerings/util/MessageBundle.as @@ -22,6 +22,8 @@ package com.threerings.util { import mx.resources.IResourceBundle; +import mx.resources.ResourceManager; +import mx.resources.IResourceManager; /** * A message bundle provides an easy mechanism by which to obtain @@ -37,13 +39,10 @@ public class MessageBundle * from the supplied resource bundle. The path is provided purely for * reporting purposes. */ - public function init ( - msgmgr :MessageManager, path :String, bundle :IResourceBundle, - parent :MessageBundle) :void + public function init (msgmgr :MessageManager, path :String, parent :MessageBundle) :void { _msgmgr = msgmgr; _path = path; - _bundle = bundle; _parent = parent; } @@ -65,24 +64,53 @@ public class MessageBundle } /** - * Adds all messages whose key starts with the specified prefix to the - * supplied array. + * Get all the messages that begin with the specified prefix. * * @param includeParent if true, messages from our parent bundle (and its * parent bundle, all the way up the chain will be included). */ - public function getAll ( - prefix :String, messages :Array, includeParent :Boolean = true) :void + public function getAll (prefix :String, includeParent :Boolean = true) :Array { - for (var key :String in _bundle.content) { - if (StringUtil.startsWith(key, prefix)) { - messages.push(get(key)); + var messages :Array = [] + for each (var value :Object in getAllMapped(prefix, includeParent)) { + messages.push(value); + } + return messages; + } + + /** + * Get all the messages and their keys that begin with the specified prefix. + */ + public function getAllMapped (prefix :String, includeParent :Boolean = true) :Object + { + var mgr :IResourceManager = ResourceManager.getInstance(); + + // search all locales, using the first one first, but including any + // non-overridden keys from backing locales + var messages :Object = {}; + var key :String; + for each (var locale :String in mgr.getLocales()) { + var bundle :IResourceBundle = mgr.getResourceBundle(locale, _path); + if (bundle != null) { + for (key in bundle.content) { + // preserve the message found in an earlier locale + if (StringUtil.startsWith(key, prefix) && !(key in messages)) { + messages[key] = bundle.content[key]; + } + } } } if (includeParent && _parent != null) { - _parent.getAll(prefix, messages, includeParent); + // don't let parent messages overwrite any we've found + var parentMap :Object = _parent.getAllMapped(prefix, includeParent); + for (key in parentMap) { + if (!(key in messages)) { + messages[key] = parentMap[key]; + } + } } + return messages; } /** @@ -93,32 +121,24 @@ public class MessageBundle * @param reportMissing whether or not the method should log an error * if the resource didn't exist. */ - protected function getResourceString ( - key :String, reportMissing :Boolean = true) :String + protected function getResourceString (key :String, reportMissing :Boolean = true) :String { - if (_bundle != null) { - var val :Object = _bundle.content[key]; - if (val != null) { - return String(val); - } - } + var value :String = ResourceManager.getInstance().getString(_path, key); - // if we have a parent, try getting the string from them - if (_parent != null) { - var value :String = _parent.getResourceString(key, false); - if (value != null) { - return value; + if (value == null) { + // if we have a parent, try getting the string from them + if (_parent != null) { + value = _parent.getResourceString(key, false); } // if we didn't find it in our parent, we want to fall // through and report missing appropriately + if (value == null && reportMissing) { + Log.getLog(this).warning("Missing translation message " + + "[bundle=" + _path + ", key=" + key + "]."); + } } - if (reportMissing) { - Log.getLog(this).warning("Missing translation message " + - "[bundle=" + _path + ", key=" + key + "]."); - } - - return null; + return value; } /** @@ -407,9 +427,6 @@ public class MessageBundle * obtain our messages. */ protected var _path :String; - /** The resource bundle from which we obtain our messages. */ - protected var _bundle :IResourceBundle; - /** Our parent bundle if we're not the global bundle. */ protected var _parent :MessageBundle; diff --git a/src/as/com/threerings/util/MessageManager.as b/src/as/com/threerings/util/MessageManager.as index 16ab9446e..2e043aba2 100644 --- a/src/as/com/threerings/util/MessageManager.as +++ b/src/as/com/threerings/util/MessageManager.as @@ -62,35 +62,10 @@ public class MessageManager */ public function MessageManager () { - // use the default locale - _locale = ResourceManager.getInstance().getLocales()[0]; - // load up the global bundle _global = getBundle(GLOBAL_BUNDLE); } -// /** -// * Get the locale that is being used to translate messages. -// * This may be useful if using standard translations, for example -// * new SimpleDateFormat("EEEE", getLocale()) to get the name of a weekday -// * that matches the language being used for all other client translations. -// */ -// public function getLocale () :Locale -// { -// return _locale; -// } -// -// /** -// * Sets the locale to the specified locale. Subsequent message bundles -// * fetched via the message manager will use the new locale. The -// * message bundle cache will also be cleared. -// */ -// public function setLocale (locale :Locale) :void -// { -// _locale = locale; -// _cache.clear(); -// } - /** * Fetches the message bundle for the specified path. If no bundle can * be located with the specified path, a special bundle is returned @@ -108,27 +83,18 @@ public class MessageManager return bundle; } - // if it's not cached, we'll need to resolve it - var rbundle :IResourceBundle = - ResourceManager.getInstance().getResourceBundle(_locale, path); - - // if the resource bundle contains a special resource, we'll - // interpret that as a derivation of MessageBundle to instantiate - // for handling that class - if (rbundle != null) { - var mbclass :Object = rbundle.content[MBUNDLE_CLASS_KEY]; - if (mbclass != null) { - try { - var clazz :Class = ClassUtil.getClassByName(String(mbclass)); - bundle = new clazz(); - } catch (ee :Error) { - Log.getLog(this).warning( - "Failure instantiating custom message bundle " + - "[mbclass=" + mbclass + ", error=" + ee + "]."); - } + // see if we should use a custom class for the bundle + var mbclass :String = ResourceManager.getInstance().getString(path, MBUNDLE_CLASS_KEY); + if (mbclass != null) { + try { + var clazz :Class = ClassUtil.getClassByName(String(mbclass)); + bundle = new clazz(); + } catch (ee :Error) { + Log.getLog(this).warning( + "Failure instantiating custom message bundle " + + "[mbclass=" + mbclass + ", error=" + ee + "]."); } } - // if there was no custom class, or we failed to instantiate the // custom class, use a standard message bundle if (bundle == null) { @@ -138,7 +104,7 @@ public class MessageManager // initialize our message bundle, cache it and return it (if we // couldn't resolve the bundle, the message bundle will cope with // it's null resource bundle) - bundle.init(this, path, rbundle, _global); + bundle.init(this, path, _global); _cache.put(path, bundle); return bundle; }