aljaz/btcpayserver
1
0
Fork 0
mirror of https://github.com/aljazceru/btcpayserver.git synced 2025-12-17 14:04:26 +01:00
Code Issues Packages Projects Releases Wiki Activity

Do not ignore IsAdmin settings when creating a new user via the Users page (Fix #1954)

Browse Source
This commit is contained in:
nicolas.dorier
2020-10-08 11:56:58 +09:00
parent 5131d8d328
commit 32e6643303
1 changed files with 2 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
BTCPayServer/Controllers/ServerController.Users.cs
View File

@@ -99,7 +99,8 @@ namespace BTCPayServer.Controllers
{ {
ViewData["AllowIsAdmin"] = _Options.AllowAdminRegistration; ViewData["AllowIsAdmin"] = _Options.AllowAdminRegistration;
ViewData["AllowRequestEmailConfirmation"] = _cssThemeManager.Policies.RequiresConfirmedEmail; ViewData["AllowRequestEmailConfirmation"] = _cssThemeManager.Policies.RequiresConfirmedEmail;
if (!_Options.AllowAdminRegistration)
model.IsAdmin = false;
if (ModelState.IsValid) if (ModelState.IsValid)
{ {
IdentityResult result; IdentityResult result;
@@ -107,14 +108,7 @@ namespace BTCPayServer.Controllers
if (!string.IsNullOrEmpty(model.Password)) if (!string.IsNullOrEmpty(model.Password))
{ {
result = await _UserManager.CreateAsync(user, model.Password); result = await _UserManager.CreateAsync(user, model.Password);
if (result.Succeeded)
{
TempData[WellKnownTempData.SuccessMessage] = "Account created";
return RedirectToAction(nameof(ListUsers));
}
} }
else else
{ {