RE: Trails 1.2 & OGNL issues

Sounds like modeling and interfacing lapses with the ID's...
 
regardless
 
Not all folks will be able to juggle and articulate documented issues with OGNL and then be pressed to resolve each individual case given their situation. Re-write feels kind like a howard re-write. Painful to what lies ahead. So I think trails should try to accomodate the best case that supports a runnable 1.2 framework.
 
I will be happy to maintain 1.2 (at my own pace) while you guys go off and re-do 1.3. Something to think about.
 
I recently logged an ognl issue and it took out the whole app... just from a complicated expression... no exception thrown either. I interpretted it as internal corruption... as event handlers were not even being invoked and the app ran until ognl completely gave out.

A list of best-careful-practices are definitely in order with other workarounds.
 
Caching disabled is unusable my version with caching on running pojo gallery is slow enough.

Best regards
Ken in nashua

---

Date: Mon, 5 May 2008 16:26:18 -0700
From: kalle.o.korhonen@...
To: users@...
CC: spedra@...
Subject: Re: [trails-users] Trails 1.2 & OGNL issues

On Mon, May 5, 2008 at 3:22 PM, Alejandro Scandroli <alejandroscandroli@...> wrote:

I've been testing Trails 1.2 for a while (over two months) now, and
OGNL is giving me a huge headache.
Although there are workarounds for all of them and they seem like not
so big a deal, they are huge when you depend on Trails development
speed and you have to spent most of your time working around these
issues.
On this particular project we are planning to move to T5 asap, but I
wonder what to do with the Trails 1.2 release.
What should we do?

We can't revert back to OGNL 2.6.x.
We can't use tapestry-prop.
We could remove all the "defaultValue"s and we could declare
interfaces for every known property (eg: classDescriptor,
propertyDescriptor, model, modelNew, blockFinder) but that's way too
verbose.
We could disable caching (org.apache.tapestry.disable-caching=true)
but that will not perform well at all.
We could wait until OGNL enables a "don't ever compile" option.
We could release 1.2 anyway

I'm using 1.2-SNAPSHOTs. I think we should release 1.2 anyway, and just properly document (and link to OGNL documentation that explains) compiled expressions and why you can and should always use #this when a compiled expression doesn't work. I know that compiled OGNL and using mixed types doesn't work and it's given me headaches as well, but I'm not willing to let go of the improved performance that compiled OGNL brings for the majority of the use cases. Disabling caching is not an option for other than development; when disabled, it'll pollute permgen space over time no matter how big it is.

Kalle

---

Windows Live SkyDrive lets you share files with faraway friends. Start sharing.