Changeset 2620 in josm


Ignore:
Timestamp:
Dec 12, 2009 5:10:01 PM (4 years ago)
Author:
jttt
Message:

Remove OsmPrimitive.setIncomplete()

Location:
trunk
Files:
14 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r2618 r2620  
    1919import java.util.Collections; 
    2020import java.util.LinkedList; 
     21import java.util.List; 
    2122import java.util.Map; 
    2223import java.util.Properties; 
    … …  
    6263 
    6364    private static class DefaultPreferenceChangeEvent implements PreferenceChangeEvent { 
    64         private String key; 
    65         private String oldValue; 
    66         private String newValue; 
     65        private final String key; 
     66        private final String oldValue; 
     67        private final String newValue; 
    6768 
    6869        public DefaultPreferenceChangeEvent(String key, String oldValue, String newValue) { 
    … …  
    7576            return key; 
    7677        } 
    77         public void setKey(String key) { 
    78             this.key = key; 
    79         } 
    8078        public String getOldValue() { 
    8179            return oldValue; 
    8280        } 
    83         public void setOldValue(String oldValue) { 
    84             this.oldValue = oldValue; 
    85         } 
    8681        public String getNewValue() { 
    8782            return newValue; 
    88         } 
    89         public void setNewValue(String newValue) { 
    90             this.newValue = newValue; 
    9183        } 
    9284    } 
    … …  
    713705    } 
    714706    synchronized public void removeFromCollection(String key, String value) { 
    715         ArrayList<String> a = new ArrayList<String>(getCollection(key, null)); 
    716         if(a != null) 
    717         { 
    718             a.remove(value); 
    719             putCollection(key, a); 
    720         } 
     707        List<String> a = new ArrayList<String>(getCollection(key, null)); 
     708        a.remove(value); 
     709        putCollection(key, a); 
    721710    } 
    722711    synchronized public boolean putCollection(String key, Collection<String> val) { 

  • trunk/src/org/openstreetmap/josm/data/osm/Filters.java

    r2180 r2620  
    44import static org.openstreetmap.josm.tools.I18n.trc; 
    55 
    6 import javax.swing.table.AbstractTableModel; 
    7  
     6import java.util.Collection; 
    87import java.util.LinkedList; 
    98import java.util.List; 
    10 import java.util.Collection; 
    119import java.util.Map; 
    1210 
    13 import org.openstreetmap.josm.data.osm.Filter; 
     11import javax.swing.table.AbstractTableModel; 
     12 
    1413import org.openstreetmap.josm.Main; 
     14import org.openstreetmap.josm.actions.search.SearchAction; 
    1515import org.openstreetmap.josm.actions.search.SearchAction.Function; 
    16 import org.openstreetmap.josm.actions.search.SearchAction; 
    1716 
    1817/** 
    … …  
    2221public class Filters extends AbstractTableModel{ 
    2322 
    24    public int disabledCount, hiddenCount; 
    25  
    26    public Filters(){ 
    27       loadPrefs(); 
    28    } 
    29  
    30    private List<Filter> filters = new LinkedList<Filter>(); 
    31    public void filter(){ 
    32       Collection<OsmPrimitive> seld = new LinkedList<OsmPrimitive> (); 
    33       Collection<OsmPrimitive> self = new LinkedList<OsmPrimitive> (); 
    34       if(Main.main.getCurrentDataSet() == null)return; 
    35       Main.main.getCurrentDataSet().setFiltered(); 
    36       Main.main.getCurrentDataSet().setDisabled(); 
    37       for (Filter flt : filters){ 
     23    public int disabledCount, hiddenCount; 
     24 
     25    public Filters(){ 
     26        loadPrefs(); 
     27    } 
     28 
     29    private List<Filter> filters = new LinkedList<Filter>(); 
     30    public void filter(){ 
     31        Collection<OsmPrimitive> seld = new LinkedList<OsmPrimitive> (); 
     32        Collection<OsmPrimitive> self = new LinkedList<OsmPrimitive> (); 
     33        if(Main.main.getCurrentDataSet() == null)return; 
     34        Main.main.getCurrentDataSet().setFiltered(); 
     35        Main.main.getCurrentDataSet().setDisabled(); 
     36        for (Filter flt : filters){ 
    3837            if(flt.enable){ 
    39                SearchAction.getSelection(flt, seld, new Function(){ 
    40                   public Boolean isSomething(OsmPrimitive o){ 
    41                      return o.isDisabled(); 
    42                   } 
    43                }); 
    44                if(flt.hide) { 
    45                   SearchAction.getSelection(flt, self, new Function(){ 
    46                      public Boolean isSomething(OsmPrimitive o){ 
    47                         return o.isFiltered(); 
    48                      } 
    49                   }); 
    50                } 
     38                SearchAction.getSelection(flt, seld, new Function(){ 
     39                    public Boolean isSomething(OsmPrimitive o){ 
     40                        return o.isDisabled(); 
     41                    } 
     42                }); 
     43                if(flt.hide) { 
     44                    SearchAction.getSelection(flt, self, new Function(){ 
     45                        public Boolean isSomething(OsmPrimitive o){ 
     46                            return o.isFiltered(); 
     47                        } 
     48                    }); 
     49                } 
    5150            } 
    52       } 
    53       disabledCount = seld.size() - self.size(); 
    54       hiddenCount = self.size(); 
    55       Main.main.getCurrentDataSet().setFiltered(self); 
    56       Main.main.getCurrentDataSet().setDisabled(seld); 
    57       Main.map.mapView.repaint(); 
    58    } 
    59  
    60    private void loadPrefs(){ 
    61       Map<String,String> prefs = Main.pref.getAllPrefix("filters.filter"); 
    62       for (String value : prefs.values()) { 
    63          Filter filter = new Filter(value); 
    64          if(filter!=null) 
    65             filters.add(filter); 
    66       } 
    67    } 
    68  
    69    private void savePrefs(){ 
    70       Map<String,String> prefs = Main.pref.getAllPrefix("filters.filter"); 
    71       for (String key : prefs.keySet()) { 
    72          String[] sts = key.split("\\."); 
    73          if (sts.length != 3)throw new Error("Incompatible filter preferences"); 
    74          Main.pref.put("filters.filter." + sts[2], null); 
    75       } 
    76  
    77       int i = 0; 
    78       for (Filter flt : filters){ 
    79          Main.pref.put("filters.filter." + i++, flt.getPrefString()); 
    80       } 
    81    } 
    82  
    83    private void savePref(int i){ 
    84       if(i >= filters.size()) 
    85          Main.pref.put("filters.filter." + i, null); 
    86       else 
    87          Main.pref.put("filters.filter." + i, filters.get(i).getPrefString()); 
    88    } 
    89  
    90    public void addFilter(Filter f){ 
    91       filters.add(f); 
    92       savePref(filters.size()-1); 
    93       filter(); 
    94       fireTableRowsInserted(filters.size()-1, filters.size()-1); 
    95    } 
    96  
    97    public void moveDownFilter(int i){ 
    98       if(i >= filters.size()-1) return; 
    99       filters.add(i+1, filters.remove(i)); 
    100       savePref(i); 
    101       savePref(i+1); 
    102       filter(); 
    103       fireTableRowsUpdated(i, i+1); 
    104    } 
    105  
    106    public void moveUpFilter(int i){ 
    107       if(i == 0) return; 
    108       filters.add(i-1, filters.remove(i)); 
    109       savePref(i); 
    110       savePref(i-1); 
    111       filter(); 
    112       fireTableRowsUpdated(i-1, i); 
    113    } 
    114  
    115    public void removeFilter(int i){ 
    116       filters.remove(i); 
    117       savePrefs(); 
    118       filter(); 
    119       fireTableRowsDeleted(i, i); 
    120    } 
    121  
    122    public void setFilter(int i, Filter f){ 
    123       filters.set(i, f); 
    124       savePref(i); 
    125       filter(); 
    126       fireTableRowsUpdated(i, i); 
    127    } 
    128  
    129    public Filter getFilter(int i){ 
    130       return filters.get(i); 
    131    } 
    132  
    133    public int getRowCount(){ 
    134       return filters.size(); 
    135    } 
    136  
    137    public int getColumnCount(){ 
    138       return 6; 
    139    } 
    140  
    141    public String getColumnName(int column){ 
    142       String[] names = { /* translators notes must be in front */ 
    143           /* column header: enable filter */             trc("filter","E"), 
    144           /* column header: hide filter */               tr("H"), 
    145           /* column header: filter text */               tr("Text"), 
    146           /* column header: apply filter for children */ tr("C"), 
    147           /* column header: inverted filter */           tr("I"), 
    148           /* column header: filter mode */               tr("M") 
    149       }; 
    150       return names[column]; 
    151    } 
    152  
    153    public Class getColumnClass(int column){ 
    154       Class[] classes = { Boolean.class, Boolean.class, String.class, Boolean.class, Boolean.class, String.class }; 
    155       return classes[column]; 
    156    } 
    157  
    158    public boolean isCellEnabled(int row, int column){ 
    159       if(!filters.get(row).enable && column!=0) return false; 
    160       return true; 
    161    } 
    162  
    163    public boolean isCellEditable(int row, int column){ 
    164       if(!filters.get(row).enable && column!=0) return false; 
    165       if(column < 5)return true; 
    166       return false; 
    167    } 
    168  
    169    public void setValueAt(Object aValue, int row, int column){ 
    170       Filter f = filters.get(row); 
    171       switch(column){ 
    172          case 0: f.enable = (Boolean)aValue; 
    173                  savePref(row); 
    174                  filter(); 
    175                  fireTableRowsUpdated(row, row); 
    176                  break; 
    177          case 1: f.hide = (Boolean)aValue; 
    178                  savePref(row); 
    179                  filter(); 
    180                  break; 
    181          case 2: f.text = (String)aValue; 
    182                  savePref(row); 
    183                  break; 
    184          case 3: f.applyForChildren = (Boolean)aValue; 
    185                  savePref(row); 
    186                  filter(); 
    187                  break; 
    188          case 4: f.inverted = (Boolean)aValue; 
    189                  savePref(row); 
    190                  filter(); 
    191                  break; 
    192       } 
    193       if(column!=0)fireTableCellUpdated(row, column); 
    194    } 
    195  
    196    public Object getValueAt(int row, int column){ 
    197       Filter f = filters.get(row); 
    198       switch(column){ 
    199          case 0: return f.enable; 
    200          case 1: return f.hide; 
    201          case 2: return f.text; 
    202          case 3: return f.applyForChildren; 
    203          case 4: return f.inverted; 
    204          case 5: 
    205                  switch(f.mode){ /* translators notes must be in front */ 
    206                     case replace:      /* filter mode: replace */      return tr("R"); 
    207                     case add:          /* filter mode: add */          return tr("A"); 
    208                     case remove:       /* filter mode: remove */       return tr("D"); 
    209                     case in_selection: /* filter mode: in selection */ return tr("F"); 
    210                  } 
    211       } 
    212       return null; 
    213    } 
     51        } 
     52        disabledCount = seld.size() - self.size(); 
     53        hiddenCount = self.size(); 
     54        Main.main.getCurrentDataSet().setFiltered(self); 
     55        Main.main.getCurrentDataSet().setDisabled(seld); 
     56        Main.map.mapView.repaint(); 
     57    } 
     58 
     59    private void loadPrefs(){ 
     60        Map<String,String> prefs = Main.pref.getAllPrefix("filters.filter"); 
     61        for (String value : prefs.values()) { 
     62            filters.add(new Filter(value)); 
     63        } 
     64    } 
     65 
     66    private void savePrefs(){ 
     67        Map<String,String> prefs = Main.pref.getAllPrefix("filters.filter"); 
     68        for (String key : prefs.keySet()) { 
     69            String[] sts = key.split("\\."); 
     70            if (sts.length != 3)throw new Error("Incompatible filter preferences"); 
     71            Main.pref.put("filters.filter." + sts[2], null); 
     72        } 
     73 
     74        int i = 0; 
     75        for (Filter flt : filters){ 
     76            Main.pref.put("filters.filter." + i++, flt.getPrefString()); 
     77        } 
     78    } 
     79 
     80    private void savePref(int i){ 
     81        if(i >= filters.size()) { 
     82            Main.pref.put("filters.filter." + i, null); 
     83        } else { 
     84            Main.pref.put("filters.filter." + i, filters.get(i).getPrefString()); 
     85        } 
     86    } 
     87 
     88    public void addFilter(Filter f){ 
     89        filters.add(f); 
     90        savePref(filters.size()-1); 
     91        filter(); 
     92        fireTableRowsInserted(filters.size()-1, filters.size()-1); 
     93    } 
     94 
     95    public void moveDownFilter(int i){ 
     96        if(i >= filters.size()-1) return; 
     97        filters.add(i+1, filters.remove(i)); 
     98        savePref(i); 
     99        savePref(i+1); 
     100        filter(); 
     101        fireTableRowsUpdated(i, i+1); 
     102    } 
     103 
     104    public void moveUpFilter(int i){ 
     105        if(i == 0) return; 
     106        filters.add(i-1, filters.remove(i)); 
     107        savePref(i); 
     108        savePref(i-1); 
     109        filter(); 
     110        fireTableRowsUpdated(i-1, i); 
     111    } 
     112 
     113    public void removeFilter(int i){ 
     114        filters.remove(i); 
     115        savePrefs(); 
     116        filter(); 
     117        fireTableRowsDeleted(i, i); 
     118    } 
     119 
     120    public void setFilter(int i, Filter f){ 
     121        filters.set(i, f); 
     122        savePref(i); 
     123        filter(); 
     124        fireTableRowsUpdated(i, i); 
     125    } 
     126 
     127    public Filter getFilter(int i){ 
     128        return filters.get(i); 
     129    } 
     130 
     131    public int getRowCount(){ 
     132        return filters.size(); 
     133    } 
     134 
     135    public int getColumnCount(){ 
     136        return 6; 
     137    } 
     138 
     139    @Override 
     140    public String getColumnName(int column){ 
     141        String[] names = { /* translators notes must be in front */ 
     142                /* column header: enable filter */             trc("filter","E"), 
     143                /* column header: hide filter */               tr("H"), 
     144                /* column header: filter text */               tr("Text"), 
     145                /* column header: apply filter for children */ tr("C"), 
     146                /* column header: inverted filter */           tr("I"), 
     147                /* column header: filter mode */               tr("M") 
     148        }; 
     149        return names[column]; 
     150    } 
     151 
     152    @Override 
     153    public Class<?> getColumnClass(int column){ 
     154        Class<?>[] classes = { Boolean.class, Boolean.class, String.class, Boolean.class, Boolean.class, String.class }; 
     155        return classes[column]; 
     156    } 
     157 
     158    public boolean isCellEnabled(int row, int column){ 
     159        if(!filters.get(row).enable && column!=0) return false; 
     160        return true; 
     161    } 
     162 
     163    @Override 
     164    public boolean isCellEditable(int row, int column){ 
     165        if(!filters.get(row).enable && column!=0) return false; 
     166        if(column < 5)return true; 
     167        return false; 
     168    } 
     169 
     170    @Override 
     171    public void setValueAt(Object aValue, int row, int column){ 
     172        Filter f = filters.get(row); 
     173        switch(column){ 
     174        case 0: f.enable = (Boolean)aValue; 
     175        savePref(row); 
     176        filter(); 
     177        fireTableRowsUpdated(row, row); 
     178        break; 
     179        case 1: f.hide = (Boolean)aValue; 
     180        savePref(row); 
     181        filter(); 
     182        break; 
     183        case 2: f.text = (String)aValue; 
     184        savePref(row); 
     185        break; 
     186        case 3: f.applyForChildren = (Boolean)aValue; 
     187        savePref(row); 
     188        filter(); 
     189        break; 
     190        case 4: f.inverted = (Boolean)aValue; 
     191        savePref(row); 
     192        filter(); 
     193        break; 
     194        } 
     195        if(column!=0) { 
     196            fireTableCellUpdated(row, column); 
     197        } 
     198    } 
     199 
     200    public Object getValueAt(int row, int column){ 
     201        Filter f = filters.get(row); 
     202        switch(column){ 
     203        case 0: return f.enable; 
     204        case 1: return f.hide; 
     205        case 2: return f.text; 
     206        case 3: return f.applyForChildren; 
     207        case 4: return f.inverted; 
     208        case 5: 
     209            switch(f.mode){ /* translators notes must be in front */ 
     210            case replace:      /* filter mode: replace */      return tr("R"); 
     211            case add:          /* filter mode: add */          return tr("A"); 
     212            case remove:       /* filter mode: remove */       return tr("D"); 
     213            case in_selection: /* filter mode: in selection */ return tr("F"); 
     214            } 
     215        } 
     216        return null; 
     217    } 
    214218} 

  • trunk/src/org/openstreetmap/josm/data/osm/Node.java

    r2612 r2620  
    6868    public Node(long id) { 
    6969        super(id, false); 
     70    } 
     71 
     72    /** 
     73     * Create new complete node 
     74     * @param id 
     75     * @param version 
     76     */ 
     77    public Node(long id, int version) { 
     78        super(id, false); 
     79        setOsmId(id, version); 
    7080    } 
    7181 

  • trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

    r2615 r2620  
    245245 
    246246    /** 
    247      * Creates a new primitive for the given id. If the id > 0, the primitive is marked 
    248      * as incomplete. 
    249      * 
    250      * @param id the id. > 0 required 
    251      * @param allowNegativeId Allows to set negative id. For internal use 
     247     * Creates a new primitive for the given id. 
     248     *  
     249     * If allowNegativeId is set, provided id can be < 0 and will be set to primitive without any processing. 
     250     * If allowNegativeId is not set, then id will have to be 0 (in that case new unique id will be generated) or 
     251     * positive number. 
     252     * 
     253     * @param id the id 
     254     * @param allowNegativeId 
    252255     * @throws IllegalArgumentException thrown if id < 0 and allowNegativeId is false 
    253256     */ 
    … …  
    12061209    } 
    12071210 
    1208     //TODO This method should not be necessary, incomplete state should be handled internally by OsmPrimitive 
    1209     public void setIncomplete(boolean incomplete) { 
     1211    private void setIncomplete(boolean incomplete) { 
    12101212        this.incomplete = incomplete; 
    12111213    } 

  • trunk/src/org/openstreetmap/josm/data/osm/Relation.java

    r2579 r2620  
    169169    public Relation(long id) throws IllegalArgumentException { 
    170170        super(id, false); 
     171    } 
     172 
     173    /** 
     174     * Creates new complete relation 
     175     * @param id 
     176     * @param version 
     177     */ 
     178    public Relation(long id, int version) { 
     179        super(id, false); 
     180        setOsmId(id, version); 
    171181    } 
    172182 

  • trunk/src/org/openstreetmap/josm/data/osm/Way.java

    r2609 r2620  
    173173    /** 
    174174     * Creates a new way for the given id. If the id > 0, the way is marked 
    175      * as incomplete. 
    176      * 
    177      * @param id the id. > 0 required 
     175     * as incomplete. If id == 0 then way is marked as new 
     176     * 
     177     * @param id the id. >= 0 required 
    178178     * @throws IllegalArgumentException thrown if id < 0 
    179179     */ 
    180180    public Way(long id) throws IllegalArgumentException { 
    181181        super(id, false); 
     182    } 
     183 
     184    /** 
     185     * Creates new way with given id and version. Way is marked as complete 
     186     * @param id 
     187     * @param version 
     188     */ 
     189    public Way(long id, int version) { 
     190        super(id, false); 
     191        setOsmId(id, version); 
    182192    } 
    183193 

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitor.java

    r2578 r2620  
    4848    } 
    4949 
     50    protected boolean isInSelectionBase(OsmPrimitive primitive) { 
     51        return selectionBase.getSelected().contains(primitive); 
     52    } 
     53 
     54    protected boolean isAlreadyRemembered(OsmPrimitive primitive) { 
     55        return mappedPrimitives.keySet().contains(primitive); 
     56    } 
     57 
    5058    /** 
    5159     * Remebers a node in the "hull" 
    … …  
    6977            return; 
    7078        Way clone = new Way(w); 
    71         List<Node> newNodes = new ArrayList<Node>(); 
     79        List<Node> newNodes = new ArrayList<Node>(w.getNodesCount()); 
    7280        for (Node n: w.getNodes()) { 
    7381            newNodes.add((Node)mappedPrimitives.get(n)); 
    … …  
    8492    protected void rememberRelation(Relation r) { 
    8593        Relation clone; 
    86         if (mappedPrimitives.keySet().contains(r)) { 
     94        if (isAlreadyRemembered(r)) { 
    8795            clone = (Relation)mappedPrimitives.get(r); 
    8896            clone.cloneFrom(r); 
    … …  
    99107        clone.setMembers(newMembers); 
    100108 
    101         if (! mappedPrimitives.keySet().contains(r)) { 
     109        if (!isAlreadyRemembered(r)) { 
    102110            mappedPrimitives.put(r, clone); 
    103111        } 
    … …  
    127135 
    128136    protected void rememberNodeIncomplete(Node n) { 
    129         if (isAlreadyRemembered(n)) 
    130             return; 
    131         Node clone = new Node(n); 
    132         clone.setIncomplete(true); 
    133         mappedPrimitives.put(n, clone); 
     137        if (!isAlreadyRemembered(n)) { 
     138            mappedPrimitives.put(n, new Node(n.getId())); 
     139        } 
    134140    } 
    135141 
    136142    protected void rememberWayIncomplete(Way w) { 
    137         if (isAlreadyRemembered(w)) 
    138             return; 
    139         Way clone = new Way(w); 
    140         clone.setNodes(null); 
    141         clone.setIncomplete(true); 
    142         mappedPrimitives.put(w, clone); 
     143        if (!isAlreadyRemembered(w)) { 
     144            mappedPrimitives.put(w, new Way(w.getId())); 
     145        } 
    143146    } 
    144147 
    145148    protected void rememberRelationIncomplete(Relation r) { 
    146         Relation clone; 
    147         if (isAlreadyRemembered(r)) { 
    148             clone = (Relation)mappedPrimitives.get(r); 
    149             clone.cloneFrom(r); 
    150         } else { 
    151             clone = new Relation(r); 
    152         } 
    153         clone.setMembers(null); 
    154         clone.setIncomplete(true); 
    155         if (! isAlreadyRemembered(r)) { 
    156             mappedPrimitives.put(r, clone); 
     149        if (!isAlreadyRemembered(r)) { 
     150            mappedPrimitives.put(r, new Relation(r.getId())); 
    157151        } 
    158152    } 
    … …  
    166160        // 
    167161        for (Node n: w.getNodes()) { 
    168             if (! isAlreadyRemembered(n)) { 
    169                 n.visit(this); 
    170             } 
     162            n.visit(this); 
    171163        } 
    172164        // ... and the way itself 
    173165        rememberWay(w); 
    174     } 
    175  
    176     protected boolean isInSelectionBase(OsmPrimitive primitive) { 
    177         return selectionBase.getSelected().contains(primitive); 
    178     } 
    179  
    180     protected boolean isAlreadyRemembered(OsmPrimitive primitive) { 
    181         return mappedPrimitives.keySet().contains(primitive); 
    182166    } 
    183167 

  • trunk/src/org/openstreetmap/josm/data/projection/NTV2GridShiftFile.java

    r2507 r2620  
    2020package org.openstreetmap.josm.data.projection; 
    2121 
     22import java.io.IOException; 
    2223import java.io.InputStream; 
    23 import java.io.IOException; 
    24 import java.io.RandomAccessFile; 
    2524import java.io.Serializable; 
    2625import java.util.ArrayList; 
    … …  
    6261public class NTV2GridShiftFile implements Serializable { 
    6362 
    64     private static final int REC_SIZE = 16; 
    6563    private String overviewHeaderCountId; 
    6664    private int overviewHeaderCount; 
    … …  
    270268    } 
    271269 
    272     public void unload() throws IOException { 
     270    public void unload() { 
    273271        topLevelSubGrid = null; 
    274272    } 

  • trunk/src/org/openstreetmap/josm/io/GpxWriter.java

    r2327 r2620  
    5050        out.println("<gpx version=\"1.1\" creator=\"JOSM GPX export\" xmlns=\"http://www.topografix.com/GPX/1/1\"\n" + 
    5151                "    xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\" \n" + 
    52                 "    xsi:schemaLocation=\"http://www.topografix.com/GPX/1/1 http://www.topografix.com/GPX/1/1/gpx.xsd\">"); 
     52        "    xsi:schemaLocation=\"http://www.topografix.com/GPX/1/1 http://www.topografix.com/GPX/1/1/gpx.xsd\">"); 
    5353        indent = "  "; 
    5454        writeMetaData(); 
    … …  
    6060    } 
    6161 
     62    @SuppressWarnings("unchecked") 
    6263    private void writeAttr(Map<String, Object> attr) { 
    6364        // FIXME this loop is evil, because it does not assure the 
    … …  
    6768            String k = ent.getKey(); 
    6869            if (k.equals(GpxData.META_LINKS)) { 
    69                 for (Object link : (Collection) ent.getValue()) { 
    70                     gpxLink((GpxLink) link); 
     70                for (GpxLink link : (Collection<GpxLink>) ent.getValue()) { 
     71                    gpxLink(link); 
    7172                } 
    7273            } else { 
    … …  
    7677    } 
    7778 
     79    @SuppressWarnings("unchecked") 
    7880    private void writeMetaData() { 
    7981        Map<String, Object> attr = data.attr; 
    … …  
    8183 
    8284        // write the description 
    83         if (attr.containsKey(GpxData.META_DESC)) simpleTag("desc", (String)attr.get(GpxData.META_DESC)); 
     85        if (attr.containsKey(GpxData.META_DESC)) { 
     86            simpleTag("desc", (String)attr.get(GpxData.META_DESC)); 
     87        } 
    8488 
    8589        // write the author details 
    … …  
    116120        // write links 
    117121        if(attr.containsKey(GpxData.META_LINKS)) { 
    118             for (Object link : (Collection) attr.get(GpxData.META_LINKS)) { 
    119                 gpxLink((GpxLink) link); 
     122            for (GpxLink link : (Collection<GpxLink>) attr.get(GpxData.META_LINKS)) { 
     123                gpxLink(link); 
    120124            } 
    121125        } 
    122126 
    123127        // write keywords 
    124         if (attr.containsKey(GpxData.META_KEYWORDS)) simpleTag("keywords", (String)attr.get(GpxData.META_KEYWORDS)); 
     128        if (attr.containsKey(GpxData.META_KEYWORDS)) { 
     129            simpleTag("keywords", (String)attr.get(GpxData.META_KEYWORDS)); 
     130        } 
    125131 
    126132        Bounds bounds = data.recalculateBounds(); 
    … …  
    128134        { 
    129135            String b = "minlat=\"" + bounds.getMin().lat() + "\" minlon=\"" + bounds.getMin().lon() + 
    130                 "\" maxlat=\"" + bounds.getMax().lat() + "\" maxlon=\"" + bounds.getMax().lon() + "\"" ; 
     136            "\" maxlat=\"" + bounds.getMax().lat() + "\" maxlon=\"" + bounds.getMax().lon() + "\"" ; 
    131137            inline("bounds", b); 
    132138        } 

  • trunk/src/org/openstreetmap/josm/plugins/PluginInformation.java

    r2512 r2620  
    2121 
    2222import org.openstreetmap.josm.Main; 
    23 import org.openstreetmap.josm.actions.AboutAction; 
    2423import org.openstreetmap.josm.data.Version; 
    2524import org.openstreetmap.josm.tools.LanguageInfo; 
    … …  
    125124        { 
    126125            int myv = Version.getInstance().getVersion(); 
    127             for(Map.Entry entry : attr.entrySet()) 
     126            for(Map.Entry<Object, Object> entry : attr.entrySet()) 
    128127            { 
    129128                try { 

  • trunk/test/unit/org/openstreetmap/josm/data/osm/APIDataSetTest.java

    r2578 r2620  
    7474        r3.put("name", "r3"); 
    7575 
    76         Relation r4 = new Relation(1); 
     76        Relation r4 = new Relation(1, 1); 
    7777        r4.put("name", "r4"); 
    78         r4.setIncomplete(false); 
    7978        r4.setModified(true); 
    8079 

  • trunk/test/unit/org/openstreetmap/josm/data/osm/DataSetMergerTest.java

    r2587 r2620  
    191191        DataSet my = new DataSet(); 
    192192        my.setVersion("0.6"); 
    193         Node n = new Node(1); 
     193        Node n = new Node(1, 1); 
    194194        n.setCoor(new LatLon(0,0)); 
    195         n.setIncomplete(false); 
    196195        n.setDeleted(true); 
    197196        n.put("key1", "value1"); 
    … …  
    201200        their.setVersion("0.6"); 
    202201        Node n1 = new Node(new LatLon(0,0)); 
    203         n1.setOsmId(1,1); 
     202        n1.setOsmId(1,2); 
    204203        n1.setModified(false); 
    205204        n1.put("key1", "value1-new"); 
    … …  
    437436        DataSet my = new DataSet(); 
    438437        my.setVersion("0.6"); 
    439         Node n = new Node(); 
    440         n.setCoor(new LatLon(0,0)); 
    441         n.setOsmId(1,1); 
    442         n.setIncomplete(true); 
     438        Node n = new Node(1); 
    443439        my.addPrimitive(n); 
    444440 
    … …  
    10571053        my.addPrimitive(n5); 
    10581054 
    1059         Way w6 = new Way(3); 
    1060         w6.setIncomplete(false); 
     1055        Way w6 = new Way(3, 1); 
    10611056        w6.setNodes(Arrays.asList(n4,n5)); 
    10621057        my.addPrimitive(w6); 
    … …  
    10971092 
    10981093        // an complete node 
    1099         Node n1 = new Node(1); 
     1094        Node n1 = new Node(1, 1); 
    11001095        n1.setCoor(new LatLon(1,1)); 
    1101         n1.setIncomplete(false); 
    11021096        source.addPrimitive(n1); 
    11031097 
    11041098        // another complete node 
    1105         Node n2 = new Node(2); 
     1099        Node n2 = new Node(2, 1); 
    11061100        n2.setCoor(new LatLon(2,2)); 
    1107         n2.setIncomplete(false); 
    11081101        source.addPrimitive(n2); 
    11091102 
    1110         // --- target daset 
     1103        // --- target dataset 
    11111104        DataSet target = new DataSet(); 
    11121105        target.setVersion("0.6"); 
    … …  
    11181111        target.addPrimitive(n5); 
    11191112 
    1120         Way w6 = new Way(3); 
    1121         w6.setIncomplete(false); 
     1113        Way w6 = new Way(3, 1); 
    11221114        w6.addNode(n4); 
    11231115        w6.addNode(n5); 
    1124         w6.setIncomplete(true); 
    11251116        target.addPrimitive(w6); 
    11261117 
    … …  
    11431134        assertNotNull(w); 
    11441135        assertFalse(w.hasIncompleteNodes()); 
    1145         assertFalse(w.isUsable()); 
     1136        assertTrue(w.isUsable()); 
    11461137        assertEquals(2, w.getNodesCount()); 
    11471138        assertEquals(1, w.getNode(0).getId()); 

  • trunk/test/unit/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitorTest.java

    r2578 r2620  
    7171        Node n1 = new Node(1); 
    7272        Node n2 = new Node(2); 
    73         Way w1 = new Way(3); 
    74         w1.setIncomplete(false); 
     73        Way w1 = new Way(3, 1); 
    7574        w1.addNode(n1); 
    7675        w1.addNode(n2); 
    … …  
    104103        Node n1 = new Node(1); 
    105104        Node n2 = new Node(2); 
    106         Way w1 = new Way(3); 
    107         w1.setIncomplete(false); 
     105        Way w1 = new Way(3, 1); 
    108106        w1.addNode(n1); 
    109107        w1.addNode(n2); 
    … …  
    154152        DataSet source = new DataSet(); 
    155153        Relation r1 = new Relation(1); 
    156         Node n20 = new Node(20); 
     154        Node n20 = new Node(20, 1); 
    157155        r1.addMember(new RelationMember("node-20",n20)); 
    158         r1.setIncomplete(false); 
    159         Way w30 = new Way(30); 
    160         w30.setIncomplete(false); 
     156        Way w30 = new Way(30, 1); 
    161157        Node n21; 
    162158        w30.addNode(n21 = new Node(21)); 
    … …  
    216212        Node n20 = new Node(20); 
    217213        r1.addMember(new RelationMember("node-20",n20)); 
    218         Way w30 = new Way(30); 
    219         w30.setIncomplete(false); 
     214        Way w30 = new Way(30, 1); 
    220215        Node n21; 
    221216        w30.addNode(n21 = new Node(21)); 

  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/nodes/NodeListMergeModelTest.java

    r2578 r2620  
    2929    } 
    3030 
    31     protected DefaultListSelectionModel inspectListSelectionModel(NodeListMergeModel model, String name) throws NoSuchFieldException, IllegalAccessException { 
     31    protected DefaultListSelectionModel inspectListSelectionModel(NodeListMergeModel model, String name) { 
    3232        return field(name).ofType(DefaultListSelectionModel.class) 
    3333        .in(model) 
    … …  
    6161 
    6262    @Test 
    63     public void test_copyMyNodesToTop_1() throws IllegalAccessException, NoSuchFieldException { 
    64         NodeListMergeModel model = new NodeListMergeModel(); 
    65  
    66         Way myWay = new Way(1); 
    67         myWay.setIncomplete(false); 
     63    public void test_copyMyNodesToTop_1() { 
     64        NodeListMergeModel model = new NodeListMergeModel(); 
     65 
     66        Way myWay = new Way(1, 1); 
    6867        myWay.addNode(new Node(2)); 
    6968        myWay.addNode(new Node(3)); 
    … …  
    8584 
    8685    @Test 
    87     public void test_copyMyNodesToTop_2() throws IllegalAccessException, NoSuchFieldException { 
    88         NodeListMergeModel model = new NodeListMergeModel(); 
    89  
    90  
    91         Way myWay = new Way(1); 
    92         myWay.setIncomplete(false); 
     86    public void test_copyMyNodesToTop_2() { 
     87        NodeListMergeModel model = new NodeListMergeModel(); 
     88 
     89 
     90        Way myWay = new Way(1, 1); 
    9391        myWay.addNode(new Node(2)); 
    9492        myWay.addNode(new Node(3)); 
    … …  
    117115 
    118116    @Test 
    119     public void test_copyMyNodesToTop_3() throws IllegalAccessException, NoSuchFieldException { 
    120         NodeListMergeModel model = new NodeListMergeModel(); 
    121  
    122  
    123         Way myWay = new Way(1); 
    124         myWay.setIncomplete(false); 
     117    public void test_copyMyNodesToTop_3() { 
     118        NodeListMergeModel model = new NodeListMergeModel(); 
     119 
     120 
     121        Way myWay = new Way(1, 1); 
    125122        myWay.addNode(new Node(2)); 
    126123        myWay.addNode(new Node(3)); 
    … …  
    145142 
    146143    @Test 
    147     public void test_copyMyNodesToTop_4() throws IllegalAccessException, NoSuchFieldException { 
    148         NodeListMergeModel model = new NodeListMergeModel(); 
    149  
    150  
    151         Way myWay = new Way(1); 
    152         myWay.setIncomplete(false); 
     144    public void test_copyMyNodesToTop_4() { 
     145        NodeListMergeModel model = new NodeListMergeModel(); 
     146 
     147 
     148        Way myWay = new Way(1, 1); 
    153149        myWay.addNode(new Node(2)); 
    154150        myWay.addNode(new Node(3)); 
    … …  
    175171 
    176172    @Test 
    177     public void test_copyMyNodesToEnd_1() throws IllegalAccessException, NoSuchFieldException { 
    178         NodeListMergeModel model = new NodeListMergeModel(); 
    179  
    180         Way myWay = new Way(1); 
    181         myWay.setIncomplete(false); 
     173    public void test_copyMyNodesToEnd_1() { 
     174        NodeListMergeModel model = new NodeListMergeModel(); 
     175 
     176        Way myWay = new Way(1, 1); 
    182177        myWay.addNode(new Node(2)); 
    183178        myWay.addNode(new Node(3)); 
    … …  
    197192 
    198193    @Test 
    199     public void test_copyMyNodesToEnd_2() throws IllegalAccessException, NoSuchFieldException { 
    200         NodeListMergeModel model = new NodeListMergeModel(); 
    201  
    202         Way myWay = new Way(1); 
    203         myWay.setIncomplete(false); 
     194    public void test_copyMyNodesToEnd_2() { 
     195        NodeListMergeModel model = new NodeListMergeModel(); 
     196 
     197        Way myWay = new Way(1, 1); 
    204198        myWay.addNode(new Node(2)); 
    205199        myWay.addNode(new Node(3)); 
    … …  
    223217 
    224218    @Test 
    225     public void test_copyMyNodesToEnd_3() throws IllegalAccessException, NoSuchFieldException { 
    226         NodeListMergeModel model = new NodeListMergeModel(); 
    227  
    228  
    229         Way myWay = new Way(1); 
    230         myWay.setIncomplete(false); 
     219    public void test_copyMyNodesToEnd_3() { 
     220        NodeListMergeModel model = new NodeListMergeModel(); 
     221 
     222 
     223        Way myWay = new Way(1, 1); 
    231224        myWay.addNode(new Node(2)); 
    232225        myWay.addNode(new Node(3)); 
    … …  
    251244 
    252245    @Test 
    253     public void test_copyMyNodesToEnd_4() throws IllegalAccessException, NoSuchFieldException { 
    254         NodeListMergeModel model = new NodeListMergeModel(); 
    255  
    256  
    257         Way myWay = new Way(1); 
    258         myWay.setIncomplete(false); 
     246    public void test_copyMyNodesToEnd_4() { 
     247        NodeListMergeModel model = new NodeListMergeModel(); 
     248 
     249 
     250        Way myWay = new Way(1, 1); 
    259251        myWay.addNode(new Node(2)); 
    260252        myWay.addNode(new Node(3)); 
    … …  
    285277 
    286278    @Test 
    287     public void test_copyMyNodesBeforeCurrent_1() throws IllegalAccessException, NoSuchFieldException { 
    288         NodeListMergeModel model = new NodeListMergeModel(); 
    289  
    290         Way myWay = new Way(1); 
    291         myWay.setIncomplete(false); 
     279    public void test_copyMyNodesBeforeCurrent_1() { 
     280        NodeListMergeModel model = new NodeListMergeModel(); 
     281 
     282        Way myWay = new Way(1, 1); 
    292283        myWay.addNode(new Node(1)); 
    293284        myWay.addNode(new Node(2)); 
    … …  
    314305 
    315306    @Test 
    316     public void test_copyMyNodesBeforeCurrent_2() throws IllegalAccessException, NoSuchFieldException { 
    317         NodeListMergeModel model = new NodeListMergeModel(); 
    318  
    319         Way myWay = new Way(1); 
    320         myWay.setIncomplete(false); 
     307    public void test_copyMyNodesBeforeCurrent_2() { 
     308        NodeListMergeModel model = new NodeListMergeModel(); 
     309 
     310        Way myWay = new Way(1, 1); 
    321311        myWay.addNode(new Node(1)); 
    322312        myWay.addNode(new Node(2)); 
    … …  
    343333 
    344334    @Test 
    345     public void test_copyMyNodesBeforeCurrent_3() throws IllegalAccessException, NoSuchFieldException { 
    346         NodeListMergeModel model = new NodeListMergeModel(); 
    347  
    348         Way myWay = new Way(1); 
    349         myWay.setIncomplete(false); 
     335    public void test_copyMyNodesBeforeCurrent_3() { 
     336        NodeListMergeModel model = new NodeListMergeModel(); 
     337 
     338        Way myWay = new Way(1, 1); 
    350339        myWay.addNode(new Node(1)); 
    351340        myWay.addNode(new Node(2)); 
    … …  
    377366    /* ----------------------------------------------------------------------------- */ 
    378367    @Test 
    379     public void test_copyMyNodesAfterCurrent_1() throws IllegalAccessException, NoSuchFieldException { 
    380         NodeListMergeModel model = new NodeListMergeModel(); 
    381  
    382         Way myWay = new Way(1); 
    383         myWay.setIncomplete(false); 
     368    public void test_copyMyNodesAfterCurrent_1() { 
     369        NodeListMergeModel model = new NodeListMergeModel(); 
     370 
     371        Way myWay = new Way(1, 1); 
    384372        myWay.addNode(new Node(1)); 
    385373        myWay.addNode(new Node(2)); 
    … …  
    406394 
    407395    @Test 
    408     public void test_copyMyNodesAfterCurrent_2() throws IllegalAccessException, NoSuchFieldException { 
    409         NodeListMergeModel model = new NodeListMergeModel(); 
    410  
    411         Way myWay = new Way(1); 
    412         myWay.setIncomplete(false); 
     396    public void test_copyMyNodesAfterCurrent_2() { 
     397        NodeListMergeModel model = new NodeListMergeModel(); 
     398 
     399        Way myWay = new Way(1, 1); 
    413400        myWay.addNode(new Node(1)); 
    414401        myWay.addNode(new Node(2)); 
    … …  
    437424 
    438425    @Test 
    439     public void test_copyMyNodesAfterCurrent_3() throws IllegalAccessException, NoSuchFieldException { 
    440         NodeListMergeModel model = new NodeListMergeModel(); 
    441  
    442         Way myWay = new Way(1); 
    443         myWay.setIncomplete(false); 
     426    public void test_copyMyNodesAfterCurrent_3() { 
     427        NodeListMergeModel model = new NodeListMergeModel(); 
     428 
     429        Way myWay = new Way(1, 1); 
    444430        myWay.addNode(new Node(1)); 
    445431        myWay.addNode(new Node(2)); 
    … …  
    467453 
    468454    @Test 
    469     public void test_copyMyNodesAfterCurrent_4() throws IllegalAccessException, NoSuchFieldException { 
    470         NodeListMergeModel model = new NodeListMergeModel(); 
    471  
    472         Way myWay = new Way(1); 
    473         myWay.setIncomplete(false); 
     455    public void test_copyMyNodesAfterCurrent_4() { 
     456        NodeListMergeModel model = new NodeListMergeModel(); 
     457 
     458        Way myWay = new Way(1, 1); 
    474459        myWay.addNode(new Node(1)); 
    475460        myWay.addNode(new Node(2)); 
    … …  
    501486    /* ----------------------------------------------------------------------------- */ 
    502487    @Test 
    503     public void test_moveUpMergedNodes_1() throws IllegalAccessException, NoSuchFieldException { 
    504         NodeListMergeModel model = new NodeListMergeModel(); 
    505  
    506         Way myWay = new Way(1); 
    507         myWay.setIncomplete(false); 
     488    public void test_moveUpMergedNodes_1() { 
     489        NodeListMergeModel model = new NodeListMergeModel(); 
     490 
     491        Way myWay = new Way(1, 1); 
    508492        myWay.addNode(new Node(1)); 
    509493        myWay.addNode(new Node(2)); 
    … …  
    528512 
    529513    @Test 
    530     public void test_moveUpMergedNodes_2() throws IllegalAccessException, NoSuchFieldException { 
    531         NodeListMergeModel model = new NodeListMergeModel(); 
    532  
    533         Way myWay = new Way(1); 
    534         myWay.setIncomplete(false); 
     514    public void test_moveUpMergedNodes_2() { 
     515        NodeListMergeModel model = new NodeListMergeModel(); 
     516 
     517        Way myWay = new Way(1, 1); 
    535518        myWay.addNode(new Node(1)); 
    536519        myWay.addNode(new Node(2)); 
    … …  
    559542 
    560543    @Test 
    561     public void test_moveUpMergedNodes_3() throws IllegalAccessException, NoSuchFieldException { 
    562         NodeListMergeModel model = new NodeListMergeModel(); 
    563  
    564         Way myWay = new Way(1); 
    565         myWay.setIncomplete(false); 
     544    public void test_moveUpMergedNodes_3() { 
     545        NodeListMergeModel model = new NodeListMergeModel(); 
     546 
     547        Way myWay = new Way(1, 1); 
    566548        myWay.addNode(new Node(1)); 
    567549        myWay.addNode(new Node(2)); 
    … …  
    593575    /* ----------------------------------------------------------------------------- */ 
    594576    @Test 
    595     public void test_moveDownMergedNodes_1() throws IllegalAccessException, NoSuchFieldException { 
    596         NodeListMergeModel model = new NodeListMergeModel(); 
    597  
    598         Way myWay = new Way(1); 
    599         myWay.setIncomplete(false); 
     577    public void test_moveDownMergedNodes_1() { 
     578        NodeListMergeModel model = new NodeListMergeModel(); 
     579 
     580        Way myWay = new Way(1, 1); 
    600581        myWay.addNode(new Node(1)); 
    601582        myWay.addNode(new Node(2)); 
    … …  
    620601 
    621602    @Test 
    622     public void test_moveDownMergedNodes_2() throws IllegalAccessException, NoSuchFieldException { 
    623         NodeListMergeModel model = new NodeListMergeModel(); 
    624  
    625         Way myWay = new Way(1); 
    626         myWay.setIncomplete(false); 
     603    public void test_moveDownMergedNodes_2() { 
     604        NodeListMergeModel model = new NodeListMergeModel(); 
     605 
     606        Way myWay = new Way(1, 1); 
    627607        myWay.addNode(new Node(1)); 
    628608        myWay.addNode(new Node(2)); 
    … …  
    651631 
    652632    @Test 
    653     public void test_moveDownMergedNodes_3() throws IllegalAccessException, NoSuchFieldException { 
    654         NodeListMergeModel model = new NodeListMergeModel(); 
    655  
    656         Way myWay = new Way(1); 
    657         myWay.setIncomplete(false); 
     633    public void test_moveDownMergedNodes_3() { 
     634        NodeListMergeModel model = new NodeListMergeModel(); 
     635 
     636        Way myWay = new Way(1, 1); 
    658637        myWay.addNode(new Node(1)); 
    659638        myWay.addNode(new Node(2)); 
Note: See TracChangeset for help on using the changeset viewer.