Merge pull request #3539 from peppy/fix-last-visit-regression

Fix last visit not supporting null values
This commit is contained in:
Dean Herbert 2018-10-05 00:39:43 +09:00 committed by GitHub
commit 9d518ff3ab
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 5 deletions

View File

@ -386,10 +386,13 @@ namespace osu.Game.Overlays.Profile
infoTextLeft.AddText(new DrawableJoinDate(user.JoinDate), boldItalic);
}
infoTextLeft.NewLine();
infoTextLeft.AddText("Last seen ", lightText);
infoTextLeft.AddText(new DrawableDate(user.LastVisit), boldItalic);
infoTextLeft.NewParagraph();
if (user.LastVisit.HasValue)
{
infoTextLeft.NewLine();
infoTextLeft.AddText("Last seen ", lightText);
infoTextLeft.AddText(new DrawableDate(user.LastVisit.Value), boldItalic);
infoTextLeft.NewParagraph();
}
if (user.PlayStyle?.Length > 0)
{

View File

@ -84,7 +84,7 @@ namespace osu.Game.Users
public string Location;
[JsonProperty(@"last_visit")]
public DateTimeOffset LastVisit;
public DateTimeOffset? LastVisit;
[JsonProperty(@"twitter")]
public string Twitter;