You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by gp...@apache.org on 2010/06/28 22:24:38 UTC

svn commit: r958706 - /myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java

Author: gpetracek
Date: Mon Jun 28 20:24:38 2010
New Revision: 958706

URL: http://svn.apache.org/viewvc?rev=958706&view=rev
Log:
EXTCDI-21 cleanup

Modified:
    myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java

Modified: myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java?rev=958706&r1=958705&r2=958706&view=diff
==============================================================================
--- myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java (original)
+++ myfaces/extensions/cdi/trunk/jse-modules/message-module/impl/src/main/java/org/apache/myfaces/extensions/cdi/message/impl/DefaultFormatterFactory.java Mon Jun 28 20:24:38 2010
@@ -51,8 +51,11 @@ public class DefaultFormatterFactory imp
 
     public synchronized FormatterFactory add(Formatter formatter)
     {
-        this.formatters.add(formatter);
-        formatterCache = null;
+        if(!this.formatters.contains(formatter))
+        {
+            this.formatters.add(formatter);
+            formatterCache = null;
+        }
         return this;
     }