From 2fbf709459dd51fd7467ccfe2577c0c91faf41bd Mon Sep 17 00:00:00 2001 From: John Blum Date: Thu, 24 Sep 2020 13:40:23 -0700 Subject: [PATCH] DATACMNS-1804 - Correct grammar in Exception messages thrown by Repository classes. Original pull request: #467. --- .../core/support/QueryExecutorMethodInterceptor.java | 7 ++++--- .../repository/core/support/RepositoryFactorySupport.java | 3 ++- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/springframework/data/repository/core/support/QueryExecutorMethodInterceptor.java b/src/main/java/org/springframework/data/repository/core/support/QueryExecutorMethodInterceptor.java index d58b9cb13..5300e2ffd 100644 --- a/src/main/java/org/springframework/data/repository/core/support/QueryExecutorMethodInterceptor.java +++ b/src/main/java/org/springframework/data/repository/core/support/QueryExecutorMethodInterceptor.java @@ -45,6 +45,7 @@ import org.springframework.util.ConcurrentReferenceHashMap; * @author Oliver Gierke * @author Mark Paluch * @author Christoph Strobl + * @author John Blum */ class QueryExecutorMethodInterceptor implements MethodInterceptor { @@ -75,9 +76,9 @@ class QueryExecutorMethodInterceptor implements MethodInterceptor { if (!queryLookupStrategy.isPresent() && repositoryInformation.hasQueryMethods()) { - throw new IllegalStateException("You have defined query method in the repository but " - + "you don't have any query lookup strategy defined. The " - + "infrastructure apparently does not support query methods!"); + throw new IllegalStateException("You have defined query methods in the repository" + + " but do not have any query lookup strategy defined." + + " The infrastructure apparently does not support query methods!"); } this.queries = queryLookupStrategy // diff --git a/src/main/java/org/springframework/data/repository/core/support/RepositoryFactorySupport.java b/src/main/java/org/springframework/data/repository/core/support/RepositoryFactorySupport.java index 86ebeb49a..3536181c1 100644 --- a/src/main/java/org/springframework/data/repository/core/support/RepositoryFactorySupport.java +++ b/src/main/java/org/springframework/data/repository/core/support/RepositoryFactorySupport.java @@ -73,6 +73,7 @@ import org.springframework.util.ObjectUtils; * @author Mark Paluch * @author Christoph Strobl * @author Jens Schauder + * @author John Blum */ public abstract class RepositoryFactorySupport implements BeanClassLoaderAware, BeanFactoryAware { @@ -449,7 +450,7 @@ public abstract class RepositoryFactorySupport implements BeanClassLoaderAware, if (composition.isEmpty()) { throw new IllegalArgumentException( - String.format("You have custom methods in %s but not provided a custom implementation!", + String.format("You have custom methods in %s but have not provided a custom implementation!", repositoryInformation.getRepositoryInterface())); }