package org.opendaylight.yangtools.yang.data.impl.schema.tree;
import com.google.common.base.Optional;
+import com.google.common.base.Preconditions;
import com.google.common.base.Predicate;
import java.util.HashMap;
+import javax.annotation.Nonnull;
import org.opendaylight.yangtools.concepts.Identifiable;
import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.PathArgument;
import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
public static final Predicate<ModifiedNode> IS_TERMINAL_PREDICATE = new Predicate<ModifiedNode>() {
@Override
- public boolean apply(final ModifiedNode input) {
+ public boolean apply(final @Nonnull ModifiedNode input) {
+ Preconditions.checkNotNull(input);
switch (input.getType()) {
case DELETE:
case MERGE:
*
*/
public void delete() {
+ final ModificationType newType;
+
+ switch (modificationType) {
+ case DELETE:
+ case UNMODIFIED:
+ // We need to record this delete.
+ newType = ModificationType.DELETE;
+ break;
+ case MERGE:
+ case SUBTREE_MODIFIED:
+ case WRITE:
+ /*
+ * We are canceling a previous modification. This is a bit tricky,
+ * as the original write may have just introduced the data, or it
+ * may have modified it.
+ *
+ * As documented in BUG-2470, a delete of data introduced in this
+ * transaction needs to be turned into a no-op.
+ */
+ newType = original.isPresent() ? ModificationType.DELETE : ModificationType.UNMODIFIED;
+ break;
+ default:
+ throw new IllegalStateException("Unhandled deletion of node with " + modificationType);
+ }
+
clearSnapshot();
- updateModificationType(ModificationType.DELETE);
children.clear();
this.value = null;
+ updateModificationType(newType);
}
/**