[ 
https://issues.apache.org/jira/browse/CONFIGURATION-848?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17862419#comment-17862419
 ] 

Gary D. Gregory edited comment on CONFIGURATION-848 at 7/2/24 12:41 PM:
------------------------------------------------------------------------

Hello [~laszlohujber] 

Why is your version ignoring the {{delimiter}} in:

{code:java}
@Override
protected Iterator<String> getKeysInternal(final String key, final String 
delimiter) {
   return getKeysInternal(key);
} 
{code}
?
Please see git master.
 


was (Author: garydgregory):
Hello [~laszlohujber] 

Why is your version ignoring the {{delimiter}} in:

{code:java}
@Override
protected Iterator<String> getKeysInternal(final String key, final String 
delimiter) {
   return getKeysInternal(key);
} 
{code}
?
 

> SubsetConfiguration does not account for delimiters as it did in 2.9.0
> ----------------------------------------------------------------------
>
>                 Key: CONFIGURATION-848
>                 URL: https://issues.apache.org/jira/browse/CONFIGURATION-848
>             Project: Commons Configuration
>          Issue Type: Bug
>    Affects Versions: 2.10.0, 2.10.1, 2.11.0
>            Reporter: Laszlo Hujber
>            Assignee: Gary D. Gregory
>            Priority: Major
>
> A bug occurs when using {{SubsetConfiguration}} with concrete subclasses of 
> {{{}AbstractHierarchicalConfiguration{}}}. This issue likely happens because 
> {{AbstractHierarchicalConfiguration}} does not override:
> {code:java}
> protected Iterator<String> getKeysInternal(String prefix, String delimiter) {
> }
> {code}
> The relevant pull request that introduced this bug is 
> [#300|https://github.com/apache/commons-configuration/pull/300].
> Refer to the test case that passes in version 2.9.
> {code:java}
>     @Test
>     public void testSubsetWithJSONConfiguration() throws 
> FileNotFoundException, ConfigurationException {
>         final JSONConfiguration jsonConfiguration = new JSONConfiguration();
>         jsonConfiguration.read(new 
> FileReader(ConfigurationAssert.getTestFile("test.json").getAbsolutePath()));
>         final SubsetConfiguration subset = new 
> SubsetConfiguration(jsonConfiguration, "capitals(0)", ".");
>         assertFalse(subset.isEmpty());
>         assertEquals(2, subset.size());
>         assertEquals("USA", subset.getString("country"));
>     }
> {code}
> Actual Behavior:
>  * {{isEmpty()}} returns {{{}true{}}}.
>  * {{size()}} returns {{{}0{}}}.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to