Send Email not running

[Eclektics]

Hi everyone,

I'm at a bit of a loss with this one as the code I'm using I've used before without issue. My goal is to have Excel move a worksheet to a new workbook, save that file and send it and then kill the file. All the code is working apart from the code to send an email, when I've took out of this code from my main module and ran it independently and nothing happens. Any help will be massively appreciated.

 Sub System_Issue_Mail()

    Dim OutApp As Object
    Dim OutMail As Object
    Dim strbody As String


    Set OutApp = CreateObject("Outlook.Application")
    Set OutMail = OutApp.CreateItem(0)


    strbody = "Hi" & vbNewLine & vbNewLine & _
              "Please find attached a new archived certificate request." & vbNewLine & _
              "Thanks"


    On Error Resume Next
    With OutMail
        .To = "System.support@.org.uk"
        .CC = ""
        .BCC = ""
        .Subject = "A New System Issue has been raised by user" & usr
        .Body = ""
        .Attachments.Add ("C:\Users\" & usr & "\Documents\TEMPFILETBR.xlsx")
        .Display   '.Send
    End With
    On Error GoTo 0


    Set OutMail = Nothing
    Set OutApp = Nothing
    
End Sub

 

[Eclektics]

Update: The code will send the email if I use .send but it's not including the attachment?

 

[KOKOSEK]

Hi

I suggest to check first that file exist, if exists, attached, if not, sent without attachment.

 Sub System_Issue_Mail()


    Dim OutApp As Object
    Dim OutMail As Object
    Dim strbody As String
[U][B]    Dim AttFile As String[/B][/U]
[U][B]    AttFile = "C:\Users\" & usr & "\Documents\TEMPFILETBR.xlsx"[/B][/U]


    Set OutApp = CreateObject("Outlook.Application")
    Set OutMail = OutApp.CreateItem(0)




    strbody = "Hi" & vbNewLine & vbNewLine & _
              "Please find attached a new archived certificate request." & vbNewLine & _
              "Thanks"




    On Error Resume Next
    With OutMail
        .To = "System.support@.org.uk"
        .CC = ""
        .BCC = ""
        .Subject = "A New System Issue has been raised by user" & usr
        .Body = ""
[U][B]        If Len(Dir(AttFile)) <> 0 Then[/B][/U]
[U][B]            .Attachments.Add AttFile[/B][/U]
[U][B]        End If[/B][/U]
        .Display   '.Send
    End With
    On Error GoTo 0




    Set OutMail = Nothing
    Set OutApp = Nothing
    
End Sub

 

[Eclektics]

Thanks KOKOSEK, the code is working when sending an email now. The issue only happens when I ask excel to display the email rather than send, rather than showing the email it does nothing.

 

[RoryA]

Try commenting out the On Error Resume Next so that you can see what's actually happening.

 

[KOKOSEK]

Thanks KOKOSEK, the code is working when sending an email now. The issue only happens when I ask excel to display the email rather than send, rather than showing the email it does nothing.

As you see in my code I've displayed email and i don't have any problems.
If file exists is attached, if file does not exists, email is shown without attachment.