From 9990668476c2cfbbdff79eafea5b894bb72fc352 Mon Sep 17 00:00:00 2001
From: Lee Trujillo <lee.trujillo@forgerock.com>
Date: Wed, 01 Apr 2015 17:50:20 +0000
Subject: [PATCH] CR-6560 OPENDJ-1909: make diffEntries use delete+add

---
 opendj-ldap-sdk/src/main/java/org/forgerock/opendj/ldap/Entries.java |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/opendj-ldap-sdk/src/main/java/org/forgerock/opendj/ldap/Entries.java b/opendj-ldap-sdk/src/main/java/org/forgerock/opendj/ldap/Entries.java
index b54e641..fac1328 100644
--- a/opendj-ldap-sdk/src/main/java/org/forgerock/opendj/ldap/Entries.java
+++ b/opendj-ldap-sdk/src/main/java/org/forgerock/opendj/ldap/Entries.java
@@ -22,7 +22,7 @@
  *
  *
  *      Copyright 2010 Sun Microsystems, Inc.
- *      Portions copyright 2011-2013 ForgeRock AS
+ *      Portions copyright 2011-2015 ForgeRock AS
  */
 
 package org.forgerock.opendj.ldap;
@@ -422,18 +422,18 @@
                 // and removed. We won't replace the attribute because this is
                 // not
                 // reversible.
-                final Attribute addedValues = new LinkedAttribute(ato);
-                addedValues.removeAll(afrom);
-                if (!addedValues.isEmpty()) {
-                    request.addModification(new Modification(ModificationType.ADD, addedValues));
-                }
-
                 final Attribute deletedValues = new LinkedAttribute(afrom);
                 deletedValues.removeAll(ato);
                 if (!deletedValues.isEmpty()) {
                     request.addModification(new Modification(ModificationType.DELETE, deletedValues));
                 }
 
+                final Attribute addedValues = new LinkedAttribute(ato);
+                addedValues.removeAll(afrom);
+                if (!addedValues.isEmpty()) {
+                    request.addModification(new Modification(ModificationType.ADD, addedValues));
+                }
+
                 afrom = ifrom.hasNext() ? ifrom.next() : null;
                 ato = ito.hasNext() ? ito.next() : null;
             } else if (cmp < 0) {

--
Gitblit v1.10.0