Changeset 1760 in josm


Ignore:
Timestamp:
2009-07-11T06:38:06+02:00 (15 years ago)
Author:
Gubaer
Message:

fixed #2882: Handling error message

Location:
trunk/src/org/openstreetmap/josm
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/UploadAction.java

    r1750 r1760  
    530530                return;
    531531            }
     532            // any other API exception
     533            //
     534            else {
     535                ex.printStackTrace();
     536                String msg = tr("<html>Uploading <strong>failed</strong>."
     537                        + "<br>"
     538                        + "{0}"
     539                        + "</html>",
     540                        ex.getDisplayMessage()
     541                );
     542                JOptionPane.showMessageDialog(
     543                        Main.map,
     544                        msg,
     545                        tr("Upload to OSM API failed"),
     546                        JOptionPane.ERROR_MESSAGE
     547                );
     548                return;
     549            }
    532550        }
    533551
    534552        // For any other exception just notify the user
    535553        //
    536         String msg;
    537         if (e.getMessage() == null) {
     554        String msg = e.getMessage();
     555        if (msg == null) {
    538556            msg = e.toString();
    539         } else {
    540             msg = e.getMessage();
    541557        }
    542558        e.printStackTrace();
    543559        JOptionPane.showMessageDialog(
    544                 null,
     560                Main.map,
    545561                msg,
    546562                tr("Upload to OSM API failed"),
  • trunk/src/org/openstreetmap/josm/io/OsmApiException.java

    r1749 r1760  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.io;
     3import static org.openstreetmap.josm.tools.I18n.tr;
    34
    45public class OsmApiException extends OsmTransferException {
     
    6566        }
    6667        if (errorBody != null) {
    67             errorBody = errorBody.replaceAll("^[ \n\t\r]+", "").replaceAll("[ \n\t\r]+$", "");
     68            errorBody = errorBody.trim();
    6869            if(!errorBody.equals(errorHeader)) {
    6970                sb.append(", Error Body=<")
     
    7475        return sb.toString();
    7576    }
     77
     78    /**
     79     * Replies a message suitable to be displayed in a message dialog
     80     *
     81     * @return a message which is suitable to be displayed in a message dialog
     82     */
     83    public String getDisplayMessage() {
     84        StringBuilder sb = new StringBuilder();
     85        if (errorHeader != null) {
     86            sb.append(tr(errorHeader));
     87            sb.append(tr("(Code={0})", responseCode));
     88        } else if (errorBody != null) {
     89            errorBody = errorBody.trim();
     90            sb.append(tr(errorBody));
     91            sb.append(tr("(Code={0})", responseCode));
     92        } else {
     93            sb.append(tr("The server replied an error with code {0}", responseCode));
     94        }
     95        return sb.toString();
     96    }
    7697}
Note: See TracChangeset for help on using the changeset viewer.