Messagebox popping up twice?

[82280zx]

I'm setting up one of my forms to make sure the user enters data into the fields and generates a error in a message box if the value is null.

What I have so far is this



If IsNull(Me.txtModel) Then
MsgBox "Please enter the Model Information", vbCritical, "Canceling Update"
Me.txtModel.SetFocus
Cancel = True
End If

So when the user go's to save the new asset if one of the fields that I require (for reports and queries to work correctly) is left blank, warn the user that it was left blank and don't allow the user to continue until something is entered in, the code works but I receive two pop up box's and I should only be seeing one. I'm putting this in BeforeUpdate as well.

[June7]

What event(s) are used? If there is MsgBox code in multiple events and those events run then there will be multiple popups.

[atuhacek]

What are you using to call your function? Before update for your form or a control.

[82280zx]

What are you using to call your function? Before update for your form or a control.

BeforeUpdate on my Form.

[82280zx]

What event(s) are used? If there is MsgBox code in multiple events and those events run then there will be multiple popups.

There are multiple events being used on the form but I only have the MsgBox in the BeforeUpdate event.

[June7]

The popups are identical? Would have to review code. If you want to provide db, follow instructions at bottom of my post.

[atuhacek]

use the after update call. the before update call happens before anything else. it wont matter if the data is entered or not it is going to make the call before the info has had a chance to be entered at all.

[June7]

I disagree with using AfterUpdate. The record needs to be validated before committing to table and while the record still has focus.

[atuhacek]

You might be right but before is before the record has a chance. i normally run a check on after update. Access doesn't "save" or "commit" is commits on entry so using the after update is like using your save or commit. It has worked for me in the past. Put an msg in a before update event. it fires on first action.

Correction! The form before update won't. I was thinking of the control level.

[82280zx]

Yea I see two of the same pop up's, not sure why. I've got to clear out some stuff then I'll upload my db file, it's been a while since I've had it reviewed I'm sure it will have lots of fun stuff I'm going to need to fix .

[Missinglinq]

You probably need to post the entire code for your Form_BeforeUpdate event.

Linq ;0)>

[82280zx]

The popups are identical? Would have to review code. If you want to provide db, follow instructions at bottom of my post.

The file is greater than two meg so I had to put it on my box account but heres my blank db https://app.box.com/s/gxwqtgd4kcuf11fplg30 I hope it's not to scary, I'm still learning.

Thanks!

[82280zx]

The file is greater than two meg so I had to put it on my box account but heres my blank db https://app.box.com/s/gxwqtgd4kcuf11fplg30 I hope it's not to scary, I'm still learning.

Thanks!

Also the code in question is under Manage Assets Form, in the Form_BeforeUpdate event but I removed the code because of the problem I was having in the one I uploaded, I thought I had left it in.

[June7]

So you're saying there isn't any code to analyze? I tried entering record on that form and get error that record is required in Agency. There are no records to test with. Provide db with code and dummy records and instructions on how to recreate the issue. Try zipping the db and see if that is small enough - 2MB zip allowed.

[Missinglinq]

It would really help if you would post your entire code for your Form_BeforeUpdate event, as I requested earlier, as not everyone can run an Access 2013 file.

Linq ;0)>