Ignore:
Timestamp:
2012-06-24T21:02:36+02:00 (12 years ago)
Author:
Don-vip
Message:

see #4899, see #7266, see #7333: Resolved NPE in conflict manager when copying a member created by merging two layers

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/ListMergeModel.java

    r5297 r5298  
    1515import java.util.HashMap;
    1616import java.util.List;
     17import java.util.Map;
    1718import java.util.Observable;
    1819
    … …  
    2122import javax.swing.DefaultListSelectionModel;
    2223import javax.swing.JOptionPane;
     24import javax.swing.JTable;
     25import javax.swing.ListSelectionModel;
    2326import javax.swing.table.DefaultTableModel;
    2427import javax.swing.table.TableModel;
    2528
    2629import org.openstreetmap.josm.Main;
     30import org.openstreetmap.josm.data.osm.DataSet;
    2731import org.openstreetmap.josm.data.osm.OsmPrimitive;
    2832import org.openstreetmap.josm.data.osm.PrimitiveId;
    … …  
    3034import org.openstreetmap.josm.gui.help.HelpUtil;
    3135import org.openstreetmap.josm.gui.widgets.OsmPrimitivesTableModel;
     36import org.openstreetmap.josm.tools.CheckParameterUtil;
    3237
    3338/**
    … …  
    6368 * @see ListMerger
    6469 */
    65 public abstract class ListMergeModel<T> extends Observable {
     70public abstract class ListMergeModel<T extends PrimitiveId> extends Observable {
    6671    public static final String FROZEN_PROP = ListMergeModel.class.getName() + ".frozen";
    6772
    … …  
    8186    private boolean isFrozen = false;
    8287    private final ComparePairListModel comparePairListModel;
     88   
     89    private DataSet myDataset;
     90    private Map<PrimitiveId, PrimitiveId> mergedMap;
    8391
    8492    /**
    … …  
    118126     * @return Primitive from my dataset referenced by entry
    119127     */
    120     protected abstract OsmPrimitive getMyPrimitive(T entry);
     128    protected OsmPrimitive getMyPrimitive(T entry) {
     129        return getMyPrimitiveDefault(entry);
     130    }
     131   
     132    protected final OsmPrimitive getMyPrimitiveDefault(PrimitiveId entry) {
     133        OsmPrimitive result = myDataset.getPrimitiveById(entry);
     134        if (result == null && mergedMap != null) {
     135            PrimitiveId id = mergedMap.get(entry);
     136            if (id == null && entry instanceof OsmPrimitive) {
     137                id = mergedMap.get(((OsmPrimitive)entry).getPrimitiveId());
     138            }
     139            if (id != null) {
     140                result = myDataset.getPrimitiveById(id);
     141            }
     142        }
     143        return result;
     144    }
    121145
    122146    protected void buildMyEntriesTableModel() {
    … …  
    310334        getMergedEntries().clear();
    311335        fireModelDataChanged();
     336    }
     337   
     338    protected final void initPopulate(OsmPrimitive my, OsmPrimitive their, Map<PrimitiveId, PrimitiveId> mergedMap) {
     339        CheckParameterUtil.ensureParameterNotNull(my, "my");
     340        CheckParameterUtil.ensureParameterNotNull(their, "their");
     341        this.myDataset = my.getDataSet();
     342        this.mergedMap = mergedMap;
     343        getMergedEntries().clear();
     344        getMyEntries().clear();
     345        getTheirEntries().clear();
    312346    }
    313347

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/ListMerger.java

    r5297 r5298  
    3636import javax.swing.event.ListSelectionListener;
    3737
     38import org.openstreetmap.josm.data.osm.PrimitiveId;
    3839import org.openstreetmap.josm.gui.widgets.OsmPrimitivesTable;
    3940import org.openstreetmap.josm.tools.CheckParameterUtil;
    … …  
    4647 * @see ListMergeModel
    4748 */
    48 public abstract class ListMerger<T> extends JPanel implements PropertyChangeListener, Observer {
     49public abstract class ListMerger<T extends PrimitiveId> extends JPanel implements PropertyChangeListener, Observer {
    4950    protected OsmPrimitivesTable myEntriesTable;
    5051    protected OsmPrimitivesTable mergedEntriesTable;

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMergeModel.java

    r5266 r5298  
    55
    66import java.util.ArrayList;
     7import java.util.Map;
    78
    89import javax.swing.table.DefaultTableModel;
    … …  
    1011import org.openstreetmap.josm.command.WayNodesConflictResolverCommand;
    1112import org.openstreetmap.josm.data.conflict.Conflict;
    12 import org.openstreetmap.josm.data.osm.DataSet;
    1313import org.openstreetmap.josm.data.osm.Node;
    1414import org.openstreetmap.josm.data.osm.OsmPrimitive;
     15import org.openstreetmap.josm.data.osm.PrimitiveId;
    1516import org.openstreetmap.josm.data.osm.Way;
    1617import org.openstreetmap.josm.gui.conflict.pair.ListMergeModel;
    1718import org.openstreetmap.josm.gui.conflict.pair.ListRole;
    18 import org.openstreetmap.josm.tools.CheckParameterUtil;
    1919
    2020public class NodeListMergeModel extends ListMergeModel<Node>{
    21     private DataSet myDataset;
    2221
    2322    /**
    … …  
    2726     * @param my  my way (i.e. the way in the local dataset)
    2827     * @param their their way (i.e. the way in the server dataset)
     28     * @param mergedMap The map of merged primitives if the conflict results from merging two layers
    2929     * @exception IllegalArgumentException thrown, if my is null
    3030     * @exception IllegalArgumentException  thrown, if their is null
    3131     */
    32     public void populate(Way my, Way their) {
    33         this.myDataset = my.getDataSet();
     32    public void populate(Way my, Way their, Map<PrimitiveId, PrimitiveId> mergedMap) {
     33        initPopulate(my, their, mergedMap);
    3434
    35         CheckParameterUtil.ensureParameterNotNull(my, "my");
    36         CheckParameterUtil.ensureParameterNotNull(their, "their");
    37         getMergedEntries().clear();
    38         getMyEntries().clear();
    39         getTheirEntries().clear();
    4035        for (Node n : my.getNodes()) {
    4136            getMyEntries().add(n);
    … …  
    8580        return (Node) getMyPrimitive(entry);
    8681    }
    87 
    88     @Override
    89     protected OsmPrimitive getMyPrimitive(Node entry) {
    90         return myDataset.getPrimitiveById(entry);
    91     }
    92 
    9382}

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMerger.java

    r5297 r5298  
    5757        Way myWay = (Way)conflict.getMy();
    5858        Way theirWay = (Way)conflict.getTheir();
    59         ((NodeListMergeModel)model).populate(myWay, theirWay);
     59        ((NodeListMergeModel)model).populate(myWay, theirWay, conflict.getMergedMap());
    6060        myEntriesTable.setLayer(findLayerFor(myWay));
    6161        theirEntriesTable.setLayer(findLayerFor(theirWay));

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberListMergeModel.java

    r4191 r5298  
    55
    66import java.util.List;
     7import java.util.Map;
    78
    89import javax.swing.table.DefaultTableModel;
    910
    1011import org.openstreetmap.josm.command.RelationMemberConflictResolverCommand;
    11 import org.openstreetmap.josm.data.osm.DataSet;
    1212import org.openstreetmap.josm.data.osm.OsmPrimitive;
     13import org.openstreetmap.josm.data.osm.PrimitiveId;
    1314import org.openstreetmap.josm.data.osm.Relation;
    1415import org.openstreetmap.josm.data.osm.RelationMember;
    … …  
    2122 */
    2223public class RelationMemberListMergeModel extends ListMergeModel<RelationMember>{
    23     private DataSet myDataset;
    2424
    2525    @Override
    … …  
    6060     * @param my my relation. Must not be null.
    6161     * @param their their relation. Must not be null.
     62     * @param mergedMap The map of merged primitives if the conflict results from merging two layers
    6263     *
    6364     * @throws IllegalArgumentException if my is null
    6465     * @throws IllegalArgumentException if their is null
    6566     */
    66     public void populate(Relation my, Relation their) {
    67         this.myDataset = my.getDataSet();
    68 
    69         CheckParameterUtil.ensureParameterNotNull(my, "my");
    70         CheckParameterUtil.ensureParameterNotNull(their, "their");
    71 
    72         getMergedEntries().clear();
    73         getMyEntries().clear();
    74         getTheirEntries().clear();
     67    public void populate(Relation my, Relation their, Map<PrimitiveId, PrimitiveId> mergedMap) {
     68        initPopulate(my, their, mergedMap);
    7569
    7670        for (RelationMember n : my.getMembers()) {
    … …  
    9488    @Override
    9589    protected RelationMember cloneEntryForMergedList(RelationMember entry) {
    96         OsmPrimitive primitive = getMyPrimitive(entry);
    97         return new RelationMember(entry.getRole(), primitive);
     90        return new RelationMember(entry.getRole(), getMyPrimitive(entry));
    9891    }
    9992
    10093    @Override
    10194    protected OsmPrimitive getMyPrimitive(RelationMember entry) {
    102         return myDataset.getPrimitiveById(entry.getMember());
     95        return getMyPrimitiveDefault(entry.getMember());
    10396    }
    10497

  • trunk/src/org/openstreetmap/josm/gui/conflict/pair/relation/RelationMemberMerger.java

    r5297 r5298  
    4848    public void populate(Conflict<? extends OsmPrimitive> conflict) {
    4949        RelationMemberListMergeModel model = (RelationMemberListMergeModel)getModel();
    50         model.populate((Relation)conflict.getMy(), (Relation)conflict.getTheir());
     50        model.populate((Relation)conflict.getMy(), (Relation)conflict.getTheir(), conflict.getMergedMap());
    5151    }
    5252

  • trunk/src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

    r5297 r5298  
    705705        @Override
    706706        public void updateEnabledState() {
    707             GuiHelper.runInEDT(new Runnable() {
     707            GuiHelper.runInEDTAndWait(new Runnable() {
    708708                @Override
    709709                public void run() {
Note: See TracChangeset for help on using the changeset viewer.