Run Time Errors

C

croth68

Background: I originally had 6 tables in this database, I cut it down to
three tables and added a large amount of items to one.

I ran across a website: allenbrowne.com from these forums. I found the
perfect search form, here is the link http://allenbrowne.com/ser-62.html I am
having a little bit of trouble with this.

Hard to explain but here it goes. I built a continuous form and for starters
added one combo box and 5 text boxes (more will be added later). The combo
box is a unique item because it comes from a different table and has two
fields, one for the id number and the other for the name. The reason both
columns need to be in the combo box is because one person may know the id
number but not the name and vice versa. The table that my form is linked to
(for searching) only contains the id number field. When I make a selection I
get a runtime error 2001 and Me.FilterOn = True is highlighted in my code. I
changed the triple quote in the
If Not IsNull(Me.Combo44) Then
strWhere = strWhere & "([Training Aid ID] = """ & Me.Combo44 & """)
AND "
End If
to a " single quote and got a runtime error of 2448 with the Me.Filter =
strWhere highlited. Here is a copy of all the code:

Option Compare Database
Option Explicit

Private Sub cmdFilter_Click()
'Purpose: Build up the criteria string form the non-blank search boxes,
and apply to the form's Filter.
'Notes: 1. We tack " AND " on the end of each condition so you can
easily add more search boxes; _
we remove the trailing " AND " at the end.
' 2. The date range works like this: _
Both dates = only dates between (both inclusive.
_
Start date only = all dates from this one onwards; _
End date only = all dates up to (and including this
one).
Dim strWhere As String 'The criteria string.
Dim lngLen As Long 'Length of the criteria string to
append to.
Const conJetDate = "\#mm\/dd\/yyyy\#" 'The format expected for dates in
a JET query string.

'***********************************************************************
'Look at each search box, and build up the criteria string from the non-
blank ones.
'***********************************************************************
'Text field example. Use quotes around the value in the string.
If Not IsNull(Me.Combo44) Then
strWhere = strWhere & "([Training Aid ID] = ""*" & Me.Combo44 & """)
AND "
End If

'Another text field example. Use Like to find anywhere in the field.
If Not IsNull(Me.Text46) Then
strWhere = strWhere & "([Dealer ID] Like ""*" & Me.Text46 & "*"") AND
"
End If

'Number field example. Do not add the extra quotes.
If Not IsNull(Me.Text52) Then
strWhere = strWhere & "([Class Name] = " & Me.Text52 & ") AND "
End If

'Yes/No field and combo example. If combo is blank or contains "ALL", we
do nothing.
'If Me.cboFilterIsCorporate = -1 Then
'strWhere = strWhere & "([IsCorporate] = True) AND "
'ElseIf Me.cboFilterIsCorporate = 0 Then
'strWhere = strWhere & "([IsCorporate] = False) AND "
'End If

'Date field example. Use the format string to add the # delimiters and
get the right international format.
If Not IsNull(Me.Text48) Then
strWhere = strWhere & "([Date Out] >= " & Format(Me.Text48,
conJetDate) & ") AND "
End If

'Another date field example. Use "less than the next day" since this
field has times as well as dates.
If Not IsNull(Me.Text50) Then 'Less than the next day.
strWhere = strWhere & "([Date Out] < " & Format(Me.Text50 + 1,
conJetDate) & ") AND "
End If

'***********************************************************************
'Chop off the trailing " AND ", and use the string as the form's Filter.
'***********************************************************************
'See if the string has more than 5 characters (a trailng " AND ") to
remove.
lngLen = Len(strWhere) - 5
If lngLen <= 0 Then 'Nah: there was nothing in the string.
MsgBox "No criteria", vbInformation, "Nothing to do."
Else 'Yep: there is something there, so remove the "
AND " at the end.
strWhere = Left$(strWhere, lngLen)
'For debugging, remove the leading quote on the next line. Prints to
Immediate Window (Ctrl+G).
'Debug.Print strWhere

'Finally, apply the string as the form's Filter.
Me.Filter = strWhere
Me.FilterOn = True
End If
End Sub

Private Sub cmdReset_Click()
'Purpose: Clear all the search boxes in the Form Header, and show all
records again.
Dim ctl As Control

'Clear all the controls in the Form Header section.
For Each ctl In Me.Section(acHeader).Controls
Select Case ctl.ControlType
Case acTextBox, acComboBox
ctl.Value = Null
'Case acCheckBox
' ctl.Value = False
End Select
Next

'Remove the form's filter.
Me.FilterOn = False
End Sub

Private Sub Form_BeforeInsert(Cancel As Integer)
'To avoid problems if the filter returns no records, we did not set its
AllowAdditions to No.
'We prevent new records by cancelling the form's BeforeInsert event
instead.
'The problems are explained at http://allenbrowne.com/bug-06.html
Cancel = True
MsgBox "You cannot add new clients to the search form.", vbInformation,
"Permission denied."
End Sub

Private Sub Form_Open(Cancel As Integer)
'Remove the single quote from these lines if you want to initially show
no records.
'Me.Filter = "(False)"
'Me.FilterOn = True
End Sub

Private Sub txtFormFilter_BeforeUpdate(Cancel As Integer)

End Sub

Any help would be greatly appreciated, I hope I explained this well enough.
Thanks.
 
J

Jeanette Cunningham

croth,
A form can be filtered using a selection from a combo box.
However, what you are doing is asking the form to find the record that
matches the record that you selected in the combo.
A combo has a property called bound the Bound Column.
This is the only part of the combo that is used to find the matching record
in the form.
Even when the user clicks the name, the combo supplies the ID to the form
and the form uses the ID to find the matching record.
If your ID is a number data type, use
strWhere = "([Training Aid ID] = " & Me.Combo44 & ") - single quotes

if your ID is a text data type, use
strWhere = "([Training Aid ID] = """ & Me.Combo44 & """) - three quotes

It doesn't matter whether the user uses the ID or the name to search the
combo, you don't change strWhere.
Your code only cares about the value of the bound column which is usually
the first column - in this case the ID field.
Check that it is the first column that is the bound column - Form property
dialog | Data tab.


Jeanette Cunningham


croth68 said:
Background: I originally had 6 tables in this database, I cut it down to
three tables and added a large amount of items to one.

I ran across a website: allenbrowne.com from these forums. I found the
perfect search form, here is the link http://allenbrowne.com/ser-62.html I
am
having a little bit of trouble with this.

Hard to explain but here it goes. I built a continuous form and for
starters
added one combo box and 5 text boxes (more will be added later). The combo
box is a unique item because it comes from a different table and has two
fields, one for the id number and the other for the name. The reason both
columns need to be in the combo box is because one person may know the id
number but not the name and vice versa. The table that my form is linked
to
(for searching) only contains the id number field. When I make a selection
I
get a runtime error 2001 and Me.FilterOn = True is highlighted in my code.
I
changed the triple quote in the
If Not IsNull(Me.Combo44) Then
strWhere = strWhere & "([Training Aid ID] = """ & Me.Combo44 & """)
AND "
End If
to a " single quote and got a runtime error of 2448 with the Me.Filter =
strWhere highlited. Here is a copy of all the code:

Option Compare Database
Option Explicit

Private Sub cmdFilter_Click()
'Purpose: Build up the criteria string form the non-blank search
boxes,
and apply to the form's Filter.
'Notes: 1. We tack " AND " on the end of each condition so you can
easily add more search boxes; _
we remove the trailing " AND " at the end.
' 2. The date range works like this: _
Both dates = only dates between (both
inclusive.
_
Start date only = all dates from this one onwards;
_
End date only = all dates up to (and including
this
one).
Dim strWhere As String 'The criteria string.
Dim lngLen As Long 'Length of the criteria string
to
append to.
Const conJetDate = "\#mm\/dd\/yyyy\#" 'The format expected for dates
in
a JET query string.


'***********************************************************************
'Look at each search box, and build up the criteria string from the
non-
blank ones.

'***********************************************************************
'Text field example. Use quotes around the value in the string.
If Not IsNull(Me.Combo44) Then
strWhere = strWhere & "([Training Aid ID] = ""*" & Me.Combo44 &
""")
AND "
End If

'Another text field example. Use Like to find anywhere in the field.
If Not IsNull(Me.Text46) Then
strWhere = strWhere & "([Dealer ID] Like ""*" & Me.Text46 & "*"")
AND
"
End If

'Number field example. Do not add the extra quotes.
If Not IsNull(Me.Text52) Then
strWhere = strWhere & "([Class Name] = " & Me.Text52 & ") AND "
End If

'Yes/No field and combo example. If combo is blank or contains "ALL",
we
do nothing.
'If Me.cboFilterIsCorporate = -1 Then
'strWhere = strWhere & "([IsCorporate] = True) AND "
'ElseIf Me.cboFilterIsCorporate = 0 Then
'strWhere = strWhere & "([IsCorporate] = False) AND "
'End If

'Date field example. Use the format string to add the # delimiters and
get the right international format.
If Not IsNull(Me.Text48) Then
strWhere = strWhere & "([Date Out] >= " & Format(Me.Text48,
conJetDate) & ") AND "
End If

'Another date field example. Use "less than the next day" since this
field has times as well as dates.
If Not IsNull(Me.Text50) Then 'Less than the next day.
strWhere = strWhere & "([Date Out] < " & Format(Me.Text50 + 1,
conJetDate) & ") AND "
End If


'***********************************************************************
'Chop off the trailing " AND ", and use the string as the form's
Filter.

'***********************************************************************
'See if the string has more than 5 characters (a trailng " AND ") to
remove.
lngLen = Len(strWhere) - 5
If lngLen <= 0 Then 'Nah: there was nothing in the string.
MsgBox "No criteria", vbInformation, "Nothing to do."
Else 'Yep: there is something there, so remove the "
AND " at the end.
strWhere = Left$(strWhere, lngLen)
'For debugging, remove the leading quote on the next line. Prints
to
Immediate Window (Ctrl+G).
'Debug.Print strWhere

'Finally, apply the string as the form's Filter.
Me.Filter = strWhere
Me.FilterOn = True
End If
End Sub

Private Sub cmdReset_Click()
'Purpose: Clear all the search boxes in the Form Header, and show all
records again.
Dim ctl As Control

'Clear all the controls in the Form Header section.
For Each ctl In Me.Section(acHeader).Controls
Select Case ctl.ControlType
Case acTextBox, acComboBox
ctl.Value = Null
'Case acCheckBox
' ctl.Value = False
End Select
Next

'Remove the form's filter.
Me.FilterOn = False
End Sub

Private Sub Form_BeforeInsert(Cancel As Integer)
'To avoid problems if the filter returns no records, we did not set its
AllowAdditions to No.
'We prevent new records by cancelling the form's BeforeInsert event
instead.
'The problems are explained at http://allenbrowne.com/bug-06.html
Cancel = True
MsgBox "You cannot add new clients to the search form.", vbInformation,
"Permission denied."
End Sub

Private Sub Form_Open(Cancel As Integer)
'Remove the single quote from these lines if you want to initially show
no records.
'Me.Filter = "(False)"
'Me.FilterOn = True
End Sub

Private Sub txtFormFilter_BeforeUpdate(Cancel As Integer)

End Sub

Any help would be greatly appreciated, I hope I explained this well
enough.
Thanks.
 
C

croth68 via AccessMonster.com

Jeanette,

Thank you very much, worked like a charm. I am learning a ton from this forum
and asking questions.
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Top