Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 329379 - Extract Local Variable support
Summary: Extract Local Variable support
Status: RESOLVED FIXED
Alias: None
Product: DLTK
Classification: Technology
Component: JavaScript (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: dltk.common-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-03 13:24 EDT by Vladislav Kuzkokov CLA
Modified: 2010-11-25 06:33 EST (History)
1 user (show)

See Also:

Attachments
implementation (1.29 MB, patch)
2010-11-03 13:28 EDT, Vladislav Kuzkokov CLA 		no flags Details | Diff
minor fixes (28.30 KB, patch)
2010-11-10 04:47 EST, Vladislav Kuzkokov CLA 		no flags Details | Diff
scope lookup functionality (24.15 KB, patch)
2010-11-11 05:27 EST, Vladislav Kuzkokov CLA 		no flags Details | Diff
Support for "Replace all occurences" capability (52.90 KB, patch)
2010-11-18 11:36 EST, Vladislav Kuzkokov CLA 		no flags Details | Diff
Support for "Replace all occurences" capability (63.78 KB, patch)
2010-11-18 12:11 EST, Vladislav Kuzkokov CLA 		no flags Details | Diff
Show Obsolete (1) 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 Vladislav Kuzkokov CLA 2010-11-03 13:24:24 EDT 
Build Identifier: M20100909-0800

Initial implementation of functionality resembling the one in JDT.

Reproducible: Always

Steps to Reproduce:
Inapplicable
Comment 1 Vladislav Kuzkokov CLA 2010-11-03 13:28:06 EDT 
Created attachment 182313 [details]
implementation
Comment 2 Vladislav Kuzkokov CLA 2010-11-10 04:47:40 EST 
Created attachment 182796 [details]
minor fixes

Better lookup for references.
Glitch in rename (Preview->Back->Preview) fixed.
Comment 3 Vladislav Kuzkokov CLA 2010-11-11 05:27:35 EST 
Created attachment 182890 [details]
scope lookup functionality

Extract local will issue a warning if name is already defined in visible scope.
Comment 4 Vladislav Kuzkokov CLA 2010-11-18 11:36:15 EST 
Created attachment 183399 [details]
Support for "Replace all occurences" capability

"Replace all occurences... " checkbox added. Unwanted replacements can be deselected on the last page of the wizard.
Comment 5 Vladislav Kuzkokov CLA 2010-11-18 12:11:57 EST 
Created attachment 183404 [details]
Support for "Replace all occurences" capability

Previous patch would break other refactorings.
Comment 6 Alex Panchenko CLA 2010-11-25 06:33:55 EST 
Committed. Thanks!