You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by tr...@apache.org on 2007/11/26 06:25:41 UTC

svn commit: r598131 - /mina/trunk/integration-ognl/src/main/java/org/apache/mina/integration/ognl/AbstractPropertyAccessor.java

Author: trustin
Date: Sun Nov 25 21:25:31 2007
New Revision: 598131

URL: http://svn.apache.org/viewvc?rev=598131&view=rev
Log:
Fixed a bug where AbstractPropertyAccessor doesn't always attempt to use PropertyEditor

Modified:
    mina/trunk/integration-ognl/src/main/java/org/apache/mina/integration/ognl/AbstractPropertyAccessor.java

Modified: mina/trunk/integration-ognl/src/main/java/org/apache/mina/integration/ognl/AbstractPropertyAccessor.java
URL: http://svn.apache.org/viewvc/mina/trunk/integration-ognl/src/main/java/org/apache/mina/integration/ognl/AbstractPropertyAccessor.java?rev=598131&r1=598130&r2=598131&view=diff
==============================================================================
--- mina/trunk/integration-ognl/src/main/java/org/apache/mina/integration/ognl/AbstractPropertyAccessor.java (original)
+++ mina/trunk/integration-ognl/src/main/java/org/apache/mina/integration/ognl/AbstractPropertyAccessor.java Sun Nov 25 21:25:31 2007
@@ -70,12 +70,12 @@
         Object answer = getProperty0((OgnlContext) context, target, name);
         if (answer == OgnlRuntime.NotFound) {
             answer = super.getPossibleProperty(context, target, name);
-            
-            PropertyEditor pe = PropertyEditorFactory.getInstance(answer);
-            if (pe != null) {
-                pe.setValue(answer);
-                answer = pe.getAsText();
-            }
+        }
+
+        PropertyEditor pe = PropertyEditorFactory.getInstance(answer);
+        if (pe != null) {
+            pe.setValue(answer);
+            answer = pe.getAsText();
         }
         return answer;
     }