Actually, the extras I was referring to was for the commons-validator project itself as I mistook the ticket for a validator ticket. The Beanshell and xpath validators required only the commons-validator project and had nothing to do with Struts. Ideally, all validators could be added to the commons-validator project and be immediately reusable in environments like Struts and Spring. Unfortunately, the code is currently too muddled for that so Struts has to create a wrapper for every new validator/validation.
Don On 5/10/05, Joe Germuska <[EMAIL PROTECTED]> wrote: > This seems like a discussion which should happen more visibly than in > a Bugzilla ticket, so I'm addressing it more clearly to the > [EMAIL PROTECTED] list: > > Regarding a JEXL-based validator to plug in to Struts' > commons-validator bridge: > >http://issues.apache.org/bugzilla/show_bug.cgi?id=34849 > > > >------- Additional Comments From [EMAIL PROTECTED] 2005-05-10 21:11 ------- > >I put out two other similar patches which used jxpath and beanshell > >respectively. It was suggested validator should have an "extras" section, > >and > >if created, perhaps that would be a good place for this as well. > > I'd be ok with putting these in some package other than struts-core, > although I don't think we have anything really obvious right now. > Then again, I've been wanting to get a JEXL dependency into Struts to > suit my vision of auto-loading per-action (and per-forward) commands > -- but those could go in this other kit as well. We could probably > factor out most everything under o.a.struts.actions as well -- is it > right to push it all into its own soup? What would we call it? > > Is it worth having a validator-only extras package? Or just lob 'em > all in one? If Validator itself were factored out of the core, would > people still feel strongly that these things wouldn't belong in a > validator JAR artifact? > > Joe > > -- > Joe Germuska > [EMAIL PROTECTED] > http://blog.germuska.com > "Narrow minds are weapons made for mass destruction" -The Ex > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
