Changeset 12575 in josm for trunk/src/org


Ignore:
Timestamp:
2017-08-06T20:24:40+02:00 (8 years ago)
Author:
michael2402
Message:

See #15057: Fix checkstye and sonar issues.

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/OverpassDownloadAction.java

    r12574 r12575  
    106106         * want to save the last query in OverpassQueryList.
    107107         */
    108         Consumer<Collection> errorReporter = (errors) -> {
     108        Consumer<Collection<Object>> errorReporter = errors -> {
    109109
    110110            boolean onlyNoDataError = errors.size() == 1 &&
    … …  
    221221                @Override
    222222                public void focusLost(FocusEvent e) {
    223 
     223                    // ignored
    224224                }
    225225            });
    … …  
    233233                ? BasicArrowButton.EAST
    234234                : BasicArrowButton.WEST);
    235             arrowButton.addActionListener(e ->  {
     235            arrowButton.addActionListener(e -> {
    236236                if (overpassQueryList.isVisible()) {
    237237                    overpassQueryList.setVisible(false);
    … …  
    288288    private static final class QueryWizardDialog extends ExtendedDialog {
    289289
     290        private static final String HEADLINE_START = "<h3>";
     291        private static final String HEADLINE_END = "</h3>";
     292        private static final String TR_START = "<tr>";
     293        private static final String TR_END = "</tr>";
     294        private static final String TD_START = "<td>";
     295        private static final String TD_END = "</td>";
    290296        private static QueryWizardDialog dialog;
    291297        private final HistoryComboBox queryWizard;
    … …  
    329335
    330336            JLabel searchLabel = new JLabel(tr("Search :"));
    331             JTextComponent descPane = this.buildDescriptionSection();
     337            JTextComponent descPane = buildDescriptionSection();
    332338            JScrollPane scroll = GuiHelper.embedInVerticalScrollPane(descPane);
    333339            scroll.getVerticalScrollBar().setUnitIncrement(10); // make scrolling smooth
    … …  
    422428        }
    423429
    424         private JTextComponent buildDescriptionSection() {
    425             JEditorPane descriptionSection = new JEditorPane("text/html", this.getDescriptionContent());
     430        private static JTextComponent buildDescriptionSection() {
     431            JEditorPane descriptionSection = new JEditorPane("text/html", getDescriptionContent());
    426432            descriptionSection.setEditable(false);
    427433            descriptionSection.addHyperlinkListener(e -> {
    … …  
    434440        }
    435441
    436         private String getDescriptionContent() {
     442        private static String getDescriptionContent() {
    437443            return new StringBuilder("<html>")
    438444                    .append(DESCRIPTION_STYLE)
    439445                    .append("<body>")
    440                     .append("<h3>")
     446                    .append(HEADLINE_START)
    441447                    .append(tr("Query Wizard"))
    442                     .append("</h3>")
     448                    .append(HEADLINE_END)
    443449                    .append("<p>")
    444450                    .append(tr("Allows you to interact with <i>Overpass API</i> by writing declarative, human-readable terms."))
    … …  
    447453                    .append(tr("<a href=\"{0}\">OSM Wiki</a>.", Main.getOSMWebsite() + "/wiki/Overpass_turbo/Wizard"))
    448454                    .append("</p>")
    449                     .append("<h3>").append(tr("Hints")).append("</h3>")
    450                     .append("<table>").append("<tr>").append("<td>")
     455                    .append(HEADLINE_START).append(tr("Hints")).append(HEADLINE_END)
     456                    .append("<table>").append(TR_START).append(TD_START)
    451457                    .append(Utils.joinAsHtmlUnorderedList(Arrays.asList("<i>type:node</i>", "<i>type:relation</i>", "<i>type:way</i>")))
    452                     .append("</td>").append("<td>")
     458                    .append(TD_END).append(TD_START)
    453459                    .append("<span>").append(tr("Download objects of a certain type.")).append("</span>")
    454                     .append("</td>").append("</tr>")
    455                     .append("<tr>").append("<td>")
     460                    .append(TD_END).append(TR_END)
     461                    .append(TR_START).append(TD_START)
    456462                    .append(Utils.joinAsHtmlUnorderedList(
    457463                            Arrays.asList("<i>key=value in <u>location</u></i>",
    458464                                    "<i>key=value around <u>location</u></i>",
    459465                                    "<i>key=value in bbox</i>")))
    460                     .append("</td>").append("<td>")
     466                    .append(TD_END).append(TD_START)
    461467                    .append(tr("Download object by specifying a specific location. For example,"))
    462468                    .append(Utils.joinAsHtmlUnorderedList(Arrays.asList(
    … …  
    469475                    .append(tr("Instead of <i>location</i> any valid place name can be used like address, city, etc."))
    470476                    .append("</span>")
    471                     .append("</td>").append("</tr>")
    472                     .append("<tr>").append("<td>")
     477                    .append(TD_END).append(TR_END)
     478                    .append(TR_START).append(TD_START)
    473479                    .append(Utils.joinAsHtmlUnorderedList(Arrays.asList("<i>key=value</i>", "<i>key=*</i>", "<i>key~regex</i>",
    474480                            "<i>key!=value</i>", "<i>key!~regex</i>", "<i>key=\"combined value\"</i>")))
    475                     .append("</td>").append("<td>")
     481                    .append(TD_END).append(TD_START)
    476482                    .append(tr("<span>Download objects that have some concrete key/value pair, only the key with any contents for the value, " +
    477483                            "the value matching some regular expression. 'Not equal' operators are supported as well.</span>"))
    478                     .append("</td>").append("</tr>")
    479                     .append("<tr>").append("<td>")
     484                    .append(TD_END).append(TR_END)
     485                    .append(TR_START).append(TD_START)
    480486                    .append(Utils.joinAsHtmlUnorderedList(Arrays.asList(
    481487                            tr("<i>expression1 {0} expression2</i>", "or"),
    482488                            tr("<i>expression1 {0} expression2</i>", "and"))))
    483                     .append("</td>").append("<td>")
     489                    .append(TD_END).append(TD_START)
    484490                    .append("<span>")
    485491                    .append(tr("Basic logical operators can be used to create more sophisticated queries. Instead of 'or' - '|', '||' " +
    486492                            "can be used, and instead of 'and' - '&', '&&'."))
    487493                    .append("</span>")
    488                     .append("</td>").append("</tr>").append("</table>")
     494                    .append(TD_END).append(TR_END).append("</table>")
    489495                    .append("</body>")
    490496                    .append("</html>")

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

    r12574 r12575  
    3030    private final DownloadTask task;
    3131    private final Future<?> future;
    32     private Consumer<Collection> errorReporter;
     32    private Consumer<Collection<Object>> errorReporter;
    3333
    3434    /**
    35      * constructor
     35     * Creates a new {@link PostDownloadHandler}
    3636     * @param task the asynchronous download task
    3737     * @param future the future on which the completion of the download task can be synchronized
    … …  
    4343
    4444    /**
    45      * constructor
     45     * Creates a new {@link PostDownloadHandler} using a custom error reporter
    4646     * @param task the asynchronous download task
    4747     * @param future the future on which the completion of the download task can be synchronized
    … …  
    4949     *                      task
    5050     */
    51     public PostDownloadHandler(DownloadTask task, Future<?> future, Consumer<Collection> errorReporter) {
     51    public PostDownloadHandler(DownloadTask task, Future<?> future, Consumer<Collection<Object>> errorReporter) {
    5252        this(task, future);
    5353        this.errorReporter = errorReporter;

  • trunk/src/org/openstreetmap/josm/gui/download/OverpassQueryList.java

    r12574 r12575  
    5656public final class OverpassQueryList extends SearchTextResultListPanel<OverpassQueryList.SelectorItem> {
    5757
    58     private final DateTimeFormatter format = DateTimeFormatter.ofPattern("HH:mm:ss, dd-MM-yyyy");
     58    private static final DateTimeFormatter FORMAT = DateTimeFormatter.ofPattern("HH:mm:ss, dd-MM-yyyy");
    5959
    6060    /*
    … …  
    8585        this.target = target;
    8686        this.componentParent = parent;
    87         this.items = this.restorePreferences();
     87        this.items = restorePreferences();
    8888
    8989        OverpassQueryListMouseAdapter mouseHandler = new OverpassQueryListMouseAdapter(lsResult, lsResultModel);
    … …  
    134134        if (!historicExist) {
    135135            SelectorItem item = new SelectorItem(
    136                     "history " + LocalDateTime.now().format(this.format),
     136                    "history " + LocalDateTime.now().format(FORMAT),
    137137                    query);
    138138
    … …  
    261261     * @return A set of the user saved items.
    262262     */
    263     private Map<String, SelectorItem> restorePreferences() {
     263    private static Map<String, SelectorItem> restorePreferences() {
    264264        Collection<Map<String, String>> toRetrieve =
    265265                Main.pref.getListOfStructs(PREFERENCE_ITEMS, Collections.emptyList());
    … …  
    279279    private class OverpassQueryListMouseAdapter extends MouseAdapter {
    280280
    281         private final JList list;
    282         private final ResultListModel model;
     281        private final JList<SelectorItem> list;
     282        private final ResultListModel<SelectorItem> model;
    283283        private final JPopupMenu emptySelectionPopup = new JPopupMenu();
    284284        private final JPopupMenu elementPopup = new JPopupMenu();
    285         private final JPopupMenu queryLookup = new JPopupMenu();
    286 
    287         OverpassQueryListMouseAdapter(JList list, ResultListModel listModel) {
     285
     286        OverpassQueryListMouseAdapter(JList<SelectorItem> list, ResultListModel<SelectorItem> listModel) {
    288287            this.list = list;
    289288            this.model = listModel;
    … …  
    331330            }
    332331
    333             SelectorItem item = (SelectorItem) model.getElementAt(idx);
     332            SelectorItem item = model.getElementAt(idx);
    334333            list.setToolTipText("<html><pre style='width:300px;'>" +
    335334                    Utils.escapeReservedCharactersHTML(Utils.restrictStringLines(item.getQuery(), 9)));
    … …  
    607606
    608607        @Override
    609         public boolean equals(Object o) {
    610             if (this == o) return true;
    611             if (!(o instanceof SelectorItem)) return false;
    612 
    613             SelectorItem that = (SelectorItem) o;
    614 
    615             return itemKey.equals(that.itemKey) &&
    616                     query.equals(that.getKey());
     608        public int hashCode() {
     609            final int prime = 31;
     610            int result = 1;
     611            result = prime * result + ((itemKey == null) ? 0 : itemKey.hashCode());
     612            result = prime * result + ((query == null) ? 0 : query.hashCode());
     613            return result;
    617614        }
    618615
    619616        @Override
    620         public int hashCode() {
    621             return itemKey.hashCode();
     617        public boolean equals(Object obj) {
     618            if (this == obj) {
     619                return true;
     620            }
     621            if (obj == null) {
     622                return false;
     623            }
     624            if (getClass() != obj.getClass()) {
     625                return false;
     626            }
     627            SelectorItem other = (SelectorItem) obj;
     628            if (itemKey == null) {
     629                if (other.itemKey != null) {
     630                    return false;
     631                }
     632            } else if (!itemKey.equals(other.itemKey)) {
     633                return false;
     634            }
     635            if (query == null) {
     636                if (other.query != null) {
     637                    return false;
     638                }
     639            } else if (!query.equals(other.query)) {
     640                return false;
     641            }
     642            return true;
    622643        }
    623644    }
Note: See TracChangeset for help on using the changeset viewer.