--- Rob Leland <[EMAIL PROTECTED]> wrote: > David Graham wrote: > > >--- Rob Leland <[EMAIL PROTECTED]> wrote: > > > > > >>Ted Husted wrote: > >> > >> > >> > >>>I use many utilities Actions like these, and the result is that fewer > > >>>*custom* Actions are needed. I think increasing the number of > standard > >>> > >>> > >>>Actions in the distribution is a very good idea. It makes Struts more > > >>>accessible to newcomers, saves everyone from reimplementing the same > >>>design, and leverages the fact that Actions are singletons. > >>> > >>>-Ted. > >>> > >>> > >>How about struts-tools distribution named after the velocity-tools. > >>These would be classes that could be used stand-alone or in pairs. > >>Tnen we could place those orphan RequestUtils absoluteURL, and other > >>utilities there. > >>It would have a package structure o.a.s.tools.(util, actions, ....) > >> > >> > > > >I think unused methods/classes should be deprecated and removed. We > >shouldn't be carrying around excess unused utility code just because a > >user *might* be using them. > > > > > I agree they should be moved from the core package. > And moved to an optional package.
I have a few concerns with this. First, it's more work to maintain this new optional package with build files, tests, distribution, etc. Second, it's likely that the unused code would decay because Struts isn't using it. Also, it's confusing to have some actions in o.a.s.tools.actions and some in o.a.s.actions. I think *all* standard actions that are distributed with Struts should live in o.a.s.actions. David > > > > -- > ----- > Rob Leland (703-525-3580) > > > Choose a job you love, and you will never have to work a day of your > life. > > -Confucius. > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > __________________________________ Do you Yahoo!? Yahoo! SiteBuilder - Free, easy-to-use web site design software http://sitebuilder.yahoo.com --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
