You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sis.apache.org by js...@apache.org on 2016/08/08 15:18:30 UTC

svn commit: r1755510 - /sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/builder/AttributeTypeBuilder.java

Author: jsorel
Date: Mon Aug  8 15:18:30 2016
New Revision: 1755510

URL: http://svn.apache.org/viewvc?rev=1755510&view=rev
Log:
Fix identifier count not decremented when removing identifier property

Modified:
    sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/builder/AttributeTypeBuilder.java

Modified: sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/builder/AttributeTypeBuilder.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/builder/AttributeTypeBuilder.java?rev=1755510&r1=1755509&r2=1755510&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/builder/AttributeTypeBuilder.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/builder/AttributeTypeBuilder.java [UTF-8] Mon Aug  8 15:18:30 2016
@@ -654,6 +654,17 @@ public final class AttributeTypeBuilder<
     }
 
     /**
+     * {@inheritDoc}
+     */
+    @Override
+    public void remove() {
+        if (isIdentifier && owner()!=null) {
+            owner().identifierCount--;
+        }
+        super.remove();
+    }
+
+    /**
      * Builds the attribute type from the information specified to this builder.
      * If a type has already been built and this builder state has not changed since the type creation,
      * then the previously created {@code AttributeType} instance is returned.