On 27/11/11 20:03, Benson Margulies wrote:
THanks. What do you think of renaming the new param to be simply 'required'?

+1, I won't have time to handle it today, please do it if you get a chance

Sergey

On Sun, Nov 27, 2011 at 1:52 PM, Sergey Beryozkin<[email protected]>  wrote:
I enabled it, multipart/form-data payloads which are mapped to primitive
parameters are handled as form payloads so it took me awhile to trace why
the test was not passing

Sergey

On 24/11/11 22:20, Benson Margulies wrote:

org.apache.cxf.systest.jaxrs.JAXRSMultipartTest.testNullableParams()

Sergey, I can't figure out why this doesn't work. It's checked in with
an Ignore. If this is a mess let me know and I'll rip it all back out.


On Thu, Nov 24, 2011 at 11:19 AM, Sergey Beryozkin<[email protected]>
  wrote:

OK, please do, I'm assuming 'false' is default

Sergey

On 24/11/11 16:10, Benson Margulies wrote:

I was thinking

    @Multipart(value = "somepart", nullIfMissing = true)



On Thu, Nov 24, 2011 at 11:08 AM, Sergey Beryozkin<[email protected]>
  wrote:

Hi
On 24/11/11 15:53, Benson Margulies wrote:

Sergey,

Thanks.

I have another closely-related whine in mind.

I got into this area because I forgot to add one of the form elements
to a form pointed at a function annotated with @Multipart.

I had sort of hoped that this would lead, optionally, to a null
instead of an exception.

I note that Multipart is 'one of ours'. Would you object to my added a
boolean flag that maps 'no such part' to null instead of throwing?

Do you mean adding a property to MultipartProvider indicating to it
that
it
should just return null ? Sounds good, I guess if one uses @Multipart
and
a
valid payload may not necessarily include a given part  then 'null'
(assuming the user has configured the provider) is valid,

Cheers, Sergey





--
Sergey Beryozkin

http://sberyozkin.blogspot.com

Talend Community Coders
http://coders.talend.com/

Reply via email to