Updating Visible Cells

reberryjr

Well-known Member
Joined
Mar 16, 2017
Messages
701
Office Version
  1. 365
Platform
  1. Windows
I have some code that filters column H for a series of values. Then, I need to update the filtered results to "Denied". For some reason, the code I'm using is updating everything, including the header row to "Denied". It's odd, b/c I use this snippet elsewhere, and there isn't an issue. When I look up how to perform the update I'm trying to, everything I've found points to using the .SpecialCells(xlCellTypeVisible).
VBA Code:
If mDLR > 1 Then mD.Range("H2:H" & mDLR).SpecialCells(xlCellTypeVisible).Value = "Denied"

  • The filter is working correctly.
  • The filter returns 1 value, making the Last Row (mDLR) = 2.
I can't imagine why, but the only thing I can think of, is the last row being row 2 is causing the issue. It's the only difference between the successful iterations and this one that's failing.
 

Excel Facts

What is the fastest way to copy a formula?
If A2:A50000 contain data. Enter a formula in B2. Select B2. Double-click the Fill Handle and Excel will shoot the formula down to B50000.
I ran your code on some cobbled up filtered data and I'm not seeing that behavior. All is as expected, even with the last row at 2. Suggest you add debug code to let you see if the range addresses are as you expect.
VBA Code:
    Dim rng As Range
    Dim Msg As String
    If mDLR > 1 Then
        Set rng = mD.Range("H2", mD.Range("H" & mD.Rows.Count).End(xlUp))
        Msg = "Range: " & rng.Address & vbCr
        Msg = Msg & "Visible Range: " & rng.SpecialCells(xlCellTypeVisible).Address & vbCr
        Msg = Msg & "mDLR: " & mDLR
        MsgBox Msg, , "Debug"
        
        mD.Range("H2:H" & mDLR).SpecialCells(xlCellTypeVisible).Value = "Denied"
    End If
 
Upvote 0
I ran your code on some cobbled up filtered data and I'm not seeing that behavior. All is as expected, even with the last row at 2. Suggest you add debug code to let you see if the range addresses are as you expect.
VBA Code:
    Dim rng As Range
    Dim Msg As String
    If mDLR > 1 Then
        Set rng = mD.Range("H2", mD.Range("H" & mD.Rows.Count).End(xlUp))
        Msg = "Range: " & rng.Address & vbCr
        Msg = Msg & "Visible Range: " & rng.SpecialCells(xlCellTypeVisible).Address & vbCr
        Msg = Msg & "mDLR: " & mDLR
        MsgBox Msg, , "Debug"
       
        mD.Range("H2:H" & mDLR).SpecialCells(xlCellTypeVisible).Value = "Denied"
    End If
This is the result of the snippet you provided.
1708172695160.png
 
Upvote 0
This is the result of the snippet you provided.
View attachment 107015
If I sort the data so that the cells that will be updated are not in row 2, the code works fine. It seems to only fail if the cell being updated is in row 2. Obviously, I don't want to always have to sort data just to filter it, so is there some sort of solution involving row2?
 
Upvote 0
You could modify your code so it tests for a row 2 scenario.
VBA Code:
    Dim rng As Range
    Dim rngVisible As Range
    
    If mDLR > 1 Then
        Set rng = mD.Range("H2", mD.Range("H" & mD.Rows.Count).End(xlUp))
        Set rngVisible = rng.SpecialCells(xlCellTypeVisible)
        
        'Special case test
        If rng.Address(0, 0) <> "H1:H2" Then
            If rng.Rows.Count = 1 Then
                Set rngVisible = rng 'Special case
            End If
            rngVisible.Value = "Denied"
        Else
            'null filter result, do nothing
        End If
    End If
 
Upvote 0
You could modify your code so it tests for a row 2 scenario.
VBA Code:
    Dim rng As Range
    Dim rngVisible As Range
   
    If mDLR > 1 Then
        Set rng = mD.Range("H2", mD.Range("H" & mD.Rows.Count).End(xlUp))
        Set rngVisible = rng.SpecialCells(xlCellTypeVisible)
       
        'Special case test
        If rng.Address(0, 0) <> "H1:H2" Then
            If rng.Rows.Count = 1 Then
                Set rngVisible = rng 'Special case
            End If
            rngVisible.Value = "Denied"
        Else
            'null filter result, do nothing
        End If
    End If
I do a lot of filtering. I'm not keen on having to add this much code every time I filter. I appreciate the help!!!
 
Upvote 0
Not sure how you made the jump from what I posted to "every time I filter". This is to solve the particular problem you posted.
 
Upvote 0

Forum statistics

Threads
1,215,110
Messages
6,123,146
Members
449,098
Latest member
Doanvanhieu

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top