bugs fix
This commit is contained in:
parent
599e9a8865
commit
84f9f63407
44 changed files with 560 additions and 92 deletions
|
@ -22,7 +22,7 @@ using WebsitePanel.WebDav.Core;
|
|||
namespace WebsitePanel.WebDavPortal.Controllers
|
||||
{
|
||||
[LdapAuthorization]
|
||||
public class AccountController : Controller
|
||||
public class AccountController : BaseController
|
||||
{
|
||||
private readonly ICryptography _cryptography;
|
||||
private readonly IAuthenticationService _authenticationService;
|
||||
|
@ -94,11 +94,21 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
|
||||
int result = UpdateUserProfile(WspContext.User.ItemId, WspContext.User.AccountId, model);
|
||||
|
||||
model.AddMessage(MessageType.Success, Resources.UI.UserProfileSuccessfullyUpdated);
|
||||
AddMessage(MessageType.Success, Resources.UI.UserProfileSuccessfullyUpdated);
|
||||
|
||||
return View(model);
|
||||
}
|
||||
|
||||
public JsonResult PhoneNumberIsAvailible()
|
||||
{
|
||||
var value = Request.QueryString.AllKeys.Any() ? Request.QueryString.Get(0) :string.Empty;
|
||||
|
||||
var result = !WspContext.Services.Organizations.CheckPhoneNumberIsInUse(WspContext.User.ItemId,
|
||||
value, WspContext.User.Login);
|
||||
|
||||
return Json(result, JsonRequestBehavior.AllowGet);
|
||||
}
|
||||
|
||||
[HttpGet]
|
||||
public ActionResult PasswordChange()
|
||||
{
|
||||
|
@ -118,7 +128,7 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
|
||||
if (_authenticationService.ValidateAuthenticationData(WspContext.User.Login, model.OldPassword) == false)
|
||||
{
|
||||
model.AddMessage(MessageType.Error, Resources.Messages.OldPasswordIsNotCorrect);
|
||||
AddMessage(MessageType.Error, Resources.Messages.OldPasswordIsNotCorrect);
|
||||
|
||||
return View(model);
|
||||
}
|
||||
|
@ -127,6 +137,12 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
WspContext.User.ItemId, WspContext.User.AccountId,
|
||||
model.PasswordEditor.NewPassword);
|
||||
|
||||
var user = _authenticationService.LogIn(WspContext.User.Login, model.PasswordEditor.NewPassword);
|
||||
|
||||
_authenticationService.CreateAuthenticationTicket(user);
|
||||
|
||||
AddMessage(MessageType.Success, Resources.Messages.PasswordSuccessfullyChanged);
|
||||
|
||||
return RedirectToRoute(AccountRouteNames.UserProfile);
|
||||
}
|
||||
|
||||
|
@ -152,7 +168,7 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
|
||||
if (exchangeAccount == null)
|
||||
{
|
||||
model.AddMessage(MessageType.Error, Resources.Messages.AccountNotFound);
|
||||
AddMessage(MessageType.Error, Resources.Messages.AccountNotFound);
|
||||
|
||||
return View(model);
|
||||
}
|
||||
|
@ -175,7 +191,7 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
|
||||
if (model.IsTokenExist == false)
|
||||
{
|
||||
model.AddMessage(MessageType.Error, Resources.Messages.IncorrectPasswordResetUrl);
|
||||
AddMessage(MessageType.Error, Resources.Messages.IncorrectPasswordResetUrl);
|
||||
|
||||
return View(model);
|
||||
}
|
||||
|
@ -184,8 +200,14 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
{
|
||||
var user = WspContext.Services.Organizations.GetUserGeneralSettings(accessToken.ItemId, accessToken.AccountId);
|
||||
|
||||
var response = _smsAuthService.SendRequestMessage(user.MobilePhone);
|
||||
WspContext.Services.Organizations.SetAccessTokenResponse(accessToken.AccessTokenGuid, response);
|
||||
if (SendPasswordResetSms(accessToken.AccessTokenGuid, user.MobilePhone))
|
||||
{
|
||||
AddMessage(MessageType.Success, Resources.Messages.SmsWasSent);
|
||||
}
|
||||
else
|
||||
{
|
||||
AddMessage(MessageType.Error, Resources.Messages.SmsWasNotSent);
|
||||
}
|
||||
}
|
||||
|
||||
return View(model);
|
||||
|
@ -210,7 +232,7 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
return RedirectToRoute(AccountRouteNames.PasswordResetFinalStep);
|
||||
}
|
||||
|
||||
model.AddMessage(MessageType.Error, Resources.Messages.IncorrectSmsResponse);
|
||||
AddMessage(MessageType.Error, Resources.Messages.IncorrectSmsResponse);
|
||||
|
||||
return View(model);
|
||||
}
|
||||
|
@ -245,7 +267,7 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
|
||||
if (_smsAuthService.VerifyResponse(token, smsResponse) == false)
|
||||
{
|
||||
model.AddMessage(MessageType.Error, Resources.Messages.IncorrectSmsResponse);
|
||||
AddMessage(MessageType.Error, Resources.Messages.IncorrectSmsResponse);
|
||||
|
||||
return RedirectToRoute(AccountRouteNames.PasswordResetSms);
|
||||
}
|
||||
|
@ -258,6 +280,8 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
|
||||
WspContext.Services.Organizations.DeletePasswordresetAccessToken(token);
|
||||
|
||||
AddMessage(MessageType.Success, Resources.Messages.PasswordSuccessfullyChanged);
|
||||
|
||||
return RedirectToRoute(AccountRouteNames.Login);
|
||||
}
|
||||
|
||||
|
@ -275,14 +299,34 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
var user = WspContext.Services.Organizations.GetUserGeneralSettings(accessToken.ItemId,
|
||||
accessToken.AccountId);
|
||||
|
||||
var response = _smsAuthService.SendRequestMessage(user.MobilePhone);
|
||||
WspContext.Services.Organizations.SetAccessTokenResponse(accessToken.AccessTokenGuid, response);
|
||||
|
||||
if (SendPasswordResetSms(accessToken.AccessTokenGuid, user.MobilePhone))
|
||||
{
|
||||
AddMessage(MessageType.Success, Resources.Messages.SmsWasSent);
|
||||
}
|
||||
else
|
||||
{
|
||||
AddMessage(MessageType.Error, Resources.Messages.SmsWasNotSent);
|
||||
}
|
||||
|
||||
return RedirectToRoute(AccountRouteNames.PasswordResetSms);
|
||||
}
|
||||
|
||||
#region Helpers
|
||||
|
||||
private bool SendPasswordResetSms(Guid token, string mobilePhone)
|
||||
{
|
||||
var response = _smsAuthService.SendRequestMessage(mobilePhone);
|
||||
|
||||
if (string.IsNullOrEmpty(response))
|
||||
{
|
||||
return false;
|
||||
}
|
||||
WspContext.Services.Organizations.SetAccessTokenResponse(token, response);
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
private UserProfile GetUserProfileModel(int itemId, int accountId)
|
||||
{
|
||||
var user = WspContext.Services.Organizations.GetUserGeneralSettings(itemId, accountId);
|
||||
|
|
|
@ -0,0 +1,33 @@
|
|||
using System;
|
||||
using System.Collections.Generic;
|
||||
using System.Linq;
|
||||
using System.Web;
|
||||
using System.Web.Mvc;
|
||||
using WebsitePanel.WebDavPortal.Models.Common;
|
||||
using WebsitePanel.WebDavPortal.Models.Common.Enums;
|
||||
|
||||
namespace WebsitePanel.WebDavPortal.Controllers
|
||||
{
|
||||
public class BaseController : Controller
|
||||
{
|
||||
public const string MessagesKey = "messagesKey";
|
||||
|
||||
public void AddMessage(MessageType type, string value)
|
||||
{
|
||||
var messages = TempData[MessagesKey] as List<Message>;
|
||||
|
||||
if (messages == null)
|
||||
{
|
||||
messages = new List<Message>();
|
||||
}
|
||||
|
||||
messages.Add(new Message
|
||||
{
|
||||
Type = type,
|
||||
Value = value
|
||||
});
|
||||
|
||||
TempData[MessagesKey] = messages;
|
||||
}
|
||||
}
|
||||
}
|
|
@ -42,7 +42,7 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
{
|
||||
[ValidateInput(false)]
|
||||
[LdapAuthorization]
|
||||
public class FileSystemController : Controller
|
||||
public class FileSystemController : BaseController
|
||||
{
|
||||
private readonly ICryptography _cryptography;
|
||||
private readonly IWebDavManager _webdavManager;
|
||||
|
@ -276,7 +276,7 @@ namespace WebsitePanel.WebDavPortal.Controllers
|
|||
|
||||
if (filePathes == null)
|
||||
{
|
||||
model.AddMessage(MessageType.Error, Resources.UI.NoFilesAreSelected);
|
||||
AddMessage(MessageType.Error, Resources.UI.NoFilesAreSelected);
|
||||
|
||||
return Json(model);
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue