Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#12006 closed defect (bug) (fixed)

Comment template functions should accept optional comment ID argument.

Reported by: filosofo Owned by: ryan
Priority: normal Milestone: 3.0
Component: Comments Version: 3.0
Severity: normal Keywords: comments has-patch
Cc: Focuses:

Description

Otherwise, to get basic comment info you have to define, populate, and reset afterwards a global $comment variable, which is unnecessarily messy.

Patch adds a comment ID variable where helpful and backwards-compatibility safe.

Attachments (2)

comment_ID.argument.12006.diff (9.6 KB ) - added by filosofo 17 years ago.
comment_ID.arugment-sans-ternary.12006.diff (8.1 KB ) - added by filosofo 17 years ago.

Download all attachments as: .zip

Change History (5)

#1 follow-up: @ryan
17 years ago

Since get_comment() returns the global comment if passed on empty ID, can we lose the empty() ternaries?

#2 in reply to: ↑ 1 @filosofo
17 years ago

Replying to ryan:

Since get_comment() returns the global comment if passed on empty ID, can we lose the empty() ternaries?

Absolutely. Second patch drops ternaries.

#3 @ryan
17 years ago

  • Resolutionfixed
  • Status newclosed
Note: See TracTickets for help on using tickets.