Car Event db 95% finnished - please review a few small errors

[dfwcomputer]

This database is for a car event that's held once a year. It has been a work in progress for about 15 weeks now and is basically finished. There is only a couple of errors and not sure where to start. The database is so big it is getting a little out of reach for me to organize everything.

Idea of the database is that I can enter entrant's information, there car details and any merchandise they want.

Main errors I’ve noticed are;

1. In email form if you select one user it still emails everyone.
2. In the car details form the induction box does not show the value list I created in the car table.
3. when entering a t-shirt in the products form you select the shirt size. Is this the best way to lay this out.
4. In the Order Form what is the best way to add a freight box between Sub-Total & Total where you can type the freight in and it's automatically added to the Total Box

Have a look over the db and let me know what ya's think.

Thanks
Dean

 

[dfwcomputer]

As this is pretty urgent now. (Because I stuffed around so long)

Does anyone know of any database companies in Australia that can finish it. Preferably with competitive pricing.

Thanks

 

[dfwcomputer]

Can anybody have a look

 

[bonekrusher]

You should simplify the the code that sends the email. Try this:

DoCmd.SendObject acSendNoObject, "", "", Me.EMAILaddress, "", "", Me![Subject], Me![MessageBody]

I couldn't find on the form where you give the Email address. So for the code I assume there is a field Me.EMAILaddress.

This one code should send out your email.

 

[dfwcomputer]

Ok,

Finished off the forms for the switchboard and stuff. Looking quite good know fixed some of the errors.

1. In email form if you select one user it still emails everyone.
2. In the car details form the induction box does not show the value list I created in the car table. (fixed)
3. when entering a t-shirt in the products form you select the shirt size. Is this the best way to lay this out.
4. In the Order Form what is the best way to add a freight box between Sub-Total & Total where you can type the freight in and it's automatically added to the Total Box

As for simplifying the code for emails I would not even know where to start as one of the guys on here helped me with it.

Let me know what ya's think.

 

[dfwcomputer]

Anybody got time to have a quick look?

 

[Jon K]

1. In email form if you select one user it still emails everyone.

I use Outlook Express, so I can't test the code in Outlook.

But a quick look at the "Email Messages form" and its controls reveals that the SendOptions.Value's are set to 2 and 3, not 1 and 2. And there is a syntax error in the "whereClause" in the first Case statement.


You need to change:
the Option Value of Option35 to 1
the Option Value of Option37 to 2

and the LookForID AfterUpdate event code to
SendOptions.Value = 2

and change the WhereClause to

    Select Case SendOptions.Value
       
        Case 1  'All Customers
                whereClause = [b]" WHERE [Email] Is Not Null"[/b]
        Case 2
                whereClause = " WHERE EntrantID= " & Me![LookForID].Value
    End Select

and ensure that the "LookForID" combo box contains only entrants who have an email address:
by adding the Email field in the "contact lookup query"
and modifying the Row Source of the LookForID combo box to include Is Not Null
[color=white].[/color]

 

[dfwcomputer]

Jon K,

Worked great, much appreciated.Makes sense they way you wrote it but never would have figured it out on my own.

Thanks again

Also how is the attachment part suppose to work.Shouldn't it be a browse button to add the attachment.In the code it say's.

If Len(Me![Attachment]) > 0 Then
.Attachments.Add (Me![Attachment])
End If

Not sure about this one

Dean

 

[Jon K]

It seems the Attachment text box would require a filepath in full.


Here's an example of returning filepath using a browse button:-
Common Dialog Box examples (Post #9)
http://www.access-programmers.co.uk/forums/showthread.php?t=97434

In the example, the "Open Common Dialog Box" browse button on the form uses the code in the basCommonDialog module. To use a similar button in your database, you need to paste the code from the basCommonDialog module in the example to a module in your database as well.
.

 

[dfwcomputer]

It seems the Attachment text box would require a filepath in full.


Here's an example of returning filepath using a browse button:-
Common Dialog Box examples (Post #9)
http://www.access-programmers.co.uk/...ad.php?t=97434

In the example, the "Open Common Dialog Box" browse button on the form uses the code in the basCommonDialog module. To use a similar button in your database, you need to paste the code from the basCommonDialog module in the example to a module in your database as well.

I couldn't seem to work this out but apart from that I have got the database to a stage where I think it's finished. Beautified it a bit and added some extra's. Let me know what you's think.

Thanks for all the input it's much appreciated.

 

[ghudson]

You should give credit [or thanks to the author] where it is due when you are using code that you have found on the internet [within this forum].

Just a constructive critique... Your forms do not have a consistent look and feel. The navigation between the forms is rough. You db is not protected. Anybody will be able to open up and mess with the design of the db objects and the raw data. I did not test the functions or the accuracy of the data entry so I can not offer any advice on that. You need to use a naming convention for your db objects. Not a bad start for your first db. You just need to polish it off.

 

[dfwcomputer]

You should give credit [or thanks to the author] where it is due when you are using code that you have found on the internet [within this forum].

Some of the code i have had laying around for nearly 12 months so i am not sure who initially wrote the code but of course it goes with out saying, THANKYOU.

Your forms do not have a consistent look and feel. The navigation between the forms is rough.

Any ideas on improving this.

as for the rest i will search around and see what i can come up with.

Thanks

 

[dfwcomputer]

By searching the forums i found this naming convention in which i attached in case it's usefull to someone else.

My question is i am running office 2003 and it has name auto correct which tracks name changes, or automatically correct common side effects that occur when you rename things.If i start to change names of my forms and such to bring it inline with the LESZYNSKI Naming Convention will it create problems or rename evrything it needs to.

Thanks Dean

 

[dfwcomputer]

I think it is finished,

So here it is, let me know what you all think. The only thing I would like to do know is add a report that shows how many people actually attended an event. Any ideas on setting up the query for this.

Thanks
Dean

 

[ghudson]

I see that you have not addressed my original comments...

You have a lot of code you have found within this forum yet you are not giving credit or thanks [or noting credit to the author in your modules] where it is due when you are using code that you have found on the internet [within this forum].

Your forms do not have a consistent look and feel. The navigation between the forms is rough. You db is not protected. Anybody will be able to open up and mess with the design of the db objects and the raw data. I did not test the functions or the accuracy of the data entry so I can not offer any advice on that. I noticed you are now using a naming convention for your db forms, tables etc. but not for the objects within the forms and reports like the text boxes and labels.

 

[dfwcomputer]

I see that you have not addressed my original comments...

You have a lot of code you have found within this forum yet you are not giving credit or thanks [or noting credit to the author in your modules] where it is due when you are using code that you have found on the internet [within this forum].

As i have said i haven't got a clue who's code it was as some of it i have had laying around for 12 months or more and some of it even came from dbforums.But as i said i thank anyone who's code I have used and if anyone see's there code let me know and i will gladly add it to the modules.Thanks again.

I tried to give the forms a consistent look with the background colours and stuff is this what you meant.

also I had trouble renamimg everything as a lot of stuff started to play up.And there is so much in this database for a newbie that i lost confidence.I suppose that's a lesson for the future always start out naming things the way they should be named.

Thanks