[switchyard-issues] [JBoss JIRA] (SWITCHYARD-619) Need a convention for plugin "create" methods and optional parameters
Tom Cunningham (JIRA)
jira-events at lists.jboss.org
Mon Jan 16 12:55:18 EST 2012
Tom Cunningham created SWITCHYARD-619:
-----------------------------------------
Summary: Need a convention for plugin "create" methods and optional parameters
Key: SWITCHYARD-619
URL: https://issues.jboss.org/browse/SWITCHYARD-619
Project: SwitchYard
Issue Type: Task
Components: tooling
Affects Versions: 0.4
Reporter: Tom Cunningham
Fix For: 0.4
BPMServicePlugin.java has a create method with 7 parameters - many of which are optional. Many of our other plugins have a similar situation - the ClojurePlugin's create has 6 parameters, and the RulesServicePlugin's create has 3 optional parameters as well.
Need a strategy for dealing with this - should the optional parameters be pulled out and put into their own separate modifyX methods?
from Keith from irc :
12:37:57 PM) kcbabo: tcunning: I'm not sure, there are a couple of options with separate pros and cons
(12:38:10 PM) kcbabo: tcunning: one thing with forge is that you can "cd" into a resource, IIRC
(12:38:20 PM) kcbabo: tcunning: which gives you context-senstive command access
(12:38:42 PM) kcbabo: tcunning: so you could CD into a service component (for example) and have localized commands to modify stuff
(12:38:57 PM) kcbabo: tcunning: another option is to simply add more commands that can be used to modify stuff
(12:39:10 PM) kcbabo: tcunning: the drawback there is that you could end up with loads of commands
(12:39:27 PM) kcbabo: tcunning: I think a JIRA is a good idea - not sure we'll reach agreement in time for 0.4, but you never know :-)
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the switchyard-issues
mailing list