Changeset 15591 in josm

Timestamp:

2019-12-13T23:04:23+01:00 (6 years ago)

Author:

Don-vip

Message:

fix #18408 - Add trim_list mapCSS function (patch by taylor.smock)

Location:

trunk

Files:

• src/org/openstreetmap/josm/gui/mappaint/mapcss/Functions.java (modified) (1 diff)
• test/unit/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSParserTest.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Functions.java

@@ -983 +983 @@
 
     /**
+     * Trim whitespaces from the strings {@code strings}.
+     *
+     * @param strings The list of strings to strip
+     * @return The resulting string
+     * @see Utils#strip
+     * @since 15591
+     */
+    public static List<String> trim_list(List<String> strings) {
+        return strings.stream().map(Utils::strip).filter(str -> !str.isEmpty()).collect(Collectors.toList());
+    }
+
+    /**
      * Check if two strings are similar, but not identical, i.e., have a Levenshtein distance of 1 or 2.
      * @param string1 first string to compare

trunk/test/unit/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSParserTest.java

@@ -325 +325 @@
     }
 
+    /**
+     * Test case for {@link Functions#trim_list}
+     */
+    @Test
+    public void testTrimList() {
+        List<String> trimmed = Functions.trim_list(Arrays.asList(" A1 ", "A2", " A3", "A4 ", ""));
+        assertEquals(4, trimmed.size());
+        assertEquals("A1", trimmed.get(0));
+        assertEquals("A2", trimmed.get(1));
+        assertEquals("A3", trimmed.get(2));
+        assertEquals("A4", trimmed.get(3));
+    }
+
     @Test
     public void testTicket8568() throws Exception {