Make WordPress Core

Opened 21 years ago

Closed 21 years ago

Last modified 20 years ago

#1698 closed defect (bug) (fixed)

Comment notification emails are not being sent

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

Description

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

Attachments (1)

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

Download all attachments as: .zip

Change History (10)

#1 @MtDewVirus
21 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.

@ringmaster
21 years ago

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

#2 @ringmaster
21 years ago

  • Keywords bg|patch added

#3 @markjaquith
21 years ago

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

#4 @markjaquith
21 years ago

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

#5 @skippy
21 years ago

  • Keywords bg|squash added

I second -- this fixes the problem.

#6 @skippy
21 years ago

  • Keywords bg|squashed added; bg|squash removed

#7 @ryan
21 years ago

Fixed by [3004]

#8 @ryan
21 years ago

  • Milestone1.6
  • Resolutionfixed
  • Status assignedclosed

#9 @(none)
20 years ago

  • Milestone 2.0

Milestone 2.0 deleted

Note: See TracTickets for help on using tickets.