Ignore:
Timestamp:
2016-09-28T02:01:20+02:00 (8 years ago)
Author:
donvip
Message:

don't reinvent the wheel: replace KeyValuePair by Tag class

Location:
applications/editors/josm/plugins/tageditor/src/org/openstreetmap/josm/plugins/tageditor
Files:
1 deleted
6 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/tageditor/src/org/openstreetmap/josm/plugins/tageditor/TagEditorDialog.java

    r32959 r33016  
    3030import org.openstreetmap.josm.data.osm.DataSet;
    3131import org.openstreetmap.josm.data.osm.OsmPrimitive;
     32import org.openstreetmap.josm.data.osm.Tag;
    3233import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionList;
    3334import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionManager;
    … …  
    3839import org.openstreetmap.josm.plugins.tageditor.preset.ui.IPresetSelectorListener;
    3940import org.openstreetmap.josm.plugins.tageditor.preset.ui.TabularPresetSelector;
    40 import org.openstreetmap.josm.plugins.tageditor.tagspec.KeyValuePair;
    4141import org.openstreetmap.josm.plugins.tageditor.tagspec.ui.ITagSelectorListener;
    4242import org.openstreetmap.josm.plugins.tageditor.tagspec.ui.TabularTagSelector;
    … …  
    176176                new ITagSelectorListener() {
    177177                    @Override
    178                     public void itemSelected(KeyValuePair pair) {
     178                    public void itemSelected(Tag pair) {
    179179                        tagEditor.stopEditing();
    180180                        tagEditor.getModel().applyKeyValuePair(pair);

  • applications/editors/josm/plugins/tageditor/src/org/openstreetmap/josm/plugins/tageditor/editor/TagEditorModel.java

    r32959 r33016  
    1414import org.openstreetmap.josm.command.SequenceCommand;
    1515import org.openstreetmap.josm.data.osm.OsmPrimitive;
     16import org.openstreetmap.josm.data.osm.Tag;
    1617import org.openstreetmap.josm.gui.tagging.TagModel;
    1718import org.openstreetmap.josm.gui.tagging.presets.TaggingPreset;
    1819import org.openstreetmap.josm.plugins.tageditor.preset.AdvancedTag;
    19 import org.openstreetmap.josm.plugins.tageditor.tagspec.KeyValuePair;
    2020
    2121@SuppressWarnings("serial")
    … …  
    8484
    8585    /**
    86      * applies a tag given by a {@see KeyValuePair} to the model
     86     * applies a tag given by a {@see Tag} to the model
    8787     *
    8888     * @param pair the key value pair
    8989     */
    90     public void applyKeyValuePair(KeyValuePair pair) {
     90    public void applyKeyValuePair(Tag pair) {
    9191        TagModel tagModel = get(pair.getKey());
    9292        if (tagModel == null) {

  • applications/editors/josm/plugins/tageditor/src/org/openstreetmap/josm/plugins/tageditor/tagspec/TagSpecifications.java

    r32959 r33016  
    1212import java.util.logging.Logger;
    1313
     14import org.openstreetmap.josm.data.osm.Tag;
    1415import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionItemPriority;
    1516import org.openstreetmap.josm.gui.tagging.ac.AutoCompletionListItem;
    … …  
    163164     * @return the list
    164165     */
    165     public ArrayList<KeyValuePair> asList() {
    166         ArrayList<KeyValuePair> entries = new ArrayList<>();
     166    public ArrayList<Tag> asList() {
     167        ArrayList<Tag> entries = new ArrayList<>();
    167168
    168169        for (TagSpecification s : tagSpecifications) {
    169170            for (LabelSpecification l : s.getLables()) {
    170                 entries.add(new KeyValuePair(s.getKey(), l.getValue()));
     171                entries.add(new Tag(s.getKey(), l.getValue()));
    171172            }
    172173        }

  • applications/editors/josm/plugins/tageditor/src/org/openstreetmap/josm/plugins/tageditor/tagspec/ui/ITagSelectorListener.java

    r32959 r33016  
    22package org.openstreetmap.josm.plugins.tageditor.tagspec.ui;
    33
    4 import org.openstreetmap.josm.plugins.tageditor.tagspec.KeyValuePair;
     4import org.openstreetmap.josm.data.osm.Tag;
    55
    66public interface ITagSelectorListener {
    7     void itemSelected(KeyValuePair pair);
     7    void itemSelected(Tag pair);
    88}

  • applications/editors/josm/plugins/tageditor/src/org/openstreetmap/josm/plugins/tageditor/tagspec/ui/TabularTagSelector.java

    r32959 r33016  
    3030import javax.swing.event.ListSelectionListener;
    3131
    32 import org.openstreetmap.josm.plugins.tageditor.tagspec.KeyValuePair;
     32import org.openstreetmap.josm.data.osm.Tag;
    3333
    3434public class TabularTagSelector extends JPanel {
    … …  
    9797                int rowNum = tagsTable.getSelectedRow();
    9898                if (rowNum >= 0) {
    99                     KeyValuePair item = getModel().getVisibleItem(rowNum);
     99                    Tag item = getModel().getVisibleItem(rowNum);
    100100                    fireItemSelected(item);
    101101                }
    … …  
    124124                        int row = tagsTable.getSelectedRow();
    125125                        if (row >= 0) {
    126                             KeyValuePair item = getModel().getVisibleItem(row);
     126                            Tag item = getModel().getVisibleItem(row);
    127127                            fireItemSelected(item);
    128128                        }
    … …  
    236236    }
    237237
    238     protected void fireItemSelected(KeyValuePair pair) {
     238    protected void fireItemSelected(Tag pair) {
    239239        synchronized (this.listeners) {
    240240            for (ITagSelectorListener listener: listeners) {
    … …  
    249249            if (e.getClickCount() == 2) {
    250250                int rowNum = tagsTable.rowAtPoint(e.getPoint());
    251                 KeyValuePair pair = getModel().getVisibleItem(rowNum);
     251                Tag pair = getModel().getVisibleItem(rowNum);
    252252                fireItemSelected(pair);
    253253            }

  • applications/editors/josm/plugins/tageditor/src/org/openstreetmap/josm/plugins/tageditor/tagspec/ui/TagsTableModel.java

    r32959 r33016  
    1212import javax.swing.table.AbstractTableModel;
    1313
    14 import org.openstreetmap.josm.plugins.tageditor.tagspec.KeyValuePair;
     14import org.openstreetmap.josm.data.osm.Tag;
    1515import org.openstreetmap.josm.plugins.tageditor.tagspec.TagSpecifications;
    1616
    … …  
    1919    private static Logger logger = Logger.getLogger(TagsTableModel.class.getName());
    2020
    21     private ArrayList<KeyValuePair> items = null;
    22     private ArrayList<KeyValuePair> visibleItems = null;
     21    private ArrayList<Tag> items = null;
     22    private ArrayList<Tag> visibleItems = null;
    2323
    2424    public TagsTableModel() {
    … …  
    3030        Collections.sort(
    3131                items,
    32                 new Comparator<KeyValuePair>() {
     32                new Comparator<Tag>() {
    3333                    @Override
    34                     public int compare(KeyValuePair self,
    35                             KeyValuePair other) {
     34                    public int compare(Tag self, Tag other) {
    3635                        int ret = self.getKey().compareToIgnoreCase(other.getKey());
    3736
    … …  
    6362        items = spec.asList();
    6463        sort();
    65         for (KeyValuePair item : items) {
     64        for (Tag item : items) {
    6665            visibleItems.add(item);
    6766        }
    … …  
    8079    @Override
    8180    public Object getValueAt(int row, int col) {
    82         KeyValuePair pair = visibleItems.get(row);
     81        Tag pair = visibleItems.get(row);
    8382        switch(col) {
    8483        case 0: return pair.getKey();
    … …  
    9493            if (filter == null || filter.trim().equals("")) {
    9594                visibleItems.clear();
    96                 for (KeyValuePair pair: items) {
     95                for (Tag pair: items) {
    9796                    visibleItems.add(pair);
    9897                }
    … …  
    10099                visibleItems.clear();
    101100                filter = filter.toLowerCase();
    102                 for (KeyValuePair pair: items) {
     101                for (Tag pair: items) {
    103102                    if (pair.getKey().toLowerCase().trim().startsWith(filter)
    104103                            || pair.getValue().toLowerCase().trim().startsWith(filter)) {
    … …  
    117116    }
    118117
    119     public KeyValuePair getVisibleItem(int row) {
     118    public Tag getVisibleItem(int row) {
    120119        if (row < 0 || row >= visibleItems.size())
    121120            throw new IndexOutOfBoundsException("row is out of bound: row=" + row);
Note: See TracChangeset for help on using the changeset viewer.