Now that it's in the trunk we can move things around a bit. Here's how I'd divide what's (today) in the .action package:

- ActionBeans -- these are the "business logic" beans. Probably should stay in the .action package.

- WikiContextFactory -- partially replaces CommandResolver; makes WikiContexts. This should go somewhere else, maybe as a top-level class.

- WikiActionBeanContext, WikiInterceptor, *TypeConverter, WikiExceptionHandler, WikiRuntimeConfiguration -- these are all pure Stripes "glue" classes that allow JSPWiki and Stripes to work together. Will also include a LocalePicker implementation (later) to read Dirk's locale cookies to help pick the correct language. Should go to its own package (sub-package of .action? or .ui?)

- WikiRequestContext, HandlerInfo, and HandlerPermission: annotations for (1) associating a Stripes event handler method with a WikiContext request context and (2) specifying the permissions a user needs to execute that handler method. HandlerPermission is a cached "holding bag" that ties these things together. Should probably go into the same package as the pure Stripes classes.

About the only thing I know for sure is that the ActionBeans should be separate from everything else...

Andrew

On Dec 3, 2008, at 2:39 PM, Janne Jalkanen wrote:

Hi (and especially Andrew)!

One thing which has been minorly bugging me about the Stripes integration is that all classes are in one ginormous package. This makes it a bit more difficult (at least for me) to visualize the structure. Anybody have any suggestions on how those could be rearranged to be a bit more clear?

/Janne

Reply via email to