Changeset 17401 in osm for applications/editors/josm/plugins/DirectUpload/src/org

Timestamp:

2009-08-31T10:03:20+02:00 (15 years ago)

Author:

guggis

Message:

applied #3339: patch by xeen: Plugin DirectUpload breaks up tags containing spaces

File:

• applications/editors/josm/plugins/DirectUpload/src/org/openstreetmap/josm/plugins/DirectUpload/UploadDataGui.java (modified) (10 diffs)

applications/editors/josm/plugins/DirectUpload/src/org/openstreetmap/josm/plugins/DirectUpload/UploadDataGui.java

@@ -28 +28 @@
 import java.util.Date;
 
-import javax.swing.JCheckBox;
+import javax.swing.JComboBox;
 import javax.swing.JLabel;
 import javax.swing.JPanel;
@@ -45 +45 @@
 import org.openstreetmap.josm.tools.Base64;
 import org.openstreetmap.josm.tools.GBC;
+import org.openstreetmap.josm.tools.UrlLabel;
 
 /**
@@ -51 +52 @@
  */
 public class UploadDataGui extends ExtendedDialog {
+    /**
+     * This enum contains the possible values for the visibility field and their
+     * explanation. Provides some methods for easier handling.
+     */
+    private enum visibility {
+        PRIVATE      (tr("Private (only shared as anonymous, unordered points)")),
+        PUBLIC       (tr("Public (shown in trace list and as anonymous, unordered points)")),
+        TRACKABLE    (tr("Trackable (only shared as anonymous, ordered points with timestamps)")),
+        IDENTIFIABLE (tr("Identifiable (shown in trace list and as identifiable, ordered points with timestamps)"));
+        
+        public final String description;
+        visibility(String description) {
+            this.description = description;
+        }
+        
+        /**
+         * "Converts" a given description into the actual enum. Returns null if no matching description
+         * is found.
+         * @param desc The description to look for
+         * @return visibility or null
+         */
+        public static visibility desc2visi(Object desc) {
+            for (visibility v : visibility.values()) {
+                if(desc.equals((String)v.description))
+                    return v;
+            }
+            return null;
+        }
+        
+        public String toString() {
+            return this.name().toLowerCase();
+        }
+    }
+
+        
     // User for log in when uploading trace
     private String username = Main.pref.get("osm-server.username");
@@ -62 +98 @@
     private JTextField descriptionField = new JTextField(50);
     private JTextField tagsField = new JTextField(50);
-    private JCheckBox publicCheckbox = new JCheckBox();
+    private JComboBox visibilityCombo = new JComboBox();
 
     // Constants used when generating upload request
@@ -96 +132 @@
      */
     private JPanel initComponents() {
-        publicCheckbox.setText(tr("Public"));
-        publicCheckbox.setToolTipText(tr("Selected makes your trace public in openstreetmap.org"));
+        JLabel visibilityLabel = new JLabel(tr("Visibility"));
+        visibilityLabel.setToolTipText(tr("Defines the visibility of your trace for other OSM users."));
+        for(visibility v : visibility.values()) {
+                visibilityCombo.addItem((String)v.description);
+        }
+        UrlLabel visiUrl = new UrlLabel(tr("http://wiki.openstreetmap.org/wiki/Visibility_of_GPS_traces"), tr("(What does that mean?)"));
 
         JLabel descriptionLabel = new JLabel(tr("Description"));
         descriptionField.setToolTipText(tr("Please enter Description about your trace."));
 
-        JLabel tagsLabel = new JLabel(tr("Tags"));
+        JLabel tagsLabel = new JLabel(tr("Tags (comma delimited)"));
         tagsField.setToolTipText(tr("Please enter tags about your trace."));
 
@@ -116 +156 @@
         p.add(descriptionField, GBC.eol().fill(GBC.HORIZONTAL));
 
-        p.add(publicCheckbox, GBC.eol());
+        p.add(visibilityLabel, GBC.std().insets(0,10,0,0));
+        p.add(visiUrl, GBC.eol().insets(0,10,0,0));
+        p.add(visibilityCombo, GBC.eol());
 
         return p;
@@ -160 +202 @@
      * @param GpxData The GPX Data to upload
      */
-    private void upload(String description, String tags, Boolean isPublic, GpxData gpxData, ProgressMonitor progressMonitor) throws IOException {
+    private void upload(String description, String tags, String visi, GpxData gpxData, ProgressMonitor progressMonitor) throws IOException {
         progressMonitor.beginTask(null);
         try {
@@ -168 +210 @@
                 // Clean description/tags from disallowed chars
                 description = description.replaceAll("[&?/\\\\]"," ");
-                tags = tags.replaceAll("[&?/\\\\.,;]"," ");
+                tags = tags.replaceAll("[&?/\\\\.;]"," ");
 
                 // Set progress dialog to indeterminate while connecting
@@ -179 +221 @@
                         writeField(baos, "description", description);
                         writeField(baos, "tags", (tags != null && tags.length() > 0) ? tags : "");
-                        writeField(baos, "public", isPublic ? "1" : "0");
+                        writeField(baos, "visibility", visi);
                         writeString(baos, "--" + BOUNDARY + "--" + LINE_END);
 
@@ -378 +420 @@
                   upload(descriptionField.getText(),
                          tagsField.getText(),
-                         publicCheckbox.isSelected(),
+                         visibility.desc2visi(visibilityCombo.getSelectedItem()).toString(),
                          ((GpxLayer)Main.map.mapView.getActiveLayer()).data,
                          progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false)