Changeset 10804 in josm

Timestamp:

2016-08-15T14:28:24+02:00 (8 years ago)

Author:

Don-vip

Message:

fix #13350 - Copy&pasting an object is not considered a change (patch by michael2402, regression) - gsoc-core

Location:

trunk

Files:

• src/org/openstreetmap/josm/command/AddPrimitivesCommand.java (modified) (1 diff)
• test/unit/org/openstreetmap/josm/command/AddPrimitivesCommandTest.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/command/AddPrimitivesCommand.java

@@ -101 +101 @@
                 }
             }
+            newPrimitives.stream().forEach(p -> p.setModified(true));
         } else { // redo
             // When redoing this command, we have to add the same objects, otherwise

trunk/test/unit/org/openstreetmap/josm/command/AddPrimitivesCommandTest.java

@@ -4 +4 @@
 import static org.junit.Assert.assertArrayEquals;
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertSame;
 import static org.junit.Assert.assertTrue;
@@ -218 +217 @@
         assertEquals(2, layer1.data.getNodes().size());
         assertEquals(1, layer1.data.getWays().size());
-        assertEquals(0, layer1.data.allModifiedPrimitives().size());
+        assertEquals(3, layer1.data.allModifiedPrimitives().size());
         for (OsmPrimitive n : layer1.data.allPrimitives()) {
             assertEquals("test", n.get("test"));
-            assertFalse(n.isModified());
+            assertTrue(n.isModified());
         }