Extract Method is ignorant of indentation

[ThunderFrame]

The indentation of the extracted lines is preserved, and the indentation of the call to the new method is seemingly always 0 spaces.

Also note the error in the new method line placement - see comment in code below...

Before:

Sub foo()

  Dim x As String
  If True Then
    If True Then
      If True Then
        x = "foo"
      End If
    End If
  End If

End Sub

After:

Sub foo()

  If True Then
    If True Then
      If True Then
      End If
NewMethod 'Note this is also on the wrong line - it should be 1 line higher up
    End If
  End If

End Sub
Private Sub NewMethod()
  Dim x As String
        x = "foo"
End Sub

[retailcoder]

This is by design, sort of - the refactoring was written before the indenter was finished - it's not a feature of the extract method refactoring to know how to indent code.

Changing from [bug] to [enhancement], since all that's needed is to call into the indenter interface when generating the extracted method.

[bclothier]

@Vogel612, please assign to me.