Ok for me. I'll work on this tonight and send a patch ASAP. (How do I
differenciate Jackrabbit's patches to Core's one in our naming scheme)?
Nicolas
On 7/13/06, Jukka Zitting <[EMAIL PROTECTED]> wrote:
Hi,
On 7/12/06, Nicolas <[EMAIL PROTECTED]> wrote:
> I need to modify the code of the ConfigurationParser class to handle my
XML
> config file. Here is what I can do:
>
> - add a parseBackupConfig() method?
> - generify the ConfigurationParser class and extend it for backup and
> repository
> - create a new class in o.a.j.backup (with a lot of cut and paste).
I'd go for option 2 with the following inheritance structure:
* o.a.j.core.config.ConfigurationParser
* o.a.j.core.config.RepositoryConfigurationParser
* o.a.j.backup.BackupConfigurationParser
That should allow for best reuse of the existing BeanConfig
configuration mechanism while still keeping the backup stuff outside
core for now.
BR,
Jukka Zitting
--
Yukatan - http://yukatan.fi/ - [EMAIL PROTECTED]
Software craftsmanship, JCR consulting, and Java development
--
a+
Nico
my blog! http://www.deviant-abstraction.net !!
