Refactor>Move Closer To Usage removes member calls

[IvenBach]

Code before:

Sub TestRDMoveCloserToBefore()

    Dim someRange As Range
    
    Dim someOtherRange As Range
    

    Dim someRange As Range
    Set someRange = Selection
    
    Set someOtherRange = ActiveWorkbook.Sheets(2).Range("A1")
    
    Set someRange = someRange.Resize(someRange.Count)
    
End Sub

Code after: .Resize is gone as is .Count

Sub TestRDMoveCloserToAfter()

    'Moved someRange
    Dim someOtherRange As Range
    

    Dim someRange As Range

    Dim someRange As Range
    Set someRange = Selection
    
    Set someOtherRange = ActiveWorkbook.Sheets(2).Range("A1")
    
    Set someRange = someRange(someRange)
    
End Sub

[retailcoder]

😨

[comintern]

someRange is declared twice in the top example. Is it possible that the refactor is running twice without updating the selection cache? Note that even though the "Before" code doesn't compile due to "Duplicate declaration in current scope", the RD parser has absolutely no issue with it. That means that every usage of someRange has too many references pointed at it:

[retailcoder]

@comintern shouldn't that throw a resolver error?

[comintern]

I stand corrected. The top declaration has zero references:

[IvenBach]

Looks like the extra Dim someRange as Range was from me not Undoing enough from a first move. I can't get it to reproduce the error now. When I do run it without the extra Dim it works fine.

Can't figure out how I got that to occur in the first place.