0

I have the code below (courtesy of allenbrowne, his info are in the code). When I am filtering using one of the columns (One_or_Two_Pearl) it is giving me a (Syntax error missing operator in query expression). I can not trace the problem as everything look fine. That column is defined as Text type and contains data (1, 2, NA). This problem is only occurring for this column, and when I debug the yellow mark is indicating the (Me.Filter = strWhere) part. Thank you in advance for your help.

'http://allenbrowne.com/ser-62.html
'Purpose:   This module illustrates how to create a search form, _
            where the user can enter as many or few criteria as they wish, _
            and results are shown one per line.
'Note:      Only records matching ALL of the criteria are returned.
'Author:    Allen Browne ([email protected]), June 2006.
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.cboxprojphase) Then
        strWhere = strWhere & "([Project_Phase] = """ & Me.cboxprojphase & """) AND "
    End If

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

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

    'Text field example. Use quotes around the value in the string.
    If Not IsNull(Me.cboxadmupc) Then
        strWhere = strWhere & "([ADM/UPC] = """ & Me.cboxadmupc & """) AND "
    End If

    'Text field example. Use quotes around the value in the string.
    If Not IsNull(Me.cboxpearl) Then
        strWhere = strWhere & "([One_or_Two_Pearl] = """ & Me.cboxpearl & """) "
    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
    Me.OrderByOn = 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 Records 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

  Dim strURL As String
  Dim objIE As Object
  Dim arrSites(2) As String
  Dim i As Integer

     arrSites(0) = "http://google.com"
     arrSites(1) = "http://google.com"

     Set objIE = CreateObject("InternetExplorer.Application")
     For i = 0 To 1 Step 1
        strURL = arrSites(i)
        If i = 0 Then
            objIE.Navigate strURL
        Else
            objIE.Navigate2 strURL, 2048
        End If
     Next i
     objIE.Visible = True
     Set objIE = Nothing
     'objIE.Quit

End Sub


Private Sub optSortorder_AfterUpdate()
If Me.optSortorder = 1 Then
     Me.OrderBy = Me.cboSortField
Else
     Me.OrderBy = Me.cboSortField & " DESC"
End If
Me.OrderByOn = True
End Sub

1 Answer 1

1

Both your problems are here:

strWhere = Left$(strWhere, lngLen)
'For debugging, remove the leading quote on the next line. Prints to Immediate Window (Ctrl+G).
'Debug.Print strWhere
  • Uncomment the Debug.Print line to see what is happening:
  • it tries to remove a trailing " AND " from your [One_or_Two_Pearl] line, but there is none.
Sign up to request clarification or add additional context in comments.

1 Comment

You are correct. I did add " AND " at the end of the last filtering condition and it worked. Thank you for your efforts.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.