Changeset 12341 in josm for trunk/src/org/openstreetmap

Timestamp:

2017-06-08T01:04:57+02:00 (7 years ago)

Author:

michael2402

Message:

Move common style update code to new method in map paint styles.

File:

• trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java

@@ -339 +339 @@
         @Override
         protected void finish() {
-            SwingUtilities.invokeLater(() -> {
-                fireMapPaintSylesUpdated();
-                styles.clearCached();
-
-                // Trigger a repaint of all data layers
-                Main.getLayerManager().getLayers()
-                    .stream()
-                    .filter(layer -> layer instanceof OsmDataLayer)
-                    .forEach(Layer::invalidate);
-            });
+            fireMapPaintSylesUpdated();
+            afterStyleUpdate();
         }
 
@@ -386 +378 @@
         MapPaintPrefHelper.INSTANCE.put(data);
         fireMapPaintSylesUpdated();
-        styles.clearCached();
-        Main.map.mapView.repaint();
+        afterStyleUpdate();
+    }
+
+    /**
+     * Manually trigger for now. TODO: Move this to a listener
+     */
+    private static void afterStyleUpdate() {
+        SwingUtilities.invokeLater(() -> {
+            styles.clearCached();
+
+            // Trigger a repaint of all data layers
+            Main.getLayerManager().getLayers()
+                .stream()
+                .filter(layer -> layer instanceof OsmDataLayer)
+                .forEach(Layer::invalidate);
+        });
     }
 
@@ -416 +422 @@
             fireMapPaintSylesUpdated();
         }
-        styles.clearCached();
-        Main.map.mapView.repaint();
+        afterStyleUpdate();
     }
 
@@ -448 +453 @@
         MapPaintPrefHelper.INSTANCE.put(styles.getStyleSources());
         fireMapPaintSylesUpdated();
-        styles.clearCached();
-        if (Main.isDisplayingMapView()) {
-            Main.map.mapView.repaint();
-        }
+        afterStyleUpdate();
     }