[jboss-dev-forums] [JBoss ESB Development] - Annotation based Action classes

[Keith Babo]

Keith Babo [http://community.jboss.org/people/kcbabo] replied to the discussion

"Annotation based Action classes"

To view the discussion, visit: http://community.jboss.org/message/548544#548544

--------------------------------------------------------------
I think everyone agrees that this is a great thing to do.  A few pieces of feedback:

o It would be great to get a basic wiki page/article built around this functionality.  You have most of the content already between this thread and the implementation in your workspace.  It's just easier to get a sense of the scope of the feature when it's all in one place.  We will need this anyway to document the feature for users.

o  Looks like this is the current list of supported annotations.  Did I miss any?
* @OnExceptionMethod
* @OnSuccessMethod
* @ProcessMethod
* @ConfigProperty
* @Destroy
* @Initialize
* @BodyParam
* @PropertyParam

o Recommend dropping "Method" from the above annotion names.

o We could take this one step further and add an @Action annotation at the class level.  This would eliminate the need to extend AbstractActionPipelineProcessor or implement the Lifecycle and PipelineProcessor interfaces; you have already defined annotations for all the methods on these interfaces.  All we need to do is add a basic delegate implementation that would wrap the annotated action and dispatch to annotated methods as appropriate.

o I did not see @BodyParam and @PropertyParam in the fisheye changeset you published.  In fact, they are not in your workspace either AFAICT.  Do I have to pay extra for those? ;-)  I checked out the processing in BeanContainerAction and it looks like you support both a "name" property in the annotation as well as search based on the annotated parameter's type.  For examples, I would use the former syntax as it's much clearer where the body and property are coming from.

--------------------------------------------------------------

Reply to this message by going to Community
[http://community.jboss.org/message/548544#548544]

Start a new discussion in JBoss ESB Development at Community
[http://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2032]

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jboss-dev-forums/attachments/20100617/c6fa4b7d/attachment.html