castor

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

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.1.2.1
  • Fix Version/s: 1.3
  • Component/s: General
  • Labels:
    None
  • Number of attachments :
    1

Description

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

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ralf Joachim added a comment -

Final patch.

Show
Ralf Joachim added a comment - Final patch.
Hide
Permalink
Werner Guttmann added a comment -

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) ?

Show
Werner Guttmann added a comment - 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) ?
Hide
Permalink
Werner Guttmann added a comment -

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.

Show
Werner Guttmann added a comment - 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.
Hide
Permalink
Werner Guttmann added a comment -

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

Show
Werner Guttmann added a comment - Actually, too late anyhow as you have already committed this. Never mind ... .

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.