Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#30930 closed defect (bug) (fixed)

Duplicate strings in comment notification

Reported by: pavelevap's profile pavelevap Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.2 Priority: normal
Severity: normal Version: 4.1
Component: I18N Keywords:
Focuses: Cc:

Description

Ticket #16271 was fixed in [30015] only partially for wp_notify_postauthor(), but missed for wp_notify_moderator(). In the end there were created several duplicate strings. We can remove 6 strings, see attached patch.

P.S. I am not sure if space before comma was intented and should be removed now? See string with IP: %2$s , %3$s.

Attachments (1)

30930.patch (3.2 KB) - added by pavelevap 9 years ago.

Download all attachments as: .zip

Change History (5)

@pavelevap
9 years ago

#1 @SergeyBiryukov
9 years ago

  • Milestone changed from Awaiting Review to 4.2

#2 @SergeyBiryukov
9 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 31059:

Remove padding from the comment notification emails in wp_notify_moderator().

See [30015] for wp_notify_postauthor().

props pavelevap.
fixes #30930.

#3 @SergeyBiryukov
9 years ago

In 31060:

Remove space before comma in wp_notify_postauthor() and wp_notify_moderator().

see #30930.

#4 @pavelevap
9 years ago

Thank you, Sergey!

Note: See TracTickets for help on using tickets.