Issue Details (XML | Word | Printable)

Key: CASTOR-2184
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Ralf Joachim
Reporter: Werner Guttmann
Votes: 0
Watchers: 0
Operations

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

Rename all *Configuration classes (such as XMLConfiguration, CPAConfiguration, ... ) to *Properties

Created: 04/Oct/07 09:57 AM   Updated: 26/Jan/09 04:11 AM   Resolved: 25/Jan/09 06:58 PM
Return to search
Component/s: General
Affects Version/s: 1.1.2.1
Fix Version/s: 1.3

Time Tracking:
Not Specified

File Attachments: 1. Text File patch-C2184-20090126.txt (210 kB)



 Description  « Hide

This is to somehow make it clearer that those classes deal with handling of properties only.



Ralf Joachim added a comment - 25/Jan/09 06:57 PM

Final patch.


Werner Guttmann added a comment - 26/Jan/09 04:05 AM

Great, As I am about to commit one or two more patches that will introduce a new property, how about committing this as soon as possible (before we make our life harder than necessary) ?


Werner Guttmann added a comment - 26/Jan/09 04:06 AM

One more thing: this patch unfortunately changes public API, as there's people out there who manually set e.g. XML properties on the (Un-)Marshaller using e.g. XMLConfiguration.SOME_CONSTANT. As bad as this is, I think there's no way aorund this. Unless, of course, you want to just deprecate the *Configuration classes. I leave this up to you.


Werner Guttmann added a comment - 26/Jan/09 04:11 AM

Actually, too late anyhow as you have already committed this. Never mind ... .