[#OGNL-7] Full Java 5 Support for OGNL

OGNL

Full Java 5 Support for OGNL

Created: 01/Feb/07 10:27 PM Updated: 19/Nov/07 02:45 PM

Component/s:

Core Runtime

Affects Version/s:

None

Fix Version/s:

2.7.2

Issue Links:

Duplicate

This issue is duplicated by:

~~OGNL-1~~ Type conversion improvements

Description

« Hide

Although xwork has been able to work around it, it seems like it would be better if OGNL supported 1.5 out-of-the-box. Particularly it seems like there are places where we should be passing ParameterizedType instead of Class so the generic type is passed along with the base class type.

All

Comments

Change History

Sort Order:

[ Permlink | « Hide ]

Tom Schneider added a comment - 02/Feb/07 10:11 AM

Just an FYI, I'll probably be taking a look at implementing this at some point in the future. I'll post a patch if I make any progress.

[ Permlink | « Hide ]

Jesse Kuhnert added a comment - 17/Feb/07 11:54 AM

Sounds like a plan, though I think we should do this in such a way that the 1.5 support is optional - so as not to exclude 1.4 jre's from continuing to use the basic functionality of the library. Perhaps something like how TestNG handles 1.5 vs 1.4 specific stuff.

[ Permlink | « Hide ]

Tom Schneider added a comment - 04/Nov/07 06:58 PM

As you might be able to tell, I didn't make much progress with this. :) This becomes less relevant for struts 2 now that we've implemented a experimental unified expression value stack in struts 2. Adding Java 5 support would probably be an update to the OGNL language itself. A pretty significant effort at this point.

[ Permlink | « Hide ]

Jesse Kuhnert added a comment - 19/Nov/07 02:26 PM

Though probably irrelevant now, I believe most 1.5-isms are currently being supported by OGNL now.

[ Permlink | « Hide ]

Tom Schneider added a comment - 19/Nov/07 02:45 PM

Really, even things like varargs, type safe enums, and generics? I would have assumed this might have required language changes.