tomfmason Posted July 13, 2006 Share Posted July 13, 2006 I have writen a contact form that saves the results to a database instead of mailing it. I did this for several reasons. One is the ammount of people that I see asking how to send mail via php and two is a friend of my asked me if I could. Anyways, I have a test version up and running and would like some suggestions on adding features or possiblely any problems with the script. [b]*Note[/b] this is an extreme beta version..lol. But is functional. The only thing that is not functional is the reply script. This is for security reasons.Here is a link to the script. Please, no comments requarding the generic layout. Please stick to functional criticisms.[url=http://www.owpt.biz/home/test/inbox.php]http://www.owpt.biz/home/test/inbox.php[/url]Any suggestions would be great. Link to comment https://forums.phpfreaks.com/topic/14452-suggestions-for-added-features/ Share on other sites More sharing options...
Barand Posted July 13, 2006 Share Posted July 13, 2006 Moved to Website Critique Link to comment https://forums.phpfreaks.com/topic/14452-suggestions-for-added-features/#findComment-57148 Share on other sites More sharing options...
brown2005 Posted July 13, 2006 Share Posted July 13, 2006 looks ok, maybe a rollover on the table row, to show wat message your actioning on more easily.. Link to comment https://forums.phpfreaks.com/topic/14452-suggestions-for-added-features/#findComment-57149 Share on other sites More sharing options...
moberemk Posted July 13, 2006 Share Posted July 13, 2006 You know, you might want to (a) move the "message deleted" message inside the <body> tag, and (b) validate your code a bit more. Oh, and maybe style it a little. Link to comment https://forums.phpfreaks.com/topic/14452-suggestions-for-added-features/#findComment-57247 Share on other sites More sharing options...
steviewdr Posted July 13, 2006 Share Posted July 13, 2006 Buuuuuuggggggyyyyyyyy :'( I broke itI got this:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Reilly', 'steve'[email protected]', 'Test "this"', 'Test's and more testsand ' at line 2 onhttp://www.owpt.biz/home/test/send.phpYou dont cancel quotes, and you allow html characters >:(Use htmlentities to tidy up and store the values.Im not sure are quotes allowed in email addresses - but if they are you better plan for them.Other than that - I do like the concept.Rgds,steve Link to comment https://forums.phpfreaks.com/topic/14452-suggestions-for-added-features/#findComment-57287 Share on other sites More sharing options...
tomfmason Posted July 13, 2006 Author Share Posted July 13, 2006 Thanks Steve that is exactly what I was looking for. I will fix the bugs and report back Link to comment https://forums.phpfreaks.com/topic/14452-suggestions-for-added-features/#findComment-57600 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.