-
Notifications
You must be signed in to change notification settings - Fork 683
Add RepositoryMetadataAccess
interface
#3145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Repositories that detect fragment implementations implementing RepositoryMetadataAccess enable metadata exposure regardless of the exposeMetadata flag.
RepositoryMetadataAccess
RepositoryMetadataAccess
interface
* @param implementation | ||
* @deprecated since 3.4 - use {@link ImplementedRepositoryFragment(Class, Object)} instead. | ||
*/ | ||
@Deprecated(since = "3.4") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add a forRemoval=true
so that we clean up the API with Spring Data 4.0
} | ||
return (Class<T>) implementation.getClass(); | ||
}); | ||
if(interfaceClass != null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Formatting
@@ -216,7 +228,7 @@ public RepositoryFragment<T> withImplementation(T implementation) { | |||
public String toString() { | |||
|
|||
return String.format("ImplementedRepositoryFragment %s%s", | |||
interfaceClass.map(ClassUtils::getShortName).map(it -> it + ":").orElse(""), | |||
interfaceClass != null ? (ClassUtils.getShortName(interfaceClass) + ";") : "", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: A slight change in :
vs. ;
Update documentation. Additional logging for repository bootstrap procedure. Limit usage of Optional in RepositoryFragment. Original Pull Request: #3145
Repositories that detect fragment implementations implementing RepositoryMetadataAccess enable metadata exposure regardless of the exposeMetadata flag. See: spring-projects#3090 Original Pull Request: spring-projects#3145
Update documentation. Additional logging for repository bootstrap procedure. Limit usage of Optional in RepositoryFragment. Original Pull Request: spring-projects#3145
Repositories that detect fragment implementations implementing
RepositoryMetadataAccess
enable metadata exposure regardless of theexposeMetadata
flag on the repository factory.