DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=34849>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=34849 ------- Additional Comments From [EMAIL PROTECTED] 2005-05-26 11:18 ------- Clarifying based on a question in a private email: the added dependencies Collections and Lang are from how this validator is written; JEXL has no external dependencies of its own. It was suggested that commons-el might be an alternative to JEXL, and it would be consistent with JSTL and Tomcat -- this is true; however, one of the nice features of JEXL is its ability to invoke methods in addition to referencing bean properties. This ability is why the code in this ticket has a commons-lang dependency: the implementation adds instances of StringUtils and ArrayUtils to the expression evaluation context so that you can write richer expressions. The Collections dependency is on IteratorEnumeration, and is just an implementation detail of the PageContextImpl class attached to this ticket. It would be annoying to have to essentially rewrite that functionality, but there may be reasons why it would be justified, especially given the version compatibility issues which were circulating around commons-collections. I originally copied the taglib "EnumeratedMap" abstract class into a private class file defined within the ExpressionValidator class (my preferred name for what Lukas submitted as ELValidator). We *could* do this for all of these nagging dependencies, and/or copy them to public classes in an o.a.s.validator.expression package. This strategy would return us to a single dependency on JEXL at the cost of adding three or four classes to Struts which are really only for supporting this one (complex but useful) function. I kind of like squirreling them all inside the one class, to avoid the risk of people casually using them as public classes, but it makes for a very big class file. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
