Ticket #6542: 6542.patch

src/org/openstreetmap/josm/actions/DownloadPrimitiveAction.java

@@ -9 +9 @@
 import java.awt.Font;
 import java.awt.GridBagLayout;
 import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
 import java.awt.event.KeyEvent;
 import java.lang.reflect.InvocationTargetException;
 import java.util.Collections;
@@ -93 +94 @@
         layout.setAutoCreateContainerGaps(true);
 
         JLabel lbl1 = new JLabel(tr("Object type:"));
-        OsmPrimitiveTypesComboBox cbType = new OsmPrimitiveTypesComboBox();
+        final OsmPrimitiveTypesComboBox cbType = new OsmPrimitiveTypesComboBox();
         cbType.addItem(trc("osm object types", "mixed"));
         cbType.setToolTipText(tr("Choose the OSM object type"));
         JLabel lbl2 = new JLabel(tr("Object ID:"));
@@ -112 +113 @@
         JCheckBox layer = new JCheckBox(tr("Separate Layer"));
         layer.setToolTipText(tr("Select if the data should be downloaded into a new layer"));
         layer.setSelected(Main.pref.getBoolean("download.newlayer"));
-        JCheckBox referrers = new JCheckBox(tr("Download referrers"));
-        referrers.setToolTipText(tr("Select if the referrers of the object should be downloaded as well"));
+        final JCheckBox referrers = new JCheckBox(tr("Download referrers (parent relations)"));
+        referrers.setToolTipText(tr("Select if the referrers of the object should be downloaded as well, i.e.,"
+                + "parent relations and for nodes, additionally, parent ways"));
         referrers.setSelected(Main.pref.getBoolean("downloadprimitive.referrers"));
+        JCheckBox full = new JCheckBox(tr("Download relation members"));
+        full.setToolTipText(tr("Select if the members of a relation should be downloaded as well"));
+        full.setSelected(Main.pref.getBoolean("downloadprimitive.full", true));
         HtmlPanel help = new HtmlPanel(tr("Object IDs can be separated by comma or space.<br/>"
                 + " Examples: <b><ul><li>1 2 5</li><li>1,2,5</li></ul><br/></b>"
                 + " In mixed mode, specify objects like this: <b>w123, n110, w12, r15</b><br/>"));
@@ -128 +133 @@
                 .addComponent(lbl2)
                 .addComponent(cbId, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE))
             .addComponent(referrers)
+            .addComponent(full)
             .addComponent(layer)
             .addComponent(help)
         );
 
+        cbType.addActionListener(new ActionListener() {
+
+            @Override
+            public void actionPerformed(ActionEvent ae) {
+                referrers.setText(cbType.getType() == OsmPrimitiveType.NODE
+                        ? tr("Download referrers (parent relations and ways)")
+                        : tr("Download referrers (parent relations)"));
+            }
+        });
+
         layout.setHorizontalGroup(layout.createParallelGroup()
             .addGroup(layout.createSequentialGroup()
                 .addGroup(layout.createParallelGroup()
@@ -143 +159 @@
                     .addComponent(cbId))
                 )
             .addComponent(referrers)
+            .addComponent(full)
             .addComponent(layer)
             .addComponent(help)
         );
@@ -178 +195 @@
             return;
         }
         remindPrimitivesHistory(cbId);
-        processItems(layer.isSelected(), tfId.getIds(), referrers.isSelected());
+        processItems(layer.isSelected(), tfId.getIds(), referrers.isSelected(), full.isSelected());
     }
 
-    void processItems(boolean newLayer, final List<PrimitiveId> ids, boolean downloadReferrers) {
+    void processItems(boolean newLayer, final List<PrimitiveId> ids, boolean downloadReferrers, boolean full) {
         OsmDataLayer layer = getEditLayer();
         if ((layer == null) || newLayer) {
             layer = new OsmDataLayer(new DataSet(), OsmDataLayer.createNewName(), null);
             Main.main.addLayer(layer);
         }
-        final DownloadPrimitivesTask task = new DownloadPrimitivesTask(layer, ids);
+        final DownloadPrimitivesTask task = new DownloadPrimitivesTask(layer, ids, full);
         Main.worker.submit(task);
 
         if (downloadReferrers) {

src/org/openstreetmap/josm/gui/io/DownloadPrimitivesTask.java

@@ -38 +38 @@
     private Set<PrimitiveId> missingPrimitives;
 
     private OsmDataLayer layer;
+    private boolean fullRelation;
     private MultiFetchServerObjectReader multiObjectReader;
     private OsmServerObjectReader objectReader;
 
@@ -47 +48 @@
      * @param layer the layer in which primitives are updated. Must not be null.
      * @param toUpdate a collection of primitives to update from the server. Set to
      * the empty collection if null.
+     * @param fullRelation true if a full download is required, i.e.,
+     * a download including the immediate children of a relation.
      * @throws IllegalArgumentException thrown if layer is null.
      */
-    public DownloadPrimitivesTask(OsmDataLayer layer, List<PrimitiveId>  ids) throws IllegalArgumentException {
+    public DownloadPrimitivesTask(OsmDataLayer layer, List<PrimitiveId>  ids, boolean fullRelation) throws IllegalArgumentException {
         super(tr("Download objects"), false /* don't ignore exception */);
         ensureParameterNotNull(layer, "layer");
         this.ids = ids;
         this.layer = layer;
+        this.fullRelation = fullRelation;
     }
 
     @Override
@@ -144 +148 @@
                 if (r.hasIncompleteMembers()) {
                     synchronized(this) {
                         if (canceled) return;
-                        objectReader = new OsmServerObjectReader(r.getId(), OsmPrimitiveType.RELATION, true /* full */);
+                        objectReader = new OsmServerObjectReader(r.getId(), OsmPrimitiveType.RELATION, fullRelation);
                     }
                     theirDataSet = objectReader.parseOsm(progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false));
                     synchronized (this) {