Changeset 1546 in josm for trunk/src/org/openstreetmap/josm/actions

Timestamp:

2009-04-22T02:39:47+02:00 (16 years ago)

Author:

framm

Message:

• streamline uploading by requesting commit message as part of the upload confirmation dialog; allow user to re-use latest commit message

File:

• trunk/src/org/openstreetmap/josm/actions/UploadAction.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/actions/UploadAction.java

@@ -15 +15 @@
 import javax.swing.JPanel;
 import javax.swing.JScrollPane;
+import javax.swing.JTextField;
 
 import org.openstreetmap.josm.Main;
@@ -36 +37 @@
 public class UploadAction extends JosmAction {
 
+    /**
+     * Last commit message used for uploading changes.
+     * FIXME save this in preferences, or even offer list of 10 last recently used comments?
+     * FIXME ugly hack; value is filled here and retrieved in the OsmApi class; find better way
+     */
+    public static String lastCommitComment; 
+    
     /** Upload Hook */
     public interface UploadHook {
@@ -98 +106 @@
                     p.add(new JScrollPane(l), GBC.eol().fill());
                 }
+                
+                p.add(new JLabel(tr("Provide a brief comment for the changes you are uploading:")), GBC.eol().insets(0, 5, 10, 3));
+                final JTextField cmt = new JTextField(lastCommitComment);
+                p.add(cmt, GBC.eol().fill(GBC.HORIZONTAL));
 
-                return new ExtendedDialog(Main.parent, 
+                while(true) {
+                    int result = new ExtendedDialog(Main.parent, 
                         tr("Upload these changes?"), 
                         p,
                         new String[] {tr("Upload Changes"), tr("Cancel")}, 
-                        new String[] {"upload.png", "cancel.png"}).getValue() == 1;  
+                        new String[] {"upload.png", "cancel.png"}).getValue();
+                    
+                    // cancel pressed
+                    if (result != 1) return false;
+                    
+                    // don't allow empty commit message
+                    if (cmt.getText().trim().length() < 3) continue;
+                    
+                    lastCommitComment = cmt.getText().trim();
+                    break;
+                }
+                return true;
             }
         });