Fix crash when status date is null (#1480)

* Fix crash when status date is null

* Fix crash when status date is null
main
Konrad Pozniak 5 years ago committed by GitHub
parent 83311b7f08
commit 18dadc843a
  1. 17
      app/src/main/java/com/keylesspalace/tusky/adapter/StatusBaseViewHolder.java
  2. 6
      app/src/main/java/com/keylesspalace/tusky/adapter/StatusDetailedViewHolder.java

@ -252,9 +252,12 @@ public abstract class StatusBaseViewHolder extends RecyclerView.ViewHolder {
}
protected void setCreatedAt(@NonNull Date createdAt) {
protected void setCreatedAt(Date createdAt) {
if (useAbsoluteTime) {
timestampInfo.setText(getAbsoluteTime(createdAt));
} else {
if(createdAt == null) {
timestampInfo.setText("?m");
} else {
long then = createdAt.getTime();
long now = System.currentTimeMillis();
@ -262,8 +265,12 @@ public abstract class StatusBaseViewHolder extends RecyclerView.ViewHolder {
timestampInfo.setText(readout);
}
}
}
private String getAbsoluteTime(@NonNull Date createdAt) {
private String getAbsoluteTime(Date createdAt) {
if(createdAt == null) {
return "??:??:??";
}
if (DateUtils.isToday(createdAt.getTime())) {
return shortSdf.format(createdAt);
} else {
@ -271,13 +278,16 @@ public abstract class StatusBaseViewHolder extends RecyclerView.ViewHolder {
}
}
private CharSequence getCreatedAtDescription(@NonNull Date createdAt) {
private CharSequence getCreatedAtDescription(Date createdAt) {
if (useAbsoluteTime) {
return getAbsoluteTime(createdAt);
} else {
/* This one is for screen-readers. Frequently, they would mispronounce timestamps like "17m"
* as 17 meters instead of minutes. */
if(createdAt == null) {
return "? minutes";
} else {
long then = createdAt.getTime();
long now = System.currentTimeMillis();
return DateUtils.getRelativeTimeSpanString(then, now,
@ -285,6 +295,7 @@ public abstract class StatusBaseViewHolder extends RecyclerView.ViewHolder {
DateUtils.FORMAT_ABBREV_RELATIVE);
}
}
}
protected void setIsReply(boolean isReply) {
if (isReply) {

@ -67,10 +67,14 @@ class StatusDetailedViewHolder extends StatusBaseViewHolder {
}
@Override
protected void setCreatedAt(@NonNull Date createdAt) {
protected void setCreatedAt(Date createdAt) {
if(createdAt == null) {
timestampInfo.setText("");
} else {
DateFormat dateFormat = DateFormat.getDateTimeInstance(DateFormat.DEFAULT, DateFormat.SHORT);
timestampInfo.setText(dateFormat.format(createdAt));
}
}
private void setReblogAndFavCount(int reblogCount, int favCount, StatusActionListener listener) {

Loading…
Cancel
Save