Ignore:
Timestamp:
2011-09-24T12:09:32+02:00 (15 years ago)
Author:
simon04
Message:

fix #3951 - user should be warned when unglue-ing two ways outside the download area

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/command/DeleteCommand.java

    r4325 r4458  
    22package org.openstreetmap.josm.command;
    33
     4import java.awt.geom.Area;
    45import static org.openstreetmap.josm.tools.I18n.marktr;
    56import static org.openstreetmap.josm.tools.I18n.tr;
    67import static org.openstreetmap.josm.tools.I18n.trn;
    78
    8 import java.awt.GridBagLayout;
    9 import java.awt.geom.Area;
    109import java.util.ArrayList;
    1110import java.util.Collection;
    … …  
    2120
    2221import javax.swing.JLabel;
    23 import javax.swing.JOptionPane;
    24 import javax.swing.JPanel;
    25 
    26 import org.openstreetmap.josm.Main;
     22
    2723import org.openstreetmap.josm.actions.SplitWayAction;
    2824import org.openstreetmap.josm.data.osm.Node;
    … …  
    3430import org.openstreetmap.josm.data.osm.Way;
    3531import org.openstreetmap.josm.data.osm.WaySegment;
    36 import org.openstreetmap.josm.gui.ConditionalOptionPaneUtil;
    3732import org.openstreetmap.josm.gui.DefaultNameFormatter;
    3833import org.openstreetmap.josm.gui.actionsupport.DeleteFromRelationConfirmationDialog;
    … …  
    234229        if (parents.isEmpty())
    235230            return null;
    236         if (!silent && !checkAndConfirmOutlyingDeletes(layer,parents))
     231        if (!silent && !checkAndConfirmOutlyingDelete(layer, parents, null))
    237232            return null;
    238233        return new DeleteCommand(layer,parents);
    … …  
    331326        }
    332327
    333         if (!silent && !checkAndConfirmOutlyingDeletes(layer,primitivesToDelete))
     328        if (!silent && !checkAndConfirmOutlyingDelete(layer, primitivesToDelete, null))
    334329            return null;
    335330
    … …  
    427422    }
    428423
    429     /**
    430      * Check whether user is about to delete data outside of the download area. Request confirmation
    431      * if he is.
    432      *
    433      * @param layer the layer in whose context data is deleted
    434      * @param primitivesToDelete the primitives to delete
    435      * @return true, if deleting outlying primitives is OK; false, otherwise
    436      */
    437     private static boolean checkAndConfirmOutlyingDeletes(OsmDataLayer layer, Collection<OsmPrimitive> primitivesToDelete) {
    438         Area a = layer.data.getDataSourceArea();
    439         boolean outside = false;
    440         boolean incomplete = false;
    441         if (a != null) {
    442             for (OsmPrimitive osm : primitivesToDelete) {
    443                 if (osm.isIncomplete()) {
    444                     incomplete = true;
    445                 } else if (osm instanceof Node && !osm.isNewOrUndeleted()
    446                         && !a.contains(((Node) osm).getCoor())) {
    447                     outside = true;
    448                 }
    449             }
    450         }
    451         else
    452         {
    453             for (OsmPrimitive osm : primitivesToDelete)
    454                 if (osm.isIncomplete()) {
    455                     incomplete = true;
    456                 }
    457         }
    458         if(outside)
    459         {
    460             JPanel msg = new JPanel(new GridBagLayout());
    461             msg.add(new JLabel(
    462                     "<html>" +
    463                     // leave message in one tr() as there is a grammatical
    464                     // connection.
    465                     tr("You are about to delete nodes outside of the area you have downloaded."
    466                             + "<br>"
    467                             + "This can cause problems because other objects (that you do not see) might use them."
    468                             + "<br>" + "Do you really want to delete?") + "</html>"));
    469             boolean answer = ConditionalOptionPaneUtil.showConfirmationDialog(
    470                     "delete_outside_nodes",
    471                     Main.parent,
    472                     msg,
    473                     tr("Delete confirmation"),
    474                     JOptionPane.YES_NO_OPTION,
    475                     JOptionPane.QUESTION_MESSAGE,
    476                     JOptionPane.YES_OPTION
    477             );
    478             if(!answer)
    479                 return false;
    480         }
    481         if(incomplete)
    482         {
    483             JPanel msg = new JPanel(new GridBagLayout());
    484             msg.add(new JLabel(
    485                     "<html>" +
    486                     // leave message in one tr() as there is a grammatical
    487                     // connection.
    488                     tr("You are about to delete incomplete objects."
    489                             + "<br>"
    490                             + "This will cause problems because you don''t see the real object."
    491                             + "<br>" + "Do you really want to delete?") + "</html>"));
    492             boolean answer = ConditionalOptionPaneUtil.showConfirmationDialog(
    493                     "delete_incomplete",
    494                     Main.parent,
    495                     msg,
    496                     tr("Delete confirmation"),
    497                     JOptionPane.YES_NO_OPTION,
    498                     JOptionPane.QUESTION_MESSAGE,
    499                     JOptionPane.YES_OPTION
    500             );
    501             if(!answer)
    502                 return false;
    503         }
    504         return true;
    505     }
     424    public static boolean checkAndConfirmOutlyingDelete(OsmDataLayer layer, Collection<? extends OsmPrimitive> primitives, OsmPrimitive ignore) {
     425        return checkAndConfirmOutlyingDelete(layer.data.getDataSourceArea(), primitives, ignore);
     426    }
     427
     428    public static boolean checkAndConfirmOutlyingDelete(Area area, Collection<? extends OsmPrimitive> primitives, OsmPrimitive ignore) {
     429        return Command.checkAndConfirmOutlyingOperation("delete",
     430                tr("Delete confirmation"),
     431                tr("You are about to delete nodes outside of the area you have downloaded."
     432                        + "<br>"
     433                        + "This can cause problems because other objects (that you do not see) might use them."
     434                        + "<br>"
     435                        + "Do you really want to delete?"),
     436                tr("You are about to delete incomplete objects."
     437                        + "<br>"
     438                        + "This will cause problems because you don''t see the real object."
     439                        + "<br>" + "Do you really want to delete?"),
     440                area, primitives, ignore);
     441    }
     442
    506443}
Note: See TracChangeset for help on using the changeset viewer.