Ignore:
Timestamp:
2016-10-09T04:03:32+02:00 (3 years ago)
Author:
Don-vip
Message:

sonar - squid:S1871 - Two branches in the same conditional structure should not have exactly the same implementation

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

Legend:

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

    r11108 r11109  
    161161                // Among existing nodes, try to keep the oldest used one
    162162                if (!n.getReferrers().isEmpty()) {
    163                     if (targetNode == null) {
    164                         targetNode = n;
    165                     } else if (n.getId() < targetNode.getId()) {
     163                    if (targetNode == null || n.getId() < targetNode.getId()) {
    166164                        targetNode = n;
    167165                    }
    168                 } else if (oldestNode == null) {
    169                     oldestNode = n;
    170                 } else if (n.getId() < oldestNode.getId()) {
     166                } else if (oldestNode == null || n.getId() < oldestNode.getId()) {
    171167                    oldestNode = n;
    172168                }
     
    200196                if (!nodesToDelete.contains(n) && !n.equals(targetNode)) {
    201197                    newNodes.add(n);
    202                 } else if (newNodes.isEmpty()) {
    203                     newNodes.add(targetNode);
    204                 } else if (!newNodes.get(newNodes.size()-1).equals(targetNode)) {
     198                } else if (newNodes.isEmpty() || !newNodes.get(newNodes.size()-1).equals(targetNode)) {
    205199                    // make sure we collapse a sequence of deleted nodes
    206200                    // to exactly one occurrence of the merged target node
  • trunk/src/org/openstreetmap/josm/actions/SessionSaveAsAction.java

    r10420 r11109  
    131131        boolean zip;
    132132        FileFilter ff = fc.getFileFilter();
    133         if (zipRequired) {
    134             zip = true;
    135         } else if (joz.equals(ff)) {
     133        if (zipRequired || joz.equals(ff)) {
    136134            zip = true;
    137135        } else if (jos.equals(ff)) {
  • trunk/src/org/openstreetmap/josm/actions/SimplifyWayAction.java

    r10662 r11109  
    101101                alertSelectAtLeastOneWay();
    102102                return;
    103             } else if (!confirmWayWithNodesOutsideBoundingBox(ways)) {
    104                 return;
    105             } else if (ways.size() > 10 && !confirmSimplifyManyWays(ways.size())) {
     103            } else if (!confirmWayWithNodesOutsideBoundingBox(ways) || (ways.size() > 10 && !confirmSimplifyManyWays(ways.size()))) {
    106104                return;
    107105            }
  • trunk/src/org/openstreetmap/josm/actions/UploadSelectionAction.java

    r10601 r11109  
    7979        Set<OsmPrimitive> ret = new HashSet<>();
    8080        for (OsmPrimitive p: primitives) {
    81             if (p.isNewOrUndeleted()) {
    82                 ret.add(p);
    83             } else if (p.isModified() && !p.isIncomplete()) {
     81            if (p.isNewOrUndeleted() || (p.isModified() && !p.isIncomplete())) {
    8482                ret.add(p);
    8583            }
Note: See TracChangeset for help on using the changeset viewer.