Mailing-List: contact users-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: users@camel.apache.org
Received-SPF: pass (nike.apache.org: domain of willem.jiang@gmail.com
 designates 209.85.220.43 as permitted sender)
Date: Sat, 13 Jul 2013 11:01:42 +0800
From: Willem jiang <willem.jiang@gmail.com>
To: users@camel.apache.org
Message-ID: <35B14211E7CE4B83A50F60A68B832098@gmail.com>
In-Reply-To: <1373650042058-5735576.post@n5.nabble.com>
References: <1373650042058-5735576.post@n5.nabble.com>
Subject: Re: Bean Method Invocation regression in 2.11
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

Hi,
Thanks for reporting this.
I just filled a JIRA=5B1=5D for it.

=5B1=5Dhttps://issues.apache.org/jira/browse/CAMEL-6544 =20

-- =20
Willem Jiang

Red Hat, Inc.
=46useSource is now part of Red Hat
Web: http://www.fusesource.com =7C http://www.redhat.com
Blog: http://willemjiang.blogspot.com (http://willemjiang.blogspot.com/) =
(English)
          http://jnn.iteye.com (http://jnn.javaeye.com/) (Chinese)
Twitter: willemjiang =20
Weibo: =E5=A7=9C=E5=AE=81willem


On Saturday, July 13, 2013 at 1:27 AM, Ashwin Pejavar wrote:

> I'm testing my upgrade to 2.11 and noticed a bug in how beanMethod
> invocations are parsed. =20
> =20
> I'm using Spring to define my routes.
> =20
> The following invocation expression: exec(*, '', 'arg3') gets parsed as=

> arg1: *
> arg2: =22=22
> arg3: =22,arg3=22 (notice the comma).
> =20
> I tracked down the bug to the class org.apache.camel.util.StringQuoteHe=
lper.
> The code-block
> =7D else if (ch =3D=3D separator) =7B
> // add as answer if we are not in a quote
> if (=21singleQuoted && =21doubleQuoted && sb.length() > 0) =7B
> String text =3D sb.toString();
> if (trim) =7B
> text =3D text.trim();
> =7D
> answer.add(text);
> sb.setLength(0);
> continue;
> =7D
> =7D
> =20
> Should instead be
> =7D else if (ch =3D=3D separator) =7B
> // add as answer if we are not in a quote
> if (=21singleQuoted && =21doubleQuoted) =7B
> if (sb.length() > 0) =7B
> String text =3D sb.toString();
> if (trim) =7B
> text =3D text.trim();
> =7D
> answer.add(text);
> sb.setLength(0);
> =7D
> continue; //always skip separator if not in quotes
> =7D
> =7D
> =20
> =20
> =20
> --
> View this message in context: http://camel.465427.n5.nabble.com/Bean-Me=
thod-Invocation-regression-in-2-11-tp5735576.html
> Sent from the Camel - Users mailing list archive at Nabble.com (http://=
Nabble.com).