Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 200748

Summary: [move method] Move method refactoring appends unnecessary argument
Product: [Eclipse Project] JDT Reporter: David Saff <david>
Component: UIAssignee: JDT-UI-Inbox <jdt-ui-inbox>
Status: CLOSED WONTFIX QA Contact:
Severity: normal    
Priority: P3 CC: martinae
Version: 3.4   
Target Milestone: ---   
Hardware: PC   
OS: All   
Whiteboard: stalebug

Description David Saff CLA 2007-08-21 22:31:19 EDT
Build ID: I20070809-1105

Steps To Reproduce:
1. Create the following class:

public class BugTest {
  public static class PropertyName {
    public String propertyName;

    public PropertyName(String propertyName) {
      this.propertyName = propertyName;
    }
  }

  public String initialCaps(PropertyName pname) {
    return pname.propertyName.substring(0, 1).toUpperCase()
        + pname.propertyName.substring(1);
  }
}


2. Move method initialCaps to parameter PropertyName pname.
3. The result:

public String initialCaps(BugTest bugTest) {
  return propertyName.substring(0, 1).toUpperCase()
        + propertyName.substring(1);
}

The "BugTest bugTest" parameter is useless.

More information:
Comment 1 Eclipse Genie CLA 2020-02-24 04:16:00 EST
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.