Clarify javadoc

The Javadoc wrongly referred to a behaviour that is not (and should not
be) implemented. List items are split into indexed keys and the
comma-separated value is not retained.

Issues: SPR-13257
master
Stephane Nicoll 9 years ago
parent 20c4f82d6b
commit b7c2881a4f
  1. 6
      spring-beans/src/main/java/org/springframework/beans/factory/config/YamlPropertiesFactoryBean.java
  2. 4
      spring-beans/src/test/java/org/springframework/beans/factory/config/YamlPropertiesFactoryBeanTests.java

@ -1,5 +1,5 @@
/*
* Copyright 2002-2014 the original author or authors.
* Copyright 2002-2015 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.
@ -48,8 +48,7 @@ import org.springframework.beans.factory.InitializingBean;
* environments.prod.name=My Cool App
* </pre>
*
* Lists are represented as comma-separated values (useful for simple String
* values) and also as property keys with <code>[]</code> dereferencers, for
* Lists are split as property keys with <code>[]</code> dereferencers, for
* example this YAML:
*
* <pre class="code">
@ -61,7 +60,6 @@ import org.springframework.beans.factory.InitializingBean;
* becomes Java Properties like this:
*
* <pre class="code">
* servers=dev.bar.com,foo.bar.com
* servers[0]=dev.bar.com
* servers[1]=foo.bar.com
* </pre>

@ -1,5 +1,5 @@
/*
* Copyright 2002-2014 the original author or authors.
* Copyright 2002-2015 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.
@ -215,6 +215,7 @@ public class YamlPropertiesFactoryBeanTests {
Properties properties = factory.getObject();
assertThat(properties.getProperty("foo[0]"), equalTo("bar"));
assertThat(properties.getProperty("foo[1]"), equalTo("baz"));
assertThat(properties.get("foo"), is(nullValue()));
}
@Test
@ -229,6 +230,7 @@ public class YamlPropertiesFactoryBeanTests {
assertThat(properties.getProperty("foo[1]"), equalTo("baz"));
assertThat(properties.getProperty("foo[2].one"), equalTo("two"));
assertThat(properties.getProperty("foo[2].three"), equalTo("four"));
assertThat(properties.get("foo"), is(nullValue()));
}
@SuppressWarnings("unchecked")

Loading…
Cancel
Save