How do I get rid of the message box telling me I am going to append a record?

The database I am working on is in Access 2007. I want the user to be able to add activities that he/she has performed during the week. The code below will add the activities to either one or two lists, depending on if the activity has been added previously.

Right now, for ever record that is being added to either list, I get a message box telling me I am about to append a record to the table. Is there a way to get rid of this? I don't want my users to have to click "Ok" 15 times when they fill out the form.

Thank you!





Private Sub btnAdd_Click()


Dim strSQL As String
Dim db As Database
Dim rst As DAO.Recordset
Dim LngItem As Long

Set db = CurrentDb()




Dim rowc As Integer


With Me.LstNewAct
  For rowc = 0 To .ListCount - 1
    strSQL = "SELECT * FROM tbl_Activities WHERE actName='" & .Column(0, rowc) & "'"
    
    Set rst = db.OpenRecordset(strSQL)
    If (rst.BOF And rst.EOF) Then ' There are no records if Beginning-Of-File and End-Of-File are both true.
      DoCmd.RunSQL "INSERT INTO tbl_Activities (actName) VALUES ('" & .Column(0, rowc) & "')"
      rst.Close
    End If
    strSQL = "SELECT actID FROM tbl_Activities WHERE actName='" & .Column(0, rowc) & "'"
    Set rst = db.OpenRecordset(strSQL)
    DoCmd.RunSQL "INSERT INTO tbl_ActCmb1 (toID, stoID, actID, actDate, actType) VALUES (cmbTO, cmbsto, '" & rst.Fields(0).Value & "', WkDate, actType1)"
  Next rowc
End With



With Me.LstAct
  For Each varItem In .ItemsSelected
    LngItem = .Column(0, varItem)
    strSQL = "INSERT INTO tbl_ActCmb1 (toID, stoID, actID, actDate, actType) VALUES (cmbTO, cmbsto, " & LngItem & ", WkDate, actType1)"
    DoCmd.RunSQL strSQL, dbFailOnError
    
  Next varItem
End With



End Sub

Open in new window





Any and all help is greatly appretiated!!!!!
MeginAsked:
Who is Participating?
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

peter57rCommented:
Use DoCmd.Execute strsql, dbfailonerror   ( instead of RunSql)
0
Dale FyeCommented:
I agree with Peter.

The best technique is to use the execute method, and don't forget to add the dbFailOnError argument, which will raise an error if something in your SQL query generates an error.  You will also need to add an error handler to address possible errors.
0
Rey Obrero (Capricorn1)Commented:
i suggest that you use

currentdb.execute strSQL, dbFailOnError
0
Ultimate Tool Kit for Technology Solution Provider

Broken down into practical pointers and step-by-step instructions, the IT Service Excellence Tool Kit delivers expert advice for technology solution providers. Get your free copy now.

Rey Obrero (Capricorn1)Commented:
i suggest that you use

currentdb.execute strSQL, dbFailOnError


or

db.execute strSQL, dbFailOnError
0
Jeffrey CoachmanMIS LiasonCommented:
<No Points wanted>

 what the Experts above have posted is the most reliable method...

To be clear, there is a setting that does what you are asking for directly, but "managing" this setting can become confusing.

;-)

JeffCoachman
0
MeginAuthor Commented:
I have added that piece of code, but now it is coming up with errors and highlighting that piece of code when I try to run it.

The message I get is "The expression On Click you entered as the event property setting produced the following error: Method or data member not found"

?????
0
Jeffrey CoachmanMIS LiasonCommented:
Post the exact, full code you tried please...
0
MeginAuthor Commented:
It is in my original post, but I will post it again (I put the code that gives me the append record notification back in because, while it is annoying, it doesn't give me an error message):

Private Sub btnAdd_Click()


Dim strSQL As String
Dim db As Database
Dim rst As DAO.Recordset
Dim LngItem As Long

Set db = CurrentDb()




Dim rowc As Integer


With Me.LstNewAct
  For rowc = 0 To .ListCount - 1
    strSQL = "SELECT * FROM tbl_Activities WHERE actName='" & .Column(0, rowc) & "'"
    
    Set rst = db.OpenRecordset(strSQL)
    If (rst.BOF And rst.EOF) Then ' There are no records if Beginning-Of-File and End-Of-File are both true.
      DoCmd.RunSQL "INSERT INTO tbl_Activities (actName) VALUES ('" & .Column(0, rowc) & "')"
      rst.Close
    End If
    strSQL = "SELECT actID FROM tbl_Activities WHERE actName='" & .Column(0, rowc) & "'"
    Set rst = db.OpenRecordset(strSQL)
    DoCmd.RunSQL "INSERT INTO tbl_ActCmb1 (toID, stoID, actID, actDate, actType) VALUES (cmbTO, cmbsto, '" & rst.Fields(0).Value & "', WkDate, actType1)"
  Next rowc
End With



With Me.LstAct
  For Each varItem In .ItemsSelected
    LngItem = .Column(0, varItem)
    strSQL = "INSERT INTO tbl_ActCmb1 (toID, stoID, actID, actDate, actType) VALUES (cmbTO, cmbsto, " & LngItem & ", WkDate, actType1)"
    DoCmd.RunSQL strSQL, dbFailOnError
    
  Next varItem
End With



End Sub

Open in new window

0
Dale FyeCommented:
use either:

currentdb.execute strSQL, dbFailOnError

or

DoCmd.Execute strsql, dbfailonerror

Or, you could declare a variable for the datbase object and instantiate it:

dim db as DAO.database
set db = currentDb()

But if you do this, you should also set it to nothing at the end of the code module, to clean-up after yourself

set db = nothing

If you have a bunch of queries to execute in the same code module, adding the db object and declaration will probably speed up your code some.
0
Dale FyeCommented:
Change your declaration statement to:

Dim db as DAO.Database
0
MeginAuthor Commented:
I made the changes, but now I am getting the error message "Run-time error '3061': Too Few parameters. Expected 4"

Here is what I now have for code. I am still getting one 'append' message when I try to update the report. Then I get the error message.

I am sure this is just me making the wrong changes. I am really, really new to this. Thank you for all of the help!

Private Sub btnAdd_Click()


Dim strSQL As String
Dim db As DAO.Database
Dim rst As DAO.Recordset
Dim LngItem As Long

Set db = CurrentDb()




Dim rowc As Integer


With Me.LstNewAct
  For rowc = 0 To .ListCount - 1
    strSQL = "SELECT * FROM tbl_Activities WHERE actName='" & .Column(0, rowc) & "'"
    
    Set rst = db.OpenRecordset(strSQL)
    If (rst.BOF And rst.EOF) Then ' There are no records if Beginning-Of-File and End-Of-File are both true.
      DoCmd.RunSQL "INSERT INTO tbl_Activities (actName) VALUES ('" & .Column(0, rowc) & "')"
      rst.Close
    End If
    strSQL = "SELECT actID FROM tbl_Activities WHERE actName='" & .Column(0, rowc) & "'"
    Set rst = db.OpenRecordset(strSQL)
    DoCmd.RunSQL "INSERT INTO tbl_ActCmb1 (toID, stoID, actID, actDate, actType) VALUES (cmbTO, cmbsto, '" & rst.Fields(0).Value & "', WkDate, actType1)"
  Next rowc
End With



With Me.LstAct
  For Each varItem In .ItemsSelected
    LngItem = .Column(0, varItem)
    strSQL = "INSERT INTO tbl_ActCmb1 (toID, stoID, actID, actDate, actType) VALUES (cmbTO, cmbsto, " & LngItem & ", WkDate, actType1)"
    CurrentDb.Execute strSQL, dbFailOnError

  Set db = Nothing
  Next varItem
End With



End Sub

Open in new window

0
MeginAuthor Commented:
This was solved by using the code:

DoCmd.SetWarnings False at the beginning and
DoCmd.SetWarnings True at the end.

Thank you to everyone who took the time to help me out!
0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
Dale FyeCommented:
Megin,

I would recommend against using SetWarnings False and True unless you have really good error handling.  If you don't you could exit the code module without reseting it to true, and then you would see NO warnings whatsoever, including when you are debugging your code or writing queries...

Jeff's post

"To be clear, there is a setting that does what you are asking for directly, but "managing" this setting can become confusing."

is a vague indicator of the existence of this (SetWarnings) technique, but most of the experts recommend against using it.
0
MeginAuthor Commented:
I haven't been able to make anything else work. Can you tell me what is wrong with it now that the codes I have been trying aren't working? I am sure I don't have really good error handling.
0
MeginAuthor Commented:
Also, thank you for the warning!
0
Dale FyeCommented:
Megan,

Is the code you posted just above your latest code?
0
MeginAuthor Commented:
Yes, with the exception of the DoCmd.SetWarnings False.
0
MeginAuthor Commented:
I received help from a co-worker and, now that the question was answered, I didn't want it sitting here open.
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
Microsoft Access

From novice to tech pro — start learning today.

Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.