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

Bug 356956

Summary: [RBD]Refactor on package name doesn't change references
Product: z_Archived Reporter: broy2
Component: EDTAssignee: pf yu <pfyu>
Status: NEW --- QA Contact:
Severity: normal    
Priority: P3 CC: chenzhh, jqian, yuhaodl
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Refactor wizard none

Description broy2 CLA 2011-09-07 11:39:02 EDT
Build Identifier: 20110907right

Refactor should rename the part and its references.
In RBD, Refactor was not a choice on a package, just Rename.
Refactor should bring up a Rename wizard with a checkbox Update references and a Preview button.  See attached.
So if Refactor on a package is an enhancement in EDT, then it's broken and should be fixed.
Otherwise, the choice should be Rename rather than Refactor.

Reproducible: Always

Steps to Reproduce:
1.Create an EGL project with package.
2.Create a program in that package.
3.Right click on the package in the Project Explorer.
4.Select Refactor > Rename and change the name of the package.
5.This creates error in the program:
IWN.VAL.3049.e 1/1 The name of the package given in the file must match the directory structure in which the file resides. The match must be exact, including case.
Comment 1 broy2 CLA 2011-09-07 11:39:42 EDT
Created attachment 202912 [details]
Refactor wizard
Comment 2 Jing Qian CLA 2011-09-07 11:41:57 EDT
I think if we give Refactor as menu, then user would expect it behaves like refactor (taking care of all the references).  If we do not support such behavior, we should not call it refactor.
Comment 3 broy2 CLA 2011-11-01 14:27:28 EDT
This is a bug, not an enhancement.
Refactor should be removed from the menu since it doesn't work.

The enhancement would be to support refactor on a package in the future.
Comment 4 Lisa Lasher CLA 2011-11-03 13:14:33 EDT
*** Bug 362758 has been marked as a duplicate of this bug. ***