Changeset 6202 in osm for applications

Timestamp:

2007-12-31T16:37:22+01:00 (17 years ago)

Author:

gabriel

Message:

validator plugin: Don't combine the fix commands into a sequence command.

This is just a hack so that we don't step on our toes when fixing duplicate
nodes, i.e. our changes to the ways canceling each other out.

Location:

applications/editors/josm

Files:

• dist/validator.jar (modified) ( previous)
• plugins/validator/src/org/openstreetmap/josm/plugins/validator/ValidatorDialog.java (modified) (2 diffs)

applications/editors/josm/plugins/validator/src/org/openstreetmap/josm/plugins/validator/ValidatorDialog.java

@@ -103 +103 @@
             return;
         
-        Bag<String, Command> commands = new Bag<String, Command>();
         Set<DefaultMutableTreeNode> processedNodes = new HashSet<DefaultMutableTreeNode>();
         for( TreePath path : selectionPaths )
@@ -126 +125 @@
                                 if( fixCommand != null )
                                 {
-                                        commands.add(error.getMessage(), fixCommand);
+                                        Main.main.undoRedo.add(fixCommand);
                                 }
                         }
                 }
         }
-                
-                Command fixCommand = null;
-                if( commands.size() == 0 )
-                        return;
                 
-                List<Command> allComands = new ArrayList<Command>(50);
-                for( Entry<String, List<Command>> errorType : commands.entrySet())
-                {
-                        String description = errorType.getKey();
-                        List<Command> errorCommands = errorType.getValue();
-                        allComands.add( new SequenceCommand("Fix " + description, errorCommands) );
-                }
-                
-                if( allComands.size() > 1 )
-                        fixCommand = new SequenceCommand("Fix errors", allComands);
-                else 
-                        fixCommand = allComands.get(0);
-                
-                Main.main.undoRedo.add( fixCommand );
                 Main.map.repaint();
                 DataSet.fireSelectionChanged(Main.ds.getSelected());