This is getting tiresome, so I will try to be quick. I created another attachment (15777) to http://issues.apache.org/bugzilla/show_bug.cgi?id=30292 . It is less compact that McGrady's version because it uses prefix not suffix, and event-to-method map. It actually was inspired by LookupDispatchAction, only I did not want to use button caption to locate methods. The code is not copied or stolen whatever the accusations are. Image button handling was added after discussion in the mailing list, and turned out to be something that Ted Husted already told about.
Event-to-method map can be considered by some as a hassle, by others as a bit of protection from direct calling of action's methods. Some would like to move mapping outside of the code. Yet other would like to use external util class, callable from a regular action class. If someone still wants to discuss all these great options, I can join. In its current form SelectAction works as a basis for my dialog stuff, which I hope to submit to Struts later. I just had decided to start from one class, which can be used by many. I just want to comment Martin Cooper's words: "IMHO, dispatch actions, whatever flavour, are a bad idea in the first place. They are essentially second-level controllers. What for? You already have a perfectly good controller in the Struts ActionServlet, so why have your own additional levels of controller below that?" http://marc.theaimsgroup.com/?l=struts-dev&m=109538996604541&w=2 Exactly, they are a second-level controllers, what is wrong with that? This event model is not much different from other frameworks, it allows to reduce number of classes, to have compact and clean config file, and most important, to easily create web components. One action for all component methods also allows to have only one address, I consider this very important for user experience. But Frank already hates me for having the same arguments day after day. Michael. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
