Return-Path: Delivered-To: apmail-incubator-cxf-dev-archive@locus.apache.org Received: (qmail 72171 invoked from network); 3 Mar 2008 11:55:22 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 3 Mar 2008 11:55:22 -0000 Received: (qmail 78986 invoked by uid 500); 3 Mar 2008 11:55:17 -0000 Delivered-To: apmail-incubator-cxf-dev-archive@incubator.apache.org Received: (qmail 78936 invoked by uid 500); 3 Mar 2008 11:55:17 -0000 Mailing-List: contact cxf-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cxf-dev@incubator.apache.org Delivered-To: mailing list cxf-dev@incubator.apache.org Received: (qmail 78927 invoked by uid 99); 3 Mar 2008 11:55:17 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 03 Mar 2008 03:55:17 -0800 X-ASF-Spam-Status: No, hits=2.0 required=10.0 tests=HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of barfitzgerald@gmail.com designates 209.85.198.188 as permitted sender) Received: from [209.85.198.188] (HELO rv-out-0910.google.com) (209.85.198.188) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 03 Mar 2008 11:54:42 +0000 Received: by rv-out-0910.google.com with SMTP id k20so3810846rvb.0 for ; Mon, 03 Mar 2008 03:54:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; bh=9RlU+c7rgnSh0Nz7OOzOU3tlBdhR1rF74Jy0oRqkrTk=; b=IXKJZmFTAEKnxlMAlolKoHfCZesL1lWzeU2S1+MCEuphKR5ksi1xaohGpx/io6e0DCFJrtAZyG3h4VrcwoI1nA9afOxeNicsWQKvIKfJ6IeMUt6MRDSYDNWo13uCkIeCVVX4BJKmN+25IPsuKQo8S1Fp9kVE5jALTT8DcslIaRA= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; b=FKSAFwMiZRd71b2acw5JYXhKfMIoR1ZJB7xAtR7UlXzRCjdC16owWWVaJtBVKNo49eAujx5mXDp1Y13gzZa3uQ8WfcgH3urLvhguca2JVZgaDW0ftP71T7nHZOQF51rh1dBAmnFmfBc+8RM5Ll9aovFIqT/Ovj9k4CBuP+dWEXM= Received: by 10.141.167.5 with SMTP id u5mr7606837rvo.71.1204545291093; Mon, 03 Mar 2008 03:54:51 -0800 (PST) Received: by 10.140.208.6 with HTTP; Mon, 3 Mar 2008 03:54:51 -0800 (PST) Message-ID: <6f5a2e920803030354k6b71217p13218c7fb3c037d9@mail.gmail.com> Date: Mon, 3 Mar 2008 11:54:51 +0000 From: "Barry Fitzgerald" To: cxf-dev@incubator.apache.org Subject: Re: JAX-RS Form Encoding Provider In-Reply-To: <012101c879f9$2959aea0$e002050a@pcgroupiona.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_19392_8520950.1204545291093" References: <6f5a2e920802271002w5d64c81av3e90bb304b386942@mail.gmail.com> <00cf01c8796f$64556d30$e002050a@pcgroupiona.com> <6f5a2e920802280237m6c067499r22f38ae342603a80@mail.gmail.com> <012101c879f9$2959aea0$e002050a@pcgroupiona.com> X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_19392_8520950.1204545291093 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline Hi, Patch added at: https://issues.apache.org/jira/browse/CXF-1455 This also adds the provider to the list of default providers which I presume is needed for the spec. Barry On Thu, Feb 28, 2008 at 11:00 AM, Sergey Beryozkin < sergey.beryozkin@iona.com> wrote: > Hi Barry > > This is great. Perhaps you may still want to support Map > as well, I was wrong suggesting it would spec uncompliant, the spec requires > MultivaluedMap, but as long as there's a provider to support > Map, then it would be fine... > > The only possible problem with Map, is that, as far as I > rememeber, one may have mutiple values for a given key in a form request : > > input=bar&input=foo, thus a String value will make it more difficult to > handle such cases. > > MultivaluedMap is really a Map>. It > has methods like getSingle, so if you know that in your own application no > multiple values are possible, then you can use getSingle() to avoid dealing > with Lists. > From the provider's perspective, perhaps add(key, value) should be used, > as it will append values to the list if it already exists > > Thanks, Sergey > > > > Thanks Sergey, > > > > I've created a sub task for this: > > https://issues.apache.org/jira/browse/CXF-1455 > > > > I'll tidy up my code, change it to use MultivaluedMap and create a > patch. > > > > Barry > > ---------------------------- > IONA Technologies PLC (registered in Ireland) > Registered Number: 171387 > Registered Address: The IONA Building, Shelbourne Road, Dublin 4, Ireland > ------=_Part_19392_8520950.1204545291093--