Add "in reply to" in comment moderation email notification

[galbaras]

The comment moderation notification email provides quick ways to moderate comments, which can save time and hassle.

Sadly, the "in reply to" information, which is available on the moderation page, is missing from these emails, so moderators lack some of the context of the comment.

As you can imagine, the meaning of a comment is vastly different when it relates to the post, as opposed to some stupid comment that someone made, and may make a difference to the moderation outcome.

Therefore, at least when reply threading is enabled, moderation emails should include the "in reply to" comment link and author name.

While we're at it, both the moderation page and email would benefit from the actual replied-to comment to be shown, in order to avoid the hassle of another page load to enable proper moderation.

This can be done in combination with #33627, where every change done on the moderation page is also reflected in the emails.

[danieltj]

I'm for adding an in reply to link with a link to the comment within the wp-admin, but I'm against any other information. Considering the fact you're moderating the reply, having the replied to comment itself doesn't seem necessary here.

[danieltj]

Include post parent URL in notify mod email

[danieltj]

I've added 43805.diff​ which includes the URL to the parent comment if one has been set. I was going to change the milestone to Future Release but let's see what people think. Haven't managed to test the patch yet, but if someone wants to give it a try and provide any feedback, please do.

[zodiac1978]

Related #43429

[davidbaumwald]

This ticket hasn't received much attention this cycle. With the deadline for enhancements in 5.3 now passed with Beta 1, this is being moved to Future Release.

[andraganescu]

Refreshed patch 43805.diff

[andraganescu]

I refreshed the patch in 43805.2.diff because the existing one did not apply cleanly with grunt patch:43805.

Still needs testing.

[imath]

Hi @andraganescu

Thanks a lot for the patch and for refreshing it.

1. I advise you to use the --no-prefix option when building your patch so that people using SVN can apply it.
2. I believe it's best to cast the comment_parent as an integer for the strict comparison as get_comment() is returning a string for the comment_parent property.
3. I guess if we include the "in reply to" information for post authors, we should probably also include this information for moderators.

43805.3.diff is adding these 3 points and 4 unit tests. I've also tested the patch regularly with success.

[imath]

Hi !

I've tested the patch once more to be sure. I confirm it behaves as expected and I believe it's Ok to commit it. Thanks everyone for your contributions on this ticket.

[SergeyBiryukov]

In 47114:

Comments: Add In reply to: %s parent comment link to new comment email notifications.

This information was previously available on the Moderate Comment screen in the admin, but was missing from moderation emails.

Props imath, danieltj, andraganescu, galbaras, SergeyBiryukov.
Fixes #43805. See #43429.