MsgBox - Unload Form Predicament

[hshah]

Okay, I have the following code in my Form_Load

If datanavigator.Recordset.RecordCount = 0 Then
MsgBox "There are No Invoices to be Viewed/Printed!", vbCritical, "No Records"
If vbOK = True Then
Unload Me
End If
End If

So it basically checks if the ADO (datanavigator) is empty and if it is, it brings up the message box, and when the person clicks Ok, it unloads the form, but here is the problem.

Using the code above, when the person clicks the ok, the message box stays there regardless of how many times u press ok and the app crashes.

How can I upload the form?

[ahara]

it should work if you remove the inner if condition - just let them click OK and then unload the form.

[demotivater]

Try this instead

Call MsgBox("There are no invoices...", vbExclamation + vbDefaultButton1, "No records")
Unload Me

[hshah]

tried both ways - still dies!

[chemicalNova]

Have you tried using just End?

If datanavigator.Recordset.RecordCount = 0 Then
MsgBox "There are No Invoices to be Viewed/Printed!", vbCritical, "No Records"
End
End If

Phreak

[hshah]

well at least with the "end" the app doesn't crash, but the whole app gets closed down, which is not what i want - btw the form in question is part of a mdiform, so i don't want the whole thing to get closed down!

[TheVader]

It looks like the problem has more to do with closing the form that with handling the messagebox call. To see if this is right, place an empty MessageBox call in the Form_Unload event (MsgBox ""). If your app crashes after clicking the OK button of both boxes the problem lies with the unloading.

In that case, make sure you close everything: set all objects to nothing etc.

[hshah]

okay, i tried the thing in form_unload and it closed fine - no problems there!

[leinad31]

Hmmm.... and I thought you can't unload a form in its load event. What I usually do is set a flag in the load event then unload the form in the activate event if the flag is true.

[hshah]

how do i do that?

[leinad31]

Private booForceUnload As Boolean  'module level
 
Private Sub Form_Load()
 
booForceUnlaod = False
If datanavigator.Recordset.RecordCount = 0 Then
   If vbOK = MsgBox ("There are No Invoices to be Viewed/Printed!", vbCritical + vbOkOnly, "No Records") Then
      booForceUnload = True
   End If
End If
'
'
End Sub
 
Private Sub Form_Activate()
   If booForceUnload = True Then 
      Unload Me
      Exit Sub
   End If
'
'
'
End Sub
 
Private Sub Form_Unload()
   booForceUnload = False  'reset. just in case for succeeding form load
End Sub

[Shaggy Hiker]

You can unload in the load event, it will just raise an error. One possible solution is to catch that error and exit the sub.

[NicePackage]

You could also check that condition before you allow the form to load. That would also look cleaner to the end user.