Ignore:
Timestamp:
2014-04-29T03:24:57+02:00 (5 years ago)
Author:
Don-vip
Message:

Sonar - fix various issues

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

Legend:

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

    r7005 r7025  
    184184                    BugReportExceptionHandler.handleException(e);
    185185                }
    186 
    187186            }
    188187        };
     
    200199                // reload if the history is not in the cache yet
    201200                return true;
    202             else if (!p.isNew() && h.getByVersion(p.getUniqueId()) == null)
     201            else
    203202                // reload if the history object of the selected object is not in the cache yet
    204                 return true;
    205             else
    206                 return false;
     203                return (!p.isNew() && h.getByVersion(p.getUniqueId()) == null);
    207204        }
    208205    };
     
    215212        }
    216213    };
    217 
    218214}
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r7005 r7025  
    427427            case 2:
    428428                return isCurrentPointInTime(row);
    429             case 3: {
    430                 HistoryOsmPrimitive p = getPrimitive(row);
    431                 if (p != null && p.getTimestamp() != null)
    432                     return DateFormat.getDateTimeInstance(DateFormat.SHORT, DateFormat.SHORT).format(p.getTimestamp());
     429            case 3:
     430                HistoryOsmPrimitive p3 = getPrimitive(row);
     431                if (p3 != null && p3.getTimestamp() != null)
     432                    return DateFormat.getDateTimeInstance(DateFormat.SHORT, DateFormat.SHORT).format(p3.getTimestamp());
    433433                return null;
    434             }
    435             case 4: {
    436                 HistoryOsmPrimitive p = getPrimitive(row);
    437                 if (p != null) {
    438                     User user = p.getUser();
     434            case 4:
     435                HistoryOsmPrimitive p4 = getPrimitive(row);
     436                if (p4 != null) {
     437                    User user = p4.getUser();
    439438                    if (user != null)
    440439                        return user.getName();
    441440                }
    442441                return null;
    443             }
    444442            }
    445443            return null;
Note: See TracChangeset for help on using the changeset viewer.