Ignore:
Timestamp:
2016-02-17T21:40:45+01:00 (3 years ago)
Author:
Don-vip
Message:

add more unit tests, javadoc, fix code style issues

File:
1 edited

Legend:

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

    r9078 r9816  
    5555    private NodeListPopupMenu popupMenu;
    5656
     57    /**
     58     * Constructs a new {@code NodeListViewer}.
     59     * @param model history browser model
     60     */
     61    public NodeListViewer(HistoryBrowserModel model) {
     62        setModel(model);
     63        build();
     64    }
     65
    5766    protected JScrollPane embeddInScrollPane(JTable table) {
    5867        JScrollPane pane = new JScrollPane(table);
     
    96105            public void tableChanged(TableModelEvent e) {
    97106                if (e.getSource() instanceof DiffTableModel) {
    98                     final DiffTableModel model = (DiffTableModel) e.getSource();
    99                     if (reversed == null || reversed != model.isReversed()) {
    100                         reversed = model.isReversed();
     107                    final DiffTableModel mod = (DiffTableModel) e.getSource();
     108                    if (reversed == null || reversed != mod.isReversed()) {
     109                        reversed = mod.isReversed();
    101110                        columnModel.getColumn(0).setHeaderValue(reversed ? reversedText : nonReversedText);
    102111                        table.getTableHeader().setToolTipText(
     
    164173    }
    165174
    166     public NodeListViewer(HistoryBrowserModel model) {
    167         setModel(model);
    168         build();
    169     }
    170 
    171175    protected void unregisterAsObserver(HistoryBrowserModel model) {
    172176        if (currentInfoPanel != null) {
     
    187191    }
    188192
     193    /**
     194     * Sets the history browser model.
     195     * @param model the history browser model
     196     */
    189197    public void setModel(HistoryBrowserModel model) {
    190198        if (this.model != null) {
     
    231239        @Override
    232240        public void actionPerformed(ActionEvent e) {
    233             if (!isEnabled()) return;
     241            if (!isEnabled())
     242                return;
    234243            OsmPrimitive p = getPrimitiveToZoom();
    235244            if (p != null) {
     
    248257
    249258        protected OsmPrimitive getPrimitiveToZoom() {
    250             if (primitiveId == null) return null;
     259            if (primitiveId == null)
     260                return null;
    251261            OsmDataLayer editLayer = Main.main.getEditLayer();
    252             if (editLayer == null) return null;
     262            if (editLayer == null)
     263                return null;
    253264            return editLayer.data.getPrimitiveById(primitiveId);
    254265        }
     
    277288        @Override
    278289        public void actionPerformed(ActionEvent e) {
    279             if (!isEnabled()) return;
    280             run();
     290            if (isEnabled()) {
     291                run();
     292            }
    281293        }
    282294
     
    314326        DiffTableModel castedModel = (DiffTableModel) model;
    315327        Long id = (Long) castedModel.getValueAt(row, 0).value;
    316         if (id == null) return null;
    317         return new SimplePrimitiveId(id, OsmPrimitiveType.NODE);
     328        return id == null ? null : new SimplePrimitiveId(id, OsmPrimitiveType.NODE);
    318329    }
    319330
     
    342353        @Override
    343354        public void mouseClicked(MouseEvent e) {
    344             if (e.getClickCount() < 2) return;
     355            if (e.getClickCount() < 2)
     356                return;
    345357            int row = table.rowAtPoint(e.getPoint());
    346             if (row <= 0) return;
     358            if (row <= 0)
     359                return;
    347360            PrimitiveId pid = primitiveIdAtRow(table.getModel(), row);
    348361            if (pid == null || pid.isNew())
Note: See TracChangeset for help on using the changeset viewer.