Scott, I would say that you are okay overriding the defaultStack since it is a plugin. For a user to actively install your plugin, then they are seeking the functionality you are providing. I don't know much about the plugin you are writing, but if it works similar to other interceptors, meaning that the user has to implement an interface to get the functionality (like, if you were building a breadcrumb plugin and actions have to implement com.gmail.stanlick.Crumbable, or have to annotated by @com.gmail.stanlick.Crumb), then it should have no side-effects. The advantage of overriding the defaultStack is that most users will have less steps to be up and running, but the caveat is that users who don't use defaultStack will have to include your interceptor manually. Of course, you'll have all of this documented and users in the second scenario will most likely know where to look to get your plugin up and running.
-W On 10/10/07, stanlick <[EMAIL PROTECTED]> wrote: > > I am writing a plugin that consists of a new interceptor. I would like its > struts-plugin.xml to append to the defaultStack so the interceptor works out > of the box for packages leveraging the default interceptor stack. What is > the ethical thing to do as it relates to a plugin modifying the default > interceptor stack? Overriding the stack does not pass the tummy test, but > in the spirit of plugin drop-n-go, I also feel odd about including a list of > modifications you need to make to get the plugin to play! > > Scott > -- > View this message in context: > http://www.nabble.com/Struts-2-Plugin-tf4603263.html#a13143714 > Sent from the Struts - User mailing list archive at Nabble.com. > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Wesley Wannemacher President, Head Engineer/Consultant WanTii, Inc. http://www.wantii.com --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
