Changeset 4864 in josm for trunk/src/org/openstreetmap


Ignore:
Timestamp:
2012-01-24T12:45:14+01:00 (8 years ago)
Author:
Don-vip
Message:

see #7258 - IllegalStateException when attempting download in Changeset Management Dialog (error message has to be translated after stabilization)

Location:
trunk/src/org/openstreetmap/josm/gui/dialogs/changeset
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManager.java

    r3083 r4864  
    366366                return;
    367367
    368             ChangesetQuery query = dialog.getChangesetQuery();
    369             if (query == null) return;
    370             ChangesetQueryTask task = new ChangesetQueryTask(ChangesetCacheManager.this, query);
    371             ChangesetCacheManager.getInstance().runDownloadTask(task);
     368            try {
     369                ChangesetQuery query = dialog.getChangesetQuery();
     370                if (query == null) return;
     371                ChangesetQueryTask task = new ChangesetQueryTask(ChangesetCacheManager.this, query);
     372                ChangesetCacheManager.getInstance().runDownloadTask(task);
     373            } catch (IllegalStateException e) {
     374                JOptionPane.showMessageDialog(ChangesetCacheManager.this, e.getMessage(), tr("Error"), JOptionPane.ERROR_MESSAGE);
     375            }
    372376        }
    373377    }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/BasicChangesetQueryPanel.java

    r3530 r4864  
    233233                query = query.forUser(im.getUserId()).beingOpen(true);
    234234            } else
    235                 // anonymous -- should not happen. Message doesn't have to be translated.
     235                // anonymous -- can happen with a fresh config. FIXME: Message have to be translated.
    236236                throw new IllegalStateException("Cannot create changeset query for open changesets of anonymous user");
    237237        }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/ChangesetQueryDialog.java

    r3757 r4864  
    177177
    178178        public void actionPerformed(ActionEvent arg0) {
    179             switch(tpQueryPanels.getSelectedIndex()) {
    180             case 0:
    181                 // currently, query specifications can't be invalid in the basic query panel.
    182                 // We select from a couple of predefined queries and there is always a query
    183                 // selected
    184                 break;
    185             case 1:
    186                 if (getChangesetQuery() == null) {
    187                     alertInvalidChangesetQuery();
    188                     pnlUrlBasedQueries.startUserInput();
    189                     return;
     179            try {
     180                switch(tpQueryPanels.getSelectedIndex()) {
     181                case 0:
     182                    // currently, query specifications can't be invalid in the basic query panel.
     183                    // We select from a couple of predefined queries and there is always a query
     184                    // selected
     185                    break;
     186                case 1:
     187                    if (getChangesetQuery() == null) {
     188                        alertInvalidChangesetQuery();
     189                        pnlUrlBasedQueries.startUserInput();
     190                        return;
     191                    }
     192                    break;
     193   
     194                case 2:
     195                    if (getChangesetQuery() == null) {
     196                        pnlAdvancedQueries.displayMessageIfInvalid();
     197                        return;
     198                    }
    190199                }
    191                 break;
    192 
    193             case 2:
    194                 if (getChangesetQuery() == null) {
    195                     pnlAdvancedQueries.displayMessageIfInvalid();
    196                     return;
    197                 }
     200                setCanceled(false);
     201                setVisible(false);
     202            } catch (IllegalStateException e) {
     203                JOptionPane.showMessageDialog(ChangesetQueryDialog.this, e.getMessage(), tr("Error"), JOptionPane.ERROR_MESSAGE);
    198204            }
    199             setCanceled(false);
    200             setVisible(false);
    201205        }
    202206    }
Note: See TracChangeset for help on using the changeset viewer.