svn commit: r1097404 - /incubator/isis/trunk/applib/src/main/java/org/apache/isis/applib/Identifier.java

[rm...@apache.org]

Author: rmatthews
Date: Thu Apr 28 10:31:07 2011
New Revision: 1097404

URL: http://svn.apache.org/viewvc?rev=1097404&view=rev
Log:
The class and name string identifier did not distinguish between an action and a property.  Now an action uses empty braces ("()") to indicate it is different from property.

Modified:
    incubator/isis/trunk/applib/src/main/java/org/apache/isis/applib/Identifier.java

Modified: incubator/isis/trunk/applib/src/main/java/org/apache/isis/applib/Identifier.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/applib/src/main/java/org/apache/isis/applib/Identifier.java?rev=1097404&r1=1097403&r2=1097404&view=diff
==============================================================================
--- incubator/isis/trunk/applib/src/main/java/org/apache/isis/applib/Identifier.java (original)
+++ incubator/isis/trunk/applib/src/main/java/org/apache/isis/applib/Identifier.java Thu Apr 28 10:31:07 2011
@@ -225,7 +225,11 @@ public class Identifier implements Compa
     }
 
     public StringBuilder toClassAndNameIdentityString(final StringBuilder buf) {
-        return toClassIdentityString(buf).append("#").append(memberName);
+        StringBuilder builder = toClassIdentityString(buf).append("#").append(memberName);
+        if (type == Type.ACTION) {
+            builder.append("()");
+        }
+        return builder;
     }
 
     public String toParmsIdentityString() {
@@ -261,7 +265,7 @@ public class Identifier implements Compa
                 identityString = toClassIdentityString();
             } else {
                 final StringBuilder buf = new StringBuilder();
-                toClassAndNameIdentityString(buf);
+                toClassIdentityString(buf).append("#").append(memberName);
                 toParmsIdentityString(buf);
                 identityString = buf.toString();
             }