Ignore:
Timestamp:
2017-06-01T01:20:37+02:00 (2 years ago)
Author:
Don-vip
Message:

sonar - squid:S3516 - Methods returns should not be invariant

File:
1 edited

Legend:

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

    r11913 r12288  
    186186    @Override
    187187    public boolean checkUpload(APIDataSet apiDataSet) {
    188         if (!Main.pref.getBoolean("fix.data.on.upload", true))
    189             return true;
    190 
    191         List<OsmPrimitive> objectsToUpload = apiDataSet.getPrimitives();
    192         Collection<Command> cmds = new LinkedList<>();
    193 
    194         for (OsmPrimitive osm : objectsToUpload) {
    195             Map<String, String> keys = new HashMap<>(osm.getKeys());
    196             if (!keys.isEmpty()) {
    197                 boolean modified = false;
    198                 for (FixData fix : deprecated) {
    199                     if (fix.fixKeys(keys, osm))
    200                         modified = true;
     188        if (Main.pref.getBoolean("fix.data.on.upload", true)) {
     189            Collection<Command> cmds = new LinkedList<>();
     190
     191            for (OsmPrimitive osm : apiDataSet.getPrimitives()) {
     192                Map<String, String> keys = new HashMap<>(osm.getKeys());
     193                if (!keys.isEmpty()) {
     194                    boolean modified = false;
     195                    for (FixData fix : deprecated) {
     196                        if (fix.fixKeys(keys, osm))
     197                            modified = true;
     198                    }
     199                    if (modified) {
     200                        cmds.add(new ChangePropertyCommand(Collections.singleton(osm), keys));
     201                    }
    201202                }
    202                 if (modified)
    203                     cmds.add(new ChangePropertyCommand(Collections.singleton(osm), keys));
    204             }
    205         }
    206 
    207         if (!cmds.isEmpty())
    208             Main.main.undoRedo.add(new SequenceCommand(tr("Fix deprecated tags"), cmds));
     203            }
     204
     205            if (!cmds.isEmpty()) {
     206                Main.main.undoRedo.add(new SequenceCommand(tr("Fix deprecated tags"), cmds));
     207            }
     208        }
    209209        return true;
    210210    }
Note: See TracChangeset for help on using the changeset viewer.