CAY-2896 Inserting two identical objects into two datamaps stores both objects in the last used datamap

 - release notes and minor cleaup

Author: stariy95

RELEASE-NOTES.txt

@@ -27,6 +27,7 @@ CAY-2876 Memory leak in the ObjectStore
 CAY-2879 Negative number for non parameterized ObjectSelect query not processed correctly
 CAY-2883 License and notice templates are not processed by the Gradle build
 CAY-2885 Modeler: DbImport fails to load DB schema view
+CAY-2896 Inserting two identical objects into two datamaps stores both objects in the last used datamap
 
 ----------------------------------
 Release: 5.0-M1

cayenne/src/main/java/org/apache/cayenne/access/flush/operation/BaseDbRowOp.java

@@ -77,15 +77,13 @@ public String toString() {
         return entity.getName() + " " + changeId;
     }
 
+    /**
+     * Internal check used in the batching ops logic
+     * @param rowOp to compare with
+     * @return true if another op has the same entity name and datamap
+     */
     protected boolean entitiesHaveSameNameAndDataMap(DbRowOp rowOp) {
-        return entitiesHaveSameName(rowOp) && entitiesHaveSameDataMap(rowOp);
-    }
-
-    boolean entitiesHaveSameName(DbRowOp rowOp) {
-        return rowOp.getEntity().getName().equals(getEntity().getName());
-    }
-
-    private boolean entitiesHaveSameDataMap(DbRowOp rowOp) {
-        return rowOp.getEntity().getDataMap().getName().equals(getEntity().getDataMap().getName());
+        return getEntity().getName().equals(rowOp.getEntity().getName())
+                && getEntity().getDataMap().getName().equals(rowOp.getEntity().getDataMap().getName());
     }
 }