You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by kw...@apache.org on 2019/05/10 11:36:25 UTC

[sling-org-apache-sling-caconfig-api] 26/31: [SLING-8272] Fixed PR Comments

This is an automated email from the ASF dual-hosted git repository.

kwin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-caconfig-api.git

commit 57117b05c362b8911b0250420d978002976e6d5b
Author: ompandey <om...@gmail.com>
AuthorDate: Thu Apr 18 12:13:55 2019 +0200

    [SLING-8272] Fixed PR Comments
---
 .../java/org/apache/sling/caconfig/ConfigurationBuilder.java  | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/main/java/org/apache/sling/caconfig/ConfigurationBuilder.java b/src/main/java/org/apache/sling/caconfig/ConfigurationBuilder.java
index 8f7035f..9cb344e 100644
--- a/src/main/java/org/apache/sling/caconfig/ConfigurationBuilder.java
+++ b/src/main/java/org/apache/sling/caconfig/ConfigurationBuilder.java
@@ -87,14 +87,13 @@ public interface ConfigurationBuilder {
     @NotNull <T> Collection<T> asAdaptableCollection(@NotNull Class<T> clazz);
 
     /**
-     * Check if the configuration node is existing in the JCR. This method checks for the configuration existence based on {@code configName} defined in the
-     * configuration definition. It checks if the configuration is available for the context path in the current resource hierarchy and if not found, also
-     * checks in the global fall-back paths configured via {@link org.apache.sling.caconfig.resource.impl.def.DefaultConfigurationResourceResolvingStrategy}
+     * Check if the configuration node is existing in the resource hierarchy. This method checks for the configuration existence based on {@code configName} 
+     * defined in the configuration definition. It checks if the configuration is available for the context path in the current resource hierarchy and if not 
+     * found, also checks in the global fall-back paths configured via {@link org.apache.sling.caconfig.resource.impl.def.DefaultConfigurationResourceResolvingStrategy}
      * which by default are {@code /conf/global}, {@code /apps/config} and {@code /libs/config}.This method does not consider the default values provided in
      * the configuration definition.
-     * @param clazz Class that can be adapted from a {@link org.apache.sling.api.resource.Resource}
-     * @param <T> Annotation class type
+     * @param configName Name of the configuration
      * @return True/False based on configuration resource node existence in JCR.
      */
-    @NotNull <T> boolean has(@NotNull Class<T> clazz);
+    boolean has(@NotNull String configName);
 }