Revision: 3743
Author: [email protected]
Date: Fri Jul 16 07:15:45 2010
Log: Removed the defaultValue property from ConstructorParameter and removed the PRIMITIVE enum from PropertyType within that annotation. Nothing actually uses it.

Originally, a primitive constructor parameter meant it was not a child or property. For example, it could be a boolean flag that indicated whether a certain operation within the constructor should be performed. However, after discussing with Matt and Thomas, we agreed that a better solution would be to just create another constructor that feeds in a default value for the flag and use that one instead.
http://code.google.com/p/power-architect/source/detail?r=3743

Modified:
/trunk/src/main/java/ca/sqlpower/architect/enterprise/DomainCategorySnapshot.java

=======================================
--- /trunk/src/main/java/ca/sqlpower/architect/enterprise/DomainCategorySnapshot.java Wed Jul 14 16:06:30 2010 +++ /trunk/src/main/java/ca/sqlpower/architect/enterprise/DomainCategorySnapshot.java Fri Jul 16 07:15:45 2010
@@ -36,7 +36,7 @@
     public DomainCategorySnapshot(
@ConstructorParameter (propertyName = "spObject") DomainCategory spObject, @ConstructorParameter (propertyName = "originalUUID") String originalUUID, - @ConstructorParameter (propertyName = "workspaceRevision", defaultValue = "0") int systemRevision) { + @ConstructorParameter (propertyName = "workspaceRevision") int systemRevision) {
         super(originalUUID, systemRevision);
         this.spObject = spObject;
     }

Reply via email to