Changeset 10452 in josm for trunk/src/org/openstreetmap/josm/command

Timestamp:

2016-06-21T23:41:42+02:00 (8 years ago)

Author:

Don-vip

Message:

fix #13019 - Make commands trigger an implicit layer redraw (patch by michael2402) - gsoc-core

Location:

trunk/src/org/openstreetmap/josm/command

Files:

• Command.java (modified) (2 diffs)
• SequenceCommand.java (modified) (1 diff)

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

@@ -148 +148 @@
      * Executes the command on the dataset. This implementation will remember all
      * primitives returned by fillModifiedData for restoring them on undo.
+     * <p>
+     * The layer should be invalidated after execution so that it can be re-painted.
      * @return true
+     * @see #invalidateAffectedLayers()
      */
     public boolean executeCommand() {
@@ -300 +303 @@
                 Objects.equals(layer, command.layer);
     }
+
+    /**
+     * Invalidate all layers that were affected by this command.
+     * @see Layer#invalidate()
+     */
+    public void invalidateAffectedLayers() {
+        OsmDataLayer layer = getLayer();
+        if (layer != null) {
+            layer.invalidate();
+        }
+    }
 }

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

@@ -126 +126 @@
 
     @Override
+    public void invalidateAffectedLayers() {
+        super.invalidateAffectedLayers();
+        for (Command c : sequence) {
+            c.invalidateAffectedLayers();
+        }
+    }
+
+    @Override
     public int hashCode() {
         return Objects.hash(super.hashCode(), Arrays.hashCode(sequence), sequenceComplete, name, continueOnError);