Grzegorz Czuba wrote:

> Simpy stated, TurbineResources.getString(key) not work because it found
> Vector object in collection.
> The identical code is found in  InitContextsAction.java.
> There are two solution.
> 1. Modify both places to check types of object in Turbine resources
> collection ( I think it is not a way for a future )
> 2. Change TurbineResources.getString to perform a conversion to a string

I'm -0 on this change. This would solve your problem, but extended properties
have their semantics (i.e.commas mean multiple values). I think that you should
not bend them. If your property value has a stray comma, escape it, when it has
an inherent form of a list (which is the case here), write the code to expect
a list. I know that you send the list further as a string, but you should word
it in the place you use it.

I need to hear other developers' opinions before I apply your change.

Rafal

PS. Please use 'cvs diff -u' to create patches.


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