Excel VAB Refuses to Sort Ascending

Shiseiji

Board Regular
Joined
Oct 23, 2009
Messages
214
Office Version
  1. 2019
Platform
  1. Windows
If I sort the column manually, accept text as number, no problem. But the code below refuses to sort the column. What am I missing???
I changed the sort range from a named range to bypass the header in my efforts to get this to run . . .

TIA!!!

Ron
NB: Had troubles with filters, so left in both possibilities.

Code:
Sub m_AD_Sort_ADATE()
'
Application.Volatile True
    Application.DisplayAlerts = False
    Application.ScreenUpdating = False
    Application.DisplayStatusBar = True
    '
    Sheets("AD").Select
    ActiveWindow.DisplayGridlines = True
    ActiveWindow.FreezePanes = False
    '
    Dim LastCol                 As Integer
    Dim LastRow                 As Long
    Dim Thiswb                  As Workbook
    Dim Thiswksht               As Worksheet
    Set thisws = ActiveSheet
    Set Thiswb = ActiveWorkbook
    '
    With Thiswksht
        LastRow = Cells(Cells.Rows.Count, "A").End(xlUp).Row
        LastCol = Cells(1, Columns.Count).End(xlToLeft).Column
    End With
    '
    If ActiveSheet.FilterMode Then
       ActiveSheet.ShowAllData
    End If
    '
    If ActiveSheet.FilterMode = True Then
      ActiveSheet.FilterMode = False
    End If
    '
    ActiveWindow.FreezePanes = False
    '
    thisws.Sort.SortFields.Clear
    '
    Range("c_AD_ADATE").Select
    '
    With Selection
        .HorizontalAlignment = xlRight
        .VerticalAlignment = xlCenter
        .NumberFormat = "dd/mm/yyyy"
    End With
    '
    Cells(2, ActiveCell.Column).Select
    Range(ActiveCell.Address, Cells(LastRow, ActiveCell.Column)).Select
    Selection.Sort Key1:=Range("c_AD_ADATE"), Order1:=xlAscending, DataOption1:=xlSortTextAsNumbers
    '
    With ActiveWorkbook.Worksheets("AD").Sort
        .SetRange Range("tbl_AD")
        .Header = xlYes
        .MatchCase = False
        .Orientation = xlTopToBottom
        .SortMethod = xlPinYin
        .Apply
    End With
    '
    ActiveWorkbook.Worksheets("AD").Sort.SortFields.Clear
    Cells(1, ActiveCell.Column).Select
    AutoFilter = False
    '
    'Cells(1, 1).Select
    'ActiveCell.Offset(1, 0).Select
    'Cells(1, 1).Select
    '
End Sub
 

Excel Facts

Does the VLOOKUP table have to be sorted?
No! when you are using an exact match, the VLOOKUP table can be in any order. Best-selling items at the top is actually the best.
I think you need to change this line:
Code:
Selection.Sort Key1:=Range("c_AD_ADATE"), Order1:=xlAscending, DataOption1:=xlSortTextAsNumbers
TO
Code:
Activesheet.Sort.SortFields.Add Key:=Range("c_AD_ADATE"), Order:=xlAscending, DataOption:=xlSortTextAsNumbers
 
Upvote 0
Thanks Jeffrey, sadly no joy. As soon as I can leave Virginia, Washington is where we're headed!

R
 
Upvote 0
Had an idea. I resorted and told Excel to sort the text and dates separately. 543 (what a number LOL) of my "dates" were text. Some of the date entries are a result of not finding a date and rather then have null cells, I enter a date outside of my range so they are always not counted. That's probably why I was seeing them on both ends of my data set. Still not sure why Excel wouldn't treat the text as a date . . .

The original data has to be converted using Text-to-Columns and who knows why these 543 cells are different. Something else to figure out. At least it's a clue. In the interest of time-management, as Excel refuses to sort the text as numbers, I think I'll try an IFERROR with DATEVALUE to see if I can clean these up so they sort properly.

Of course I'm always open to suggestions! I'm a code hack and freely admit it.
Thanks again

Ron

R
 
Upvote 0
Solution

Forum statistics

Threads
1,223,984
Messages
6,175,786
Members
452,669
Latest member
reeseann

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