Changeset 10089 in josm for trunk/src/org/openstreetmap/josm/gui/dialogs/relation/SelectionTable.java

Timestamp:

2016-03-30T22:28:45+02:00 (10 years ago)

Author:

Don-vip

Message:

add unit test, javadoc

File:

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

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

@@ -10 +10 @@
 import javax.swing.SwingUtilities;
 
-import org.openstreetmap.josm.data.osm.OsmPrimitive;
-
+/**
+ * Selection table of relation editor.
+ * @since 2563
+ */
 public class SelectionTable extends JTable {
 
     private final SelectionTableModel model;
-    private MemberTableModel memberTableModel;
+    private final MemberTableModel memberTableModel;
+
+    /**
+     * Constructs a new {@code SelectionTable}.
+     * @param model table model
+     * @param memberTableModel member table model
+     */
+    public SelectionTable(SelectionTableModel model, MemberTableModel memberTableModel) {
+        super(model, new SelectionTableColumnModel(memberTableModel));
+        this.model = model;
+        this.memberTableModel = memberTableModel;
+        build();
+    }
 
     protected void build() {
         setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
         addMouseListener(new DoubleClickAdapter());
-    }
-
-    public SelectionTable(SelectionTableModel model, SelectionTableColumnModel columnModel) {
-        super(model, columnModel);
-        this.model = model;
-        build();
-    }
-
-    public void setMemberTableModel(MemberTableModel memberTableModel) {
-        this.memberTableModel = memberTableModel;
     }
 
@@ -38 +42 @@
                 return;
             int row = rowAtPoint(evt.getPoint());
-            OsmPrimitive primitive = model.getPrimitive(row);
-            memberTableModel.selectMembersReferringTo(Collections.singleton(primitive));
+            memberTableModel.selectMembersReferringTo(Collections.singleton(model.getPrimitive(row)));
         }
     }