We went back and forth about this in regards to the Velocity/Struts integration package for a while. Finally, we put support for the ContextHelper in Struts 1.1 (which might also work with the JSTL) and the adapter for 1.0 in a new Velocity Tools package. The ContextHelper gave Gabriel and Geir what they needed to know to write the 1.0 adapter, and leaves most of the Velocity code with the Velocity team.
Since Struts has so many Commons dependencies now, one more isn't going to make any difference :) The next question might be what about Tiles integration in Struts? Cedric's Definitions are really quite neat, are coupled (to an extent) with the ActionForwards, and has hooks to the ActionServlet. It also uses an XML configuration file. Leaving it seperate seems fine, but then again, if we have everything else in struts-config ... A very cool new feature, that I'm just getting into today, is the ability for each Tile to call an Action as it is included, which makes writing portlets very simple. As soon as get through it, I'm going to take a look at it from the perspective of the new sub-application (application module?) support. -Ted. David Winterfeldt wrote: > > I've been thinking about the inclusion of the Struts > driver/interface to the Validator with the Commons > version. Some of my concerns are as follows. If > Struts is included, what is the criteria for including > another driver? I wouldn't want to be exclusive, but > I wouldn't want to get into having to maintain drivers > for frameworks I don't use or understand. One > solution I can think of would be if a Commons commiter > would volunteer to maintain that driver/interface and > I could help them. The only other issue is that if > the driver is in Commons then the example web apps and > cactus unit tests would be there too. It doesn't seem > like this should be there. Maybe that is just because > there aren't any other Commons projects doing this > yet, but there will be in the future. > > David -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
