|
|
|||||||||
|
|||||||||
|
|||||||||
| |
||
| |||||||||
![]() |
|
|
«
Previous Thread
|
Next Thread
»
|
Thread Tools | Search this Thread | Rate Thread | Display Modes |
|
#1
|
|||
|
|||
|
Http://www.totalmediaguide.com
Its a community review site of albums/books/films and games only music and games are up at the moment.
Comments suggestions about all areas of improvement. Thanks a lot! |
|
#2
|
||||
|
||||
|
I like the design, looks good
![]() when I log in and dont fill in any details I get this: Code:
Microsoft JET Database Engine error '80040e14' Extra ) in query expression '(((General_Site_Members.Member_ID)=) AND ((General_Section_Members.Section_ID)=6))'. /inc/general_menu.asp, line 14 have a check so that the user needs to fill in the username and password fields. when I registered, I got this error Code:
Microsoft JET Database Engine error '80040e14' Syntax error (missing operator) in query expression ''nofriend's',3,1,'19CF8312DBFF4CEB23E44CC35FC603E 00A310A57','AF8B','nofriends','13/07/2006 12:08:42')'. /register_process.asp, line 27 you are not replacing invalid chars such as a ' I used nofreind's to login, you need something like this to make it sql safe Code:
strName = replace(strName,"'","''") you also dont have any email validation, I registered with "asd" as my email Make sure that you are checking for invalid chars in your sql, and make sure users need to fill in values where they need to ![]() other than that it looks like a nice site ![]()
__________________
Look! Its a ShemZilla ![]() ![]()
|
|
#3
|
||||
|
||||
|
I agree with NF, the design looks good, nice job
![]()
__________________
Everyone has a photographic memory. Some don't have film. I am a nobody, nobody is perfect, therefore I am perfect.
|
![]() |
| Viewing: ASP Free Forums > Web Design > Site Reviews > Http://www.totalmediaguide.com |
| Thread Tools | Search this Thread |
| Display Modes | Rate This Thread |
|
|
|
|