Fixes attempting to attach an already loaded User while test merging

This commit is contained in:
Cyberboss
2018-10-02 10:21:33 -04:00
parent 723a9a5596
commit df454396a4
@@ -644,11 +644,16 @@ namespace Tgstation.Server.Host.Controllers
if (needToApplyRemainingPrs)
{
var contextUser = new Models.User
//an invocation of LoadRevisionInformation could have already loaded this user
var contextUser = databaseContext.Users.Local.Where(x => x.Id == AuthenticationContext.User.Id).FirstOrDefault();
if (contextUser == default)
{
Id = AuthenticationContext.User.Id
};
databaseContext.Users.Attach(contextUser);
contextUser = new Models.User
{
Id = AuthenticationContext.User.Id
};
databaseContext.Users.Attach(contextUser);
}
foreach (var I in model.NewTestMerges)
{