`SimpleJpaRepository.delete(Specification<T> spec)` throws `NullPointerException`

[xsg22]

Hi,
org.springframework.data.jpa.repository.support.SimpleJpaRepository#delete(Specification) throw s an NullPointerException when I execute in the kotlin language environment. More error messages like this

java.lang.NullPointerException: Parameter specified as non-null is null: method cn.sail.patient.controller.mutaion.ExerciseMutation.deleteExerciseRecord$lambda$0, parameter <anonymous parameter 1>

@Override
public long delete(Specification<T> spec) {

    CriteriaBuilder builder = this.em.getCriteriaBuilder();
    CriteriaDelete<T> delete = builder.createCriteriaDelete(getDomainClass());

    Predicate predicate = spec.toPredicate(delete.from(getDomainClass()), null, builder);
    
    if (predicate != null) {
	    delete.where(predicate);
    }
    
    return this.em.createQuery(delete).executeUpdate();
}

when executing "spec.toPredicate(delete.from(getDomainClass()), null, builder)", the second parameter passed in is null. However, the comment on the "Specification.toPredicate(Root root, CriteriaQuery query, CriteriaBuilder criteriaBuilder)" method clearly states that the ’query‘ parameter must not be null. If 'query' is null, the above exception will appear under kotlin because “CriteriaQuery query” is not a nullable type.

/**
* Creates a WHERE clause for a query of the referenced entity in form of a {@link Predicate} for the given
* {@link Root} and {@link CriteriaQuery}.
*
* @param root must not be {@literal null}.
* @param query must not be {@literal null}.
* @param criteriaBuilder must not be {@literal null}.
* @return a {@link Predicate}, may be {@literal null}.
*/
@Nullable
Predicate toPredicate(Root<T> root, CriteriaQuery<?> query, CriteriaBuilder criteriaBuilder);

Thanks for any ideas for smooth workarounds or other input,

Leon

[mp911de]

We have a problem in that area indeed because the query parameter is declared as non-null and yet we call the specification with null.

[mp911de]

In its actual sense, this is not a bug because the documentation says that the specification is expected to be non-null and the argument is not defined as @Nullable. However, several specifications can be null, so we should leniently accept null specifications.

[zteaterml]

@mp911de thank you for patching this issue. When do you anticipate a new version will be cut with this fix?