Changeset 6026 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2013-06-26T11:09:17+02:00 (11 years ago)

Author:

akks

Message:

Change HighlightHelper methods, adopted DeteleAction to use the same code

File:

• trunk/src/org/openstreetmap/josm/gui/util/HighlightHelper.java (modified) (7 diffs)

trunk/src/org/openstreetmap/josm/gui/util/HighlightHelper.java

@@ -3 +3 @@
 
 import java.util.Collection;
+import java.util.Collections;
 import java.util.HashSet;
+import java.util.Iterator;
 import java.util.Set;
 import org.openstreetmap.josm.Main;
@@ -20 +22 @@
      * Highlight and remember given primitives
      * @param prims - primitives to highlight/unhighlight
-     * @param flag - true to highlight
      */
-    public void highlight(Collection <? extends OsmPrimitive> prims, boolean flag) {
+    public boolean highlight(Collection <? extends OsmPrimitive> prims) {
+        return highlight(prims, false);
+    }
+    
+    /**
+     * Highlight and remember given primitives
+     * @param prims - primitives to highlight/unhighlight
+     * @param only - remove previous highlighting
+     */
+    public boolean highlight(Collection <? extends OsmPrimitive> prims, boolean only) {
+        boolean needsRepaint = false;
+        if (only) {
+            Iterator<OsmPrimitive> it = highlightedPrimitives.iterator();
+            while (it.hasNext()) {
+                OsmPrimitive p = it.next();
+                if (!prims.contains(p)) {
+                    p.setHighlighted(false);
+                    it.remove();
+                    needsRepaint = true;
+                }
+            }
+        }
         for (OsmPrimitive p: prims) {
-            highlight(p, flag);
+            needsRepaint |= setHighlight(p, true);
         }
+        //return true;
+        return needsRepaint;
     }
     
@@ -32 +56 @@
      * @param prims - primitives to highlight/unhighlight
      */
-    public void highlightOnly(Collection <? extends OsmPrimitive> prims) {
-        clear();
-        highlight(prims, true);
+    public boolean highlightOnly(Collection <? extends OsmPrimitive> prims) {
+        return highlight(prims, true);
     }
     
@@ -41 +64 @@
      * @param p - primitives to highlight/unhighlight
      */
-    public void highlightOnly(OsmPrimitive p) {
-        clear();
-        highlight(p, true);
+    public boolean highlightOnly(OsmPrimitive p) {
+        return highlight(Collections.singleton(p), true);
     }
     
@@ -51 +73 @@
      * @param flag - true to highlight
      */
-    public void highlight(OsmPrimitive p, boolean flag) {
+    public boolean setHighlight(OsmPrimitive p, boolean flag) {
         if (p instanceof Relation) {
+            boolean needRepaint = false;
             for (OsmPrimitive m: ((Relation) p).getMemberPrimitives()) {
-                highlight(m, flag);
+                needRepaint |= setHighlight(m, flag);
             }
+            return needRepaint;
         } else
         if (flag) {
             if (highlightedPrimitives.add(p)) {
                 p.setHighlighted(true);
+                return true;
             }
         } else {
             if (highlightedPrimitives.remove(p)) {
                 p.setHighlighted(false);
+                return true;
             }
         }
+        return false;
     }
     
@@ -81 +108 @@
      * Slow method to import all currently highlighted primitives into this instance
      */
-    public void findAllHighligted() {
+    public void findAllHighlighted() {
         DataSet ds = Main.main.getCurrentDataSet();
         if (ds!=null) {
@@ -89 +116 @@
     
     /**
-     * Slow method to import all currently highlighted primitives into this instance
+     * Slow method to remove highlights from all primitives
      */
-    public static void clearAllHighligted() {
+    public static void clearAllHighlighted() {
         DataSet ds = Main.main.getCurrentDataSet();
         if (ds!=null) {