THanks. What do you think of renaming the new param to be simply 'required'?
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/ >
