Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
User name:
Password:
Save Password
Forgot your password?

 All Forums
 Visual Assist
 Feature Requests
 Extract method enhancements
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

Dunemaster
New Member

Russia
8 Posts

Posted - Dec 05 2012 :  12:15:18 AM  Show Profile  Reply with Quote
1) When using 'Extract method', the new method implementation is inserted after that last method.

Most often I need it to be inserted right after the method I'm extracting from

An option (may be not in the refactoring window, but one to be set from setting windows) would be nice

The same goes for method implementation

2) Setting protection level in 'Extract method' refactoring would be very useful.

Another option would be always extract as private

Edited by - Dunemaster on Dec 05 2012 12:21:25 AM

accord
Whole Tomato Software

United Kingdom
3164 Posts

Posted - Dec 06 2012 :  3:55:07 PM  Show Profile  Reply with Quote
Point 1: we are considering doing this at some point:

case=12454

Point 2: I have put in a feature request for this:

case=71074

However it is not really clear for me, why you mention "always extract as private" separately. Don't you mean visibility by "protection level" in the previous sentence? (public, protected, private) Am I missing something?
Go to Top of Page

Dunemaster
New Member

Russia
8 Posts

Posted - Dec 06 2012 :  11:08:15 PM  Show Profile  Reply with Quote
Yes, I do mean visibility.
Maybe I'm mistaken, but it seems to me currently the visibility for extracted method matches the visibility of calling method, which is rarely the required behaviuor
Go to Top of Page

accord
Whole Tomato Software

United Kingdom
3164 Posts

Posted - Dec 07 2012 :  09:48:38 AM  Show Profile  Reply with Quote
Ok, the feature request is about the ability of setting the visibility on the dialog.
Go to Top of Page

sean
Whole Tomato Software

USA
2641 Posts

Posted - Feb 13 2015 :  8:16:06 PM  Show Profile  Reply with Quote
case=12454 is fixed in build 2059
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
© 2019 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000