Opened 3 years ago

Last modified 3 years ago

#14078 new enhancement

Don't send notifications for comments too early

Reported by: mrmist Owned by:
Priority: normal Milestone: Future Release
Component: General Version: 3.0
Severity: normal Keywords: has-patch needs-testing
Cc:

Description

Not sure if this classes as a bug or enhancement, will call it an enhancement for now.

In wp_set_comment_status the notifications are currently sent out before the work is done, and there is potential for the subsequent work to fail (due to database issue etc.)

Attached patch (attempts to) move the notification so it is only sent after the work is done.

Attachments (1)

send_notification_later.patch​ (743 bytes) - added by mrmist 3 years ago.
Move notifications

Download all attachments as: .zip

Change History (3)

mrmist — 3 years ago

Move notifications

comment:1   ocean90 — 3 years ago

  • Milestone changed from Unassigned to Future Release

I think there is a typo in your patch, you have commented $comment out.

comment:2   mrmist — 3 years ago

Think that's ok.. $comment is already set by that stage. Not sure why I left the comment in... Rush job.

Note: See TracTickets for help on using tickets.