Opened 13 years ago
Closed 10 years ago
#4360 closed enhancement (wontfix)
Allow commenting without giving an email address, but still requiring a name
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | minor | Version: | 2.9 |
Component: | Administration | Keywords: | close |
Focuses: | Cc: | ||
PR Number: |
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 (1)
Change History (19)
#2
follow-up:
↓ 3
@
13 years ago
- Milestone changed from 2.4 to 2.3
- Owner changed from anonymous to rob1n
#3
in reply to:
↑ 2
@
13 years ago
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.
#6
@
12 years ago
Email is often used for gravatar and notification stuff. So I fully agree with the checkbox solution.
#9
@
12 years ago
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.
#10
@
12 years ago
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.' ) );
#11
@
12 years ago
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.' ) );
#12
@
12 years ago
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...
#13
@
12 years ago
- Milestone 2.5 deleted
- Resolution set to wontfix
- Status changed from new to closed
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.
#14
follow-up:
↓ 17
@
10 years ago
This bug (or "feature enhancement", if you prefer) definitely should be fixed. It is unfortunate that it has stood for 2 years without being fixed. Most of that time with a patch in hand.
"If you really want this functionality you can implement it in a plugin..."
Requiring a plugin to alter basic settings is silly. Placing two options in one checkbox is also silly.
#15
@
10 years ago
- Resolution wontfix deleted
- Status changed from closed to reopened
- Version set to 2.8.4
I wasn't going to reopen this bug, but then I realized that it was only one developer who decided to ignore it, despite popular approval. Perhaps the environment has changed. Perhaps now, this bug might be fixed if the devs were more aware of it.
#16
@
10 years ago
- Version changed from 2.8.4 to 2.9
Whatever the case may be, no new features in 2.8.4
#17
in reply to:
↑ 14
@
10 years ago
- Keywords close added; has-patch needs-testing removed
Replying to WK1:
This bug (or "feature enhancement", if you prefer) definitely should be fixed. It is unfortunate that it has stood for 2 years without being fixed. Most of that time with a patch in hand.
"If you really want this functionality you can implement it in a plugin..."
Requiring a plugin to alter basic settings is silly. Placing two options in one checkbox is also silly.
Ok, now it's 2 developers. WordPress has a policy of "don't add options that aren't absolutely necessary". That's what plugins are for.
Most users won't need that much control over what is and what isn't required.
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.