How To Disable or Change Attribute According to CRUD Operation Insert Or Update

Question

I'm trying to build a small website using ASP.NET MVC With EntityFramework on Visual Studio 2013.4

I have this model:

    public Class User
    {
Public int UserID {get;set;}
        [Required]
        [ValidUserName]

        public string UserName { get; set; }

        [Required]
        public string UserPassword { get; set; }

    }

This model has a Custom Attribute which is called [ValidUserName].

public class ValidUserName : ValidationAttribute
{
    PMAIDEntities db;
    public ValidUserName()
    {
        db = new PMAIDEntities();
    }
    protected override ValidationResult IsValid(object value, ValidationContext validationContext)
    {
        if (value != null)
        {

            var valueAsString = value.ToString();
            bool IsExist = db.Users.Where(x => x.UserName == (string)value).Any();
            if (IsExist)
            {
                var errorMessage = FormatErrorMessage(validationContext.DisplayName);
                return new ValidationResult("User Name is Already Exist !!");
            }                   
        }
        return ValidationResult.Success;
    }
}

The problem is when I try to add a new record to the database the [ValidUserName] works fine, which informs the end-user if the username exists in the database or not. But, when the update operation occur on a record to change password the [ValidUserName] gives me that the username exists in the database. enter image description here

I know that the username already exists, but how can I solve this problem to let [ValidUserName] accept current username because the username shall remain as is?

This the the code in the controller:

[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Edit([Bind(Include = "UserID,UserName,UserPassword,UserRole,UserFullName,UserEmail,UserMobile,UserLastLogin,UserAddress,UserBio,UserNationality,UserGender,UserDOB,UserAge,UserDateAdded")] User user)
{
    if (ModelState.IsValid)
    {
        db.Entry(user).State = EntityState.Modified;
        db.SaveChanges();
        return RedirectToAction("Index");
    }
    ViewBag.UserNationality = new SelectList(db.Nationalities, "Id", "Name", user.UserNationality);
    return View(user);
}

edit

i use this code in model

    public class User
    { 

        public int UserID { get; set; }

    [Required]
    [Remote("IsUserNameValid", "Users", AdditionalFields = "UserID", ErrorMessage = "Username already exists.")]
        [ValidUserName]
        public string UserName { get; set; }

    [Required]  
        public string UserPassword { get; set; }
}

and in the controller i add the following code

public JsonResult IsUsernameValid(string UserName, int UserID)
{
    var isValid = true;

    if (UserName != null)
    {
        isValid = !db.Users.Where(x => x.UserName == UserName && x.UserID != UserID).Any();
    }

    return Json(isValid, JsonRequestBehavior.AllowGet);
}

the problem here is the remote attribute require to specify the controller name , in this case what should i write if is my controller located inside an area called 'Admin'

Can you post your model completely? What type is the UserID? — ataravati, May 23 '15 at 21:36

ataravati · Accepted Answer · 2015-05-23T23:11:30.510

1

You need to use Remote Validation (RemoteAttribute). Add this action method to your controller (I'm assuming your UserID is integer):

public JsonResult IsUserNameValid(string UserName, int UserID = 0)
{
    var isValid = true; 

    if (UserName != null)
    {
        isValid = !db.Users.Where(x => x.UserName == UserName && x.UserID != UserID).Any();                         
    }

    return Json(isValid, JsonRequestBehavior.AllowGet);
}

Then, use it in your model like this (updated for calling the controller in the Admin area):

public class User
{
    [Required]
    [Remote("IsUserNameValid", "Users", "Admin", 
       AdditionalFields = "UserID", ErrorMessage = "Username already exists.")]
    public string UserName { get; set; }

    [Required]
    public string UserPassword { get; set; }

    // other properties...
}

You can read more about implementing Remote Validation in asp.net MVC here.

edited May 23 '15 at 23:11

answered May 23 '15 at 21:43

ataravati

8,891
9
57
89

is there any reference should i added to solution , because [Remote] Attribute not compiling well . – Loai May 23 '15 at 22:02
What version of asp.net mvc are you using? – ataravati May 23 '15 at 22:35
thanks , you gave me the way to do it , but now i have an an issue here could you please look at the edited post . – Loai May 23 '15 at 22:43
and I've another question please , should I remove my previous Attribute [ValidUserName ] or keep it in the code ? – Loai May 23 '15 at 22:50
@Loai, I sent you the link to the RemoteAttribute page on the MSDN website. You should be able to figure that out yourself by looking at the documentation. Here's the constructor you need to use: https://msdn.microsoft.com/en-us/library/gg453821(v=vs.118).aspx – ataravati May 23 '15 at 23:10
Look at my updated answer. And, by the way, of course remove the ValidUserName attribute. You're not using it anymore, right? So, you should remove it. – ataravati May 23 '15 at 23:11
could you please see this question and help me on it . http://stackoverflow.com/questions/30423072/mvc-remote-attribute-is-not-working ataravati @ataravati – Loai May 25 '15 at 17:06

How To Disable or Change Attribute According to CRUD Operation Insert Or Update

1 Answers1