Search code examples
spring-boothibernatespring-mvcannotationsspring-annotations

Non bean property throwing Bean property is not readable or has an invalid getter method


So my issue is a bit different from others. The error that it is throwing is not a field name but rather the input in a form. I've never encountered this error before

Error. The contents of '' is what I key for password

org.springframework.beans.NotReadablePropertyException: Invalid property 'Yijian@123' of bean class [com.Alex.UserPackage.User]: Bean property 'Yijian@123' is not readable or has an invalid getter method: Does the return type of the getter match the parameter type of the setter? at org.springframework.beans.AbstractNestablePropertyAccessor.getPropertyValue(AbstractNestablePropertyAccessor.java:622) ~[spring-beans-5.2.7.RELEASE.jar:5.2.7.RELEASE] at org.springframework.beans.AbstractNestablePropertyAccessor.getPropertyValue(AbstractNestablePropertyAccessor.java:612) ~[spring-beans-5.2.7.RELEASE.jar:5.2.7.RELEASE] at org.springframework.validation.AbstractPropertyBindingResult.getActualFieldValue(AbstractPropertyBindingResult.java:104) ~[spring-context-5.2.7.RELEASE.jar:5.2.7.RELEASE]

Entity class

@Entity
@ValidPassword
public class User {
    @Pattern(regexp="[a-zA-Z]+", message = "Enter letters only!")
    private String firstName;
    @Pattern(regexp="[a-zA-Z]+", message = "Enter letters only!")
    private String lastName;
    private String password;
    private String matchingPassword;
    private String passportNumber;


    

    public String getPassword() {
        return password;
    }

    public void setPassword(String password) {
        this.password = password;
    }

    public String getMatchingPassword() {
        return matchingPassword;
    }

    public void setMatchingPassword(String matchingPassword) {
        this.matchingPassword = matchingPassword;
    }


}

@ValidPassword custom annotation. The error started to occur after I used the getters for password and matchingPassword

private String message;

@Override
public boolean isValid(User user, ConstraintValidatorContext context) {
    String password = user.getPassword();
    String matchingPassword = user.getMatchingPassword();
    if (password== null || matchingPassword == null) {
    return false;
    }
    
    System.out.println("PASSWORDS: " + password + matchingPassword);
    
    boolean flag = Pattern.matches("^(?=.*?[A-Z])(?=.*?[a-z])(?=.*?[0-9])(?=.*?[#?!@$%^&*-]).{8,}$", password);
    boolean flag1 = password.equals(matchingPassword);
    
    if ( !flag1 ) {
        message = "Passwords do not match!";
    }
    
    
     context.disableDefaultConstraintViolation();
        context.buildConstraintViolationWithTemplate(message)
                .addPropertyNode(password).addConstraintViolation();
        
    return flag && flag1;
    
}

//Show default message if no special message is set
@Override
public void initialize(ValidPassword validPassword) {
    
     message = validPassword.message();
}

Form password portion

<div class = "row">
            <div class="col-sm-6 form-group">
                <label>Password : </label> <Input type="password"
                    th:field="*{password}" th:required="required" class="form-control" />
                    <p th:if="${#fields.hasErrors('password')}"
                    th:errors="*{password}" class="alert alert-danger"></p>
            </div>

Solution

  • You are passing the property value instead of the property name to addPropertyNode(password).

    Replace the following:

    context.buildConstraintViolationWithTemplate(message)
                    .addPropertyNode(password).addConstraintViolation();
    

    with:

    context.buildConstraintViolationWithTemplate(message)
                    .addPropertyNode("password").addConstraintViolation();