Ticket #4360 (closed enhancement: wontfix)

Opened 2 years ago

Last modified 9 months ago

Allow commenting without giving an email address, but still requiring a name

Reported by: jameslab6 Assigned to:
Priority: normal Milestone:
Component: Administration Version:
Severity: minor Keywords: has-patch needs-testing
Cc:

Description

There is a Discussion option "Comment author must fill out name and e-mail". I want every commenter to have a name, but I don't care about them giving an email address (nobody gives real addresses anyway). So I think these two requirements should be decoupled.

Attachments

4360.diff (12.5 kB) - added by rob1n on 06/05/07 04:20:42.

Change History

05/29/07 18:11:26 changed by foolswisdom

  • milestone changed from 2.3 to 2.4.

(follow-up: ↓ 3 ) 05/29/07 20:38:57 changed by rob1n

  • owner changed from anonymous to rob1n.
  • milestone changed from 2.4 to 2.3.

How about we change the logic so that we provide checkboxes for what information needs to be included?

So we have checkboxes for Name, Email and URL. The user can selectively choose what he wants filled out.

(in reply to: ↑ 2 ) 05/30/07 12:40:33 changed by jameslab6

Replying to rob1n:

How about we change the logic so that we provide checkboxes for what information needs to be included? So we have checkboxes for Name, Email and URL. The user can selectively choose what he wants filled out.

Yes, that sounds perfect.

06/05/07 04:20:42 changed by rob1n

  • attachment 4360.diff added.

06/05/07 04:21:19 changed by rob1n

  • keywords set to has-patch needs-testing.

06/05/07 04:21:25 changed by rob1n

  • status changed from new to assigned.

06/21/07 09:18:33 changed by Stargazer

Email is often used for gravatar and notification stuff. So I fully agree with the checkbox solution.

09/12/07 02:47:43 changed by ryan

  • milestone changed from 2.3 to 2.4 (next).

12/19/07 07:26:35 changed by rob1n

  • owner deleted.
  • status changed from assigned to new.

01/09/08 09:06:47 changed by spencerp

I'm willing to test this out if someone can provide an updated patch for it? I'm not a coder, so I'm not sure how this "code" works? Or, unless everyone thinks it's not necessary anymore, maybe just close this ticket out and someone create a plugin for it instead? Just was wondering.

01/09/08 09:08:20 changed by spencerp

But as for testing this out on latest trunk, here's what I got:

Here's what I got so far:

a) Options don't "stick". I selected to use just commenter name, and hit update options button. It's not sticking, the check mark isn't "in" the check box after hitting update options... resulting in b) b) After submitting the comment, I get following error message: "Error: please enter a valid email address." c) In wp-comments.post.php file..

this line needs to changed from: wp_die( ( 'Error: please complete the URL field.' );

to (get rid of syntax error or whatever): wp_die( ( 'Error: please complete the URL field.' ) );

01/09/08 09:13:43 changed by spencerp

Oops, forgot code blocks above.

this line needs to changed from:

wp_die( __( 'Error: please complete the URL field.' );

to (get rid of syntax error or whatever):

wp_die( __( 'Error: please complete the URL field.' ) );

01/15/08 11:34:05 changed by spencerp

Can someone do something with this one? I COULD make a partial up-to-date patch for this, it's just where I'd have to make the patch for the update the WordPress version that'd throw me off. I'm not sure how to do that. The other thing that I'm not sure about, is, the whole thing where the current patch wasn't having the "option stick"? =/

I tested out the current patch, but when choosing "name" for required rather then email address, it threw an error and the option wasn't "sticking". I put the check mark in the box for "Name" or whatever, hit update button and the check mark was no longer in the check box and the option wasn't "sticking" like supposed too. Hence for getting the error I mentioned above.

Can someone (with code wisdom) maybe do an up-to-date patch for this and perhaps figure out why the patch wasn't working? Like I said, I could make an up-to-date partial patch. But I need someone with more coding experience to figure out the rest...

03/15/08 23:06:14 changed by westi

  • status changed from new to closed.
  • resolution set to wontfix.
  • milestone deleted.

Marking as wontfix.

We're not going to add extra options here.

If you really want this functionality you can implement it in a plugin which works with the core option set to allow comments without email_address and name and the plugin doing the check for a name on the preprocess_comment hook and using wp_die if it's not supplied.