>FYI the code was working fine when i had outlook 2003
>also just found out that if outlook 2012 was running the error is no more, noting that this was not the case with 2003

BTW, unrelated observation - but you don't need to use macro for the attachment. You can use name variable in your code.

Also, I agree with Tamar - try using Send as a method.