Make WordPress Core

Opened 19 years ago

Closed 19 years ago

Last modified 17 years ago

#1698 closed defect (bug) (fixed)

Comment notification emails are not being sent

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


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

Attachments (1)

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

Download all attachments as: .zip

Change History (10)

#1 @MtDewVirus
19 years ago

With, 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.

19 years ago

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

#2 @ringmaster
19 years ago

  • Keywords bg|patch added

#3 @markjaquith
19 years ago

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

#4 @markjaquith
19 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
19 years ago

  • Keywords bg|squash added

I second -- this fixes the problem.

#6 @skippy
19 years ago

  • Keywords bg|squashed added; bg|squash removed

#7 @ryan
19 years ago

Fixed by [3004]

#8 @ryan
19 years ago

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

#9 @(none)
17 years ago

  • Milestone 2.0 deleted

Milestone 2.0 deleted

Note: See TracTickets for help on using tickets.