Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 326349 - [refactoring] Refactoring array name fails
Summary: [refactoring] Refactoring array name fails
Status: RESOLVED FIXED
Alias: None
Product: JSDT
Classification: WebTools
Component: General (show other bugs)
Version: 3.2.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2.2 P   Edit
Assignee: Chris Jaun CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords:
Depends on: 269009
Blocks:
  Show dependency tree
 
Reported: 2010-09-27 16:22 EDT by Chris Jaun CLA
Modified: 2010-10-19 14:21 EDT (History)
2 users (show)

See Also:
thatnitind: review+

Attachments
patch (4.00 KB, patch)
2010-09-27 16:23 EDT, Chris Jaun CLA 		no flags Details | Diff
patch part 2 (4.43 KB, patch)
2010-10-18 13:50 EDT, Chris Jaun CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Jaun CLA 2010-09-27 16:22:40 EDT 
+++ This bug was initially created as a clone of Bug #269009 +++

Build ID: WTP 3.1 m6

Steps To Reproduce:
Type the following into the JS editor:

var b = new Array();
b[0] = 1;
b[1] = 2;
b[2] = 3;
b[3] = 4;

1. Highlight the 'b' in the fight line.
2. Right-click
3. Refactor -> Rename
4. Type a new name - notice that all the b's in the array and its element definitions update as you type.
5. Press the Enter key.

An error dialog pops up and the name changes revert back to the original.

"The refactoring 'Rename Local Variable' (org.eclipse.wst.jsdt.ui.rename.local.variable) cannot be performed, since its input does not exist."



More information:
Comment 1 Chris Jaun CLA 2010-09-27 16:23:16 EDT 
Created attachment 179684 [details]
patch
Comment 2 Chris Jaun CLA 2010-09-29 17:54:42 EDT 
Checked into patches build by Nitin.
Comment 3 Chris Jaun CLA 2010-10-18 13:50:07 EDT 
Created attachment 181117 [details]
patch part 2

Need to apply part 2 of patch.
Comment 4 Chris Jaun CLA 2010-10-18 13:50:39 EDT 
Need to apply part 2 of patch. The first part is already included.
Comment 5 Carl Anderson CLA 2010-10-19 14:21:10 EDT 
Committed patch part 2 to R3_2_2_patches