Re: EL issues and 6.0.x release

2010-02-01 Thread Konstantin Kolinko
2010/1/31 Mark Thomas : > On 30/01/2010 17:41, Mark Thomas wrote: >> On 30/01/2010 07:33, Konstantin Kolinko wrote: >>> Regarding the implementation, AttributeParser.java class. I think >>> that, based on the above, we can fix it to solve bug 48627. Other >>> parts of the new implementation will re

Re: EL issues and 6.0.x release

2010-01-30 Thread Mark Thomas
On 30/01/2010 17:41, Mark Thomas wrote: > On 30/01/2010 07:33, Konstantin Kolinko wrote: >> Regarding the implementation, AttributeParser.java class. I think >> that, based on the above, we can fix it to solve bug 48627. Other >> parts of the new implementation will remain unchanged. > > I'll take

Re: EL issues and 6.0.x release

2010-01-30 Thread Mark Thomas
On 30/01/2010 07:33, Konstantin Kolinko wrote: > Regarding the implementation, AttributeParser.java class. I think > that, based on the above, we can fix it to solve bug 48627. Other > parts of the new implementation will remain unchanged. I'll take another look at this. I thought that this wouldn

Re: EL issues and 6.0.x release

2010-01-29 Thread Konstantin Kolinko
2010/1/8 Mark Thomas : > I think I have got to the bottom of why the EL has been so fragile and > why the fixes to the various edge case bug fixes have invariably created > new edge cases. It is probably easier to explain by considering an > example. Using the echo tag from the newly added EL test

Re: EL issues and 6.0.x release

2010-01-08 Thread Mark Thomas
On 08/01/2010 12:18, Tim Funk wrote: > +1 to trunk > -0 for backport to 6.0.x > > Is this legal? ${'\'}${1+1} > Shouldn't it be: ${'\\'}${1+1} Yes it should. and the Java code to append the \ is: result.append("${''}"); This is part of what makes this so tricky to get right. The quoting rule

Re: EL issues and 6.0.x release

2010-01-08 Thread Mark Thomas
On 08/01/2010 11:53, Remy Maucherat wrote: > On Fri, 2010-01-08 at 11:36 +, Mark Thomas wrote: >> I have some rough code that implements this scheme. I am in the process >> of integrating it into Jasper for trunk. I can see this taking little >> while to iron out the wrinkles before I'll be rea

Re: EL issues and 6.0.x release

2010-01-08 Thread Tim Funk
+1 to trunk -0 for backport to 6.0.x Is this legal? ${'\'}${1+1} Shouldn't it be: ${'\\'}${1+1} Or phrased another way, what should I see on the screen with this: ${1+1}${'\t'}${1+1} "2 2" or "2\t2" -Tim On 1/8/2010 6:36 AM, Mark Thomas wrote: One area where help would be appreciated is in

Re: EL issues and 6.0.x release

2010-01-08 Thread Remy Maucherat
On Fri, 2010-01-08 at 11:36 +, Mark Thomas wrote: > I have some rough code that implements this scheme. I am in the process > of integrating it into Jasper for trunk. I can see this taking little > while to iron out the wrinkles before I'll be ready t propose a > back-port for 6.0.x. I don't th

EL issues and 6.0.x release

2010-01-08 Thread Mark Thomas
I think I have got to the bottom of why the EL has been so fragile and why the fixes to the various edge case bug fixes have invariably created new edge cases. It is probably easier to explain by considering an example. Using the echo tag from the newly added EL test cases consider the following JS