SendObject Loop does not loop through all records!!

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

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.Recordsetclone 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

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.Recordsetclone 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?

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).

Expand|Select|Wrap|Line Numbers

1. Do Until rst.EOF
2. ...
3. strSendTo = Me.Suppliers_EmailAddress
4. ...
5. strMessageText = "To: " & Me.SupplierName & vbCrLf _
6. ... 
7. rst.MoveNext
8. Loop
9.  

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

-Stewart

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_EmailAddress] 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

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_emailaddress]

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

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

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

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):
-Stewart

Thank you for you help.

I will post my code below as a finished product for this Sub.

Thank you again for all of your help. The forum is fantastic!!!