Issue Details (XML | Word | Printable)

Key: CONFIGURATION-349
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Oliver Heger
Reporter: Ralph Goers
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Commons Configuration

XMLConfigurationProvider cannot create a class that extends XMLConfiguration

Created: 10/Nov/08 08:27 AM   Updated: 22/Aug/09 07:36 PM
Return to search
Component/s: None
Affects Version/s: 1.5
Fix Version/s: 1.6

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works extended.txt 2008-11-10 08:29 AM Ralph Goers 4 kB

Resolution Date: 11/Nov/08 09:25 PM


 Description  « Hide
XMLConfigurationProvider in DefaultConfigurationBuilder has default scope. When configuring a new ConfigurationProvider to use a class that extends XMLConfiguration an Exception is raised because the setConfigurationClass method cannot be called from the bean utility classes.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Ralph Goers added a comment - 10/Nov/08 08:29 AM
The attached patch provides a unit test for the problem as well as the fix.

Oliver Heger added a comment - 11/Nov/08 09:25 PM
Patch applied in revision 713166. Many thanks.

Note: I was unable to port this patch to the configuration2 branch because the new type converters cannot deal with conversions to Class objects. We will have to port this later if necessary.


Ralph Goers added a comment - 11/Nov/08 10:08 PM
Thanks. I will take a look at what to do on the configuration2 branch.

Oliver Heger added a comment - 12/Nov/08 08:54 PM
In this branch the dependency to commons-beanutils was dropped and replaced by an own converter framework (in the converter package). Obviously these converters are not yet as powerful as the ones of beanutils were. In this special case a converter for Class objects is missing.

But maybe we should create a new ticket for this issue.