PATCH returns NPE when it should be a not found [DATAREST-529]

Luan Nguyen opened DATAREST-529 and commented

Line 156 of PersistentEntityResourceHandlerMethodArgumentResolver.java possibly missed a "throw" statement. I.e. it should be "throw new ResourceNotFoundException();" instead of just "new ResourceNotFoundException();"

Test case:

@Entity
public class Person {
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    private Long id;
    private String name;

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }
}

public interface PersonRepository extends PagingAndSortingRepository<Person, Long> {
}

curl -H "Content-Type: application/json" -X PATCH -d '{"name": "John Doe"}' localhost:8080/persons/1000
{"timestamp":1430770675151,"status":500,"error":"Internal Server Error","exception":"java.lang.NullPointerException","message":"No message available"}

No further details from DATAREST-529