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

Posted to commits@tuscany.apache.org by mc...@apache.org on 2008/05/20 15:02:44 UTC

svn commit: r658228 - /incubator/tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/assembly/xml/ConstrainingTypeProcessor.java

Author: mcombellack
Date: Tue May 20 06:02:44 2008
New Revision: 658228

URL: http://svn.apache.org/viewvc?rev=658228&view=rev
Log:
Fixed equals() problem where a String was being compared to a QName (which would always fail) rather than comparing the QName with a QName (FindBugs reported issue)

Modified:
    incubator/tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/assembly/xml/ConstrainingTypeProcessor.java

Modified: incubator/tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/assembly/xml/ConstrainingTypeProcessor.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/assembly/xml/ConstrainingTypeProcessor.java?rev=658228&r1=658227&r2=658228&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/assembly/xml/ConstrainingTypeProcessor.java (original)
+++ incubator/tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/assembly/xml/ConstrainingTypeProcessor.java Tue May 20 06:02:44 2008
@@ -131,7 +131,7 @@
                         constrainingType.getProperties().add(abstractProperty);
                         policyProcessor.readPolicies(abstractProperty, reader);
                         
-                    } else if (OPERATION.equals(name)) {
+                    } else if (OPERATION_QNAME.equals(name)) {
 
                         // Read an <operation>
                         Operation operation = new OperationImpl();