WordPress.org

Make WordPress Core

Opened 10 years ago

Closed 9 years ago

Last modified 8 years ago

#1698 closed defect (bug) (fixed)

Comment notification emails are not being sent

Reported by: MtDewVirus Owned by: markjaquith
Milestone: Priority: high
Severity: major Version: 1.6
Component: Administration Keywords: comment email notification bg|has-patch bg|commit bg|squashed
Focuses: Cc:

Description

The emails to notify of new comments are not being sent.

Attachments (1)

pluggable-functions.php.diff (1.1 KB) - added by ringmaster 10 years ago.
Patch to use new caps structure for user data in this function.

Download all attachments as: .zip

Change History (10)

comment:1 @MtDewVirus10 years ago

With http://trac.wordpress.org/changeset/2827, the following line was removed from function wp_notify_postauthor

$user = $wpdb->get_row("SELECT * FROM $wpdb->users WHERE ID='$post->post_author' LIMIT 1");

This causes the next line to always return false because $user isn't set.
if ( == $user->user_email) return false; If there's no email to send the comment to
Comment notificatons will never be sent.

@ringmaster10 years ago

Patch to use new caps structure for user data in this function.

comment:2 @ringmaster10 years ago

  • Keywords bg|patch added

comment:3 @markjaquith10 years ago

  • Keywords bg|has-patch bg|commit added; bg|patch removed
  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned

comment:4 @markjaquith10 years ago

Just chiming in to say that I definitely tested this patch, and it definitely fixes the problem. Go go Gadget commit!

comment:5 @skippy9 years ago

  • Keywords bg|squash added

I second -- this fixes the problem.

comment:6 @skippy9 years ago

  • Keywords bg|squashed added; bg|squash removed

comment:7 @ryan9 years ago

Fixed by [3004]

comment:8 @ryan9 years ago

  • Milestone set to 1.6
  • Resolution set to fixed
  • Status changed from assigned to closed

comment:9 @anonymous8 years ago

  • Milestone 2.0 deleted

Milestone 2.0 deleted

Note: See TracTickets for help on using tickets.