Changeset 3518 in josm for trunk


Ignore:
Timestamp:
2010-09-10T07:45:16+02:00 (14 years ago)
Author:
jttt
Message:
  • add New relation button to Preset dialog
  • add required attribute to tagging preset (allows to specify what tags has to filled for osm primitive to qualify as having preset)
  • show list of preset also in relation dialog
Location:
trunk/src/org/openstreetmap/josm/gui
Files:
1 added
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

    r3501 r3518  
    77import java.awt.BorderLayout;
    88import java.awt.Component;
    9 import java.awt.Cursor;
    10 import java.awt.Dialog.ModalityType;
    11 import java.awt.Dimension;
    129import java.awt.Font;
    1310import java.awt.GridBagLayout;
    1411import java.awt.Point;
     12import java.awt.Dialog.ModalityType;
    1513import java.awt.event.ActionEvent;
    1614import java.awt.event.ActionListener;
    … …  
    2018import java.awt.event.MouseAdapter;
    2119import java.awt.event.MouseEvent;
    22 import java.awt.event.MouseListener;
    2320import java.util.ArrayList;
    2421import java.util.Collection;
    … …  
    6663import org.openstreetmap.josm.data.osm.Relation;
    6764import org.openstreetmap.josm.data.osm.RelationMember;
     65import org.openstreetmap.josm.data.osm.Tag;
    6866import org.openstreetmap.josm.data.osm.Way;
    6967import org.openstreetmap.josm.data.osm.event.AbstractDatasetChangedEvent;
    … …  
    7876import org.openstreetmap.josm.gui.SideButton;
    7977import org.openstreetmap.josm.gui.dialogs.ToggleDialog;
     78import org.openstreetmap.josm.gui.dialogs.properties.PresetListPanel.PresetHandler;
    8079import org.openstreetmap.josm.gui.dialogs.relation.RelationEditor;
    8180import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    82 import org.openstreetmap.josm.gui.preferences.TaggingPresetPreference;
    8381import org.openstreetmap.josm.gui.tagging.TaggingPreset;
    8482import org.openstreetmap.josm.gui.tagging.ac.AutoCompletingComboBox;
    … …  
    480478    private final SideButton btnEdit;
    481479    private final SideButton btnDel;
    482     private final JPanel presets = new JPanel(new GridBagLayout());
     480    private final PresetListPanel presets = new PresetListPanel();
    483481
    484482    private final JLabel selectSth = new JLabel("<html><p>"
    … …  
    716714    }
    717715
    718     private void checkPresets(int nodes, int ways, int relations, int closedways)
    719     {
    720         /**
    721          * Small helper class that manages the highlighting of the label on hover as well as opening
    722          * the corresponding preset when clicked
    723          */
    724         class PresetLabelML implements MouseListener {
    725             JLabel label;
    726             Font bold;
    727             Font normal;
    728             TaggingPreset tag;
    729             PresetLabelML(JLabel lbl, TaggingPreset t) {
    730                 super();
    731                 label = lbl;
    732                 lbl.setCursor(new Cursor(Cursor.HAND_CURSOR));
    733                 normal = label.getFont();
    734                 bold = normal.deriveFont(normal.getStyle() ^ Font.BOLD);
    735                 tag = t;
    736             }
    737             public void mouseClicked(MouseEvent arg0) {
    738                 tag.actionPerformed(null);
    739             }
    740             public void mouseEntered(MouseEvent arg0) {
    741                 label.setFont(bold);
    742             }
    743             public void mouseExited(MouseEvent arg0) {
    744                 label.setFont(normal);
    745             }
    746             public void mousePressed(MouseEvent arg0) {}
    747             public void mouseReleased(MouseEvent arg0) {}
    748         }
    749 
    750         presets.removeAll();
    751         int total = nodes+ways+relations+closedways;
    752         if(total == 0) {
    753             presets.setVisible(false);
    754             return;
    755         }
    756 
    757         for(TaggingPreset t : TaggingPresetPreference.taggingPresets) {
    758             if((t.types == null || !((relations > 0 && !t.types.contains("relation")) &&
    759                     (nodes > 0 && !t.types.contains("node")) &&
    760                     (ways+closedways > 0 && !t.types.contains("way")) &&
    761                     (closedways > 0 && !t.types.contains("closedway")))) && t.isShowable())
    762             {
    763                 int found = 0;
    764                 for(TaggingPreset.Item i : t.data) {
    765                     if(!(i instanceof TaggingPreset.Key)) {
    766                         continue;
    767                     }
    768                     String val = ((TaggingPreset.Key)i).value;
    769                     String key = ((TaggingPreset.Key)i).key;
    770                     // we subtract 100 if not found and add 1 if found
    771                     found -= 100;
    772                     if(key == null || !valueCount.containsKey(key)) {
    773                         continue;
    774                     }
    775 
    776                     Map<String, Integer> v = valueCount.get(key);
    777                     if(v.size() == 1 && val != null && v.containsKey(val) && v.get(val) == total) {
    778                         found += 101;
    779                     }
    780                 }
    781 
    782                 if(found <= 0) {
    783                     continue;
    784                 }
    785 
    786                 JLabel lbl = new JLabel(t.getName());
    787                 lbl.addMouseListener(new PresetLabelML(lbl, t));
    788                 presets.add(lbl, GBC.eol().fill(GBC.HORIZONTAL));
    789             }
    790         }
    791 
    792         if(presets.getComponentCount() > 0) {
    793             presets.setVisible(true);
    794             // This ensures the presets are exactly as high as needed.
    795             int height = presets.getComponentCount() * presets.getComponent(0).getHeight();
    796             Dimension size = new Dimension(presets.getWidth(), height);
    797             presets.setMaximumSize(size);
    798             presets.setMinimumSize(size);
    799         } else {
    800             presets.setVisible(false);
    801         }
    802     }
    803 
    804716    private int findRow(TableModel model, Object value) {
    805717        for (int i=0; i<model.getRowCount(); i++) {
    … …  
    809721        return -1;
    810722    }
     723
     724    private PresetHandler presetHandler = new PresetHandler() {
     725
     726        @Override
     727        public void updateTags(List<Tag> tags) {
     728            Command command = TaggingPreset.createCommand(getSelection(), tags);
     729            if (command != null) {
     730                Main.main.undoRedo.add(command);
     731            }
     732        }
     733
     734        @Override
     735        public Collection<OsmPrimitive> getSelection() {
     736            if (Main.main == null) return null;
     737            if (Main.main.getCurrentDataSet() == null) return null;
     738
     739            return Main.main.getCurrentDataSet().getSelected();
     740        }
     741    };
    811742
    812743    public void selectionChanged(Collection<? extends OsmPrimitive> newSelection) {
    … …  
    870801            propertyData.addRow(new Object[]{e.getKey(), e.getValue()});
    871802        }
    872 
    873         // re-load membership data
    874         // this is rather expensive since we have to walk through all members of all existing relationships.
    875         // could use back references here for speed if necessary.
    876803
    877804        membershipData.setRowCount(0);
    … …  
    913840        }
    914841
    915         checkPresets(nodes, ways, relations, closedways);
     842        presets.updatePresets(nodes, ways, relations, closedways, valueCount, presetHandler);
    916843
    917844        membershipTable.getTableHeader().setVisible(membershipData.getRowCount() > 0);

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

    r3508 r3518  
    2424import java.util.Collection;
    2525import java.util.Collections;
     26import java.util.HashMap;
    2627import java.util.HashSet;
    2728import java.util.Iterator;
    2829import java.util.List;
     30import java.util.Map;
    2931import java.util.Set;
    3032import java.util.logging.Logger;
    … …  
    6163import org.openstreetmap.josm.data.osm.Relation;
    6264import org.openstreetmap.josm.data.osm.RelationMember;
     65import org.openstreetmap.josm.data.osm.Tag;
    6366import org.openstreetmap.josm.gui.ConditionalOptionPaneUtil;
    6467import org.openstreetmap.josm.gui.DefaultNameFormatter;
    … …  
    6669import org.openstreetmap.josm.gui.SideButton;
    6770import org.openstreetmap.josm.gui.HelpAwareOptionPane.ButtonSpec;
     71import org.openstreetmap.josm.gui.dialogs.properties.PresetListPanel.PresetHandler;
    6872import org.openstreetmap.josm.gui.help.ContextSensitiveHelpAction;
    6973import org.openstreetmap.josm.gui.help.HelpUtil;
    7074import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    7175import org.openstreetmap.josm.gui.tagging.TagEditorPanel;
     76import org.openstreetmap.josm.gui.tagging.TagModel;
    7277import org.openstreetmap.josm.gui.tagging.ac.AutoCompletingTextField;
    7378import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionList;
    … …  
    123128        referrerModel = new ReferringRelationsBrowserModel(relation);
    124129
    125         tagEditorPanel = new TagEditorPanel();
     130        tagEditorPanel = new TagEditorPanel(new PresetHandler() {
     131
     132            @Override
     133            public void updateTags(List<Tag> tags) {
     134                Map<String, TagModel> modelTags = new HashMap<String, TagModel>();
     135                for (int i=0; i<tagEditorPanel.getModel().getRowCount(); i++) {
     136                    TagModel tagModel = tagEditorPanel.getModel().get(i);
     137                    modelTags.put(tagModel.getName(), tagModel);
     138                }
     139                for (Tag tag: tags) {
     140                    TagModel existing = modelTags.get(tag.getKey());
     141
     142                    if (tag.getValue().isEmpty()) {
     143                        if (existing != null) {
     144                            tagEditorPanel.getModel().delete(tag.getKey());
     145                        }
     146                    } else {
     147                        if (existing != null) {
     148                            tagEditorPanel.getModel().updateTagValue(existing, tag.getValue());
     149                        } else {
     150                            tagEditorPanel.getModel().add(tag.getKey(), tag.getValue());
     151                        }
     152                    }
     153
     154                }
     155            }
     156
     157            @Override
     158            public Collection<OsmPrimitive> getSelection() {
     159                Relation relation = new Relation();
     160                tagEditorPanel.getModel().applyToPrimitive(relation);
     161                return Collections.<OsmPrimitive>singletonList(relation);
     162            }
     163        });
     164
    126165        // populate the models
    127166        //

  • trunk/src/org/openstreetmap/josm/gui/io/TagSettingsPanel.java

    r3133 r3518  
    2727    protected void build() {
    2828        setLayout(new BorderLayout());
    29         add(pnlTagEditor = new TagEditorPanel(), BorderLayout.CENTER);
     29        add(pnlTagEditor = new TagEditorPanel(null), BorderLayout.CENTER);
    3030    }
    3131
    3232    /**
    3333     * Creates a new panel
    34      * 
     34     *
    3535     * @param changesetCommentModel the changeset comment model. Must not be null.
    3636     * @throws IllegalArgumentException thrown if {@code changesetCommentModel} is null

  • trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorModel.java

    r3141 r3518  
    5050     * Creates a new tag editor model. Internally allocates two selection models
    5151     * for row selection and column selection.
    52      * 
     52     *
    5353     * To create a {@see JTable} with this model:
    5454     * <pre>
    … …  
    5656     *    TagTable tbl  = new TagTabel(model);
    5757     * </pre>
    58      * 
     58     *
    5959     * @see #getRowSelectionModel()
    6060     * @see #getColumnSelectionModel()
    … …  
    6666    /**
    6767     * Creates a new tag editor model.
    68      * 
     68     *
    6969     * @param rowSelectionModel the row selection model. Must not be null.
    7070     * @param colSelectionModel the column selection model. Must not be null.
    … …  
    8585    /**
    8686     * Replies the row selection model used by this tag editor model
    87      * 
     87     *
    8888     * @return the row selection model used by this tag editor model
    8989     */
    … …  
    9494    /**
    9595     * Replies the column selection model used by this tag editor model
    96      * 
     96     *
    9797     * @return the column selection model used by this tag editor model
    9898     */
    … …  
    355355        tags.add(tag);
    356356        setDirty(false);
     357        fireTableDataChanged();
    357358    }
    358359
    … …  
    441442    /**
    442443     * Replies the the tags in this tag editor model as {@see TagCollection}.
    443      * 
     444     *
    444445     * @return the the tags in this tag editor model as {@see TagCollection}
    445446     */

  • trunk/src/org/openstreetmap/josm/gui/tagging/TagEditorPanel.java

    r3210 r3518  
    66import java.awt.GridBagLayout;
    77import java.awt.Insets;
     8import java.util.HashMap;
     9import java.util.Map;
     10import java.util.Map.Entry;
    811import java.util.logging.Logger;
    912
    … …  
    1215import javax.swing.JPanel;
    1316import javax.swing.JScrollPane;
     17import javax.swing.event.TableModelEvent;
     18import javax.swing.event.TableModelListener;
    1419
     20import org.openstreetmap.josm.gui.dialogs.properties.PresetListPanel;
     21import org.openstreetmap.josm.gui.dialogs.properties.PresetListPanel.PresetHandler;
    1522import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    1623import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionList;
    … …  
    3340    private TagTable tagTable;
    3441
     42    private PresetListPanel presetListPanel;
     43    private final PresetHandler presetHandler;
     44
    3545    private AutoCompletionManager autocomplete;
    3646    private AutoCompletionList acList;
    … …  
    4656        pnl.setLayout(new BorderLayout());
    4757        pnl.add(new JScrollPane(tagTable), BorderLayout.CENTER);
     58        if (presetHandler != null) {
     59            presetListPanel = new PresetListPanel();
     60            pnl.add(presetListPanel, BorderLayout.NORTH);
     61        }
    4862        return pnl;
    4963    }
    … …  
    98112        gc.anchor = GridBagConstraints.CENTER;
    99113        add(tablePanel,gc);
     114
     115        if (presetHandler != null) {
     116            model.addTableModelListener(new TableModelListener() {
     117                @Override
     118                public void tableChanged(TableModelEvent e) {
     119                    updatePresets();
     120                }
     121            });
     122        }
    100123    }
    101124
    … …  
    104127     * internally and can be retrieved with {@see #getModel()}.
    105128     */
    106     public TagEditorPanel() {
    107         this(null);
     129    public TagEditorPanel(PresetHandler presetHandler) {
     130        this(null, presetHandler);
    108131    }
    109132
    … …  
    111134     * Creates a new tag editor panel with a supplied model. If
    112135     * {@code model} is null, a new model is created.
    113      * 
     136     *
    114137     * @param model the tag editor model
    115138     */
    116     public TagEditorPanel(TagEditorModel model) {
     139    public TagEditorPanel(TagEditorModel model, PresetHandler presetHandler) {
    117140        this.model = model;
     141        this.presetHandler = presetHandler;
    118142        if (this.model == null) {
    119143            this.model = new TagEditorModel();
    … …  
    135159     * tag editor panel. {@code layer} is the data layer from whose data set
    136160     * tag values are proposed as auto completion items.
    137      * 
     161     *
    138162     * @param layer the data layer. Must not be null.
    139163     * @throws IllegalArgumentException thrown if {@code layer} is null
    … …  
    158182        super.setEnabled(enabled);
    159183    }
     184
     185    private void updatePresets() {
     186        Map<String, Map<String, Integer>> valuesCount = new HashMap<String, Map<String,Integer>>();
     187        for (Entry<String, String> entry: model.getTags().entrySet()) {
     188            Map<String, Integer> values = new HashMap<String, Integer>();
     189            values.put(entry.getValue(), 1);
     190            valuesCount.put(entry.getKey(), values);
     191        }
     192        presetListPanel.updatePresets(0, 0, 1, 0, valuesCount, presetHandler);
     193        validate();
     194    }
    160195}

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPreset.java

    r3486 r3518  
    1919import java.io.Reader;
    2020import java.io.UnsupportedEncodingException;
     21import java.util.ArrayList;
    2122import java.util.Arrays;
    2223import java.util.Collection;
     24import java.util.Collections;
    2325import java.util.EnumSet;
    2426import java.util.HashMap;
    … …  
    3739import javax.swing.JPanel;
    3840import javax.swing.JTextField;
     41import javax.swing.SwingUtilities;
    3942import javax.xml.transform.stream.StreamSource;
    4043
    4144import org.openstreetmap.josm.Main;
     45import org.openstreetmap.josm.command.AddCommand;
    4246import org.openstreetmap.josm.command.ChangePropertyCommand;
    4347import org.openstreetmap.josm.command.Command;
    … …  
    4751import org.openstreetmap.josm.data.osm.OsmUtils;
    4852import org.openstreetmap.josm.data.osm.Relation;
     53import org.openstreetmap.josm.data.osm.Tag;
    4954import org.openstreetmap.josm.data.osm.Way;
    5055import org.openstreetmap.josm.gui.ExtendedDialog;
    … …  
    8893            return name().toLowerCase();
    8994        }
    90 
    91     }
     95    }
     96
     97    public static final int DIALOG_ANSWER_APPLY = 1;
     98    public static final int DIALOG_ANSWER_NEW_RELATION = 2;
     99    public static final int DIALOG_ANSWER_CANCEL = 3;
    92100
    93101    public TaggingPresetMenu group = null;
    … …  
    109117        public boolean focus = false;
    110118        abstract boolean addToPanel(JPanel p, Collection<OsmPrimitive> sel);
    111         abstract void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds);
     119        abstract void addCommands(List<Tag> changedTags);
    112120        boolean requestFocusInWindow() {return false;}
    113121    }
    … …  
    175183        public boolean use_last_as_default = false;
    176184        public boolean delete_if_empty = false;
     185        public boolean required = false;
    177186
    178187        private JComponent value;
    … …  
    220229        }
    221230
    222         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {
     231        @Override public void addCommands(List<Tag> changedTags) {
    223232
    224233            // return if unchanged
    … …  
    235244                        v = null;
    236245                    }
    237                     cmds.add(new ChangePropertyCommand(sel, key, v));
     246                    changedTags.add(new Tag(key, v));
    238247        }
    239248        @Override boolean requestFocusInWindow() {return value.requestFocusInWindow();}
    … …  
    250259        public boolean default_ = false; // only used for tagless objects
    251260        public boolean use_last_as_default = false;
     261        public boolean required = false;
    252262
    253263        private QuadStateCheckBox check;
    … …  
    312322        }
    313323
    314         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {
     324        @Override public void addCommands(List<Tag> changedTags) {
    315325            // if the user hasn't changed anything, don't create a command.
    316326            if (check.getState() == initialState && !def) return;
    317327
    318328            // otherwise change things according to the selected value.
    319             cmds.add(new ChangePropertyCommand(sel, key,
     329            changedTags.add(new Tag(key,
    320330                    check.getState() == QuadStateCheckBox.State.SELECTED ? value_on :
    321331                        check.getState() == QuadStateCheckBox.State.NOT_SELECTED ? value_off :
    … …  
    339349        public boolean editable = true;
    340350        public boolean use_last_as_default = false;
     351        public boolean required = false;
    341352
    342353        private JComboBox combo;
    … …  
    432443            return true;
    433444        }
    434         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {
     445        @Override public void addCommands(List<Tag> changedTags) {
    435446            Object obj = combo.getSelectedItem();
    436447            String display = (obj == null) ? null : obj.toString();
    … …  
    464475                lastValue.put(key, value);
    465476            }
    466             cmds.add(new ChangePropertyCommand(sel, key, value));
     477            changedTags.add(new Tag(key, value));
    467478        }
    468479        @Override boolean requestFocusInWindow() {return combo.requestFocusInWindow();}
    … …  
    485496            return false;
    486497        }
    487         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {}
     498        @Override public void addCommands(List<Tag> changedTags) {}
    488499    }
    489500
    … …  
    514525            return false;
    515526        }
    516         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {}
     527        @Override public void addCommands(List<Tag> changedTags) {}
    517528    }
    518529
    … …  
    603614            return false;
    604615        }
    605         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {}
     616        @Override public void addCommands(List<Tag> changedTags) {}
    606617    }
    607618
    … …  
    614625            return false;
    615626        }
    616         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {}
     627        @Override public void addCommands(List<Tag> changedTags) {}
    617628    }
    618629
    … …  
    622633            return false;
    623634        }
    624         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {}
     635        @Override public void addCommands(List<Tag> changedTags) {}
    625636    }
    626637
    … …  
    630641
    631642        @Override public boolean addToPanel(JPanel p, Collection<OsmPrimitive> sel) { return false; }
    632         @Override public void addCommands(Collection<OsmPrimitive> sel, List<Command> cmds) {
    633             cmds.add(new ChangePropertyCommand(sel, key, value != null && !value.equals("") ? value : null));
     643        @Override public void addCommands(List<Tag> changedTags) {
     644            changedTags.add(new Tag(key, value != null && !value.equals("") ? value : null));
    634645        }
    635646    }
    … …  
    885896        }
    886897        p.add(items, GBC.eol().fill());
    887         if (selected.size() == 0) {
     898        if (selected.size() == 0 && !supportsRelation()) {
    888899            setEnabledRec(items, false);
    889900        }
    … …  
    924935        if (Main.main == null) return;
    925936        if (Main.main.getCurrentDataSet() == null) return;
    926         Collection<OsmPrimitive> sel = createSelection(Main.main.getCurrentDataSet().getSelected());
     937
     938        Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
     939        int answer = showDialog(sel, supportsRelation());
     940
     941        if (sel.size() != 0 && answer == DIALOG_ANSWER_APPLY) {
     942            Command cmd = createCommand(sel, getChangedTags());
     943            if (cmd != null) {
     944                Main.main.undoRedo.add(cmd);
     945            }
     946        } else if (answer == DIALOG_ANSWER_NEW_RELATION) {
     947            List<Command> cmds = new ArrayList<Command>(2);
     948            final Relation r = new Relation();
     949            cmds.add(new AddCommand(r));
     950            Command cmd = createCommand(Collections.<OsmPrimitive>singletonList(r), getChangedTags());
     951            if (cmd != null) {
     952                cmds.add(cmd);
     953            }
     954            Main.main.undoRedo.add(new SequenceCommand(tr("Add relation"), cmds));
     955            SwingUtilities.invokeLater(new Runnable() {
     956                @Override
     957                public void run() {
     958                    // Relation list dialog has to be updated first for selectRelation to work
     959                    Main.map.relationListDialog.selectRelation(r);
     960                }
     961            });
     962        }
     963        Main.main.getCurrentDataSet().setSelected(Main.main.getCurrentDataSet().getSelected()); // force update
     964
     965    }
     966
     967    public int showDialog(Collection<OsmPrimitive> selection, final boolean showNewRelation) {
     968        Collection<OsmPrimitive> sel = createSelection(selection);
    927969        PresetPanel p = createPanel(sel);
    928970        if (p == null)
    929             return;
     971            return DIALOG_ANSWER_CANCEL;
    930972
    931973        int answer = 1;
    … …  
    944986                    super(Main.parent,
    945987                            title,
    946                             new String[] { tr("Apply Preset"), tr("Cancel") },
    947                             true);
     988                            showNewRelation?
     989                                    new String[] { tr("Apply Preset"), tr("New relation"), tr("Cancel") }:
     990                                        new String[] { tr("Apply Preset"), tr("Cancel") },
     991                                        true);
    948992                    contentInsets = new Insets(10,5,0,5);
    949                     setButtonIcons(new String[] {"ok.png", "cancel.png" });
     993                    if (showNewRelation) {
     994                        setButtonIcons(new String[] {"ok.png", "dialogs/addrelation.png", "cancel.png" });
     995                    } else {
     996                        setButtonIcons(new String[] {"ok.png", "cancel.png" });
     997                    }
    950998                    setContent(content);
    951999                    setDefaultButton(1);
    … …  
    9591007            answer = new PresetDialog(p, title, (sel.size() == 0)).getValue();
    9601008        }
    961         if (sel.size() != 0 && answer == 1) {
    962             Command cmd = createCommand(sel);
    963             if (cmd != null) {
    964                 Main.main.undoRedo.add(cmd);
    965             }
    966         }
    967         Main.main.getCurrentDataSet().setSelected(Main.main.getCurrentDataSet().getSelected()); // force update
     1009        if (!showNewRelation && answer == 2)
     1010            return DIALOG_ANSWER_CANCEL;
     1011        else
     1012            return answer;
    9681013    }
    9691014
    … …  
    10101055    }
    10111056
    1012     private Command createCommand(Collection<OsmPrimitive> sel) {
    1013         List<Command> cmds = new LinkedList<Command>();
    1014         for (Item i : data) {
    1015             i.addCommands(sel, cmds);
    1016         }
     1057    public List<Tag> getChangedTags() {
     1058        List<Tag> result = new ArrayList<Tag>();
     1059        for (Item i: data) {
     1060            i.addCommands(result);
     1061        }
     1062        return result;
     1063    }
     1064
     1065    public static Command createCommand(Collection<OsmPrimitive> sel, List<Tag> changedTags) {
     1066        List<Command> cmds = new ArrayList<Command>();
     1067        for (Tag tag: changedTags) {
     1068            if (!tag.getValue().isEmpty()) {
     1069                cmds.add(new ChangePropertyCommand(sel, tag.getKey(), tag.getValue()));
     1070            }
     1071        }
     1072
    10171073        if (cmds.size() == 0)
    10181074            return null;
    … …  
    10231079    }
    10241080
     1081    private boolean supportsRelation() {
     1082        return types == null || types.contains(PresetType.RELATION);
     1083    }
     1084
    10251085    protected void updateEnabledState() {
    10261086        setEnabled(Main.main != null && Main.main.getCurrentDataSet() != null);
    … …  
    10381098        updateEnabledState();
    10391099    }
     1100
     1101    @Override
     1102    public String toString() {
     1103        return (types == null?"":types) + " " + name;
     1104    }
    10401105}

  • trunk/src/org/openstreetmap/josm/gui/tagging/tagging-preset.xsd

    r3479 r3518  
    105105                <attribute name="delete_if_empty" type="boolean" />
    106106                <attribute name="use_last_as_default" type="boolean" />
     107                <attribute name="required" type="boolean"/>
    107108
    108109                <attribute name="type" use="prohibited"/>
    … …  
    121122                <attribute name="delete_if_empty" type="boolean" />
    122123                <attribute name="display_values" type="string"/>
     124                <attribute name="required" type="boolean"/>
    123125
    124126                <attribute name="type" use="prohibited"/>
    … …  
    136138                <attribute name="delete_if_empty" type="boolean" />
    137139                <attribute name="use_last_as_default" type="boolean" />
     140                <attribute name="required" type="boolean"/>
    138141
    139142                <attribute name="name" use="prohibited"/>
Note: See TracChangeset for help on using the changeset viewer.