[CONFIGURATION-653] ConfigurationUtils.dump and other methods should use ImmutableConfiguration

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.1, 2.1.1
• Fix Version/s: 2.2
• Assignee: Unassigned
• Reporter: Vincent Maurin
• Created: 02/Mar/17 11:00
• Updated: 12/Oct/17 20:23
• Resolved: 07/Mar/17 21:07

Description

Since the read-only ImmutableConfiguration/ImmutableHierarchicalConfiguration has been introduced, several methods that uses Configuration in a read-only fashion should be adapted to use these ones instead of the read/write alternative.

For example :
ConfigurationUtils.dump
ConfigurationUtils.toString
ConfigurationUtils.copy (for the source)
ConfigurationUtils.append (for the source)
ConfigurationUtils.convertToHierarchical
ConfigurationConverter.getProperties
... and maybe others

It should not break any existing code as Configuration implements ImmutableConfiguration and it could enable code that manipulates read only configurations to use these utilities