Ignore:
Timestamp:
2013-12-24T21:12:40+01:00 (10 years ago)
Author:
Don-vip
Message:

global use of Utils.joinAsHtmlUnorderedList()

Location:
trunk/src/org/openstreetmap/josm/actions
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java

    r6380 r6524  
    88import java.net.URL;
    99import java.net.URLEncoder;
     10import java.util.ArrayList;
    1011import java.util.Collection;
    1112import java.util.concurrent.Future;
    … …  
    2930import org.openstreetmap.josm.io.OsmTransferCanceledException;
    3031import org.openstreetmap.josm.io.OsmTransferException;
     32import org.openstreetmap.josm.tools.Utils;
    3133import org.xml.sax.SAXException;
    3234
    … …  
    322324            if (urlString.matches(PATTERN_OSM_API_URL)) {
    323325                // TODO: proper i18n after stabilization
    324                 String message = "<ul><li>"+tr("OSM Server URL:") + " " + url.getHost() + "</li><li>" +
    325                         tr("Command")+": "+url.getPath()+"</li>";
     326                Collection<String> items = new ArrayList<String>();
     327                items.add(tr("OSM Server URL:") + " " + url.getHost());
     328                items.add(tr("Command")+": "+url.getPath());
    326329                if (url.getQuery() != null) {
    327                     message += "<li>" + tr("Request details: {0}", url.getQuery().replaceAll(",\\s*", ", ")) + "</li>";
    328                 }
    329                 message += "</ul>";
    330                 return message;
     330                    items.add(tr("Request details: {0}", url.getQuery().replaceAll(",\\s*", ", ")));
     331                }
     332                return Utils.joinAsHtmlUnorderedList(items);
    331333            }
    332334            // TODO: other APIs

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadTaskList.java

    r6336 r6524  
    3434import org.openstreetmap.josm.tools.ExceptionUtil;
    3535import org.openstreetmap.josm.tools.ImageProvider;
     36import org.openstreetmap.josm.tools.Utils;
    3637
    3738/**
    … …  
    257258            }
    258259            if (!errors.isEmpty()) {
    259                 final StringBuilder sb = new StringBuilder();
     260                final Collection<String> items = new ArrayList<String>();
    260261                for (Object error : errors) {
    261262                    if (error instanceof String) {
    262                         sb.append("<li>").append(error).append("</li>").append("<br>");
     263                        items.add((String) error);
    263264                    } else if (error instanceof Exception) {
    264                         sb.append("<li>").append(ExceptionUtil.explainException((Exception) error)).append("</li>")
    265                         .append("<br>");
     265                        items.add(ExceptionUtil.explainException((Exception) error));
    266266                    }
    267267                }
    268                 sb.insert(0, "<ul>");
    269                 sb.append("</ul>");
    270268
    271269                GuiHelper.runInEDT(new Runnable() {
    … …  
    273271                    public void run() {
    274272                        JOptionPane.showMessageDialog(Main.parent, "<html>"
    275                                 + tr("The following errors occurred during mass download: {0}", sb.toString()) + "</html>",
     273                                + tr("The following errors occurred during mass download: {0}",
     274                                        Utils.joinAsHtmlUnorderedList(items)) + "</html>",
    276275                                tr("Errors during download"), JOptionPane.ERROR_MESSAGE);
    277276                    }

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java

    r6084 r6524  
    55
    66import java.util.ArrayList;
     7import java.util.Collection;
    78import java.util.LinkedHashSet;
    89import java.util.List;
    … …  
    1516import org.openstreetmap.josm.gui.ExceptionDialogUtil;
    1617import org.openstreetmap.josm.tools.ExceptionUtil;
     18import org.openstreetmap.josm.tools.Utils;
    1719
    1820public class PostDownloadHandler implements Runnable {
    … …  
    104106        //
    105107        if (!errors.isEmpty()) {
    106             final StringBuffer sb = new StringBuffer();
     108            final Collection<String> items = new ArrayList<String>();
    107109            for (Object error:errors) {
    108110                if (error instanceof String) {
    109                     sb.append("<li>").append(error).append("</li>").append("<br>");
     111                    items.add((String) error);
    110112                } else if (error instanceof Exception) {
    111                     sb.append("<li>").append(ExceptionUtil.explainException((Exception)error)).append("</li>").append("<br>");
     113                    items.add(ExceptionUtil.explainException((Exception)error));
    112114                }
    113115            }
    114             sb.insert(0, "<html><ul>");
    115             sb.append("</ul></html>");
    116116
    117117            SwingUtilities.invokeLater(new Runnable() {
    … …  
    120120                    JOptionPane.showMessageDialog(
    121121                            Main.parent,
    122                             sb.toString(),
     122                            "<html>"+Utils.joinAsHtmlUnorderedList(items)+"</html>",
    123123                            tr("Errors during download"),
    124124                            JOptionPane.ERROR_MESSAGE);

  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r6429 r6524  
    189189                label.setToolTipText("<html>"
    190190                        + description
    191                         + (examples.length > 0 ? ("<ul><li>" + Utils.join("</li><li>", Arrays.asList(examples)) + "</li></ul>") : "")
     191                        + (examples.length > 0 ? Utils.joinAsHtmlUnorderedList(Arrays.asList(examples)) : "")
    192192                        + "</html>");
    193193            }
Note: See TracChangeset for help on using the changeset viewer.