Changeset 18080 in josm for trunk/src

Timestamp:

2021-07-22T22:43:10+02:00 (3 years ago)

Author:

Don-vip

Message:

fix #21156 - fix #20851 - see #20861 - disable value_template for multiple selection (patch by marcello)

Location:

trunk/src/org/openstreetmap/josm/gui/tagging/presets

Files:

• TaggingPresetItemGuiSupport.java (modified) (5 diffs)
• items/Text.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetItemGuiSupport.java

@@ -6 +6 @@
 import org.openstreetmap.josm.data.osm.Tagged;
 import org.openstreetmap.josm.data.osm.search.SearchCompiler;
-import org.openstreetmap.josm.gui.tagging.presets.items.KeyedItem;
 import org.openstreetmap.josm.tools.ListenerList;
 import org.openstreetmap.josm.tools.template_engine.TemplateEngineDataProvider;
@@ -14 +13 @@
 import java.util.Collections;
 import java.util.function.Supplier;
-import java.util.stream.Collectors;
 
 /**
@@ -70 +68 @@
      * Creates a new {@code TaggingPresetItemGuiSupport}
      *
+     * @param presetInitiallyMatches whether the preset initially matched
      * @param selected the selected primitives
-     * @param presetInitiallyMatches whether the preset initially matched
+     * @param changedTagsSupplier the changed tags
      * @return the new {@code TaggingPresetItemGuiSupport}
      */
@@ -82 +81 @@
      * Creates a new {@code TaggingPresetItemGuiSupport}
      *
+     * @param presetInitiallyMatches whether the preset initially matched
      * @param selected the selected primitives
-     * @param presetInitiallyMatches whether the preset initially matched
      * @return the new {@code TaggingPresetItemGuiSupport}
      */
@@ -91 +90 @@
     }
 
+    /**
+     * Get tags with values as currently shown in the dialog.
+     * If exactly one primitive is selected, get all tags of it, then 
+     * overwrite with the current values shown in the dialog.
+     * Else get only the tags shown in the dialog.
+     * @return Tags
+     */
+    public Tagged getTagged() {
+        if (selected.size() != 1) {
+            return Tagged.ofTags(changedTagsSupplier.get());
+        }
+        // if there is only one primitive selected, get its tags
+        Tagged tagged = Tagged.ofMap(selected.iterator().next().getKeys());
+        // update changed tags
+        changedTagsSupplier.get().forEach(tag -> tagged.put(tag));
+        return tagged;
+    }
+
     @Override
     public Collection<String> getTemplateKeys() {
-        return changedTagsSupplier.get().stream().map(Tag::getKey).collect(Collectors.toList());
+        return getTagged().keySet();
     }
 
     @Override
     public Object getTemplateValue(String key, boolean special) {
-        return changedTagsSupplier.get().stream()
-                .filter(tag -> key.equals(tag.getKey()))
-                .findFirst().map(Tag::getValue).orElseGet(() -> {
-                    KeyedItem.Usage usage = KeyedItem.determineTextUsage(getSelected(), key);
-                    return usage.hasUniqueValue() ? usage.getFirst() : null;
-                });
+        String value = getTagged().get(key);
+        return (value == null || value.isEmpty()) ? null : value;
     }
 
     @Override
     public boolean evaluateCondition(SearchCompiler.Match condition) {
-        Tagged tagged = Tagged.ofTags(changedTagsSupplier.get());
-        return condition.match(tagged);
+        return condition.match(getTagged());
     }
 

trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/Text.java

@@ -239 +239 @@
     }
 
+    /**
+     * Set the value template.
+     * @param pattern The value_template pattern.
+     * @throws SAXException If an error occured while parsing.
+     */
     public void setValue_template(String pattern) throws SAXException { // NOPMD
         try {
@@ -249 +254 @@
 
     private void setupListeners(AutoCompletingTextField textField, TaggingPresetItemGuiSupport support) {
-        if (valueTemplate == null) { // only fire on normal fields
+        // value_templates don't work well with multiple selected items because, 
+        // as the command queue is currently implemented, we can only save 
+        // the same value to all selected primitives, which is probably not 
+        // what you want.
+        if (valueTemplate == null || support.getSelected().size() > 1) { // only fire on normal fields
             textField.getDocument().addDocumentListener(DocumentAdapter.create(ignore ->
                     support.fireItemValueModified(this, key, textField.getText())));
         } else { // only listen on calculated fields
-            textField.setForeground(Color.BLUE);
             support.addListener((source, key, newValue) -> {
                 String valueTemplateText = valueTemplate.getText(support);
@@ -259 +267 @@
                         valueTemplate, key, source, newValue, valueTemplateText);
                 textField.setItem(valueTemplateText);
+                if (originalValue != null && !originalValue.equals(valueTemplateText)) {
+                    textField.setForeground(Color.RED);
+                } else {
+                    textField.setForeground(Color.BLUE);
+                }
             });
         }