Changeset 12578 in josm for trunk


Ignore:
Timestamp:
2017-08-06T22:38:35+02:00 (7 years ago)
Author:
Don-vip
Message:

fix some Sonar issues

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

Legend:

Unmodified
Added
Removed

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

    r12576 r12578  
    228228                    overpassQueryList.setVisible(false);
    229229                    arrowButton.setDirection(BasicArrowButton.WEST);
    230                     OVERPASS_QUERY_LIST_OPENED.put(false);
     230                    OVERPASS_QUERY_LIST_OPENED.put(Boolean.FALSE);
    231231                } else {
    232232                    overpassQueryList.setVisible(true);
    233233                    arrowButton.setDirection(BasicArrowButton.EAST);
    234                     OVERPASS_QUERY_LIST_OPENED.put(false);
     234                    OVERPASS_QUERY_LIST_OPENED.put(Boolean.FALSE);
    235235                }
    236236            });

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

    r12547 r12578  
    10031003    public static class SearchSettingsActionParameter extends ActionParameter<SearchSetting> {
    10041004
     1005        /**
     1006         * Constructs a new {@code SearchSettingsActionParameter}.
     1007         * @param name parameter name (the key)
     1008         */
    10051009        public SearchSettingsActionParameter(String name) {
    10061010            super(name);

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

    r12577 r12578  
    184184                componentParent,
    185185                tr("Edit item"),
    186                 item.getKey(),
    187                 item.getQuery(),
    188                 new String[] {tr("Save")});
     186                item,
     187                tr("Save"));
    189188        dialog.showDialog();
    190189
    … …  
    428427
    429428        EditItemDialog(Component parent, String title, String... buttonTexts) {
    430             this(parent, title, "", "", buttonTexts);
     429            this(parent, title, null, buttonTexts);
    431430        }
    432431
    … …  
    434433                Component parent,
    435434                String title,
    436                 String nameToEdit,
    437                 String queryToEdit,
     435                SelectorItem itemToEdit,
    438436                String... buttonTexts) {
    439437            super(parent, title, buttonTexts);
    440438
     439            String nameToEdit = itemToEdit != null ? itemToEdit.getKey() : "";
     440            String queryToEdit = itemToEdit != null ? itemToEdit.getQuery() : "";
    441441            this.initialNameHash = nameToEdit.hashCode();
    442442
Note: See TracChangeset for help on using the changeset viewer.