Excel email macro failing on sheet array

Hello, I am trying to adapt a macro from Ron DeBruins website to email an array of sheets but it is failing when it is trying to determine which sheets to send. The error message says "Run-time error '9': Subscript out of range" A second question I have is how do I change this to only email the visible sheets?

This is the line it is failing on:

.Sheets(Array("Supplier Instruction Sheet", "R&D Request-Protein", "Sample Analysis", "Sample Arrival Summary", "Protein R&D Formula Pricing", "Protein-Prelim Spec")).Copy

The whole macro:

Sub Prot_Mail_Sheets_Array()
'Working in 2000-2010
    Dim FileExtStr As String
    Dim FileFormatNum As Long
    Dim Sourcewb As Workbook
    Dim Destwb As Workbook
    Dim TempFilePath As String
    Dim TempFileName As String
    Dim OutApp As Object
    Dim OutMail As Object
    Dim sh As Worksheet
    Dim TheActiveWindow As Window
    Dim TempWindow As Window
    Dim cell As Range
    Dim strto As String
    
    With Application
        .ScreenUpdating = False
        .EnableEvents = False
    End With

    Set Sourcewb = ActiveWorkbook

    'Copy the sheets to a new workbook
    'We add a temporary Window to avoid the Copy problem
    'if there is a List or Table in one of the sheets and
    'if the sheets are grouped
    With Sourcewb
        Set TheActiveWindow = ActiveWindow
        Set TempWindow = .NewWindow
        .Sheets(Array("Supplier Instruction Sheet", "R&D Request-Protein", "Sample Analysis", "Sample Arrival Summary", "Protein R&D Formula Pricing", "Protein-Prelim Spec")).Copy
    End With

    'Close temporary Window
    TempWindow.Close

    Set Destwb = ActiveWorkbook

    'Determine the Excel version and file extension/format
    With Destwb
        If Val(Application.Version) < 12 Then
            'You use Excel 97-2003
            FileExtStr = ".xls": FileFormatNum = -4143
        Else
            'You use Excel 2007-2010, we exit the sub when your answer is
            'NO in the security dialog that you only see  when you copy
            'an sheet from a xlsm file with macro's disabled.
            If Sourcewb.Name = .Name Then
                With Application
                    .ScreenUpdating = True
                    .EnableEvents = True
                End With
                MsgBox "Your answer is NO in the security dialog"
                Exit Sub
            Else
                Select Case Sourcewb.FileFormat
                Case 51: FileExtStr = ".xlsx": FileFormatNum = 51
                Case 52:
                    If .HasVBProject Then
                        FileExtStr = ".xlsm": FileFormatNum = 52
                    Else
                        FileExtStr = ".xlsx": FileFormatNum = 51
                    End If
                Case 56: FileExtStr = ".xls": FileFormatNum = 56
                Case Else: FileExtStr = ".xlsb": FileFormatNum = 50
                End Select
            End If
        End If
    End With

    '    'Change all cells in the worksheets to values if you want
       For Each sh In Destwb.Worksheets
            sh.Select
            With sh.UsedRange
                .Cells.Copy
                .Cells.PasteSpecial xlPasteValues
                .Cells(1).Select
            End With
            Application.CutCopyMode = False
            Destwb.Worksheets(1).Select
        Next sh


    'Save the new workbook/Mail it/Delete it
    TempFilePath = Environ$("temp") & "\"
    TempFileName = "Part of " & Sourcewb.Name & " " _
                 & Format(Now, "dd-mmm-yy h-mm-ss")

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

    With Destwb
        .SaveAs TempFilePath & TempFileName & FileExtStr, _
                FileFormat:=FileFormatNum
        On Error Resume Next
        For Each cell In ThisWorkbook.Sheets("Ranges").Range("J3:L3")
        If cell.Value Like "?*@?*.?*" Then
            strto = strto & cell.Value & ";"
        End If
    Next cell
    If Len(strto) > 0 Then strto = Left(strto, Len(strto) - 1)
        With OutMail
            .To = strto
            .CC = ""
            .BCC = ""
            .Subject = "This is the Subject line"
            .Body = "Hi there"
            .Attachments.Add Destwb.FullName
            'You can add other files also like this
            '.Attachments.Add ("C:\test.txt")
            .display   'or use .Display
        End With
        On Error GoTo 0
        .Close SaveChanges:=False
    End With

    'Delete the file you have send
    Kill TempFilePath & TempFileName & FileExtStr

    Set OutMail = Nothing
    Set OutApp = Nothing

    With Application
        .ScreenUpdating = True
        .EnableEvents = True
    End With
End Sub

Thanks for any guidance you can give!

Clayton Grove

Leith, thank you for the assistance! I was hoping you would stumble into my thread! I ran your macro but it failed with an error that says "Compile Error: Invalid qualifier" on the section below:

 With Sourcewb
        Set TheActiveWindow = ActiveWindow
        Set TempWindow = .NewWindow
          For Each Sht In .Sheets
            If Sht.Visible = True Then
               ReDim Preserve Shts(N)
               Shts(N) = Sht.Name
               N = N + 1
            End If
          Next Sht
        Shts.Copy
    End With

On this line:

      Shts.Copy

Any ideas what is causing this?

Thanks again!

Clayton Grove

No worries! I changed that line of code and now is give me an error that says "method 'copy'of object 'Sheets' failed" on that same line.

.Sheets(Array(Shts)).Copy