Issue Details (XML | Word | Printable)

Key: JEXL-3
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: dion gillard
Votes: 1
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
Commons JEXL

[JEXL] Static method resolution and changes to context

Created: 04/Apr/06 08:37 AM   Updated: 17/Jul/09 05:55 PM
Return to search
Component/s: None
Affects Version/s: 1.0
Fix Version/s: 2.0

Time Tracking:
Not Specified

File Attachments:
  Size
Zip Archive jexl-patch.zip 2006-04-04 08:39 AM dion gillard 12 kB
Environment:
Operating System: other
Platform: Other

Bugzilla Id: 39193
Resolution Date: 17/Jul/09 05:55 PM


 Description  « Hide
From: Guido Anzuoni <ganzuoni@gmail.com>

Here I will try to explain may changes to see if you agree to include in the trunk.
1. UberspectImpl
I need to include static methods resolution in expressions (e.g.
System.currentTimeMillis()). I patched the class to allow it.

2. JexlContext
Resolving vars using a Map is not enough flexible for my needs. I have modified
it to have a single method
public Object resolveVariable(String var);

This allows me for better flexibility in the resolution process.
I have implemented a JextContext based on a java.util.Map, an extensible
JexlContext and a delegating JexlContext.
The extensible resolver is abstract and has a "parent" context to which
delegates in the case local resolution failes.
Delegating implementation extends the extensible context and uses another
JexlContext for local resolution.

More, I have removed any reference to JexlContex.getVars() in the code.

If you wish I can send the modified classes to better evaluate my changes.



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
There are no subversion log entries for this issue yet.