From 5269a7d175935d03c86f0f39a48df7d3fb58457f Mon Sep 17 00:00:00 2001 From: Oliver Gierke Date: Mon, 12 Dec 2016 16:50:38 +0100 Subject: [PATCH] DATAREST-793 - RepositoryRestConfiguration's setters return this consistently. Adapted the return type of setter methods in RepositoryRestConfiguration to consistently return the current instance to allow call chaining. --- .../core/config/RepositoryRestConfiguration.java | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/spring-data-rest-core/src/main/java/org/springframework/data/rest/core/config/RepositoryRestConfiguration.java b/spring-data-rest-core/src/main/java/org/springframework/data/rest/core/config/RepositoryRestConfiguration.java index 1dec131c4..4d8d7de1a 100644 --- a/spring-data-rest-core/src/main/java/org/springframework/data/rest/core/config/RepositoryRestConfiguration.java +++ b/spring-data-rest-core/src/main/java/org/springframework/data/rest/core/config/RepositoryRestConfiguration.java @@ -112,13 +112,15 @@ public class RepositoryRestConfiguration { * * @param basePath the basePath to set */ - public void setBasePath(String basePath) { + public RepositoryRestConfiguration setBasePath(String basePath) { Assert.isTrue(!basePath.startsWith("http"), "Use a path not a URI"); basePath = StringUtils.trimTrailingCharacter(basePath, '/'); this.basePath = URI.create(basePath.startsWith("/") ? basePath : "/".concat(basePath)); Assert.isTrue(!this.basePath.isAbsolute(), "Absolute URIs are not supported as base path!"); + + return this; } /** @@ -504,8 +506,11 @@ public class RepositoryRestConfiguration { * @param enableEnumTranslation * @see #getEnumTranslationConfiguration() */ - public void setEnableEnumTranslation(boolean enableEnumTranslation) { + public RepositoryRestConfiguration setEnableEnumTranslation(boolean enableEnumTranslation) { + this.enableEnumTranslation = enableEnumTranslation; + + return this; } /** @@ -547,9 +552,13 @@ public class RepositoryRestConfiguration { * @param repositoryDetectionStrategy can be {@literal null}. * @since 2.5 */ - public void setRepositoryDetectionStrategy(RepositoryDetectionStrategy repositoryDetectionStrategy) { + public RepositoryRestConfiguration setRepositoryDetectionStrategy( + RepositoryDetectionStrategy repositoryDetectionStrategy) { + this.repositoryDetectionStrategy = repositoryDetectionStrategy == null ? RepositoryDetectionStrategies.DEFAULT : repositoryDetectionStrategy; + + return this; } /**