You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tuscany.apache.org by fw...@apache.org on 2008/06/03 04:57:37 UTC

svn commit: r662626 - /incubator/tuscany/java/sca/modules/databinding/src/main/java/org/apache/tuscany/sca/databinding/DefaultDataBindingExtensionPoint.java

Author: fwang
Date: Mon Jun  2 19:57:37 2008
New Revision: 662626

URL: http://svn.apache.org/viewvc?rev=662626&view=rev
Log:
Traverse the databinding collection directly,the map of bindings may has duplicate binding value.

Modified:
    incubator/tuscany/java/sca/modules/databinding/src/main/java/org/apache/tuscany/sca/databinding/DefaultDataBindingExtensionPoint.java

Modified: incubator/tuscany/java/sca/modules/databinding/src/main/java/org/apache/tuscany/sca/databinding/DefaultDataBindingExtensionPoint.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/databinding/src/main/java/org/apache/tuscany/sca/databinding/DefaultDataBindingExtensionPoint.java?rev=662626&r1=662625&r2=662626&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/databinding/src/main/java/org/apache/tuscany/sca/databinding/DefaultDataBindingExtensionPoint.java (original)
+++ incubator/tuscany/java/sca/modules/databinding/src/main/java/org/apache/tuscany/sca/databinding/DefaultDataBindingExtensionPoint.java Mon Jun  2 19:57:37 2008
@@ -274,7 +274,7 @@
     public DataType introspectType(Object value) {
         loadDataBindings();
         DataType dataType = null;
-        for (DataBinding binding : bindings.values()) {
+        for (DataBinding binding : databindings) {
             // don't introspect for JavaBeansDatabinding as all javatypes will
             // anyways match to its basetype
             // which is java.lang.Object. Default to this only if no databinding