svn commit: r765487 - /myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/convert/NumberConverter.java

[ma...@apache.org]

Author: matzew
Date: Thu Apr 16 07:23:18 2009
New Revision: 765487

URL: http://svn.apache.org/viewvc?rev=765487&view=rev
Log:
TRINIDAD-1455 - NumberConverter caches only the first encountered converter for a given type. Converters for other locales are recreated every time

Thanks to Max Starets for the patch

Modified:
    myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/convert/NumberConverter.java

Modified: myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/convert/NumberConverter.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/convert/NumberConverter.java?rev=765487&r1=765486&r2=765487&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/convert/NumberConverter.java (original)
+++ myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/convert/NumberConverter.java Thu Apr 16 07:23:18 2009
@@ -981,11 +981,12 @@
         if (nfMap == null)
         {
           nfMap = new HashMap<Locale, NumberFormat>();
-          nfMap.put(locale, (NumberFormat)format.clone());
+          _numberFormatHolder.put(key, nfMap);
+          
         }
         // add this based on the type ('number','currency','percent') or
         // pattern1, pattern2.. patternN to the main holder
-        _numberFormatHolder.put(key, nfMap);
+        nfMap.put(locale, (NumberFormat)format.clone());
       }
     }
   }