Re: [Lift] Re: ConversionRules

2010-01-19 Thread Naftoli Gugenheim
Hi Marius and Tim (and anyone who has an opinion on this). Sorry for not responding earlier. To clarify, there's no question that parseXXX will continue to return a box. The call to the DateFormat's parse method is wrapped in tryo in the default implementation. The real question is if formatXXX

Re: [Lift] Re: ConversionRules

2010-01-15 Thread Timothy Perrett
+1 on the Box... that is the lift idiom. Cheers, Tim On 15 Jan 2010, at 07:18, Marius wrote: I'd strongly suggest: 1. All should return Box. In case of parsing failure for example return a Failure. 2. If a param is null, inside your method use (Box !! param) which would give you an Empty

[Lift] Re: ConversionRules

2010-01-14 Thread Marius
I'd strongly suggest: 1. All should return Box. In case of parsing failure for example return a Failure. 2. If a param is null, inside your method use (Box !! param) which would give you an Empty if param is null. We should not assume dumb default values for input that doesn't make sense. Br's,