Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#13026 closed defect (bug) (fixed)

'Your email is <em>never</em> published nor shared.' should be removed from wp-includes/comment-template.php

Reported by: lloydbudd's profile lloydbudd Owned by: nacin's profile nacin
Milestone: 3.0 Priority: lowest
Severity: minor Version: 3.0
Component: Comments Keywords: has-patch
Focuses: Cc:

Description

'Your email is <em>never</em> published nor shared.' should be removed from wp-includes/comment-template.php

ENV: WP Trunk r14092 ("3.0-beta1")

This type of (ethical) statement should be consciously made by the blog owner, and likely relates to their comment policy.
(Plugin material)

ADDITIONAL DETAILS

This is particularly problematic for multiple sites / blog networks like WordPress.com, where the trust is put in the service provider not the individual blogger.

Attachments (3)

13026.diff (1.4 KB) - added by lloydbudd 15 years ago.
Patch for wp-includes/comment-template.php removing "Your email is <em>never</em> published nor shared."
bmb_0011_t13026.patch (1.7 KB) - added by bumbu 15 years ago.
Now just 'Your email will not be published.'
13026-comment-form-text.diff (1.7 KB) - added by lloydbudd 15 years ago.
Great points nacin! We should actually make it explicit. There are so many naive people out there, and so much poor language out there regarding contact forms being "email", so here is a patch with "email address" -- you knew this was the true path based on your commit msg of "regarding publication of email addresses" ;-)

Download all attachments as: .zip

Change History (10)

@lloydbudd
15 years ago

Patch for wp-includes/comment-template.php removing "Your email is <em>never</em> published nor shared."

#1 @bumbu
15 years ago

  • Cc bmbalex@… added
  • Keywords has-patch added

#2 @nacin
15 years ago

I'd rather opt for "will not be published". People need to know that by submitting a comment, it will not show up with their comment.

Any theme that does that (such as a private site) should change the text. I agree the current text places a burden on the wrong people, but I think we should lighten its tone and promise instead of completely striking it.

#3 @bumbu
15 years ago

So what to do? To change into "will not be published" or to move this in template files?

#4 @nacin
15 years ago

My opinion is that I would like to see:

'comment_notes_before' => '<p class="comment-notes">' . __( 'Your email will not be published.' ) . ( $req ? __( ' Required fields are marked <span class="required">*</span>' ) : '' ) . '</p>',

This matches Kubrick, which was "(will not be published)" after the field label.

@bumbu
15 years ago

Now just 'Your email will not be published.'

#5 @nacin
15 years ago

(In [14132]) Switch to simpler language regarding publication of email addresses in comment_form() template. props bumbu, see #13026.

@lloydbudd
15 years ago

Great points nacin! We should actually make it explicit. There are so many naive people out there, and so much poor language out there regarding contact forms being "email", so here is a patch with "email address" -- you knew this was the true path based on your commit msg of "regarding publication of email addresses" ;-)

#6 @nacin
15 years ago

  • Owner set to nacin
  • Status changed from new to accepted

Didn't see your comment, as attachments don't trigger a notification. (I'll add that I like that it doesn't trigger a notification, but that's another story.)

#7 @nacin
15 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

(In [14195]) Clarify language for non-publication of emails. props lloydbudd, fixes #13026.

Note: See TracTickets for help on using tickets.