incubator-tamaya git commit: TAMAYA-60 Wrote more JavaDoc.

2015-03-10 Thread plexus
Repository: incubator-tamaya
Updated Branches:
  refs/heads/master 5e598cb96 -> 39e623dac


TAMAYA-60 Wrote more JavaDoc.


Project: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/commit/39e623da
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/tree/39e623da
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/diff/39e623da

Branch: refs/heads/master
Commit: 39e623dacb0e6b9ac35d63709fa96839ab27c5e2
Parents: 5e598cb
Author: Oliver B. Fischer 
Authored: Wed Mar 11 06:36:10 2015 +0100
Committer: Oliver B. Fischer 
Committed: Wed Mar 11 06:36:10 2015 +0100

--
 .../modules/builder/ConfigurationBuilder.java   | 92 ++--
 1 file changed, 66 insertions(+), 26 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-tamaya/blob/39e623da/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
--
diff --git 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
index ce26aad..945eb2f 100644
--- 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
+++ 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
@@ -45,23 +45,6 @@ import java.util.stream.Stream;
 
 import static java.lang.String.format;
 
-/* TODO LIST FOR TAMAYA-60
- *
- * - configurable loading of provided PropertyConverter DONE
- * - configurable loading of provided PropertySources DONE
- * - configurable loading of provided PropertySourceProviders DONE
- * - configurable loading of provided PropertyFilters DONE
- * - I can not overhand null in varargs
- * - Rethink the default behaviour for SPI loading
- * - Work on all TODOs for TAMAYA-60
- * - Write JavaDoc
- * - adding sources via URL DONE
- *
- *
- *
- *
- */
-
 /**
  * Builder class used for building a configuration manually without relying
  * only on the Service Provider Interface API.
@@ -112,6 +95,11 @@ public class ConfigurationBuilder {
 
 private boolean isLoadProvidedPropertyFilters = false;
 
+/**
+ * Creates a new builder instance.
+ */
+public ConfigurationBuilder() {
+}
 
 /**
  * Allows to set configuration context during unit tests.
@@ -280,15 +268,31 @@ public class ConfigurationBuilder {
 }
 
 /**
+ * Adds one or more property filter instances to the configuration to be 
build.
+ *
+ * {@code PropertyFilter quoteReplacingFilter = new QuoteFilter();
+ * PropertyFilter commaRemovingFilter = new CommaFilter();
+ *
+ * builder.addPropertyFilters(commaRemovingFilter, quoteReplacingFilter)};
+ * 
+ *
+ * @param filters list of property filter instances which should be applied
+ *to the properties of the configuration to be build.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.spi.PropertyFilter
+ * @see #disableProvidedPropertyFilters()
+ * @see #enabledProvidedPropertyFilters()
  */
-public ConfigurationBuilder addPropertyFilters(PropertyFilter... 
propertyFilters){
-Objects.requireNonNull(propertyFilters);
+public ConfigurationBuilder addPropertyFilters(PropertyFilter... filters){
+Objects.requireNonNull(filters);
 
-contextBuilder.addPropertyFilters(propertyFilters);
+contextBuilder.addPropertyFilters(filters);
 return this;
 }
 
+
 /**
  * @return the builder instance currently used
  */
@@ -298,13 +302,29 @@ public class ConfigurationBuilder {
 }
 
 /**
+ * Adds a property converter for the a given type to the configuration to
+ * be build.
+ *
+ * {@code PropertyConverter converter = value -> new 
MyType(value, 42);
+ *
+ * builder.addPropertyConverter(MyType.class, converter}
+ * 
+ *
+ * @param type the required target type the converter should be applied to
+ * @param converter the converter to be used to convert the string property
+ *  to the given target type.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.PropertyConverter
+ * @see #enableProvidedPropertyConverters()
+ * @see #disableProvidedPropertyConverters()
  */
-public  ConfigurationBuilder addPropertyConverter(Class type, 
PropertyConverter propertyConverter) {
+public  ConfigurationBuilder addPropertyConverter(Class type, 
PropertyConverter converter) {
 Objects.requireNonNull(type);
-Objects.requireNonNull(propertyConverter);
+Objects.requireNonNull(converter);
 
-re

incubator-tamaya git commit: TAMAYA-60 JavaDoc..

2015-03-10 Thread plexus
Repository: incubator-tamaya
Updated Branches:
  refs/heads/master afdd6c8c0 -> 5e598cb96


TAMAYA-60 JavaDoc..


Project: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/commit/5e598cb9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/tree/5e598cb9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/diff/5e598cb9

Branch: refs/heads/master
Commit: 5e598cb9677dc9f1f31302512036ec4f678e3830
Parents: afdd6c8
Author: Oliver B. Fischer 
Authored: Tue Mar 10 22:57:55 2015 +0100
Committer: Oliver B. Fischer 
Committed: Tue Mar 10 22:57:55 2015 +0100

--
 .../modules/builder/ConfigurationBuilder.java   | 52 ++--
 1 file changed, 49 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-tamaya/blob/5e598cb9/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
--
diff --git 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
index 173117a..ce26aad 100644
--- 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
+++ 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
@@ -122,7 +122,23 @@ public class ConfigurationBuilder {
 }
 
 /**
+ * Adds one resources with properties in an arbitrary format
+ * to the configuration to be build.
+ *
+ * If a specific format is supported depends on the available
+ * {@link org.apache.tamaya.format.ConfigurationFormat} 
implementations.
+ *
+ * {@code URL resource = new URL("file:/etc/service/config.json");
+ *
+ * builder.addPropertySources(resource);}
+ * 
+ *
+ * @param url resource with properties for the the configuration to be 
build.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.format.ConfigurationFormat
+ * @see org.apache.tamaya.format.ConfigurationFormats#getFormats()
  */
 public ConfigurationBuilder addPropertySource(URL url) {
 try {
@@ -325,7 +341,7 @@ public class ConfigurationBuilder {
 }
 
 /**
- * Disables the loading of all {@link org.apache.tamaya.PropertyConverter}
+ * Disables the automatic loading of all {@link 
org.apache.tamaya.PropertyConverter}
  * service providers.
  *
  * @return the builder instance currently used
@@ -343,7 +359,13 @@ public class ConfigurationBuilder {
 
 
 /**
+ * Enables the automatic loading of all {@link 
org.apache.tamaya.spi.PropertySource}
+ * service providers.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.spi.PropertySource
+ * @see #disableProvidedPropertySources()
  */
 public ConfigurationBuilder enableProvidedPropertySources() {
 checkBuilderState();
@@ -353,11 +375,25 @@ public class ConfigurationBuilder {
 return this;
 }
 
+/**
+ * Checks if the automatic loading of all {@link 
org.apache.tamaya.spi.PropertySource
+ * PropertySource} service providers is enabled or disabled.
+ *
+ * @return {@code true} if the automatic loading is enabled,
+ * otherwise {@code false}.
+ */
 public boolean isPropertySourcesLoadingEnabled() {
 return loadProvidedPropertySources;
 }
 
 
+/**
+ * Checks if the automatic loading of all {@link 
org.apache.tamaya.spi.PropertyFilter
+ * PropertyFilter} service providers is enabled or disabled.
+ *
+ * @return {@code true} if the automatic loading is enabled,
+ * otherwise {@code false}.
+ */
 public boolean isPropertyFilterLoadingEnabled() {
 return isLoadProvidedPropertyFilters;
 }
@@ -375,7 +411,7 @@ public class ConfigurationBuilder {
 }
 
 /**
- * Disables the loading of all {@link org.apache.tamaya.spi.PropertyFilter}
+ * Disables the automatic loading of all {@link 
org.apache.tamaya.spi.PropertyFilter}
  * service providers.
  *
  * @return the builder instance currently used
@@ -394,7 +430,13 @@ public class ConfigurationBuilder {
 }
 
 /**
+ * Disables the automatic loading of all {@link 
org.apache.tamaya.spi.PropertySource}
+ * service providers.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.spi.PropertySource
+ * @see #enableProvidedPropertySources()
  */
 public ConfigurationBuilder disableProvidedPropertySources() {
 checkBuilderState();
@@ -405,9 +447,13 @@ public class ConfigurationBuilder {
 }
 
 /**
- *
+ * Enabl

[1/2] incubator-tamaya git commit: TAMAYA-60 Worked on the JavaDoc...

2015-03-10 Thread plexus
Repository: incubator-tamaya
Updated Branches:
  refs/heads/master 0d83a6fe8 -> afdd6c8c0


TAMAYA-60 Worked on the JavaDoc...


Project: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/commit/72e119e7
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/tree/72e119e7
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/diff/72e119e7

Branch: refs/heads/master
Commit: 72e119e72f1ee230c33517c5467050fd71a14604
Parents: 0d83a6f
Author: Oliver B. Fischer 
Authored: Mon Mar 9 07:53:57 2015 +0100
Committer: Oliver B. Fischer 
Committed: Mon Mar 9 07:53:57 2015 +0100

--
 .../modules/builder/ConfigurationBuilder.java   | 51 
 1 file changed, 51 insertions(+)
--


http://git-wip-us.apache.org/repos/asf/incubator-tamaya/blob/72e119e7/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
--
diff --git 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
index 01180d0..cd55a27 100644
--- 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
+++ 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
@@ -121,6 +121,9 @@ public class ConfigurationBuilder {
 return this;
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public ConfigurationBuilder addPropertySource(URL url) {
 try {
 ConfigurationData data = getConfigurationDataFromURL(url);
@@ -179,6 +182,9 @@ public class ConfigurationBuilder {
 return this;
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public ConfigurationBuilder addPropertySource(String url, String... urls) {
 Stream.of(Collections.singletonList(url), Arrays.asList(urls))
   .flatMap(Collection::stream)
@@ -190,6 +196,9 @@ public class ConfigurationBuilder {
 return this;
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public ConfigurationBuilder addPropertySources(PropertySource... sources){
 checkBuilderState();
 
@@ -203,11 +212,17 @@ public class ConfigurationBuilder {
 }
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public ConfigurationBuilder 
addPropertySourceProviders(PropertySourceProvider... propertySourceProviders){
 contextBuilder.addPropertySourceProviders(propertySourceProviders);
 return this;
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public ConfigurationBuilder addPropertyFilters(PropertyFilter... 
propertyFilters){
 Objects.requireNonNull(propertyFilters);
 
@@ -215,11 +230,17 @@ public class ConfigurationBuilder {
 return this;
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public ConfigurationBuilder 
setPropertyValueCombinationPolicy(PropertyValueCombinationPolicy 
propertyValueCombinationPolicy){
 
contextBuilder.setPropertyValueCombinationPolicy(propertyValueCombinationPolicy);
 return this;
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public  ConfigurationBuilder addPropertyConverter(Class type, 
PropertyConverter propertyConverter) {
 Objects.requireNonNull(type);
 Objects.requireNonNull(propertyConverter);
@@ -227,6 +248,9 @@ public class ConfigurationBuilder {
 return addPropertyConverter(TypeLiteral.of(type), propertyConverter);
 }
 
+/**
+ * @return the builder instance currently used
+ */
 public  ConfigurationBuilder addPropertyConverter(TypeLiteral type, 
PropertyConverter propertyConverter){
 Objects.requireNonNull(type);
 Objects.requireNonNull(propertyConverter);
@@ -244,6 +268,8 @@ public class ConfigurationBuilder {
  * Enables the loading of all {@link org.apache.tamaya.PropertyConverter}
  * service providers.
  *
+ * @return the builder instance currently used
+ *
  * @see org.apache.tamaya.PropertyConverter
  * @see #disableProvidedPropertyConverters()
  */
@@ -259,6 +285,8 @@ public class ConfigurationBuilder {
  * Disables the loading of all {@link org.apache.tamaya.PropertyConverter}
  * service providers.
  *
+ * @return the builder instance currently used
+ *
  * @see org.apache.tamaya.PropertyConverter
  * @see #enableProvidedPropertyConverters()
  */
@@ -271,6 +299,9 @@ public class ConfigurationBuilder {
 }
 
 
+/**
+ * @return the builder instance currently used
+ */
 pub

[2/2] incubator-tamaya git commit: TAMAYA-60 JavaDoc...

2015-03-10 Thread plexus
TAMAYA-60 JavaDoc...


Project: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/commit/afdd6c8c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/tree/afdd6c8c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-tamaya/diff/afdd6c8c

Branch: refs/heads/master
Commit: afdd6c8c06651e611a9364c4162f84509246a0ba
Parents: 72e119e
Author: Oliver B. Fischer 
Authored: Tue Mar 10 22:43:39 2015 +0100
Committer: Oliver B. Fischer 
Committed: Tue Mar 10 22:43:39 2015 +0100

--
 .../modules/builder/ConfigurationBuilder.java   | 63 ++--
 1 file changed, 57 insertions(+), 6 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/incubator-tamaya/blob/afdd6c8c/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
--
diff --git 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
index cd55a27..173117a 100644
--- 
a/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
+++ 
b/modules/builder/src/main/java/org/apache/tamaya/modules/builder/ConfigurationBuilder.java
@@ -157,16 +157,16 @@ public class ConfigurationBuilder {
  * If a specific format is supported depends on the available
  * {@link org.apache.tamaya.format.ConfigurationFormat} 
implementations.
  *
- *
- * URL first = new URL("file:/etc/service/config.json");
- * URL second = new URL(file:/etc/defaults/values.properties");
+ *{@code URL first = new URL("file:/etc/service/config.json");
+ * URL second = new URL("file:/etc/defaults/values.properties");
  *
- * builder.addPropertySources(first, second);
+ * builder.addPropertySources(first, second);}
  *
  *
  * @param url first resource with properties for the the configuration to 
be build.
  * @param urls list additional of resources with properties for the 
configuration to be
  * build.
+ *
  * @return the builder instance currently used
  *
  * @see org.apache.tamaya.format.ConfigurationFormat
@@ -182,8 +182,26 @@ public class ConfigurationBuilder {
 return this;
 }
 
+
 /**
+ * Adds one or more resources with properties in an arbitrary format
+ * to the configuration to be build.
+ *
+ * If a specific format is supported depends on the available
+ * {@link org.apache.tamaya.format.ConfigurationFormat} 
implementations.
+ *
+ *{@code builder.addPropertySources("file:/etc/service/config.json",
+ *"file:/etc/defaults/values.properties");}
+ *
+ *
+ * @param url first resource with properties for the the configuration to 
be build.
+ * @param urls list additional of resources with properties for the 
configuration to be
+ * build.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.format.ConfigurationFormat
+ * @see org.apache.tamaya.format.ConfigurationFormats#getFormats()
  */
 public ConfigurationBuilder addPropertySource(String url, String... urls) {
 Stream.of(Collections.singletonList(url), Arrays.asList(urls))
@@ -197,7 +215,20 @@ public class ConfigurationBuilder {
 }
 
 /**
+ * Adds one or more property source instances to the configuration to be 
build.
+ *
+ *{@code PropertySource first = new CustomPropertySource();
+ * PropertySource second = new YetAnotherPropertySource();
+ *
+ * builder.addPropertySources(first, second)};
+ *
+ *
+ * @param sources list of property source instances with properties for the
+ *configuration to be build.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.spi.PropertySource
  */
 public ConfigurationBuilder addPropertySources(PropertySource... sources){
 checkBuilderState();
@@ -213,10 +244,22 @@ public class ConfigurationBuilder {
 }
 
 /**
+ * Adds one or more property source provider instances to the 
configuration to be build.
+ *
+ * {@code PropertySourceProvider jc = new JavaConfigurationProvider();
+ *
+ * builder.addPropertySources(jc)};
+ * 
+ *
+ * @param providers list of property source provider instances each 
providing a set
+ *  of property source instances for the configuration to 
be build.
+ *
  * @return the builder instance currently used
+ *
+ * @see org.apache.tamaya.spi.PropertySourceProvider
  */
-public ConfigurationBuilder 
addPropertySourceProviders(PropertySou