Rename "setX(param)" methods which return the Descriptor "this"
instance to "x(param)"
--------------------------------------------------------------------------------------
Key: SHRINKDESC-67
URL:
https://issues.jboss.org/browse/SHRINKDESC-67
Project: ShrinkWrap Descriptors
Issue Type: Sub-task
Reporter: Andrew Rubinger
For example:
Application5Descriptor.securityRole().setRoleName(roleName)
Here the API looks backwards. "securityRole()" is really a "get"
operation, because it doesn't return the descriptor, but instead SecurityRoleType. So
it should be named "getSecurityRole()".
"setRoleName" on the other hand is used in method chaining, so according to the
conventions we use in ShrinkWrap alongside Fowler's definition for fluent APIs in
DSLs:
http://martinfowler.com/bliki/FluentInterface.html
...then "setRoleName(param)" should become simply "roleName".
--
This message is automatically generated by JIRA.
For more information on JIRA, see:
http://www.atlassian.com/software/jira