Changeset 2289 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2009-10-15T04:45:06+02:00 (17 years ago)

Author:

Gubaer

Message:

fixed #3719: Upload dialog still lists closed changesets after JOSM knows they've been closed
fixed #3668: Upload dialog: comment textbox buggy
Added context sensitive help to upload dialog
Improved error handling for errors due to closing already closed changesets

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• ExceptionDialogUtil.java (modified) (2 diffs)
• help/ContextSensitiveHelpAction.java (added)
• io/DownloadOpenChangesetsTask.java (modified) (1 diff)
• io/UploadDialog.java (modified) (11 diffs)

trunk/src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java

@@ -144 +144 @@
 
     /**
+     * Explains a {@see OsmApiException} which was thrown because of a conflict
+     * 
+     * @param e the exception
+     */
+    public static void explainConflict(OsmApiException e) {
+        JOptionPane.showMessageDialog(Main.parent, ExceptionUtil.explainConflict(e), tr("Conflict"),
+                JOptionPane.ERROR_MESSAGE);
+    }
+
+    /**
      * Explains a {@see UnknownHostException} which has caused an {@see OsmTransferException}.
      * This is most likely happening when there is an error in the API URL or when
@@ -231 +241 @@
                 return;
             }
+            if (oae.getResponseCode() == HttpURLConnection.HTTP_CONFLICT) {
+                explainConflict(oae);
+                return;
+            }
 
         }

trunk/src/org/openstreetmap/josm/gui/io/DownloadOpenChangesetsTask.java

@@ -70 +70 @@
                 new Runnable() {
                     public void run() {
-                        model.addOrUpdate(changesets);
+                        model.setChangesets(changesets);
                     }
                 }

trunk/src/org/openstreetmap/josm/gui/io/UploadDialog.java

@@ -52 +52 @@
 import org.openstreetmap.josm.gui.OsmPrimitivRenderer;
 import org.openstreetmap.josm.gui.SideButton;
+import org.openstreetmap.josm.gui.help.ContextSensitiveHelpAction;
+import org.openstreetmap.josm.gui.help.HelpBuilder;
 import org.openstreetmap.josm.gui.tagging.TagEditorModel;
 import org.openstreetmap.josm.gui.tagging.TagEditorPanel;
@@ -64 +66 @@
  * This is a dialog for entering upload options like the parameters for
  * the upload changeset and the strategy for opening/closing a changeset.
- * 
- * 
+ *
  */
 public class UploadDialog extends JDialog {
-
 
     public static final String HISTORY_KEY = "upload.comment.history";
@@ -109 +109 @@
 
     private ChangesetSelectionPanel pnlChangesetSelection;
-
     private boolean canceled = false;
 
@@ -152 +151 @@
         southTabbedPane.setComponentAt(0, pnlChangesetSelection = new ChangesetSelectionPanel());
         southTabbedPane.setTitleAt(0, tr("Settings"));
+        southTabbedPane.setToolTipTextAt(0, tr("Decide how to upload the data and which changeset to use"));
         southTabbedPane.setTitleAt(1, tr("Tags of new changeset"));
+        southTabbedPane.setToolTipTextAt(1, tr("Apply tags to the changeset data is uploaded to"));
         southTabbedPane.addChangeListener(new TabbedPaneChangeLister());
         JPanel pnl1 = new JPanel();
@@ -191 +192 @@
                 JComponent.WHEN_IN_FOCUSED_WINDOW
         );
+
+        pnl.add(new SideButton(new ContextSensitiveHelpAction("/Help/Dialogs/UploadDialog")));
+        HelpBuilder.setHelpContext(getRootPane(),"/Help/Dialogs/UploadDialog");
         return pnl;
     }
@@ -562 +566 @@
 
     /**
-     * Listens to window closing events and processes them as cancel events
+     * Listens to window closing events and processes them as cancel events.
+     * Listens to window open events and initializes user input
      *
      */
@@ -572 +577 @@
 
         @Override
-        public void windowActivated(WindowEvent e) {
+        public void windowOpened(WindowEvent e) {
             startUserInput();
         }
@@ -621 +626 @@
             pnl.add(new JLabel(tr("Provide a brief comment for the changes you are uploading:")), GBC.eol().insets(0, 5, 10, 3));
             cmt = new HistoryComboBox();
+            cmt.setToolTipText(tr("Enter an upload comment (min. 3 characters)"));
             List<String> cmtHistory = new LinkedList<String>(Main.pref.getCollection(HISTORY_KEY, new LinkedList<String>()));
             // we have to reverse the history, because ComboBoxHistory will reverse it again
@@ -994 +1000 @@
     }
 
+    /**
+     * A combobox model for the list of open changesets
+     *
+     */
     public class OpenChangesetModel extends DefaultComboBoxModel {
         private List<Changeset> changesets;
@@ -1034 +1044 @@
         }
 
-        /**
-         * Updates the current list of open changesets with the changesets
-         * in <code>changesets</code>
-         * 
-         * @param changesets the collection of changesets. If null, removes
-         * all changesets from the current list of changesets
-         */
-        public void addOrUpdate(Collection<Changeset> changesets) {
-            if (changesets == null){
-                this.changesets.clear();
-                setSelectedItem(null);
-            }
-            for (Changeset cs: changesets) {
-                internalAddOrUpdate(cs);
+        public void setChangesets(Collection<Changeset> changesets) {
+            this.changesets.clear();
+            if (changesets != null) {
+                for (Changeset cs: changesets) {
+                    internalAddOrUpdate(cs);
+                }
             }
             fireContentsChanged(this, 0, getSize());
@@ -1055 +1057 @@
                 if (changesets.contains(getSelectedItem())) {
                     setSelectedItem(getSelectedItem());
+                } else if (!this.changesets.isEmpty()){
+                    setSelectedItem(this.changesets.get(0));
                 } else {
-                    setSelectedItem(this.changesets.get(0));
+                    setSelectedItem(null);
                 }
             } else {