Ignore:
Timestamp:
2017-08-27T17:07:54+02:00 (4 years ago)
Author:
Don-vip
Message:

see #15182 - move ConflictCollection from OsmDataLayer to DataSet. Simplifies some code where a data set is enough, and a layer is not needed

Location:
trunk/src/org/openstreetmap/josm/command
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/command/PurgeCommand.java

    r12605 r12672  
    122122                } else {
    123123                    getAffectedDataSet().removePrimitive(osm);
    124                     if (getLayer() != null) {
    125                         Conflict<?> conflict = getLayer().getConflicts().getConflictForMy(osm);
    126                         if (conflict != null) {
    127                             purgedConflicts.add(conflict);
    128                             getLayer().getConflicts().remove(conflict);
    129                         }
     124                    Conflict<?> conflict = getAffectedDataSet().getConflicts().getConflictForMy(osm);
     125                    if (conflict != null) {
     126                        purgedConflicts.add(conflict);
     127                        getAffectedDataSet().getConflicts().remove(conflict);
    130128                    }
    131129                }
    … …  
    157155
    158156        for (Conflict<?> conflict : purgedConflicts) {
    159             getLayer().getConflicts().add(conflict);
     157            getAffectedDataSet().getConflicts().add(conflict);
    160158        }
    161159    }

  • trunk/src/org/openstreetmap/josm/command/conflict/ConflictAddCommand.java

    r12663 r12672  
    1313import org.openstreetmap.josm.command.Command;
    1414import org.openstreetmap.josm.data.conflict.Conflict;
     15import org.openstreetmap.josm.data.osm.DataSet;
    1516import org.openstreetmap.josm.data.osm.DefaultNameFormatter;
    1617import org.openstreetmap.josm.data.osm.OsmPrimitive;
    … …  
    3839    }
    3940
     41    /**
     42     * Constructs a new {@code ConflictAddCommand}.
     43     * @param ds the data set. Must not be null.
     44     * @param conflict the conflict to add
     45     * @since 12672
     46     */
     47    public ConflictAddCommand(DataSet ds, Conflict<? extends OsmPrimitive> conflict) {
     48        super(ds);
     49        this.conflict = conflict;
     50    }
     51
    4052    protected void warnBecauseOfDoubleConflict() {
    4153        JOptionPane.showMessageDialog(
    … …  
    5567    public boolean executeCommand() {
    5668        try {
    57             getLayer().getConflicts().add(conflict);
     69            getAffectedDataSet().getConflicts().add(conflict);
    5870        } catch (IllegalStateException e) {
    5971            Logging.error(e);
    … …  
    7284            return;
    7385        }
    74         getLayer().getConflicts().remove(conflict);
     86        getAffectedDataSet().getConflicts().remove(conflict);
    7587    }
    7688

  • trunk/src/org/openstreetmap/josm/command/conflict/ConflictResolveCommand.java

    r12636 r12672  
    99import org.openstreetmap.josm.data.conflict.Conflict;
    1010import org.openstreetmap.josm.data.conflict.ConflictCollection;
     11import org.openstreetmap.josm.data.osm.DataSet;
    1112import org.openstreetmap.josm.gui.MainApplication;
    1213import org.openstreetmap.josm.gui.layer.OsmDataLayer;
    … …  
    5960     */
    6061    protected void reconstituteConflicts() {
    61         OsmDataLayer editLayer = getLayer();
     62        DataSet ds = getAffectedDataSet();
    6263        for (Conflict<?> c : resolvedConflicts) {
    63             if (!editLayer.getConflicts().hasConflictForMy(c.getMy())) {
    64                 editLayer.getConflicts().add(c);
     64            if (!ds.getConflicts().hasConflictForMy(c.getMy())) {
     65                ds.getConflicts().add(c);
    6566            }
    6667        }

  • trunk/src/org/openstreetmap/josm/command/conflict/ModifiedConflictResolveCommand.java

    r10216 r12672  
    5555            conflict.getMy().setModified(conflict.getTheir().isModified());
    5656        }
    57         getLayer().getConflicts().remove(conflict);
     57        getAffectedDataSet().getConflicts().remove(conflict);
    5858        rememberConflict(conflict);
    5959        return true;

  • trunk/src/org/openstreetmap/josm/command/conflict/RelationMemberConflictResolverCommand.java

    r12636 r12672  
    1111
    1212import org.openstreetmap.josm.data.conflict.Conflict;
     13import org.openstreetmap.josm.data.osm.DataSet;
    1314import org.openstreetmap.josm.data.osm.OsmPrimitive;
    1415import org.openstreetmap.josm.data.osm.Relation;
    … …  
    8182
    8283        MainApplication.getLayerManager().setActiveLayer(layer);
    83         OsmDataLayer editLayer = MainApplication.getLayerManager().getEditLayer();
     84        DataSet editDs = MainApplication.getLayerManager().getEditDataSet();
    8485
    8586        // restore the former state
    … …  
    8990        // restore a conflict if necessary
    9091        //
    91         if (!editLayer.getConflicts().hasConflictForMy(conflict.getMy())) {
    92             editLayer.getConflicts().add(conflict);
     92        if (!editDs.getConflicts().hasConflictForMy(conflict.getMy())) {
     93            editDs.getConflicts().add(conflict);
    9394        }
    9495    }

  • trunk/src/org/openstreetmap/josm/command/conflict/VersionConflictResolveCommand.java

    r10216 r12672  
    6464            }
    6565        }
    66         getLayer().getConflicts().remove(conflict);
     66        getAffectedDataSet().getConflicts().remove(conflict);
    6767        rememberConflict(conflict);
    6868        return true;
Note: See TracChangeset for help on using the changeset viewer.