SendObject Loop does not loop through all records!!

[nspader]

I am having trouble with my loop code. The code works very well. However, it only loops through 3 records and then completes without errors. I will post code below. Any help with this would be greatly appreciated.

I am using Windows 2000 with Access 2000, sending email through Outlook 2000.

 
Private Sub Form_Load()
On Error GoTo Err_Form_Load
 
    Dim rst As DAO.Recordset
    Dim bkMark As String
    Dim stDocName As String
    Dim strSendTo As String
    Dim strSubject As String
    Dim strMessageText As String
 
    Set rst = Me.Recordset
 
    bkMark = rst.Bookmark
    Me.Bookmark = bkMark
 
    rst.MoveFirst
 
 
    Do Until rst.EOF
 
        stDocName = "Request for Updated PO Info EMAIL"
        strSendTo = Me.Suppliers_EmailAddress
        strSubject = "Wesco Distribution Shipping Update Report"
        strMessageText = "To:  " & Me.SupplierName & vbCrLf _
            & "" & vbCrLf _
            & "Attached is a Shipping Update Report for certain PO numbers." & vbCrLf _
            & "" & vbCrLf _
            & "Please review the attached report and reply back to this email with the requested information." & vbCrLf _
            & "" & vbCrLf _
            & "Thank you," & vbCrLf _
            & "" & vbCrLf _
            & "Wesco Distribution Expediting Department "
 
        DoCmd.SendObject acSendReport, stDocName, acFormatRTF, strSendTo, , , strSubject, strMessageText
 
    rst.MoveNext
 
    Loop
 
    rst.Close
 
    Set rst = Nothing
 
    DoCmd.Close acForm, "Email", acSaveYes
 
Exit_Form_Click:
    Exit Sub
 
Err_Form_Load:
    MsgBox Err.Description
    Resume Exit_Form_Click
 
End Sub
 
 

Please help me figure this out. I am completely boggled by this.

Nick

[nspader]

I should also state that the purpose of this code is to first select the supplier and supplier email from a table (only selecting ones that have emails). Then it is to send an email to each supplier with its own report in it.

As of now it works perfectly for the first three records everytime. It sends the report for that supplier to the correct supplier email.

I just cannot figure out why it ends after three records when I know there are 7 recods in the recordset (via form view).

Please help me to finish this DB. I am so close to finishing this.(last piece to put together before it is a finished product)

Thank you in advance (Again)

Nick

[Stewart Ross]

Hi. At first glance there appear to be no errors in your code. However, at line 11 you are setting the recordset object rst by referring to the recordset property of the form; the norm is to use the recordsetclone method to do so.

Try set rst = Me.Recordsetclo ne instead and let us know whether this resolves the missing records issue.

If it does not resolve the missing records, you will have to check the processing going on within the loop by setting a breakpoint and checking that all records are being processed (using, say, Debug.Print on individual fields within the loop).

-Stewart

[nspader]

Hi. At first glance there appear to be no errors in your code. However, at line 11 you are setting the recordset object rst by referring to the recordset property of the form; the norm is to use the recordsetclone method to do so.

Try set rst = Me.Recordsetclo ne instead and let us know whether this resolves the missing records issue.

If it does not resolve the missing records, you will have to check the processing going on within the loop by setting a breakpoint and checking that all records are being processed (using, say, Debug.Print on individual fields within the loop).

-Stewart

I did try RecordsetClone. ..When I do that it loops on the first Record only.

Also, something I noticed, for multiple suppliers I have the same contact. It seems to stop after repeating the same contact twice in a row...The third file. Any help?

[Stewart Ross]

Hi again. Recordsetclone is the right way to go. You mention that it only sends one record out when you do so - I've reviewed your code again and notice that you are not reffering to the rst recordset object's fields at all - you are referring to the fields on the current record of your form (see extracts below).

Do Until rst.EOF
...
strSendTo = Me.Suppliers_EmailAddress
...
strMessageText = "To: " & Me.SupplierName & vbCrLf _
... 
rst.MoveNext
Loop
 

You should surely be referring to rst![Suppliers_Email Address] and rst![SupplierName], not exclusively to the form's fields if you are processing the underlying recordset within a loop.

-Stewart

[nspader]

Hi again. Recordsetclone is the right way to go. You mention that it only sends one record out when you do so - I've reviewed your code again and notice that you are not reffering to the rst recordset object's fields at all - you are referring to the fields on the current record of your form (see extracts below).
You should surely be referring to rst![Suppliers_Email Address] and rst![SupplierName], not exclusively to the form's fields if you are processing the underlying recordset within a loop.

-Stewart

I placed that into the code and now I get a return saying "Item not Found In This Collection". Any Suggestions?

Nick

[nspader]

I placed that into the code and now I get a return saying "Item not Found In This Collection". Any Suggestions?

Nick

I should have said it is on line srtTo = rst![supplier_emaila ddress]

[Stewart Ross]

I should have said it is on line srtTo = rst![supplier_emaila ddress]

Hi. The error means that the field in the underlying recordset table or query is not called by the name listed, so it cannot be found. Could you check what the field name really is?

If you would prefer your DB to be checked by us directly you could Zip a 'sanitised' version of your DB (no sensitive data in it) and attach it as a post if this would help. To do so, after sending a reply use the Edit facilities and Manage Attachments to add an attachment to your post.

-Stewart

[nspader]

Hi. The error means that the field in the underlying recordset table or query is not called by the name listed, so it cannot be found. Could you check what the field name really is?

If you would prefer your DB to be checked by us directly you could Zip a 'sanitised' version of your DB (no sensitive data in it) and attach it as a post if this would help. To do so, after sending a reply use the Edit facilities and Manage Attachments to add an attachment to your post.

-Stewart

Thank you so much for you help. I did realize that it was not named properly and it works like a charm. I do have one question pertaining to this code.

Is there any code to show a message saying "no email addresses on file for selected branch" When I select a branch from the drop down list and it has no files in it?

Something like if BOF><EOF Then show message "..."

Any help is appreaciated as always.

Nick

[Stewart Ross]

Hi Nick. It's good that it is now working for you!

To check for an empty recordset, output a message and exit from the subroutine enter something like this to do what you want. Place it above the loop (just before the DO UNTIL):

IF rst.EOF then
  MsgBox "There are no e-mail addresses in this set", vbExclamation, "No e-mail addresses"
  Exit SUB
END IF

-Stewart