[#XW-534] Updgrate Ognl dependency to 2.7 for XWork 1.2.3 release

XWork

Updgrate Ognl dependency to 2.7 for XWork 1.2.3 release

Created: 30/Jun/07 11:44 AM Updated: 13/Jun/08 06:06 AM

Component/s:

None

Affects Version/s:

None

Fix Version/s:

1.2.4

Description

« Hide

Updgrate Ognl dependency to 2.7 for XWork 1.2.3 release

Martin just pointed out there's a new version of ognl that have significant improvements, we might need to use the Ognl.compileExpression() to take advantage of the new speed improvement.

[1] - http://forums.opensymphony.com/thread.jspa?messageID=163582&#163582
[2] - http://www.theserverside.com/news/thread.tss?thread_id=45954

All

Comments

Change History

Sort Order:

[ Permlink | « Hide ]

tm_jee - [14/Jul/07 10:07 PM ]

there's an upgrade guide written by Jesse at http://cwiki.apache.org/confluence/display/WW/OGNL+Expression+Compilation

[ Permlink | « Hide ]

tm_jee - [25/Nov/07 09:21 AM ]

Fixed. Changes are at :-

Adding java/com/opensymphony/xwork/XWorkConstants.java
Sending java/com/opensymphony/xwork/util/CompoundRootAccessor.java
Sending java/com/opensymphony/xwork/util/ObjectProxyPropertyAccessor.java
Sending java/com/opensymphony/xwork/util/OgnlUtil.java
Sending test/com/opensymphony/xwork/util/CompoundRootPropertyAccessorTest.java
Sending test/com/opensymphony/xwork/util/MyPrimitiveArrayObject.java
Transmitting file data ......
Committed revision 1684.

Kindly review. Cheers.

[ Permlink | « Hide ]

tm_jee - [25/Nov/07 09:23 AM ]

The ognl expression compilation mode is off by default. By default it will use expression parsing. To turn it on we need to use

<xwork>
   <parameters>
             <parameter name="useOgnlEnhancement" value="true" />
    </parameters>
</xwork>

if expression compilation mode failed (throws expression) it will fallback to expression parsing.

[ Permlink | « Hide ]

tm_jee - [25/Nov/07 09:49 AM ]

Kindly review. :-)

Cheers.

[ Permlink | « Hide ]

Rainer Hermanns - [13/Jun/08 06:06 AM ]

code does not work when feature is enabled...
Lots of failing unit tests...

See my additional mail for more information.