Changeset 3327 in josm for trunk/src/org/openstreetmap


Ignore:
Timestamp:
2010-06-09T22:14:06+02:00 (15 years ago)
Author:
jttt
Message:

Fix some of "Keystroke %s is already assigned to %s, will be overridden by %s" warnings

Location:
trunk/src/org/openstreetmap/josm
Files:
13 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/Main.java

    r3317 r3327  
    301301        Object existing = inputMap.get(keyStroke);
    302302        if (existing != null && !existing.equals(action)) {
    303             System.out.println(String.format("Keystroke is already assigned to %s, will be overridden by %s", existing, action));
     303            System.out.println(String.format("Keystroke %s is already assigned to %s, will be overridden by %s", keyStroke, existing, action));
    304304        }
    305305        inputMap.put(keyStroke, action);
    … …  
    309309
    310310    public static void unregisterActionShortcut(Shortcut shortcut) {
    311         contentPanePrivate.getInputMap().remove(shortcut.getKeyStroke());
     311        contentPanePrivate.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).remove(shortcut.getKeyStroke());
    312312    }
    313313

  • trunk/src/org/openstreetmap/josm/actions/AutoScaleAction.java

    r3251 r3327  
    7272    }
    7373
     74    public static void autoScale(String mode) {
     75        new AutoScaleAction(mode, false).autoScale();
     76    }
     77
    7478    private final String mode;
    7579
    … …  
    101105        return shortcut;
    102106    }
     107
     108    /**
     109     *
     110     * @param mode
     111     * @param marker Used only to differentiate from default constructor
     112     */
     113    private AutoScaleAction(String mode, boolean marker) {
     114        super(false);
     115        this.mode = mode;
     116    }
     117
    103118
    104119    public AutoScaleAction(String mode) {

  • trunk/src/org/openstreetmap/josm/actions/JosmAction.java

    r3252 r3327  
    8282
    8383    public JosmAction() {
     84        this(true);
     85    }
     86
     87    public JosmAction(boolean installAdapters) {
    8488        setHelpId();
    85         installAdapters();
     89        if (installAdapters) {
     90            installAdapters();
     91        }
    8692    }
    8793

  • trunk/src/org/openstreetmap/josm/actions/MoveAction.java

    r3262 r3327  
    22package org.openstreetmap.josm.actions;
    33
     4import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    5 import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    66
    77import java.awt.event.ActionEvent;
    … …  
    2828
    2929    public enum Direction { UP, LEFT, RIGHT, DOWN }
     30
    3031    private Direction myDirection;
    3132
    … …  
    6364    public void actionPerformed(ActionEvent event) {
    6465
     66        if (!Main.isDisplayingMapView())
     67            return;
     68
    6569        // find out how many "real" units the objects have to be moved in order to
    6670        // achive an 1-pixel movement
    … …  
    7377
    7478        switch (myDirection) {
    75             case UP:
    76                 distx = 0;
    77                 disty = -disty;
    78                 break;
    79             case DOWN:
    80                 distx = 0;
    81                 break;
    82             case LEFT:
    83                 disty = 0;
    84                 distx = -distx;
    85                 break;
    86             default:
    87                 disty = 0;
     79        case UP:
     80            distx = 0;
     81            disty = -disty;
     82            break;
     83        case DOWN:
     84            distx = 0;
     85            break;
     86        case LEFT:
     87            disty = 0;
     88            distx = -distx;
     89            break;
     90        default:
     91            disty = 0;
    8892        }
    8993

  • trunk/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

    r3320 r3327  
    8888    private EastNorth currentMouseEastNorth;
    8989
     90    private Shortcut extraShortcut;
     91
    9092    public DrawAction(MapFrame mapFrame) {
    9193        super(tr("Draw"), "node/autonode", tr("Draw nodes"),
    … …  
    9496
    9597        // Add extra shortcut N
    96         Main.registerActionShortcut(this, Shortcut.registerShortcut("mapmode:drawfocus", tr("Mode: Draw Focus"), KeyEvent.VK_N, Shortcut.GROUP_EDIT));
     98        extraShortcut = Shortcut.registerShortcut("mapmode:drawfocus", tr("Mode: Draw Focus"), KeyEvent.VK_N, Shortcut.GROUP_EDIT);
     99        Main.registerActionShortcut(this, extraShortcut);
    97100
    98101        cursorCrosshair = getCursor();
    … …  
    241244        //
    242245        DataSet ds = getCurrentDataSet();
    243         if(ds != null)
     246        if(ds != null) {
    244247            ds.fireSelectionChanged();
     248        }
    245249    }
    246250
    … …  
    994998        setEnabled(getEditLayer() != null);
    995999    }
     1000
     1001    @Override
     1002    public void destroy() {
     1003        super.destroy();
     1004        Main.unregisterActionShortcut(extraShortcut);
     1005    }
    9961006}

  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r3266 r3327  
    4141import org.openstreetmap.josm.actions.JoinNodeWayAction;
    4242import org.openstreetmap.josm.actions.JosmAction;
     43import org.openstreetmap.josm.actions.JumpToAction;
    4344import org.openstreetmap.josm.actions.MergeLayerAction;
    4445import org.openstreetmap.josm.actions.MergeNodesAction;
    4546import org.openstreetmap.josm.actions.MergeSelectionAction;
    4647import org.openstreetmap.josm.actions.MirrorAction;
     48import org.openstreetmap.josm.actions.MoveAction;
    4749import org.openstreetmap.josm.actions.MoveNodeAction;
    4850import org.openstreetmap.josm.actions.NewAction;
    … …  
    175177    public final int defaultMenuPos = 5;
    176178
     179    public final JosmAction moveUpAction = new MoveAction(MoveAction.Direction.UP);
     180    public final JosmAction moveDownAction = new MoveAction(MoveAction.Direction.DOWN);
     181    public final JosmAction moveLeftAction = new MoveAction(MoveAction.Direction.LEFT);
     182    public final JosmAction moveRightAction = new MoveAction(MoveAction.Direction.RIGHT);
     183    public final JumpToAction jumpToAct = new JumpToAction();
     184
     185
    177186    /**
    178187     * Add a JosmAction to a menu.

  • trunk/src/org/openstreetmap/josm/gui/MapStatus.java

    r3177 r3327  
    1717import java.awt.Toolkit;
    1818import java.awt.event.AWTEventListener;
    19 import java.awt.event.ComponentEvent;
    2019import java.awt.event.InputEvent;
    2120import java.awt.event.KeyAdapter;
    … …  
    3938
    4039import org.openstreetmap.josm.Main;
    41 import org.openstreetmap.josm.actions.JumpToAction;
    4240import org.openstreetmap.josm.data.coor.CoordinateFormat;
    4341import org.openstreetmap.josm.data.coor.LatLon;
    … …  
    504502    private AWTEventListener awtListener = new AWTEventListener() {
    505503        public void eventDispatched(AWTEvent event) {
    506             if (event instanceof ComponentEvent &&
    507                     ((ComponentEvent)event).getComponent() == mv) {
     504            if (event instanceof InputEvent &&
     505                    ((InputEvent)event).getComponent() == mv) {
    508506                synchronized (collector) {
    509507                    mouseState.modifiers = ((InputEvent)event).getModifiersEx();
    … …  
    575573        this.collector = new Collector(mapFrame);
    576574
    577         JumpToAction JumpToAct = new JumpToAction();
    578         lonText.addMouseListener(JumpToAct);
    579         latText.addMouseListener(JumpToAct);
     575        lonText.addMouseListener(Main.main.menu.jumpToAct);
     576        latText.addMouseListener(Main.main.menu.jumpToAct);
    580577
    581578        // Listen for mouse movements and set the position text field

  • trunk/src/org/openstreetmap/josm/gui/MapView.java

    r3252 r3327  
    3434import org.openstreetmap.josm.Main;
    3535import org.openstreetmap.josm.actions.AutoScaleAction;
    36 import org.openstreetmap.josm.actions.MoveAction;
    3736import org.openstreetmap.josm.actions.mapmode.MapMode;
    3837import org.openstreetmap.josm.data.Bounds;
    … …  
    190189    public MapView(final JPanel contentPane) {
    191190        Main.pref.addPreferenceChangeListener(this);
     191
     192        //        new MoveAction(MoveAction.Direction.UP);
     193        //        new MoveAction(MoveAction.Direction.DOWN);
     194        //        new MoveAction(MoveAction.Direction.LEFT);
     195        //        new MoveAction(MoveAction.Direction.RIGHT);
     196
    192197        addComponentListener(new ComponentAdapter(){
    193198            @Override public void componentResized(ComponentEvent e) {
    … …  
    206211                    if (!zoomToDataSetBoundingBox(layer.data)) {
    207212                        // no bounding box defined
    208                         new AutoScaleAction("data").actionPerformed(null);
     213                        AutoScaleAction.autoScale("data");
    209214                    }
    210215                } else {
    211                     new AutoScaleAction("layer").actionPerformed(null);
     216                    AutoScaleAction.autoScale("layer");
    212217                }
    213218
    214219                new MapMover(MapView.this, contentPane);
    215                 new MoveAction(MoveAction.Direction.UP);
    216                 new MoveAction(MoveAction.Direction.DOWN);
    217                 new MoveAction(MoveAction.Direction.LEFT);
    218                 new MoveAction(MoveAction.Direction.RIGHT);
    219220            }
    220221        });

  • trunk/src/org/openstreetmap/josm/gui/dialogs/SelectionListDialog.java

    r3290 r3327  
    294294     * Launches the search dialog
    295295     */
    296     class SearchAction extends AbstractAction implements EditLayerChangeListener {
     296    static class SearchAction extends AbstractAction implements EditLayerChangeListener {
    297297        public SearchAction() {
    298298            putValue(NAME, tr("Search"));
    … …  
    358358
    359359        public void actionPerformed(ActionEvent e) {
    360             new AutoScaleAction("selection").autoScale();
     360            AutoScaleAction.autoScale("selection");
    361361        }
    362362
    … …  
    423423        }
    424424
     425        @Override
    425426        public void actionPerformed(ActionEvent e) {
    426427            Relation relation = (Relation)model.getSelected().toArray()[0];
    … …  
    428429            Collection<OsmPrimitive> selection = model.getAllElements();
    429430            for (RelationMember member: relation.getMembers()) {
    430                 if (selection.contains(member.getMember()))
     431                if (selection.contains(member.getMember())) {
    431432                    members.add(member);
     433                }
    432434            }
    433435            Main.map.relationListDialog.selectRelation(relation);
    434436            RelationEditor.getEditor(Main.map.mapView.getEditLayer(), relation,
    435                 members).setVisible(true);
     437                    members).setVisible(true);
    436438        }
    437439    }
    … …  
    801803            {
    802804                text.append(": ");
    803                 for(OsmPrimitive o : sel)
    804                    text.append(o.getDisplayName(df));
     805                for(OsmPrimitive o : sel) {
     806                    text.append(o.getDisplayName(df));
     807                }
    805808                setText(text.toString());
    806             }
    807             else
     809            } else {
    808810                setText(tr("Selection: {0}", text));
     811            }
    809812            addActionListener(this);
    810813        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/properties/PropertiesDialog.java

    r3306 r3327  
    5151import javax.swing.table.DefaultTableCellRenderer;
    5252import javax.swing.table.DefaultTableModel;
     53import javax.swing.table.TableColumnModel;
    5354import javax.swing.table.TableModel;
    54 import javax.swing.table.TableColumnModel;
    5555import javax.swing.text.JTextComponent;
    5656
    … …  
    157157
    158158    private DataSetListenerAdapter dataChangedAdapter = new DataSetListenerAdapter(this);
     159    private AddAction addAction = new AddAction();
     160    private Shortcut addActionShortcut = Shortcut.registerShortcut("properties:add", tr("Add Properties"), KeyEvent.VK_B,
     161            Shortcut.GROUP_MNEMONIC);
    159162
    160163    @Override
    … …  
    164167        MapView.addEditLayerChangeListener(this);
    165168        updateSelection();
     169        Main.registerActionShortcut(addAction, addActionShortcut);
    166170    }
    167171
    … …  
    171175        SelectionEventManager.getInstance().removeSelectionListener(this);
    172176        MapView.removeEditLayerChangeListener(this);
     177        Main.unregisterActionShortcut(addActionShortcut);
    173178    }
    174179
    … …  
    331336     * @param row
    332337     */
    333     @SuppressWarnings("unchecked")
    334338    void membershipEdit(int row) {
    335339        Relation relation = (Relation)membershipData.getValueAt(row, 0);
    … …  
    474478            + tr("Please select the objects you want to change properties for.") + "</p></html>");
    475479
    476     class MemberInfo {
     480    static class MemberInfo {
    477481        List<RelationMember> role = new ArrayList<RelationMember>();
    478482        List<Integer> position = new ArrayList<Integer>();
    … …  
    480484        void add(RelationMember r, Integer p)
    481485        {
    482           role.add(r);
    483           position.add(p);
     486            role.add(r);
     487            position.add(p);
    484488        }
    485489        String getPositionString()
    … …  
    496500                        ++cnt;
    497501                    } else {
    498                         if(cnt == 1)
     502                        if(cnt == 1) {
    499503                            positionString += ","+String.valueOf(last);
    500                         else if(cnt > 1)
     504                        } else if(cnt > 1) {
    501505                            positionString += "-"+String.valueOf(last);
     506                        }
    502507                        positionString += "-"+String.valueOf(cur);
    503508                        cnt = 0;
    … …  
    505510                    last = cur;
    506511                }
    507                 if(cnt == 1)
     512                if(cnt == 1) {
    508513                    positionString += ","+String.valueOf(last);
    509                 else if(cnt > 1)
     514                } else if(cnt > 1) {
    510515                    positionString += "-"+String.valueOf(last);
    511             }
    512             if(positionString.length() > 20)
    513               positionString = positionString.substring(0,17)+"...";
     516                }
     517            }
     518            if(positionString.length() > 20) {
     519                positionString = positionString.substring(0,17)+"...";
     520            }
    514521            return positionString;
    515522        }
    … …  
    589596
    590597        mod.getColumn(1).setCellRenderer(new DefaultTableCellRenderer() {
    591             @SuppressWarnings("unchecked")
    592598            @Override public Component getTableCellRendererComponent(JTable table, Object value,
    593599                    boolean isSelected, boolean hasFocus, int row, int column) {
    … …  
    619625
    620626        mod.getColumn(2).setCellRenderer(new DefaultTableCellRenderer() {
    621             @SuppressWarnings("unchecked")
    622627            @Override public Component getTableCellRendererComponent(JTable table, Object value,
    623628                    boolean isSelected, boolean hasFocus, int row, int column) {
    … …  
    667672
    668673        // -- add action and shortcut
    669         AddAction addAction = new AddAction();
    670674        this.btnAdd = new SideButton(addAction);
    671675        btnAdd.setFocusable(true);
    … …  
    673677        btnAdd.getInputMap(JComponent.WHEN_FOCUSED).put(KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0), "onEnter");
    674678        btnAdd.getActionMap().put("onEnter", addAction);
    675 
    676         Main.registerActionShortcut(addAction, Shortcut.registerShortcut("properties:add", tr("Add Properties"), KeyEvent.VK_B,
    677                 Shortcut.GROUP_MNEMONIC));
    678679
    679680        // -- edit action
    … …  
    873874                    Relation r = (Relation) ref;
    874875                    MemberInfo mi = roles.get(r);
    875                     if(mi == null)
     876                    if(mi == null) {
    876877                        mi = new MemberInfo();
     878                    }
    877879                    roles.put(r, mi);
    878880                    int i = 1;

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTable.java

    r3210 r3327  
    203203            OsmPrimitive primitive = getMemberTableModel().getReferredPrimitive(row);
    204204            layer.data.setSelected(primitive);
    205             AutoScaleAction action = new AutoScaleAction("selection");
    206             action.autoScale();
     205            AutoScaleAction.autoScale("selection");
    207206        }
    208207

  • trunk/src/org/openstreetmap/josm/gui/history/NodeListViewer.java

    r3083 r3327  
    203203            if (p!= null) {
    204204                getEditLayer().data.setSelected(p.getPrimitiveId());
    205                 new AutoScaleAction("selection").autoScale();
     205                AutoScaleAction.autoScale("selection");
    206206            }
    207207        }

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPreset.java

    r3321 r3327  
    829829        if (data == null)
    830830            return null;
    831         OsmDataLayer layer = Main.main.getEditLayer();
    832831        PresetPanel p = new PresetPanel();
    833832        LinkedList<Item> l = new LinkedList<Item>();
Note: See TracChangeset for help on using the changeset viewer.