Formate date when transfering from worksheet to worksheet

ipbr21054

Well-known Member
Joined
Nov 16, 2010
Messages
5,832
Office Version
  1. 2007
Platform
  1. Windows
I use the code below
In the array we are looking at M,E of which are dates.

In column M the date example is 05/07/2017 but when i transfer to another worksheet i see 42921 & its formated as GENERAL.
Please advise how i get to see it as DATE & 05/07/2017 dd/mm/yyyy as opposed 42921 GENERAL

Or do i need to make this happen on the worksheet its going to ?

Thanks

Rich (BB code):
Private Sub Kdx2_Click()
    
    Dim WB As Workbook, DestWB As Workbook
    Dim ws As Worksheet, DestWS As Worksheet
    Dim rng As Range, rngDest As Range
    Dim ColArr As Variant, SCol As Variant, DCol As Variant
    Dim r As Long
    Dim x

    Dim answer As Integer
    answer = MsgBox("DO YOU WISH TO TRANSFER THESE KDX2 FILES ?", vbInformation + vbYesNo, "KDX2 TRANSFER MESSAGE")
    If answer = vbNo Then
    Exit Sub
    Else
    
    If ActiveCell.Column > 1 Then Exit Sub
    
'   Grab row number of active row
    r = ActiveCell.Row

    On Error Resume Next
    Set DestWB = Application.Workbooks("CLONING-KDX2.xlsm")

    If DestWB Is Nothing Then
        Workbooks.Open fileName:="C:\Users\Ian\Desktop\REMOTES ETC\DR\EXCEL WORKSHEETS\CLONING-KDX2.xlsm"
        Set DestWB = Application.Workbooks("CLONING-KDX2.xlsm")
    End If
    On Error GoTo 0

    Set WB = ThisWorkbook
    On Error Resume Next
    Set ws = WB.Worksheets("DATABASE")
    On Error GoTo 0
    If ws Is Nothing Then
        MsgBox "Worksheet 'DATABASE' IS MISSING"
        Exit Sub
    End If
 
    Set DestWS = DestWB.Worksheets("KDX2LIST")
    ColArr = Array("A:A", "D:B", "G:C", "N:D", "M:E", "L:F", "I:G")
    
    Dim DestNextRow As Long
    With DestWS
        DestNextRow = .Range("A" & .Rows.Count).End(xlUp).Row + 1
        
    End With

    Application.ScreenUpdating = False
    For Each SCol In ColArr
        DCol = Split(SCol, ":")(1)
        SCol = Split(SCol, ":")(0)
        With ws
            Set rng = .Cells(r, SCol)
        End With

        With DestWS
            Set rngDest = .Range(DCol & DestNextRow)
        End With
        rng.Copy
        rngDest.PasteSpecial PASTE:=xlPasteValues
        
        rngDest.Borders.Weight = xlThin
        rngDest.Font.Size = 16
        rngDest.Font.Bold = True
        rngDest.HorizontalAlignment = xlCenter
        rngDest.Cells.Interior.ColorIndex = 6
        rngDest.Cells.RowHeight = 25
    Next SCol
    Application.ScreenUpdating = True
     With Sheets("KDX2LIST")
    If .AutoFilterMode Then .AutoFilterMode = False
    x = .Cells(.Rows.Count, 1).End(xlUp).Row
    .Range("A3:G" & x).Sort Key1:=.Range("A3"), Order1:=xlAscending, Header:=xlYes, DataOption1:=xlSortTextAsNumbers
    .Range("A1").Select
    ActiveWorkbook.Close savechanges:=True
  End With
  End If
End Sub
 

Excel Facts

Return population for a City
If you have a list of cities in A2:A100, use Data, Geography. Then =A2.Population and copy down.
I believe that when you copy and paste a date as a value, it may appear as a serial number (e.g., 42921) in the destination cell, and the cell format might be set to GENERAL by default.

Option 1: Insert value directly instead of copy & paste as values.

VBA Code:
    With DestWS
        Set rngDest = .Range(DCol & DestNextRow)
    End With

    rngDest.Value = rng.Value

Option 2: Reformatting after paste
VBA Code:
    If SCol = "M" Then
        rngDest.NumberFormat = "dd/mm/yyyy"
    End If
 
Upvote 0
VBA Code:
' ...
For Each SCol In ColArr
    DCol = Split(SCol, ":")(1)
    SCol = Split(SCol, ":")(0)
    With ws
        Set rng = .Cells(r, SCol)
    End With

    With DestWS
        Set rngDest = .Range(DCol & DestNextRow)
    End With
    rng.Copy
    rngDest.PasteSpecial Paste:=xlPasteValues

   'Option 2
    If SCol = "M" Then
        rngDest.NumberFormat = "dd/mm/yyyy"
    End If


    rngDest.Borders.Weight = xlThin
    rngDest.Font.Size = 16
    rngDest.Font.Bold = True
    rngDest.HorizontalAlignment = xlCenter
    rngDest.Cells.Interior.ColorIndex = 6
    rngDest.Cells.RowHeight = 25
Next SCol
' ...
 
Upvote 0
Solution

Forum statistics

Threads
1,224,818
Messages
6,181,152
Members
453,021
Latest member
Justyna P

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