Ticket #21438: 21438-TagChecker-2.patch

src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

@@ -544 +544 @@
      *  else a set which might be empty.
      */
     private static Set<String> getPresetValues(String key) {
-        Set<String> res = TaggingPresets.getPresetValues(key);
-        if (!Utils.isEmpty(res))
-            return res;
+        if (TaggingPresets.isKeyInPresets(key)) {
+            return TaggingPresets.getPresetValues(key);
+        }
         if (additionalPresetsValueData.contains(key))
             return Collections.emptySet();
         // null means key is not known
@@ -558 +558 @@
      * @param key key
      * @return {@code true} if the given key is in internal presets
      * @since 9023
+     * @deprecated Use {@link TaggingPresets#isKeyInPresets(String)} instead
      */
+    @Deprecated
     public static boolean isKeyInPresets(String key) {
-        return !Utils.isEmpty(TaggingPresets.getPresetValues(key));
+        return TaggingPresets.isKeyInPresets(key);
     }
 
     /**
@@ -882 +884 @@
         if (!checkValues || key == null || Utils.isEmpty(value))
             return;
         if (additionalPresetsValueData != null && !isTagIgnored(key, value)) {
-            if (!isKeyInPresets(key)) {
+            if (!TaggingPresets.isKeyInPresets(key)) {
                 spellCheckKey(withErrors, p, key);
             } else if (!isTagInPresets(key, value)) {
                 if (oftenUsedTags.contains(key, value)) {
@@ -906 +908 @@
         if (ignoreDataEquals.contains(prettifiedKey)) {
             fixedKey = prettifiedKey;
         } else {
-            fixedKey = isKeyInPresets(prettifiedKey) ? prettifiedKey : harmonizedKeys.get(prettifiedKey);
+            fixedKey = TaggingPresets.isKeyInPresets(prettifiedKey) ? prettifiedKey : harmonizedKeys.get(prettifiedKey);
         }
         if (fixedKey == null && ignoreDataTag.stream().anyMatch(a -> a.getKey().equals(prettifiedKey))) {
             fixedKey = prettifiedKey;

src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresets.java

@@ -214 +214 @@
     }
 
     /**
+     * Determines if the given key is in the loaded presets.
+     * @param key key
+     * @return {@code true} if the given key in the loaded presets
+     * @since xxx
+     */
+    public static boolean isKeyInPresets(String key) {
+        return PRESET_TAG_CACHE.get(key) != null;
+    }
+
+    /**
      * Replies a new collection of all presets matching the parameters.
      *
      * @param t the preset types to include