[JBoss JIRA] (FORGE-275) delete associated methods when deleting a field
by George Gastaldi (JIRA)
[ https://issues.jboss.org/browse/FORGE-275?page=com.atlassian.jira.plugin.... ]
George Gastaldi updated FORGE-275:
----------------------------------
Status: Closed (was: Pull Request Sent)
Resolution: Done
> 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
> Assignee: Ricardo Martinelli Oliveira
> Priority: Minor
> Labels: Starter
> Fix For: 1.1.2.Final
>
> 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
For more information on JIRA, see: http://www.atlassian.com/software/jira
12 years, 2 months
[JBoss JIRA] (FORGE-150) Implementations of JavaParser Interfaces should maintain equality with other implementations
by Ivan St. Ivanov (JIRA)
[ https://issues.jboss.org/browse/FORGE-150?page=com.atlassian.jira.plugin.... ]
Ivan St. Ivanov reassigned FORGE-150:
-------------------------------------
Assignee: Ivan St. Ivanov
> Implementations of JavaParser Interfaces should maintain equality with other implementations
> --------------------------------------------------------------------------------------------
>
> Key: FORGE-150
> URL: https://issues.jboss.org/browse/FORGE-150
> Project: Forge
> Issue Type: Enhancement
> Components: Parsers / File Manipulation
> Affects Versions: 1.0.0.Alpha4
> Reporter: Jason Porter
> Assignee: Ivan St. Ivanov
> Fix For: 1.1.2.Final
>
>
> Until I saw how to create an Import (which I still don't agree with having to be tied to the Implementation) I thought about creating my own implementation for a test (a mock would be even better) so I looked at the equals for the ImportImpl and it does not keep equality with other implementations because it checks to see if the class is the same. It also checks internal state instead of using the interface methods.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
12 years, 2 months