Changeset 11109 in josm


Ignore:
Timestamp:
2016-10-09T04:03:32+02:00 (8 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
Files:
1 added
5 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            }
  • trunk/test/unit/org/openstreetmap/josm/actions/MergeNodesActionTest.java

    r11108 r11109  
    33
    44import static org.junit.Assert.assertEquals;
     5import static org.junit.Assert.assertNull;
    56
    67import java.util.Arrays;
     
    4647
    4748    /**
    48      * Unit test of {@link MergeNodesAction#selectTargetLocationNode} - mode 0
     49     * Unit test of {@link MergeNodesAction#selectTargetLocationNode}
    4950     */
    5051    @Test
    51     public void testSelectTargetLocationNodeMode0() {
     52    public void testSelectTargetLocationNode() {
    5253        Main.pref.putInteger("merge-nodes.mode", 0);
    5354        assertEquals(1, MergeNodesAction.selectTargetLocationNode(Arrays.asList(new Node(0), new Node(1))).getId());
    54     }
    5555
    56     /**
    57      * Unit test of {@link MergeNodesAction#selectTargetLocationNode} - mode 1
    58      */
    59     @Test
    60     public void testSelectTargetLocationNodeMode1() {
    6156        Main.pref.putInteger("merge-nodes.mode", 1);
    6257        assertEquals(LatLon.ZERO, MergeNodesAction.selectTargetLocationNode(
    6358                Arrays.asList(new Node(LatLon.NORTH_POLE), new Node(LatLon.SOUTH_POLE))).getCoor());
    64     }
    6559
    66     /**
    67      * Unit test of {@link MergeNodesAction#selectTargetLocationNode} - mode 2 with a single node
    68      */
    69     @Test
    70     public void testSelectTargetLocationNodeMode2SingleNode() {
    7160        Main.pref.putInteger("merge-nodes.mode", 2);
    7261        assertEquals(LatLon.NORTH_POLE, MergeNodesAction.selectTargetLocationNode(
    7362                Arrays.asList(new Node(LatLon.NORTH_POLE))).getCoor());
    7463    }
     64
     65    /**
     66     * Unit test of {@link MergeNodesAction#selectTargetNode}
     67     */
     68    @Test
     69    public void testSelectTargetNode() {
     70        assertNull(MergeNodesAction.selectTargetNode(Collections.emptyList()));
     71        assertEquals(1, MergeNodesAction.selectTargetNode(Arrays.asList(new Node(1))).getId());
     72    }
    7573}
Note: See TracChangeset for help on using the changeset viewer.