It nearly works!

Tarqs

Board Regular
Joined
Feb 14, 2011
Messages
120
Hi all,

I had a similar piece of code created for me, however I now need to amend for a new requirment.

I've got it to a point and it nearly works but it only matches and copies for the first entry that it finds and not all entries?? (assumes it is unique)

code is

Code:
Option Explicit
Sub ReportMatcher()
    Dim wsDest As Worksheet
    Dim wsSource As Worksheet
    Dim rngDest As Range
    Dim rngSource As Range
    Dim rng As Range
    Dim rowMatch As Variant
 
    Set wsDest = ActiveSheet
 
    If Not Application.Dialogs(xlDialogActivate).Show Then
        Exit Sub
    End If
 
    Application.ScreenUpdating = False
 
    Set wsSource = ActiveSheet
 
    With wsDest
        Set rngDest = .Range("D:D")
    End With
 
    With wsSource
        Set rngSource = .Range("A2", .Cells(.Rows.Count, 1).End(xlUp))
    End With
 
    For Each rng In rngSource.Cells
        If Len(rng.Value) > 0 Then
            rowMatch = Application.Match(rng.Value, rngDest, 0)
 
            If IsNumeric(rowMatch) Then
 
                wsDest.Range("T" & rowMatch).Value = wsSource.Range("B" & rng.Row).Value
                wsDest.Range("U" & rowMatch).Value = wsSource.Range("D" & rng.Row).Value
            End If
        End If
    Next rng
    Application.ScreenUpdating = False
 
    MsgBox "Order Numbers Added", vbInformation
End Sub

Example, I have a list of report numbers. The list could have multiple entries for the same number. I need it paste the data against all the entries in the list and not just the first entry which it is currently doing.

Help greatfully received.

Tarqs
 

Excel Facts

Format cells as date
Select range and press Ctrl+Shift+3 to format cells as date. (Shift 3 is the # sign which sort of looks like a small calendar).

Forum statistics

Threads
1,224,598
Messages
6,179,814
Members
452,945
Latest member
Bib195

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