Changeset 17741 in josm

Timestamp:

2021-04-11T11:29:12+02:00 (4 years ago)

Author:

simon04

Message:

see #20659 - Extract LayerVisibilityAction.ColorSelector.getColors

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/layer/LayerVisibilityAction.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/layer/LayerVisibilityAction.java

@@ -610 +610 @@
             panels.put(color, colorPanel);
 
-            List<Color> colors = layerSupplier.get().stream().map(l -> l.getColor()).distinct().collect(Collectors.toList());
+            List<Color> colors = getColors();
             if (colors.size() == 1) {
                 highlightColor(colors.get(0));
@@ -616 +616 @@
         }
 
-        @Override
-        public void updateLayers() {
-            Collection<Layer> layers = layerSupplier.get();
-            List<Color> colors = layers.stream().filter(l -> l instanceof GpxLayer)
-                    .map(l -> ((GpxLayer) l).getColor())
+        private List<Color> getColors() {
+            return layerSupplier.get().stream()
+                    .filter(layer -> layer instanceof GpxLayer)
+                    .map(Layer::getColor)
                     .distinct()
                     .collect(Collectors.toList());
+        }
+
+        @Override
+        public void updateLayers() {
+            List<Color> colors = getColors();
             if (colors.size() == 1) {
                 setVisible(true);