On 9 Nov, 11:40, Joel wrote:
I'm running with excel 2003 and do not get any errors either with the
original code or my modification. I 've tried making some changes to the
worksheets to see if I get the errror and I'm not able to duplicate this
error with your code.

Can you tell me which line is failing?



"Jeegna" wrote:
On 8 Nov, 18:44, Joel wrote:
change the following

from:
r.Copy rngTo(i)
to:
r.Copy destination:=rngTo(i)

Or if it is only one cell

rngTo(i).value = r.value

" wrote:
Hi,

Very new to this so would be grateful for any help.

I'm trying to copy rows from one workbook to another when they meet a
certain criteria. But I do not want to overwrite previously copied
lines. I want to add new rows, that meet the criteria, to the end of
the older copied data.

Dont know whether its relevent but to make sure that lines that are
already on sheet 6 aren't duplicated, I have a lookup function in col
23 of sheet 1 that brings back the unique identifier if it is already
on sheet 6.

I know this is very convaluted!

Sub UnmatchedPOs()

Set rng = Sheets(1).Range("A:V").Rows

Set rngTo = Sheets(6).Range("A1").End(xlDown)

For Each r In rng
If r.Cells(3).Value = "NO" Then
If r.Cells(9).Value = DateValue("April, 01, 2007") Then
If r.Cells(9).Value < DateValue("April 01, 2008") Then
If Not r.Cells(6).Value = r.Cells(23).Value Then

i = i + 1
r.Copy rngTo(i)

End If
End If
End If
End If
Next

End Sub

When I run this I get:

1004 - Application defined or object defined error

Many Thanks,

Jig- Hide quoted text -

- Show quoted text -

Hi Joel,

I still got the above error, with the text you said to put in.

Many Thanks,

Jig- Hide quoted text -

- Show quoted text -

I'm running Excel 2000 so could that be it?

When I hit debug, the line that is highlighted is the one you asked me
to change. It was that line before I changed it as well.

Cheers,