Check Box selected to carry out multiple functions: prin a couple of forms and update fields

[neill_long]

I have an option on my form that the user ticks a check box if something has been sent to our finance department. Just now there is an event procedure on this check box as follows:





I have a button that will print out a couple of reports, which there is an embedded macro attached to it, as follows:



What I would like to try to achieve is get rid of the button and when the user selects the check box, it will:

1. Add the date to the Date_to_Finance field
2. Print the reports, as shown in the picture above.

I have replicated the macro and saved it as FinanceForms. In the event procedure I added:

Private Sub Check36_Click()
DoCmd.RunMacro "FinanceForms"
End Sub

But the form then displayed errors in some fields:

#Name?


I'm guessing what I did is confusing the database and that's why I get the errors. Am I doing something wrong or can someone suggest a different approach, possibly adding code that will do what I am wanting it to do instead of using a macro?

thanks
Alistair

[ranman256]

You want all fields in the query,
dont use references to form fields in the report,
use them in the query, the report looks at query fields,to stop #name.

[pbaldy]

The code to replicate the macro would use

DoCmd.OpenReport

with the appropriate arguments. Add that to your code.

[neill_long]

I have kind of got this working, thanks for the responses. It is now printing the reports when I click the checkbox. However it is also printing a copy of the form the checkbox is on. Here is what I have on my code, how would I amend it so that it only prints the report?

Private Sub Check36_Click()
If Me.Check36 = True Then
        Me.Refresh
    Me.Date_to_Finance = Date
    DoCmd.OpenReport "Sales Invoice Request", acViewNormal
    DoCmd.PrintOut , , , , 1
    DoCmd.OpenReport "Finance Duplicate", acViewNormal
    DoCmd.PrintOut , , , , 1
   
 Else
        Me.Date_to_Finance = Null
    End If
End Sub

thanks
Alistair

[pbaldy]

Get rid of the printout lines.