Great.
BTW, referencing a form using
Code:
Form_frmEventInput.RecordSource = "select * from tblEventInfo where " & GCriteria
is the wrong method. You should use
Code:
Forms!frmEventInput.RecordSource = "select * from tblEventInfo where " & GCriteria
And instead of changing the form record source, you might think about using the Filter property. The code would look like
Code:
Forms!frmEventInput.Filter = GCriteria
Forms!frmEventInput.FilterOn = True
Forms!frmEventInput.Caption = "Events (" & cboSearchField.Value & " contains '*" & txtSearchString & "*')"
To remove the filter, the code would be (depending on where the button is):
Code:
Me.Filter = ""
Me.FilterOn = False
This is just my preference.....
Other things I noticed:
** I would remove ALL look up FIELDS (different than look up TABLES) from your tables.
See The Evils of Look up FIELDS
http://access.mvps.org/access/lookupfields.htm
No experienced programmer (that I know of) uses look up FIELDS.
Also see
The Ten Commandments of Access
http://access.mvps.org/access/tencommandments.htm
** "Name" is a reserved word in Access and shouldn't be used for object names.
** "Option Explicit" should be at the top of EVERY code module. There is an option to require variable declarations.
** You have "ID" as the name of the PK field. This is a very poor name for a field, let alone 10 tables.
Which is easier to understand "ID" or "TimeID" (or "Time_ID")??
Take time to use a naming convention. It will help you in the future.
You might also read these
Autonumbers--What they are NOT and What They Are
http://www.utteraccess.com/wiki/index.php/Autonumbers
Microsoft Access Tables: Primary Key Tips and Techniques
http://www.fmsinc.com/free/newtips/primarykey.asp
Use Autonumbers properly
http://access.mvps.org/access/general/gen0025.htm