Changeset 14877 in josm


Ignore:
Timestamp:
2019-03-10T20:57:11+01:00 (7 weeks ago)
Author:
GerdP
Message:

fix sonar issue: Replace this if-then-else statement by a single method invocation.

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/dialogs/NotesDialog.java

    r14470 r14877  
    137137        }
    138138        openInBrowserAction.setEnabled(noteData != null && noteData.getSelectedNote() != null && noteData.getSelectedNote().getId() > 0);
    139         if (noteData == null || !noteData.isModified()) {
    140             uploadAction.setEnabled(false);
    141         } else {
    142             uploadAction.setEnabled(true);
    143         }
     139        uploadAction.setEnabled(noteData != null && noteData.isModified());
    144140        //enable sort button if any notes are loaded
    145         if (noteData == null || noteData.getNotes().isEmpty()) {
    146             sortAction.setEnabled(false);
    147         } else {
    148             sortAction.setEnabled(true);
    149         }
     141        sortAction.setEnabled(noteData != null && !noteData.getNotes().isEmpty());
    150142    }
    151143
  • trunk/src/org/openstreetmap/josm/gui/tagging/TagTable.java

    r13726 r14877  
    220220
    221221        protected final void updateEnabledState() {
    222             if (getSelectedColumnCount() >= 1 && getSelectedRowCount() >= 1) {
    223                 setEnabled(true);
    224             } else {
    225                 setEnabled(false);
    226             }
     222            setEnabled(getSelectedColumnCount() >= 1 && getSelectedRowCount() >= 1);
    227223        }
    228224    }
Note: See TracChangeset for help on using the changeset viewer.