Skip to content

Commit 54c84b5

Browse files
authored
Merge pull request #1227 from johnhenley/issues/null-createdate-1226
FIX: Replace null CreatedOnDate with minimum SQL date when creating Forum User in activeforums_UserProfiles
2 parents 4ff46d2 + 00d36df commit 54c84b5

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

Dnn.CommunityForums/Controllers/ForumUserController.cs

+4
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,10 @@ public DotNetNuke.Modules.ActiveForums.Entities.ForumUserInfo GetByUserId(int po
9696
if (user.UserInfo != null)
9797
{
9898
user.DateCreated = user.UserInfo.CreatedOnDate;
99+
if (user.DateCreated < System.Data.SqlTypes.SqlDateTime.MinValue.Value)
100+
{
101+
user.DateCreated = System.Data.SqlTypes.SqlDateTime.MinValue.Value;
102+
}
99103
this.Insert(user);
100104
}
101105
}

Dnn.CommunityForums/Entities/ForumUserInfo.cs

+2-2
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public ForumUserInfo(int moduleId)
7878

7979
public string UserCaption { get; set; }
8080

81-
public DateTime DateCreated { get; set; } = DateTime.UtcNow;
81+
public DateTime? DateCreated { get; set; } = DateTime.UtcNow;
8282

8383
public DateTime? DateUpdated { get; set; }
8484

@@ -509,7 +509,7 @@ public string GetProperty(string propertyName, string format, System.Globalizati
509509
case "displayname":
510510
return PropertyAccess.FormatString(DotNetNuke.Modules.ActiveForums.Controllers.ForumUserController.GetDisplayName(this.PortalSettings, this.MainSettings, isMod: new DotNetNuke.Modules.ActiveForums.Controllers.ForumUserController(this.ModuleId).GetByUserId(portalId: accessingUser.PortalID, userId: accessingUser.UserID).GetIsMod(this.ModuleId), isAdmin: new DotNetNuke.Modules.ActiveForums.Controllers.ForumUserController(this.ModuleId).GetByUserId(portalId: accessingUser.PortalID, userId: accessingUser.UserID).IsAdmin, this.UserId, this.Username, this.FirstName, this.LastName, this.DisplayName), format);
511511
case "datecreated":
512-
return Utilities.GetUserFormattedDateTime((DateTime?)this.DateCreated, formatProvider, accessingUser.Profile.PreferredTimeZone.GetUtcOffset(DateTime.UtcNow));
512+
return Utilities.GetUserFormattedDateTime(this.DateCreated, formatProvider, accessingUser.Profile.PreferredTimeZone.GetUtcOffset(DateTime.UtcNow));
513513
case "dateupdated":
514514
return Utilities.GetUserFormattedDateTime(this.DateUpdated, formatProvider, accessingUser.Profile.PreferredTimeZone.GetUtcOffset(DateTime.UtcNow));
515515
case "datelastpost":

0 commit comments

Comments
 (0)