Ignore:
Timestamp:
2016-07-06T22:51:51+02:00 (8 years ago)
Author:
Don-vip
Message:

fix #13115 - fix NPE, improve WMS errors handling

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/preferences/imagery/AddWMSLayerPanel.java

    r10179 r10520  
    8484                    formats.setSelectedItem(wms.getPreferredFormats());
    8585                } catch (MalformedURLException ex) {
     86                    Main.error(ex, false);
    8687                    JOptionPane.showMessageDialog(getParent(), tr("Invalid service URL."),
    8788                            tr("WMS Error"), JOptionPane.ERROR_MESSAGE);
    8889                } catch (IOException ex) {
     90                    Main.error(ex, false);
    8991                    JOptionPane.showMessageDialog(getParent(), tr("Could not retrieve WMS layer list."),
    9092                            tr("WMS Error"), JOptionPane.ERROR_MESSAGE);
    … …  
    9395                    String title = tr("WMS Error");
    9496                    String message = tr("Could not parse WMS layer list.");
    95                     Main.error("Could not parse WMS layer list. Incoming data:\n"+incomingData);
    96                     if (incomingData != null
    97                             && (incomingData.startsWith("<html>") || incomingData.startsWith("<HTML>"))
    98                             && (incomingData.endsWith("</html>") || incomingData.endsWith("</HTML>"))) {
     97                    Main.error(ex, "Could not parse WMS layer list. Incoming data:\n"+incomingData);
     98                    if ((incomingData.startsWith("<html>") || incomingData.startsWith("<HTML>"))
     99                      && (incomingData.endsWith("</html>") || incomingData.endsWith("</HTML>"))) {
    99100                        GuiHelper.notifyUserHtmlError(AddWMSLayerPanel.this, title, message, incomingData);
    100101                    } else {
     102                        if (ex.getMessage() != null) {
     103                            message += '\n' + ex.getMessage();
     104                        }
    101105                        JOptionPane.showMessageDialog(getParent(), message, title, JOptionPane.ERROR_MESSAGE);
    102106                    }
Note: See TracChangeset for help on using the changeset viewer.