Opened 12 years ago
Closed 12 years ago
#25880 closed defect (bug) (fixed)
wp_notify_postauthor() does not initialize the subject and content text for custom comment types
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.8 | Priority: | normal |
Severity: | minor | Version: | |
Component: | Comments | Keywords: | has-patch commit |
Focuses: | Cc: |
Description (last modified by )
Unlike wp_notify_moderator()
, that uses a switch/case()
statement that defaults to comment types of type 'comment', wp_notify_postauthor()
uses an if/else()
, resulting in $notify_message
and $subject
undefined var notices when these vars are concatenated later in that same function, when using a custom comment type:
Notice: Undefined variable: notify_message in ...wp-includes\pluggable.php on line 1076 Notice: Undefined variable: subject in ...\wp-includes\pluggable.php on line 1109
This can be easily fixed by initializing the vars ($var = ''
) or by replacing the if/else()
with a switch/case()
that defaults to 'comment'.
Attachments (1)
Change History (8)
Note: See
TracTickets for help on using
tickets.
Small correction. I've mentioned wp_notify_moderator() two times by mistake.
Correction:
Unlike, wp_notify_moderator() that uses a switch/case() statement that defaults to comment types of type 'comment', wp_notify_postauthor() uses an if/else()...