[
https://issues.jboss.org/browse/FORGE-275?page=com.atlassian.jira.plugin....
]
Lincoln Baxter III commented on FORGE-275:
------------------------------------------
Hi Shameera,
First, please read and follow the contributor guide for submitting changes:
https://docs.jboss.org/author/display/FORGE/Contribute+or+Get+Involved
Second, your approach is fairly good, but this should probably be abstracted from the
RmPlugin. I think that the work for deleting relevant fields or methods should probably
take place in the JavaResource itself. Possibly even prompting for user input there as
well. The RmPlugin, however, should stay generic, and not know what type of resources it
is operating on.
~Lincoln
delete associated methods when deleting a field
-----------------------------------------------
Key: FORGE-275
URL:
https://issues.jboss.org/browse/FORGE-275
Project: Forge
Issue Type: Feature Request
Components: Brainstorming, Builtin Plugins
Affects Versions: 1.0.0.Alpha2, 1.0.0.Alpha3
Reporter: Rodney Russ
Priority: Minor
Attachments: Forge-275.patch
When removing a field, it would make sense to remove the associated methods. For
example:
cd Person.java
rm firstName
*delete associated methods setFirstName() and getFirstName() (Y/n)?
--
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