Changeset 6990 in josm


Ignore:
Timestamp:
2014-04-19T00:36:43+02:00 (6 years ago)
Author:
Don-vip
Message:

sonar - place Strings literals on the left side when checking for equality (can avoid NPEs)

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

Legend:

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

    r6979 r6990  
    10561056        } else if (os.toLowerCase().startsWith("windows")) {
    10571057            platform = new PlatformHookWindows();
    1058         } else if (os.equals("Linux") || os.equals("Solaris") ||
    1059                 os.equals("SunOS") || os.equals("AIX") ||
    1060                 os.equals("FreeBSD") || os.equals("NetBSD") || os.equals("OpenBSD")) {
     1058        } else if ("Linux".equals(os) || "Solaris".equals(os) ||
     1059                "SunOS".equals(os) || "AIX".equals(os) ||
     1060                "FreeBSD".equals(os) || "NetBSD".equals(os) || "OpenBSD".equals(os)) {
    10611061            platform = new PlatformHookUnixoid();
    10621062        } else if (os.toLowerCase().startsWith("mac os x")) {
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadGpsTask.java

    r6920 r6990  
    8181            String[] table = url.split("\\?|=|&");
    8282            for (int i = 0; i<table.length; i++) {
    83                 if (table[i].equals("bbox") && i<table.length-1 )
     83                if ("bbox".equals(table[i]) && i<table.length-1 )
    8484                    return download(newLayer, new Bounds(table[i+1], ",", ParseMethod.LEFT_BOTTOM_RIGHT_TOP), progressMonitor);
    8585            }
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r6883 r6990  
    502502                String mv = null;
    503503
    504                 if (key.equals("timestamp")) {
     504                if ("timestamp".equals(key)) {
    505505                    mv = DateUtils.fromDate(osm.getTimestamp());
    506506                } else {
     
    603603            }
    604604
    605             if (regexp && key.length() > 0 && !key.equals("*")) {
     605            if (regexp && key.length() > 0 && !"*".equals(key)) {
    606606                try {
    607607                    keyPattern = Pattern.compile(key, regexFlags(false));
     
    614614                keyPattern = null;
    615615            }
    616             if (regexp && this.value.length() > 0 && !this.value.equals("*")) {
     616            if (regexp && this.value.length() > 0 && !"*".equals(this.value)) {
    617617                try {
    618618                    valuePattern = Pattern.compile(this.value, regexFlags(false));
     
    769769        private String user;
    770770        public UserMatch(String user) {
    771             if (user.equals("anonymous")) {
     771            if ("anonymous".equals(user)) {
    772772                this.user = null;
    773773            } else {
     
    13141314            value = "";
    13151315        }
    1316         if (key.equals("type"))
     1316        if ("type".equals(key))
    13171317            return new ExactType(value);
    1318         else if (key.equals("user"))
     1318        else if ("user".equals(key))
    13191319            return new UserMatch(value);
    1320         else if (key.equals("role"))
     1320        else if ("role".equals(key))
    13211321            return new RoleMatch(value);
    13221322        else
  • trunk/src/org/openstreetmap/josm/corrector/ReverseWayTagCorrector.java

    r6987 r6990  
    191191        String sign = matcher.group(1);
    192192        String rest = matcher.group(2);
    193         sign = sign.equals("-") ? "" : "-";
     193        sign = "-".equals(sign) ? "" : "-";
    194194        return sign + rest;
    195195    }
  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r6986 r6990  
    13651365            int event = parser.next();
    13661366            if (event == XMLStreamConstants.START_ELEMENT) {
    1367                 if (parser.getLocalName().equals("tag")) {
     1367                String localName = parser.getLocalName();
     1368                if ("tag".equals(localName)) {
    13681369                    settingsMap.put(parser.getAttributeValue(null, "key"), new StringSetting(parser.getAttributeValue(null, "value")));
    13691370                    jumpToEnd();
    1370                 } else if (parser.getLocalName().equals("list") ||
    1371                         parser.getLocalName().equals("collection") ||
    1372                         parser.getLocalName().equals("lists") ||
    1373                         parser.getLocalName().equals("maps")
     1371                } else if ("list".equals(localName) ||
     1372                        "collection".equals(localName) ||
     1373                        "lists".equals(localName) ||
     1374                        "maps".equals(localName)
    13741375                ) {
    13751376                    parseToplevelList();
    13761377                } else {
    1377                     throwException("Unexpected element: "+parser.getLocalName());
     1378                    throwException("Unexpected element: "+localName);
    13781379                }
    13791380            } else if (event == XMLStreamConstants.END_ELEMENT) {
     
    14041405            int event = parser.next();
    14051406            if (event == XMLStreamConstants.START_ELEMENT) {
    1406                 if (parser.getLocalName().equals("entry")) {
     1407                String localName = parser.getLocalName();
     1408                if ("entry".equals(localName)) {
    14071409                    if (entries == null) {
    14081410                        entries = new ArrayList<String>();
     
    14101412                    entries.add(parser.getAttributeValue(null, "value"));
    14111413                    jumpToEnd();
    1412                 } else if (parser.getLocalName().equals("list")) {
     1414                } else if ("list".equals(localName)) {
    14131415                    if (lists == null) {
    14141416                        lists = new ArrayList<List<String>>();
    14151417                    }
    14161418                    lists.add(parseInnerList());
    1417                 } else if (parser.getLocalName().equals("map")) {
     1419                } else if ("map".equals(localName)) {
    14181420                    if (maps == null) {
    14191421                        maps = new ArrayList<Map<String, String>>();
     
    14211423                    maps.add(parseMap());
    14221424                } else {
    1423                     throwException("Unexpected element: "+parser.getLocalName());
     1425                    throwException("Unexpected element: "+localName);
    14241426                }
    14251427            } else if (event == XMLStreamConstants.END_ELEMENT) {
     
    14341436            settingsMap.put(key, new MapListSetting(Collections.unmodifiableList(maps)));
    14351437        } else {
    1436             if (name.equals("lists")) {
     1438            if ("lists".equals(name)) {
    14371439                settingsMap.put(key, new ListListSetting(Collections.<List<String>>emptyList()));
    1438             } else if (name.equals("maps")) {
     1440            } else if ("maps".equals(name)) {
    14391441                settingsMap.put(key, new MapListSetting(Collections.<Map<String, String>>emptyList()));
    14401442            } else {
     
    14491451            int event = parser.next();
    14501452            if (event == XMLStreamConstants.START_ELEMENT) {
    1451                 if (parser.getLocalName().equals("entry")) {
     1453                if ("entry".equals(parser.getLocalName())) {
    14521454                    entries.add(parser.getAttributeValue(null, "value"));
    14531455                    jumpToEnd();
     
    14671469            int event = parser.next();
    14681470            if (event == XMLStreamConstants.START_ELEMENT) {
    1469                 if (parser.getLocalName().equals("tag")) {
     1471                if ("tag".equals(parser.getLocalName())) {
    14701472                    map.put(parser.getAttributeValue(null, "key"), parser.getAttributeValue(null, "value"));
    14711473                    jumpToEnd();
     
    15001502        @Override
    15011503        public void visit(StringSetting setting) {
    1502             if (noPassword && key.equals("osm-server.password"))
     1504            if (noPassword && "osm-server.password".equals(key))
    15031505                return; // do not store plain password.
    15041506            /* don't save default values */
  • trunk/src/org/openstreetmap/josm/data/Version.java

    r6889 r6990  
    131131        isLocalBuild = false;
    132132        value = properties.get("Is-Local-Build");
    133         if (value != null && value.trim().equalsIgnoreCase("true"))  {
     133        if (value != null && "true".equalsIgnoreCase(value.trim()))  {
    134134            isLocalBuild = true;
    135135        }
  • trunk/src/org/openstreetmap/josm/data/coor/LatLon.java

    r6920 r6990  
    136136        String sSeconds = cDmsSecondFormatter.format(tSeconds);
    137137
    138         if (sSeconds.equals(cDms60)) {
     138        if (cDms60.equals(sSeconds)) {
    139139            sSeconds = cDms00;
    140140            sMinutes = cDmsMinuteFormatter.format(tMinutes+1);
    141141        }
    142         if (sMinutes.equals("60")) {
     142        if ("60".equals(sMinutes)) {
    143143            sMinutes = "00";
    144144            sDegrees = Integer.toString(tDegree+1);
  • trunk/src/org/openstreetmap/josm/data/imagery/ImageryInfo.java

    r6968 r6990  
    765765    public void setAttribution(AbstractTileSource s) {
    766766        if (attributionText != null) {
    767             if (attributionText.equals("osm")) {
     767            if ("osm".equals(attributionText)) {
    768768                s.setAttributionText(new Mapnik().getAttributionText(0, null, null));
    769769            } else {
     
    772772        }
    773773        if (attributionLinkURL != null) {
    774             if (attributionLinkURL.equals("osm")) {
     774            if ("osm".equals(attributionLinkURL)) {
    775775                s.setAttributionLinkURL(new Mapnik().getAttributionLinkURL());
    776776            } else {
     
    791791        }
    792792        if (termsOfUseURL != null) {
    793             if (termsOfUseURL.equals("osm")) {
     793            if ("osm".equals(termsOfUseURL)) {
    794794                s.setTermsOfUseURL(new Mapnik().getTermsOfUseURL());
    795795            } else {
  • trunk/src/org/openstreetmap/josm/data/osm/ChangesetCache.java

    r6889 r6990  
    175175    }
    176176
     177    /**
     178     * Replies the list of open changesets.
     179     * @return The list of open changesets
     180     */
    177181    public List<Changeset> getOpenChangesets() {
    178182        List<Changeset> ret = new ArrayList<Changeset>();
     
    190194    @Override
    191195    public void preferenceChanged(PreferenceChangeEvent e) {
    192         if (e.getKey() == null || ! e.getKey().equals("osm-server.url"))
     196        if (e.getKey() == null || !"osm-server.url".equals(e.getKey()))
    193197            return;
    194198
  • trunk/src/org/openstreetmap/josm/data/osm/TigerUtils.java

    r6380 r6990  
    2828    }
    2929
     30    /**
     31     * Determines if the given key denotes an integer value.
     32     * @param name The key to determine
     33     * @return {@code true} if the given key denotes an integer value
     34     */
    3035    public static boolean tagIsInt(String name) {
    31         if (name.equals("tiger:tlid"))
     36        if ("tiger:tlid".equals(name))
    3237            return true;
    3338        return false;
  • trunk/src/org/openstreetmap/josm/data/projection/CustomProjection.java

    r6890 r6990  
    173173                String key = m.group(1);
    174174                // alias
    175                 if (key.equals("k")) {
     175                if ("k".equals(key)) {
    176176                    key = Param.k_0.key;
    177177                }
     
    181181                    // some aliases
    182182                    if (key.equals(Param.proj.key)) {
    183                         if (value.equals("longlat") || value.equals("latlon") || value.equals("latlong")) {
     183                        if ("longlat".equals(value) || "latlon".equals(value) || "latlong".equals(value)) {
    184184                            value = "lonlat";
    185185                        }
     
    264264                nadgridsId = nadgridsId.substring(1);
    265265            }
    266             if (nadgridsId.equals("null"))
     266            if ("null".equals(nadgridsId))
    267267                return new NullDatum(null, ellps);
    268268            NTV2GridShiftFileWrapper nadgrids = Projections.getNTV2Grid(nadgridsId);
  • trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2GridShiftFile.java

    r6830 r6990  
    169169        HashMap<String, List<NTV2SubGrid>> subGridMap = new HashMap<String, List<NTV2SubGrid>>();
    170170        for (int i = 0; i < subGrid.length; i++) {
    171             if (subGrid[i].getParentSubGridName().equalsIgnoreCase("NONE")) {
     171            if ("NONE".equalsIgnoreCase(subGrid[i].getParentSubGridName())) {
    172172                topLevelCount++;
    173173            }
     
    177177        topLevelCount = 0;
    178178        for (int i = 0; i < subGrid.length; i++) {
    179             if (subGrid[i].getParentSubGridName().equalsIgnoreCase("NONE")) {
     179            if ("NONE".equalsIgnoreCase(subGrid[i].getParentSubGridName())) {
    180180                topLevelSubGrid[topLevelCount++] = subGrid[i];
    181181            } else {
  • trunk/src/org/openstreetmap/josm/data/validation/Test.java

    r6623 r6990  
    327327    protected static final boolean isBuilding(OsmPrimitive p) {
    328328        String v = p.get("building");
    329         return v != null && !v.equals("no") && !v.equals("entrance");
     329        return v != null && !"no".equals(v) && !"entrance".equals(v);
    330330    }
    331331}
  • trunk/src/org/openstreetmap/josm/gui/DefaultNameFormatter.java

    r6986 r6990  
    445445
    446446    private String getNameTagValue(IRelation relation, String nameTag) {
    447         if (nameTag.equals("name")) {
     447        if ("name".equals(nameTag)) {
    448448            if (Main.pref.getBoolean("osm-primitives.localize-name", true))
    449449                return relation.getLocalName();
    450450            else
    451451                return relation.getName();
    452         } else if (nameTag.equals(":LocationCode")) {
     452        } else if (":LocationCode".equals(nameTag)) {
    453453            for (String m : relation.keySet()) {
    454454                if (m.endsWith(nameTag))
  • trunk/src/org/openstreetmap/josm/gui/IconToggleButton.java

    r6380 r6990  
    6262    @Override
    6363    public void propertyChange(PropertyChangeEvent evt) {
    64         if (evt.getPropertyName().equals("active")) {
     64        if ("active".equals(evt.getPropertyName())) {
    6565            setSelected((Boolean)evt.getNewValue());
    6666            requestFocusInWindow();
    67         } else if (evt.getPropertyName().equals("selected")) {
     67        } else if ("selected".equals(evt.getPropertyName())) {
    6868            setSelected((Boolean)evt.getNewValue());
    6969        }
  • trunk/src/org/openstreetmap/josm/gui/JosmUserIdentityManager.java

    r6889 r6990  
    2121
    2222/**
    23  * JosmUserStateManager is a global object which keeps track of what JOSM knows about
     23 * JosmUserIdentityManager is a global object which keeps track of what JOSM knows about
    2424 * the identity of the current user.
    2525 *
     
    3737 * of the current user.
    3838 *
    39  * The global JosmUserStateManager listens to {@link PreferenceChangeEvent}s and keeps track
     39 * The global JosmUserIdentityManager listens to {@link PreferenceChangeEvent}s and keeps track
    4040 * of what the current JOSM instance knows about the current user. Other subsystems can
    41  * let the global JosmUserStateManager know in case they fully identify the current user, see
     41 * let the global JosmUserIdentityManager know in case they fully identify the current user, see
    4242 * {@link #setFullyIdentified}.
    4343 *
    44  * The information kept by the JosmUserStateManager can be used to
     44 * The information kept by the JosmUserIdentityManager can be used to
    4545 * <ul>
    4646 *   <li>safely query changesets owned by the current user based on its user id, not on its user name</li>
     
    242242    @Override
    243243    public void preferenceChanged(PreferenceChangeEvent evt) {
    244         if (evt.getKey().equals("osm-server.username")) {
     244        if ("osm-server.username".equals(evt.getKey())) {
    245245            if (!(evt.getNewValue() instanceof StringSetting)) return;
    246246            String newValue = ((StringSetting) evt.getNewValue()).getValue();
     
    254254            return;
    255255
    256         } else if (evt.getKey().equals("osm-server.url")) {
     256        } else if ("osm-server.url".equals(evt.getKey())) {
    257257            if (!(evt.getNewValue() instanceof StringSetting)) return;
    258258            String newValue = ((StringSetting) evt.getNewValue()).getValue();
     
    263263            }
    264264
    265         } else if (evt.getKey().equals("oauth.access-token.key")) {
     265        } else if ("oauth.access-token.key".equals(evt.getKey())) {
    266266            accessTokenKeyChanged = true;
    267267
    268         } else if (evt.getKey().equals("oauth.access-token.secret")) {
     268        } else if ("oauth.access-token.secret".equals(evt.getKey())) {
    269269            accessTokenSecretChanged = true;
    270270        }
  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r6889 r6990  
    753753            @Override
    754754            public void preferenceChanged(PreferenceChangeEvent e) {
    755                 if (e.getKey().equals("audio.menuinvisible")) {
     755                if ("audio.menuinvisible".equals(e.getKey())) {
    756756                    showAudioMenu(!Boolean.parseBoolean(e.getNewValue().toString()));
    757757                }
  • trunk/src/org/openstreetmap/josm/gui/MapMover.java

    r6989 r6990  
    4141        @Override
    4242        public void actionPerformed(ActionEvent e) {
    43             if (action.equals(".") || action.equals(",")) {
     43            if (".".equals(action) || ",".equals(action)) {
    4444                Point mouse = nc.getMousePosition();
    4545                if (mouse == null)
    4646                    mouse = new Point((int)nc.getBounds().getCenterX(), (int)nc.getBounds().getCenterY());
    47                 MouseWheelEvent we = new MouseWheelEvent(nc, e.getID(), e.getWhen(), e.getModifiers(), mouse.x, mouse.y, 0, false, MouseWheelEvent.WHEEL_UNIT_SCROLL, 1, action.equals(",") ? -1 : 1);
     47                MouseWheelEvent we = new MouseWheelEvent(nc, e.getID(), e.getWhen(), e.getModifiers(), mouse.x, mouse.y, 0, false,
     48                        MouseWheelEvent.WHEEL_UNIT_SCROLL, 1, ",".equals(action) ? -1 : 1);
    4849                mouseWheelMoved(we);
    4950            } else {
    5051                EastNorth center = nc.getCenter();
    5152                EastNorth newcenter = nc.getEastNorth(nc.getWidth()/2+nc.getWidth()/5, nc.getHeight()/2+nc.getHeight()/5);
    52                 if (action.equals("left"))
     53                if ("left".equals(action))
    5354                    nc.zoomTo(new EastNorth(2*center.east()-newcenter.east(), center.north()));
    54                 else if (action.equals("right"))
     55                else if ("right".equals(action))
    5556                    nc.zoomTo(new EastNorth(newcenter.east(), center.north()));
    56                 else if (action.equals("up"))
     57                else if ("up".equals(action))
    5758                    nc.zoomTo(new EastNorth(center.east(), 2*center.north()-newcenter.north()));
    58                 else if (action.equals("down"))
     59                else if ("down".equals(action))
    5960                    nc.zoomTo(new EastNorth(center.east(), newcenter.north()));
    6061            }
  • trunk/src/org/openstreetmap/josm/gui/SelectionManager.java

    r6380 r6990  
    279279    @Override
    280280    public void propertyChange(PropertyChangeEvent evt) {
    281         if (evt.getPropertyName().equals("active") && !(Boolean)evt.getNewValue() && mousePosStart != null) {
     281        if ("active".equals(evt.getPropertyName()) && !(Boolean)evt.getNewValue() && mousePosStart != null) {
    282282            paintRect();
    283283            mousePosStart = null;
  • trunk/src/org/openstreetmap/josm/gui/dialogs/LatLonDialog.java

    r6753 r6990  
    363363                sb.append("x");     // cardinal direction
    364364                String c = m.group(7).toUpperCase();
    365                 if (c.equals("N") || c.equals("S") || c.equals("E") || c.equals("W")) {
     365                if ("N".equals(c) || "S".equals(c) || "E".equals(c) || "W".equals(c)) {
    366366                    list.add(c);
    367367                } else {
     
    461461
    462462        double coord = (coordDeg < 0 ? -1 : 1) * (Math.abs(coordDeg) + coordMin / 60 + coordSec / 3600);
    463         coord = card.equals("N") || card.equals("E") ? coord : -coord;
    464         if (card.equals("N") || card.equals("S")) {
     463        coord = "N".equals(card) || "E".equals(card) ? coord : -coord;
     464        if ("N".equals(card) || "S".equals(card)) {
    465465            latLon.lat = coord;
    466466        } else {
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/AdvancedChangesetQueryPanel.java

    r6890 r6990  
    362362            String prefRoot = "changeset-query.advanced.open-restrictions";
    363363            String v = Main.pref.get(prefRoot + ".query-type", "open");
    364             rbBoth.setSelected(v.equals("both"));
    365             rbOpenOnly.setSelected(v.equals("open"));
    366             rbClosedOnly.setSelected(v.equals("closed"));
     364            rbBoth.setSelected("both".equals(v));
     365            rbOpenOnly.setSelected("open".equals(v));
     366            rbClosedOnly.setSelected("closed".equals(v));
    367367        }
    368368    }
     
    599599            String prefRoot = "changeset-query.advanced.user-restrictions";
    600600            String v = Main.pref.get(prefRoot + ".query-type", "mine");
    601             if (v.equals("mine")) {
     601            if ("mine".equals(v)) {
    602602                JosmUserIdentityManager im = JosmUserIdentityManager.getInstance();
    603603                if (im.isAnonymous()) {
     
    606606                    rbRestrictToMyself.setSelected(true);
    607607                }
    608             } else if (v.equals("uid")) {
     608            } else if ("uid".equals(v)) {
    609609                rbRestrictToUid.setSelected(true);
    610             } else if (v.equals("username")) {
     610            } else if ("username".equals(v)) {
    611611                rbRestrictToUserName.setSelected(true);
    612612            }
     
    896896            String prefRoot = "changeset-query.advanced.open-restrictions";
    897897            String v = Main.pref.get(prefRoot + ".query-type", "closed-after");
    898             rbClosedAfter.setSelected(v.equals("closed-after"));
    899             rbClosedAfterAndCreatedBefore.setSelected(v.equals("closed-after-created-before"));
     898            rbClosedAfter.setSelected("closed-after".equals(v));
     899            rbClosedAfterAndCreatedBefore.setSelected("closed-after-created-before".equals(v));
    900900            if (!rbClosedAfter.isSelected() && !rbClosedAfterAndCreatedBefore.isSelected()) {
    901901                rbClosedAfter.setSelected(true);
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/UrlBasedQueryPanel.java

    r6920 r6990  
    169169        }
    170170        protected void feedbackValid() {
    171             if (getCurrentFeedback().equals("valid")) return;
     171            if ("valid".equals(getCurrentFeedback())) return;
    172172            lblValid.setIcon(ImageProvider.get("dialogs", "valid"));
    173173            lblValid.setToolTipText("");
     
    176176
    177177        protected void feedbackInvalid() {
    178             if (getCurrentFeedback().equals("invalid")) return;
     178            if ("invalid".equals(getCurrentFeedback())) return;
    179179            lblValid.setIcon(ImageProvider.get("warning-small"));
    180180            lblValid.setToolTipText(tr("This changeset query URL is invalid"));
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationNodeMap.java

    r6830 r6990  
    6363    public static Node firstOnewayNode(RelationMember m){
    6464        if(!m.isWay()) return null;
    65         if(m.getRole().equals("backward")) return m.getWay().lastNode();
     65        if("backward".equals(m.getRole())) {
     66            return m.getWay().lastNode();
     67        }
    6668        return m.getWay().firstNode();
    6769    }
     
    6971    public static Node lastOnewayNode(RelationMember m){
    7072        if(!m.isWay()) return null;
    71         if(m.getRole().equals("backward")) return m.getWay().firstNode();
     73        if("backward".equals(m.getRole())) {
     74            return m.getWay().firstNode();
     75        }
    7276        return m.getWay().lastNode();
    7377    }
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationSortUtils.java

    r6362 r6990  
    5050
    5151    static boolean isBackward(final RelationMember member){
    52         return member.getRole().equals("backward");
     52        return "backward".equals(member.getRole());
    5353    }
    5454
    5555    static boolean isForward(final RelationMember member){
    56         return member.getRole().equals("forward");
     56        return "forward".equals(member.getRole());
    5757    }
    5858
     
    6060        return isForward(member) || isBackward(member);
    6161    }
    62 
    6362}
  • trunk/src/org/openstreetmap/josm/gui/dialogs/validator/ValidatorTreePanel.java

    r6890 r6990  
    7878        for (KeyListener keyListener : getKeyListeners()) {
    7979            // Fix #3596 - Remove default keyListener to avoid conflicts with JOSM commands
    80             if (keyListener.getClass().getName().equals("javax.swing.plaf.basic.BasicTreeUI$Handler")) {
     80            if ("javax.swing.plaf.basic.BasicTreeUI$Handler".equals(keyListener.getClass().getName())) {
    8181                removeKeyListener(keyListener);
    8282            }
  • trunk/src/org/openstreetmap/josm/gui/download/PlaceSelection.java

    r6920 r6990  
    206206            depth++;
    207207            try {
    208                 if (qName.equals("searchresults")) {
     208                if ("searchresults".equals(qName)) {
    209209                    // do nothing
    210                 } else if (qName.equals("named") && (depth == 2)) {
     210                } else if ("named".equals(qName) && (depth == 2)) {
    211211                    currentResult = new PlaceSelection.SearchResult();
    212212                    currentResult.name = atts.getValue("name");
     
    219219                    currentResult.zoom = Integer.parseInt(atts.getValue("zoom"));
    220220                    data.add(currentResult);
    221                 } else if (qName.equals("description") && (depth == 3)) {
     221                } else if ("description".equals(qName) && (depth == 3)) {
    222222                    description = new StringBuffer();
    223                 } else if (qName.equals("named") && (depth == 4)) {
     223                } else if ("named".equals(qName) && (depth == 4)) {
    224224                    // this is a "named" place in the nearest places list.
    225225                    String info = atts.getValue("info");
     
    227227                        currentResult.nearestPlace = atts.getValue("name");
    228228                    }
    229                 } else if (qName.equals("place") && atts.getValue("lat") != null) {
     229                } else if ("place".equals(qName) && atts.getValue("lat") != null) {
    230230                    currentResult = new PlaceSelection.SearchResult();
    231231                    currentResult.name = atts.getValue("display_name");
     
    258258        @Override
    259259        public void endElement(String namespaceURI, String localName, String qName) throws SAXException {
    260             if (qName.equals("description") && description != null) {
     260            if ("description".equals(qName) && description != null) {
    261261                currentResult.description = description.toString();
    262262                description = null;
  • trunk/src/org/openstreetmap/josm/gui/help/HelpUtil.java

    r6897 r6990  
    1616import org.openstreetmap.josm.tools.LanguageInfo.LocaleType;
    1717
     18/**
     19 * Provides utility methods for help system.
     20 * @since 2252
     21 */
    1822public final class HelpUtil {
    1923
     
    140144    public static String buildAbsoluteHelpTopic(String topic, LocaleType type) {
    141145        String prefix = getHelpTopicPrefix(type);
    142         if (prefix == null || topic == null || topic.trim().length() == 0 || topic.trim().equals("/"))
     146        if (prefix == null || topic == null || topic.trim().length() == 0 || "/".equals(topic.trim()))
    143147            return prefix;
    144148        prefix += "/" + topic;
     
    220224     *  </pre>
    221225     *
    222      *
    223      * @param helpTopic
     226     * @param helpTopic Help topic to mark
     227     * @return {@code helpTopic}
    224228     */
    225229    public static String ht(String helpTopic) {
  • trunk/src/org/openstreetmap/josm/gui/io/UploadDialog.java

    r6889 r6990  
    578578    @Override
    579579    public void preferenceChanged(PreferenceChangeEvent e) {
    580         if (e.getKey() == null || ! e.getKey().equals("osm-server.url"))
     580        if (e.getKey() == null || !"osm-server.url".equals(e.getKey()))
    581581            return;
    582582        final Setting<?> newValue = e.getNewValue();
  • trunk/src/org/openstreetmap/josm/gui/io/UploadParameterSummaryPanel.java

    r6901 r6990  
    156156    public void hyperlinkUpdate(HyperlinkEvent e) {
    157157        if (HyperlinkEvent.EventType.ACTIVATED.equals(e.getEventType())) {
    158             if (e.getDescription() == null || configHandler == null)
     158            String desc = e.getDescription();
     159            if (desc == null || configHandler == null)
    159160                return;
    160             if (e.getDescription().equals("urn:changeset-configuration")) {
     161            if ("urn:changeset-configuration".equals(desc)) {
    161162                configHandler.handleChangesetConfigurationRequest();
    162             } else if (e.getDescription().equals("urn:advanced-configuration")) {
     163            } else if ("urn:advanced-configuration".equals(desc)) {
    163164                configHandler.handleUploadStrategyConfigurationRequest();
    164165            }
  • trunk/src/org/openstreetmap/josm/gui/io/UploadStrategy.java

    r6889 r6990  
    7777            }
    7878            v = v.trim().toLowerCase();
    79             if (v.equals("true"))
     79            if ("true".equals(v))
    8080                return SINGLE_REQUEST_STRATEGY;
    81             else if (v.equals("false"))
     81            else if ("false".equals(v))
    8282                return INDIVIDUAL_OBJECTS_STRATEGY;
    8383            else
  • trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java

    r6901 r6990  
    406406        }
    407407
    408         protected void valiateChunkSize() {
     408        protected void validateChunkSize() {
    409409            try {
    410410                int chunkSize = Integer.parseInt(tfChunkSize.getText().trim());
     
    430430        @Override
    431431        public void changedUpdate(DocumentEvent arg0) {
    432             valiateChunkSize();
     432            validateChunkSize();
    433433        }
    434434
    435435        @Override
    436436        public void insertUpdate(DocumentEvent arg0) {
    437             valiateChunkSize();
     437            validateChunkSize();
    438438        }
    439439
    440440        @Override
    441441        public void removeUpdate(DocumentEvent arg0) {
    442             valiateChunkSize();
     442            validateChunkSize();
    443443        }
    444444
     
    446446        public void propertyChange(PropertyChangeEvent evt) {
    447447            if (evt.getSource() == tfChunkSize
    448                     && evt.getPropertyName().equals("enabled")
     448                    && "enabled".equals(evt.getPropertyName())
    449449                    && (Boolean)evt.getNewValue()
    450450            ) {
    451                 valiateChunkSize();
     451                validateChunkSize();
    452452            }
    453453        }
  • trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java

    r6883 r6990  
    147147                        MapImage mapImage = ((NodeElemStyle) style).mapImage;
    148148                        if (mapImage != null) {
    149                             if (includeDeprecatedIcon || mapImage.name == null || !mapImage.name.equals("misc/deprecated.png")) {
     149                            if (includeDeprecatedIcon || mapImage.name == null || !"misc/deprecated.png".equals(mapImage.name)) {
    150150                                return new ImageIcon(mapImage.getDisplayedNodeIcon(false));
    151151                            } else {
  • trunk/src/org/openstreetmap/josm/gui/mappaint/MultiCascade.java

    r6561 r6990  
    1919    public Range range;
    2020
     21    /**
     22     * Constructs a new {@code MultiCascade}.
     23     */
    2124    public MultiCascade() {
    2225        layers = new HashMap<String, Cascade>();
     
    3942                // Everything that is not on the default layer is assumed to
    4043                // be a modifier. Can be overridden in style definition.
    41                 if (!layer.equals("default") && !layer.equals("*")) {
     44                if (!"default".equals(layer) && !"*".equals(layer)) {
    4245                    c.put(MODIFIER, true);
    4346                }
     
    6063        if (c == null) {
    6164            c = new Cascade();
    62             if (!layer.equals("default") && !layer.equals("*")) {
     65            if (!"default".equals(layer) && !"*".equals(layer)) {
    6366                c.put(MODIFIER, true);
    6467            }
  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Selector.java

    r6977 r6990  
    429429
    430430        public boolean matchesBase(OsmPrimitiveType type) {
    431             if (base.equals("*")) {
     431            if ("*".equals(base)) {
    432432                return true;
    433433            } else if (OsmPrimitiveType.NODE.equals(type)) {
    434                 return base.equals("node");
     434                return "node".equals(base);
    435435            } else if (OsmPrimitiveType.WAY.equals(type)) {
    436                 return base.equals("way") || base.equals("area");
     436                return "way".equals(base) || "area".equals(base);
    437437            } else if (OsmPrimitiveType.RELATION.equals(type)) {
    438                 return base.equals("area") || base.equals("relation") || base.equals("canvas");
     438                return "area".equals(base) || "relation".equals(base) || "canvas".equals(base);
    439439            }
    440440            return false;
     
    446446            } else {
    447447                if (p instanceof Relation) {
    448                     if (base.equals("area")) {
     448                    if ("area".equals(base)) {
    449449                        return ((Relation) p).isMultipolygon();
    450                     } else if (base.equals("canvas")) {
     450                    } else if ("canvas".equals(base)) {
    451451                        return p.get("#canvas") != null;
    452452                    }
  • trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java

    r6987 r6990  
    249249                    continue;
    250250                }
    251                 if (kv[0].equals("_osm_session")) {
     251                if ("_osm_session".equals(kv[0])) {
    252252                    // osm session cookie found
    253253                    String token = extractToken(connection);
  • trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

    r6844 r6990  
    487487            @Override
    488488            public void actionPerformed(ActionEvent e) {
    489                 if (e.getActionCommand().equals("<") && actionsTree.getSelectionCount() > 0) {
     489                if ("<".equals(e.getActionCommand()) && actionsTree.getSelectionCount() > 0) {
    490490
    491491                    int leadItem = selected.getSize();
     
    500500                        } else if (node.getUserObject() instanceof Action) {
    501501                            selected.add(leadItem++, new ActionDefinition((Action)node.getUserObject()));
    502 
    503502                        }
    504503                    }
    505                 } else if (e.getActionCommand().equals(">") && selectedList.getSelectedIndex() != -1) {
     504                } else if (">".equals(e.getActionCommand()) && selectedList.getSelectedIndex() != -1) {
    506505                    while (selectedList.getSelectedIndex() != -1) {
    507506                        selected.remove(selectedList.getSelectedIndex());
    508507                    }
    509                 } else if (e.getActionCommand().equals("up")) {
     508                } else if ("up".equals(e.getActionCommand())) {
    510509                    int i = selectedList.getSelectedIndex();
    511510                    Object o = selected.get(i);
     
    515514                        selectedList.setSelectedIndex(i-1);
    516515                    }
    517                 } else if (e.getActionCommand().equals("down")) {
     516                } else if ("down".equals(e.getActionCommand())) {
    518517                    int i = selectedList.getSelectedIndex();
    519518                    Object o = selected.get(i);
     
    580579        private JButton createButton(String name) {
    581580            JButton b = new JButton();
    582             if (name.equals("up")) {
     581            if ("up".equals(name)) {
    583582                b.setIcon(ImageProvider.get("dialogs", "up"));
    584             } else if (name.equals("down")) {
     583            } else if ("down".equals(name)) {
    585584                b.setIcon(ImageProvider.get("dialogs", "down"));
    586585            } else {
     
    739738                @Override
    740739                protected void exportDone(JComponent source, Transferable data, int action) {
    741                     if (movingComponent.equals("list")) {
     740                    if ("list".equals(movingComponent)) {
    742741                        try {
    743742                            List<?> draggedData = (List<?>) data.getTransferData(ACTION_FLAVOR);
     
    981980
    982981        for (String s : getToolString()) {
    983             if (s.equals("|")) {
     982            if ("|".equals(s)) {
    984983                result.add(ActionDefinition.getSeparator());
    985984            } else {
  • trunk/src/org/openstreetmap/josm/gui/preferences/server/AuthenticationPreferencesPanel.java

    r6890 r6990  
    115115    public final void initFromPreferences() {
    116116        String authMethod = Main.pref.get("osm-server.auth-method", "basic");
    117         if (authMethod.equals("basic")) {
     117        if ("basic".equals(authMethod)) {
    118118            rbBasicAuthentication.setSelected(true);
    119         } else if (authMethod.equals("oauth")) {
     119        } else if ("oauth".equals(authMethod)) {
    120120            rbOAuth.setSelected(true);
    121121        } else {
     
    140140        }
    141141        Main.pref.put("osm-server.auth-method", authMethod);
    142         if (authMethod.equals("basic")) {
     142        if ("basic".equals(authMethod)) {
    143143            // save username and password and clear the OAuth token
    144144            pnlBasicAuthPreferences.saveToPreferences();
    145145            OAuthAccessTokenHolder.getInstance().clear();
    146146            OAuthAccessTokenHolder.getInstance().save(Main.pref, CredentialsManager.getInstance());
    147         } else if (authMethod.equals("oauth")) {
     147        } else if ("oauth".equals(authMethod)) {
    148148            // clear the password in the preferences
    149149            pnlBasicAuthPreferences.clearPassword();
  • trunk/src/org/openstreetmap/josm/gui/preferences/shortcut/PrefJPanel.java

    r6798 r6990  
    322322                    if (panel.cbDisable.isSelected()) {
    323323                        sc.setAssignedModifier(-1);
    324                     } else if (panel.tfKey.getSelectedItem() == null || panel.tfKey.getSelectedItem().equals("")) {
     324                    } else if (panel.tfKey.getSelectedItem() == null || "".equals(panel.tfKey.getSelectedItem())) {
    325325                        sc.setAssignedModifier(KeyEvent.VK_CANCEL);
    326326                    } else {
  • trunk/src/org/openstreetmap/josm/gui/widgets/AbstractTextComponentValidator.java

    r6883 r6990  
    173173    @Override
    174174    public void propertyChange(PropertyChangeEvent evt) {
    175         if (evt.getPropertyName().equals("enabled")) {
     175        if ("enabled".equals(evt.getPropertyName())) {
    176176            boolean enabled = (Boolean)evt.getNewValue();
    177177            if (enabled) {
  • trunk/src/org/openstreetmap/josm/gui/widgets/JosmComboBox.java

    r6890 r6990  
    187187
    188188        @Override public void propertyChange(PropertyChangeEvent evt) {
    189             if (evt.getPropertyName().equals("editable")) {
     189            if ("editable".equals(evt.getPropertyName())) {
    190190                if (evt.getNewValue().equals(true)) {
    191191                    enableMenu();
     
    193193                    disableMenu();
    194194                }
    195             } else if (evt.getPropertyName().equals("editor")) {
     195            } else if ("editor".equals(evt.getPropertyName())) {
    196196                disableMenu();
    197197                if (isEditable()) {
  • trunk/src/org/openstreetmap/josm/gui/widgets/TextContextualPopupMenu.java

    r6792 r6990  
    4242    protected final PropertyChangeListener propertyChangeListener = new PropertyChangeListener() {
    4343        @Override public void propertyChange(PropertyChangeEvent evt) {
    44             if (evt.getPropertyName().equals("editable")) {
     44            if ("editable".equals(evt.getPropertyName())) {
    4545                removeAll();
    4646                addMenuEntries();
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r6907 r6990  
    165165            String url = st.nextToken();
    166166
    167             if (!method.equals("GET")) {
     167            if (!"GET".equals(method)) {
    168168                sendNotImplemented(out);
    169169                return;
  • trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java

    r6450 r6990  
    3737            if (imgNode.getNodeType() == Node.ELEMENT_NODE) {
    3838                Element imgElem = (Element) imgNode;
    39                 if (imgElem.getTagName().equals("geoimage")) {
     39                if ("geoimage".equals(imgElem.getTagName())) {
    4040                    ImageEntry entry = new ImageEntry();
    4141                    NodeList attrNodes = imgElem.getChildNodes();
     
    4545                            Element attrElem = (Element) attrNode;
    4646                            try {
    47                                 if (attrElem.getTagName().equals("file")) {
     47                                String attrElemName = attrElem.getTagName();
     48                                if ("file".equals(attrElemName)) {
    4849                                    entry.setFile(new File(attrElem.getTextContent()));
    49                                 } else if (attrElem.getTagName().equals("position")) {
     50                                } else if ("position".equals(attrElemName)) {
    5051                                    double lat = Double.parseDouble(attrElem.getAttribute("lat"));
    5152                                    double lon = Double.parseDouble(attrElem.getAttribute("lon"));
    5253                                    entry.setPos(new LatLon(lat, lon));
    53                                 } else if (attrElem.getTagName().equals("speed")) {
     54                                } else if ("speed".equals(attrElemName)) {
    5455                                    entry.setSpeed(Double.parseDouble(attrElem.getTextContent()));
    55                                 } else if (attrElem.getTagName().equals("elevation")) {
     56                                } else if ("elevation".equals(attrElemName)) {
    5657                                    entry.setElevation(Double.parseDouble(attrElem.getTextContent()));
    57                                 } else if (attrElem.getTagName().equals("gps-time")) {
     58                                } else if ("gps-time".equals(attrElemName)) {
    5859                                    entry.setGpsTime(new Date(Long.parseLong(attrElem.getTextContent())));
    59                                 } else if (attrElem.getTagName().equals("exif-orientation")) {
     60                                } else if ("exif-orientation".equals(attrElemName)) {
    6061                                    entry.setExifOrientation(Integer.parseInt(attrElem.getTextContent()));
    61                                 } else if (attrElem.getTagName().equals("exif-time")) {
     62                                } else if ("exif-time".equals(attrElemName)) {
    6263                                    entry.setExifTime(new Date(Long.parseLong(attrElem.getTextContent())));
    63                                 } else if (attrElem.getTagName().equals("exif-gps-time")) {
     64                                } else if ("exif-gps-time".equals(attrElemName)) {
    6465                                    entry.setExifGpsTime(new Date(Long.parseLong(attrElem.getTextContent())));
    65                                 } else if (attrElem.getTagName().equals("exif-coordinates")) {
     66                                } else if ("exif-coordinates".equals(attrElemName)) {
    6667                                    double lat = Double.parseDouble(attrElem.getAttribute("lat"));
    6768                                    double lon = Double.parseDouble(attrElem.getAttribute("lon"));
    6869                                    entry.setExifCoor(new LatLon(lat, lon));
    69                                 } else if (attrElem.getTagName().equals("exif-image-direction")) {
     70                                } else if ("exif-image-direction".equals(attrElemName)) {
    7071                                    entry.setExifImgDir(Double.parseDouble(attrElem.getTextContent()));
    71                                 } else if (attrElem.getTagName().equals("is-new-gps-data") && Boolean.parseBoolean(attrElem.getTextContent())) {
     72                                } else if ("is-new-gps-data".equals(attrElemName) && Boolean.parseBoolean(attrElem.getTextContent())) {
    7273                                    entry.flagNewGpsData();
    7374                                }
     
    7980                    }
    8081                    entries.add(entry);
    81                 } else if (imgElem.getTagName().equals("show-thumbnails")) {
     82                } else if ("show-thumbnails".equals(imgElem.getTagName())) {
    8283                    useThumbs = Boolean.parseBoolean(imgElem.getTextContent());
    8384                }
  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r6890 r6990  
    353353        String policy = Main.pref.get(togglePreferenceKey, "ask");
    354354        policy = policy.trim().toLowerCase();
    355         if (policy.equals("never")) {
     355        if ("never".equals(policy)) {
    356356            if ("pluginmanager.version-based-update.policy".equals(togglePreferenceKey)) {
    357357                Main.info(tr("Skipping plugin update after JOSM upgrade. Automatic update at startup is disabled."));
     
    362362        }
    363363
    364         if (policy.equals("always")) {
     364        if ("always".equals(policy)) {
    365365            if ("pluginmanager.version-based-update.policy".equals(togglePreferenceKey)) {
    366366                Main.info(tr("Running plugin update after JOSM upgrade. Automatic update at startup is enabled."));
     
    371371        }
    372372
    373         if (!policy.equals("ask")) {
     373        if (!"ask".equals(policy)) {
    374374            Main.warn(tr("Unexpected value ''{0}'' for preference ''{1}''. Assuming value ''ask''.", policy, togglePreferenceKey));
    375375        }
     
    13551355            String policy = Main.pref.get(preferencesKey, "ask");
    13561356            policy = policy.trim().toLowerCase();
    1357             cbDontShowAgain.setSelected(! policy.equals("ask"));
     1357            cbDontShowAgain.setSelected(!"ask".equals(policy));
    13581358        }
    13591359
Note: See TracChangeset for help on using the changeset viewer.