how to attach a worksheet and email it

[dmcgov]

i have this test code that is working as expected

Please Login or Register  to view this content.

so it picks a sheet based on the target.row, saves it and then attaches it to an email and it sends. everything so far is good.

then i tried to put it in my other spreadsheet and it doesn't work. it fails on the attachment line. what am i doing wrong?

Please Login or Register  to view this content.

here is the line it fails on:

Please Login or Register  to view this content.

[davesexcel]

Does the msgbox work?

[dmcgov]

yes it displays the proper filename and extension.

[dmcgov]

thoughts anyone?, still haven't been able to get this fixed.

[davesexcel]

What is the error?

[dmcgov]

RunTime Error 287 - application-defined or object-defined error

[davesexcel]

I can't see where you defined x

[dmcgov]

Please Login or Register  to view this content.

when i msgbox x, it tells me the correct filename and extension.

[davesexcel]

I am not sure, if your "y" works, but your "x" doesn't then there must be something missing. The msgbox box is just displaying a string, the attachment has to be something that exists.

You haven't shown the complete code, so we can't see where you dim'd the variables.

[dmcgov]

@Dave

I can verify that the file is getting saved properly. in my sample test case, i have it working just fine. just can't get it to work in my other macro. how can i debug this?

should i upload the sample and the working workbooks for you to take a look at?

[davesexcel]

Sure send a sample workbook

[dmcgov]

so now my test wb is failing on the same line. so i'll just upload the sample workbook.

[davesexcel]

Please Login or Register  to view this content.

is not part of the code, you have it commented out,

Please Login or Register  to view this content.

-errors because it is not a filename.

[dmcgov]

yup, i didn't check well enough so thanks for picking that up. so i went back to my test workbook and figured out how to make it work again. so below is what i have. the only thing that is missing is that i want it to email the selected row and attach a spreadsheet. what it is doing now is emailing the sheet and attaching the sheet, not what i want. can you suggest how to make this work like i want?

Please Login or Register  to view this content.

i tried this line but it error'd out

Please Login or Register  to view this content.

[dmcgov]

thoughts anyone?

[davesexcel]

I don't know, your code is too busy, attached is a simple example using this code.

Please Login or Register  to view this content.

[dmcgov]

yup, i tried your code and indeed it copies the row. but when i put in the attachment code, then its back to copying the worksheet. i'll keep trying to modify the code. thanks for your help.