WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#12006 closed defect (bug) (fixed)

Comment template functions should accept optional comment ID argument.

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

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 15 years ago.
comment_ID.arugment-sans-ternary.12006.diff​ (8.1 KB) - added by filosofo 15 years ago.

Download all attachments as: .zip

Change History (5)

@filosofo
15 years ago

#1 follow-up: @ryan
15 years ago

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

@filosofo
15 years ago

#2 in reply to: ↑ 1 @filosofo
15 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
15 years ago

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

[12821]

Note: See TracTickets for help on using tickets.