Address Checkstyle

Issue gh-17447

Signed-off-by: Josh Cummings <3627351+jzheaux@users.noreply.github.com>
This commit is contained in:
Josh Cummings 2025-07-31 11:09:04 -06:00
parent 93cb01612b
commit 6f1232ce79
3 changed files with 25 additions and 8 deletions

View File

@ -59,8 +59,8 @@ import org.springframework.util.PropertyPlaceholderHelper;
* {@code @HasRole} annotation found on a given {@link AnnotatedElement}. * {@code @HasRole} annotation found on a given {@link AnnotatedElement}.
* *
* <p> * <p>
* Meta-annotations that use enum values can use {@link ExpressionTemplateValueProvider} to * Meta-annotations that use enum values can use {@link ExpressionTemplateValueProvider}
* provide custom placeholder values. * to provide custom placeholder values.
* *
* <p> * <p>
* Since the process of synthesis is expensive, it is recommended to cache the synthesized * Since the process of synthesis is expensive, it is recommended to cache the synthesized
@ -176,8 +176,8 @@ final class ExpressionTemplateSecurityAnnotationScanner<A extends Annotation>
} }
@Override @Override
public Object convert(Object source, TypeDescriptor sourceType, TypeDescriptor targetType) { public @Nullable Object convert(@Nullable Object source, TypeDescriptor sourceType, TypeDescriptor targetType) {
return (source != null) ? ((ExpressionTemplateValueProvider)source).getExpressionTemplateValue() : null; return (source != null) ? ((ExpressionTemplateValueProvider) source).getExpressionTemplateValue() : null;
} }
} }

View File

@ -1,3 +1,19 @@
/*
* Copyright 2004-present the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* https://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.springframework.security.core.annotation; package org.springframework.security.core.annotation;
/** /**
@ -20,14 +36,13 @@ package org.springframework.security.core.annotation;
* } * }
* </pre> * </pre>
* *
* @since 6.5
* @author Mike Heath * @author Mike Heath
* @since 7.0
*/ */
public interface ExpressionTemplateValueProvider { public interface ExpressionTemplateValueProvider {
/** /**
* Returns the value to be used in an expression template. * Returns the value to be used in an expression template.
*
* @return the value to be used in an expression template * @return the value to be used in an expression template
*/ */
String getExpressionTemplateValue(); String getExpressionTemplateValue();

View File

@ -62,8 +62,8 @@ public class ExpressionTemplateSecurityAnnotationScannerTests {
} }
enum Permission implements ExpressionTemplateValueProvider { enum Permission implements ExpressionTemplateValueProvider {
READ,
WRITE; READ, WRITE;
@Override @Override
public String getExpressionTemplateValue() { public String getExpressionTemplateValue() {
@ -72,6 +72,7 @@ public class ExpressionTemplateSecurityAnnotationScannerTests {
case WRITE -> "'user.WRITE'"; case WRITE -> "'user.WRITE'";
}; };
} }
} }
@Documented @Documented
@ -89,6 +90,7 @@ public class ExpressionTemplateSecurityAnnotationScannerTests {
@Target({ ElementType.TYPE, ElementType.METHOD }) @Target({ ElementType.TYPE, ElementType.METHOD })
@HasAnyCustomPermissions(permissions = { Permission.READ, Permission.WRITE }) @HasAnyCustomPermissions(permissions = { Permission.READ, Permission.WRITE })
@interface HasAllCustomPermissions { @interface HasAllCustomPermissions {
} }
@Documented @Documented