Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 15 years ago

Closed 13 years ago

#17862 closed enhancement (fixed)

The second argument for wp_notify_postauthor() is unnecessary

Reported by:	scribu	Owned by:	westi
Priority:	normal	Milestone:	3.8
Component:	Comments	Version:
Severity:	normal	Keywords:	has-patch commit
Cc:		Focuses:

Description

Only the comment id is needed. See patch.

Attachments (4)

17862.diff (2.2 KB ) - added by scribu 15 years ago.
17862.2.diff (2.6 KB ) - added by scribu 15 years ago.: Clarification in wp_new_comment()
17862.3.diff (2.7 KB ) - added by wonderboymusic 13 years ago.
17862.4.diff (1.3 KB ) - added by markjaquith 13 years ago.: simplest version, for non-conflict with #25699 patches

Download all attachments as: .zip

Change History (7)

@scribu
15 years ago

Attachment 17862.diff added

@scribu
15 years ago

Attachment 17862.2.diff added

Clarification in wp_new_comment()

#1 @SergeyBiryukov
13 years ago

Milestone Future Release → 3.8

@wonderboymusic
13 years ago

Attachment 17862.3.diff added

#2 @wonderboymusic
13 years ago

Keywords commit added

.3.diff makes the patch apply against trunk and adds our new required braces

#3 @westi
13 years ago

Owner set to westi
Resolution → fixed
Status new → closed

Deprecate the second argument for wp_notify_postauthor because it is unecessary. Fixes #17862 props scribu and wonderboymusic.

@markjaquith
13 years ago

Attachment 17862.4.diff added

simplest version, for non-conflict with #25699 patches

Note: See TracTickets for help on using tickets.

Download in other formats: