From 19a13e606779d25d3472e22d59bccc87caf0380c Mon Sep 17 00:00:00 2001 From: Jens Schauder Date: Mon, 24 Jul 2017 13:10:26 +0200 Subject: [PATCH] DATAJPA-1058 - Polishing. Added content for JavaDoc @return. Reduced visibility. Original pull request: #208. --- .../query/QueryParameterSetterFactory.java | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/main/java/org/springframework/data/jpa/repository/query/QueryParameterSetterFactory.java b/src/main/java/org/springframework/data/jpa/repository/query/QueryParameterSetterFactory.java index 4c1c17212..3712c972b 100644 --- a/src/main/java/org/springframework/data/jpa/repository/query/QueryParameterSetterFactory.java +++ b/src/main/java/org/springframework/data/jpa/repository/query/QueryParameterSetterFactory.java @@ -49,9 +49,9 @@ abstract class QueryParameterSetterFactory { * Creates a new {@link QueryParameterSetterFactory} for the given {@link JpaParameters}. * * @param parameters must not be {@literal null}. - * @return + * @return A basic {@link QueryParameterSetterFactory} that can handle named and index parameters. */ - public static QueryParameterSetterFactory basic(JpaParameters parameters) { + static QueryParameterSetterFactory basic(JpaParameters parameters) { Assert.notNull(parameters, "JpaParameters must not be null!"); @@ -64,10 +64,10 @@ abstract class QueryParameterSetterFactory { * * @param parameters must not be {@literal null}. * @param metadata must not be {@literal null}. - * @return + * @return A {@link QueryParameterSetterFactory} for criteria Queries. */ - public static QueryParameterSetterFactory forCriteriaQuery(JpaParameters parameters, - List> metadata) { + static QueryParameterSetterFactory forCriteriaQuery(JpaParameters parameters, + List> metadata) { Assert.notNull(parameters, "JpaParameters must not be null!"); Assert.notNull(metadata, "ParameterMetadata must not be null!"); @@ -82,10 +82,10 @@ abstract class QueryParameterSetterFactory { * @param parser must not be {@literal null}. * @param evaluationContextProvider must not be {@literal null}. * @param parameters must not be {@literal null}. - * @return + * @return A {@link QueryParameterSetterFactory} that can handle {@link org.springframework.expression.spel.standard.SpelExpression}s. */ - public static QueryParameterSetterFactory parsing(SpelExpressionParser parser, - EvaluationContextProvider evaluationContextProvider, Parameters parameters) { + static QueryParameterSetterFactory parsing(SpelExpressionParser parser, + EvaluationContextProvider evaluationContextProvider, Parameters parameters) { Assert.notNull(parser, "SpelExpressionParser must not be null!"); Assert.notNull(evaluationContextProvider, "EvaluationContextProvider must not be null!"); @@ -165,7 +165,7 @@ abstract class QueryParameterSetterFactory { * * @param expression must not be {@literal null}. * @param values must not be {@literal null}. - * @return + * @return the result of the evaluation. */ private Object evaluateExpression(Expression expression, Object[] values) { @@ -295,7 +295,7 @@ abstract class QueryParameterSetterFactory { * * @param parameter can be {@literal null}. * @param binding must not be {@literal null}. - * @return + * @return a {@link javax.persistence.Parameter} object based on the information from the arguments. */ static javax.persistence.Parameter of(JpaParameter parameter, ParameterBinding binding) {