Changeset 8380 in josm


Ignore:
Timestamp:
2015-05-17T06:02:07+02:00 (5 years ago)
Author:
Don-vip
Message:

code style - A getX() method which returns a boolean should be named isX()

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

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/upload/UploadNotesTask.java

    r8285 r8380  
    7272                }
    7373                for (NoteComment comment : note.getComments()) {
    74                     if (comment.getIsNew()) {
     74                    if (comment.isNew()) {
    7575                        if (Main.isDebugEnabled()) {
    7676                            Main.debug("found note change to upload");
  • trunk/src/org/openstreetmap/josm/actions/upload/ValidateUploadHook.java

    r8378 r8380  
    9292                    }
    9393                }
    94                 if (!error.getIgnored()) {
     94                if (!error.isIgnored()) {
    9595                    ++nume;
    9696                }
  • trunk/src/org/openstreetmap/josm/data/notes/NoteComment.java

    r8377 r8380  
    6161    }
    6262
    63     public void setIsNew(boolean isNew) {
     63    public void setNew(boolean isNew) {
    6464        this.isNew = isNew;
    6565    }
    6666
    6767    /** @return true if this is a new comment/action and needs to be uploaded to the API */
    68     public boolean getIsNew() {
     68    public boolean isNew() {
    6969        return isNew;
    7070    }
  • trunk/src/org/openstreetmap/josm/data/osm/NoteData.java

    r8365 r8380  
    149149            }
    150150            for (NoteComment comment : note.getComments()) {
    151                 if (comment.getIsNew()) {
     151                if (comment.isNew()) {
    152152                    return true;
    153153                }
     
    170170                    @Override
    171171                    public boolean evaluate(NoteComment object) {
    172                         return object.getIsNew();
     172                        return object.isNew();
    173173                    }
    174174                });
  • trunk/src/org/openstreetmap/josm/data/validation/PaintVisitor.java

    r8285 r8380  
    112112    @Override
    113113    public void visit(TestError error) {
    114         if (error != null && !error.getIgnored()) {
     114        if (error != null && !error.isIgnored()) {
    115115            color = error.getSeverity().getColor();
    116116            selected = error.isSelected();
  • trunk/src/org/openstreetmap/josm/data/validation/TestError.java

    r8378 r8380  
    208208    }
    209209
    210     public boolean getIgnored() {
     210    public boolean isIgnored() {
    211211        return ignored;
    212212    }
  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r8378 r8380  
    486486     */
    487487    public static JMenuItem add(JMenu menu, JosmAction action, boolean isExpert, Integer index) {
    488         if (action.getShortcut().getAutomatic())
     488        if (action.getShortcut().isAutomatic())
    489489            return null;
    490490        final JMenuItem menuitem;
     
    540540     */
    541541    public static <E extends Enum<E>> JMenuItem add(JMenu menu, JosmAction action, Enum<E> group) {
    542         if (action.getShortcut().getAutomatic())
     542        if (action.getShortcut().isAutomatic())
    543543            return null;
    544544        int i = getInsertionIndexForGroup(menu, group.ordinal());
  • trunk/src/org/openstreetmap/josm/gui/dialogs/validator/ValidatorTreePanel.java

    r8378 r8380  
    182182        final Boolean other = ValidatorPreference.PREF_OTHER.get();
    183183        for (TestError e : errors) {
    184             if (e.getIgnored()) {
     184            if (e.isIgnored()) {
    185185                continue;
    186186            }
     
    321321        DataSet ds = Main.main.getCurrentDataSet();
    322322        for (TestError error : newerrors) {
    323             if (!error.getIgnored()) {
     323            if (!error.isIgnored()) {
    324324                errors.add(error);
    325325                if (ds != null) {
  • trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

    r8345 r8380  
    11211121
    11221122            // add shortcut info to the tooltip if needed
    1123             if (sc.getAssignedUser()) {
     1123            if (sc.isAssignedUser()) {
    11241124                if (tt.startsWith("<html>") && tt.endsWith("</html>")) {
    11251125                    tt = tt.substring(6,tt.length()-6);
  • trunk/src/org/openstreetmap/josm/gui/preferences/shortcut/PrefJPanel.java

    r8345 r8380  
    166166                label.setForeground(Main.pref.getUIColor("Table.foreground"));
    167167            }
    168             if(sc.getAssignedUser()) {
     168            if(sc.isAssignedUser()) {
    169169                label.setBackground(Main.pref.getColor(
    170170                        marktr("Shortcut Background: User"),
    171171                        new Color(200,255,200)));
    172             } else if(!sc.getAssignedDefault()) {
     172            } else if(!sc.isAssignedDefault()) {
    173173                label.setBackground(Main.pref.getColor(
    174174                        marktr("Shortcut Background: Modified"),
     
    289289                int row = panel.shortcutTable.convertRowIndexToModel(lsm.getMinSelectionIndex());
    290290                Shortcut sc = (Shortcut)panel.model.getValueAt(row, -1);
    291                 panel.cbDefault.setSelected(!sc.getAssignedUser());
     291                panel.cbDefault.setSelected(!sc.isAssignedUser());
    292292                panel.cbDisable.setSelected(sc.getKeyStroke() == null);
    293293                panel.cbShift.setSelected(sc.getAssignedModifier() != -1 && (sc.getAssignedModifier() & KeyEvent.SHIFT_DOWN_MASK) != 0);
  • trunk/src/org/openstreetmap/josm/io/NoteWriter.java

    r8346 r8380  
    7777            out.print("user=\"" + encode(comment.getUser().getName()) + "\" ");
    7878        }
    79         out.print("is_new=\"" + comment.getIsNew() + "\" ");
     79        out.print("is_new=\"" + comment.isNew() + "\" ");
    8080        out.print(">");
    8181        out.print(encode(comment.getText(), false));
  • trunk/src/org/openstreetmap/josm/tools/Shortcut.java

    r8378 r8380  
    8282    }
    8383
    84     public boolean getAssignedDefault() {
     84    public boolean isAssignedDefault() {
    8585        return assignedDefault;
    8686    }
    8787
    88     public boolean getAssignedUser() {
     88    public boolean isAssignedUser() {
    8989        return assignedUser;
    9090    }
    9191
    92     public boolean getAutomatic() {
     92    public boolean isAutomatic() {
    9393        return automatic;
    9494    }
     
    9898    }
    9999
    100     private boolean getReset() {
     100    private boolean isReset() {
    101101        return reset;
    102102    }
     
    167167    // get a string that can be put into the preferences
    168168    private boolean save() {
    169         if (getAutomatic() || getReset() || !getAssignedUser()) {
     169        if (isAutomatic() || isReset() || !isAssignedUser()) {
    170170            return Main.pref.putCollection("shortcut.entry."+shortText, null);
    171171        } else {
     
    316316
    317317        for(Shortcut sc : newshortcuts) {
    318             if (sc.getAssignedUser()
     318            if (sc.isAssignedUser()
    319319            && findShortcut(sc.getAssignedKey(), sc.getAssignedModifier()) == null) {
    320320                shortcuts.put(sc.getShortText(), sc);
     
    323323        // Shortcuts at their default values
    324324        for(Shortcut sc : newshortcuts) {
    325             if (!sc.getAssignedUser() && sc.getAssignedDefault()
     325            if (!sc.isAssignedUser() && sc.isAssignedDefault()
    326326            && findShortcut(sc.getAssignedKey(), sc.getAssignedModifier()) == null) {
    327327                shortcuts.put(sc.getShortText(), sc);
     
    330330        // Shortcuts that were automatically moved
    331331        for(Shortcut sc : newshortcuts) {
    332             if (!sc.getAssignedUser() && !sc.getAssignedDefault()
     332            if (!sc.isAssignedUser() && !sc.isAssignedDefault()
    333333            && findShortcut(sc.getAssignedKey(), sc.getAssignedModifier()) == null) {
    334334                shortcuts.put(sc.getShortText(), sc);
Note: See TracChangeset for help on using the changeset viewer.