fix warning for rule java:S3398
diff --git a/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java b/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java
index 4dd6ef1..e63bda9 100644
--- a/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java
+++ b/src/main/java/org/apache/sling/clam/oak/internal/NodeObservingJcrPropertyDigger.java
@@ -157,24 +157,6 @@
threadPool.submit(digTask);
}
- private Set<String> filter(final String path, final Set<String> names, final Pattern pattern) {
- final Set<String> paths = new HashSet<>();
- for (final String name : names) {
- final String p = String.format("%s/%s", path, name);
- if (pattern.matcher(p).matches()) {
- if (logger.isDebugEnabled()) {
- logger.debug("path '{}' matches pattern '{}'", p, pattern.pattern());
- }
- paths.add(p);
- } else {
- if (logger.isDebugEnabled()) {
- logger.debug("path '{}' doesn't match pattern '{}'", p, pattern.pattern());
- }
- }
- }
- return paths;
- }
-
private Set<String> concat(final Set<String> a, final Set<String> b) {
final Set<String> set = new HashSet<>(a);
set.addAll(b);
@@ -204,6 +186,24 @@
this.userId = userId;
}
+ private Set<String> filter(final String path, final Set<String> names, final Pattern pattern) {
+ final Set<String> paths = new HashSet<>();
+ for (final String name : names) {
+ final String p = String.format("%s/%s", path, name);
+ if (pattern.matcher(p).matches()) {
+ if (logger.isDebugEnabled()) {
+ logger.debug("path '{}' matches pattern '{}'", p, pattern.pattern());
+ }
+ paths.add(p);
+ } else {
+ if (logger.isDebugEnabled()) {
+ logger.debug("path '{}' doesn't match pattern '{}'", p, pattern.pattern());
+ }
+ }
+ }
+ return paths;
+ }
+
@Override
@SuppressWarnings({"java:S3776", "checkstyle:IllegalCatch", "checkstyle:NestedIfDepth"})
public void run() {