Changeset 1884 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTable.java

Timestamp:

2009-08-02T19:13:42+02:00 (15 years ago)

Author:

jttt

Message:

Fix ArrayIndexOutOfBoundsException 0:-2 in relation editor when Shift-Tab is pressed in table without selected cell

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTable.java (modified) (8 diffs)

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

@@ -1 +1 @@
 // License: GPL. For details, see LICENSE file.
 package org.openstreetmap.josm.gui.dialogs.relation;
+
+import static org.openstreetmap.josm.tools.I18n.tr;
 
 import java.awt.event.ActionEvent;
@@ -25 +27 @@
 import org.openstreetmap.josm.gui.layer.Layer.LayerChangeListener;
 
-import static org.openstreetmap.josm.tools.I18n.tr;
-
 public class MemberTable extends JTable implements IMemberModelListener {
 
@@ -39 +39 @@
     /**
      * constructor
-     * 
+     *
      * @param model
      * @param columnModel
@@ -75 +75 @@
      * adjusts the width of the columns for the tag name and the tag value to the width of the
      * scroll panes viewport.
-     * 
+     *
      * Note: {@see #getPreferredScrollableViewportSize()} did not work as expected
-     * 
+     *
      * @param scrollPaneWidth the width of the scroll panes viewport
      */
@@ -104 +104 @@
      * <li>it jumps over cells in the first column</li> <li>it automatically add a new empty row
      * when the user leaves the last cell in the table</li> <ul>
-     * 
-     * 
+     *
+     *
      */
     class SelectNextColumnCellAction extends AbstractAction {
@@ -132 +132 @@
      * Action to be run when the user navigates to the previous cell in the table, for instance by
      * pressing Shift-TAB
-     * 
+     *
      */
     class SelectPreviousColumnCellAction extends AbstractAction {
@@ -143 +143 @@
             }
 
-            if (col == 0 && row == 0) {
+            if (col <= 0 && row <= 0) {
                 // change nothing
             } else if (row > 0) {
@@ -166 +166 @@
     /**
      * Replies the popup menu for this table
-     * 
+     *
      * @return the popup menu
      */