Captcha when registering a new account
Review Request #50 - Created Jan. 17, 2009 and submitted
| Information | |
|---|---|
| Alexandru Tandrau | |
| infoarena | |
| Reviewers | |
| hackers | |
Captcha
It would be nice to have reCaptcha on the forum also for anonymous people.
Please have a look at my comments. I'm particularly concerned about the usability of the registration form.
-
trunk/config.php.sample (Diff revision 1) -
Please change the private key on the production system.
-
trunk/www/controllers/account_validator.php (Diff revision 1) -
Disable Captcha when in IA_DEVELOPMENT_MODE. Development should be possible offline.
-
trunk/www/controllers/register.php (Diff revision 1) -
$data is only for form values. Maybe use $view instead. Also, the widget should not be displayed at all when in IA_DEVELOPMENT_MODE. It relies on external resources which might not be accessible while developing offline.
-
trunk/www/views/register.php (Diff revision 1) -
"Captcha" is not self-explanatory. We should not assume people know what Captcha is. Maybe use "Verificare."
-
trunk/www/views/register.php (Diff revision 1) -
This note is confusing, especially for someone who barely knows what Captcha is. Just explain users what they need to do. The whole story about digitizing books is ReCAPTCHA marketing against other Captcha's. It doesn't belong here.
Review request changed
Change Summary:
I have modified according to Cristi's comments, except changing the private key on the production system. I believe I do not have necessary credentials to do that, but one can create a new account at www.recaptcha.net and receive new keys for the production server.
Diff: |
Revision 2 (+27 -12) |
|---|