Rafal Krzewski wrote:
> 
> [EMAIL PROTECTED] wrote:
> > AFAIK, Configurations is not repository independent so could be easily
> > used with XML files and maybe even a naming system.
> > In fact, since Magnus patch has been incorporated in Configurations,
> > the GenericResources class is exactly the same as Configurations
> > exept for the constructors...
> 
> You mean it *is* repository independent? Hmmm... I need read more
> about this class then. 

Yes, that's what I meant. 
Configurations is repository independent and uses a ConfigurationRepository
to access the properties.
ConfigurationRepository is syntax independent but define load() and save()
so is mostly useful for file-based serialization format.
ExtendedProperties is a subclass ConfigurationRepository which is
syntax dependent.

> I'm also a bit confused about GenericResources
> class. What is it's role in your implementation?
>

It was there, so I didn't remove it :) 
I guess the GenericResources was supposed to be a wrapper around 
configurations to fix the getKeys() method.
If you're going to really clean-up the resources service I think :
- GenericResources can be removed and all it's code moved in 
  TurbineResourceService (which is currently itself a wrapper aroung
  GenericResources).
- I'm not sure of the status of MessageResources and ErrorResources
  which aren't used in the code base except in TurbineResourceService.
- TurbineResourceService should be renamed to TurbineResourcesService.
                                                             ^

> > I'm +1 for the getResources() but I won't be able to work on it before at
> > least 1 week because we're readying a new release for Jetspeed so I'll
> > be mostly doing bug chasing and documentation for the next few days... :/
> 
> Sounds like fun... ;-). That's OK. It turned out that I'll have a few
> hours
> to work on it today. I'll let you know about the results.
> 

OK.

--
Rapha�l Luta - [EMAIL PROTECTED]
Vivendi Universal Networks - Services Manager / Paris


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Search: <http://www.mail-archive.com/turbine%40list.working-dogs.com/>
Problems?:           [EMAIL PROTECTED]

Reply via email to