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

Bug 329379

Summary:

Extract Local Variable support

Product:

[Technology] DLTK

Reporter:

Vladislav Kuzkokov <vladislav.kuzkokov>

Component:

JavaScript

Assignee:

dltk.common-inbox <dltk.common-inbox>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

alex.panchenko

Version:

unspecified

Target Milestone:

---

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
implementation	none
minor fixes	none
scope lookup functionality	none
Support for "Replace all occurences" capability	none
Support for "Replace all occurences" capability	none

Description Vladislav Kuzkokov

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

2010-11-03 13:28:06 EDT

Created attachment 182313 [details]
implementation

Comment 2 Vladislav Kuzkokov

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

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

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

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

2010-11-25 06:33:55 EST

Committed. Thanks!