Make WordPress Core

Opened 16 years ago

Closed 11 years ago

#10177 closed enhancement (fixed)

get_ version of comments_number()

Reported by: nbachiyski's profile nbachiyski Owned by: wonderboymusic's profile wonderboymusic
Milestone: 4.0 Priority: normal
Severity: normal Version: 2.8
Component: Comments Keywords: has-patch i18n
Focuses: template Cc:

Description

Currently: comments_number() accepts several arguments for the comment number labels and outputs them. get_comments_number(), however, returns only the number of comments.

Problem: there isn't a function to mimic the comments_number() functionality, but to return instead of echo.

The patch introduces get_comments_number_text(), which inherits the functionality of comments_number(), but returns. comments_number() just calls get_comments_number_text().

Attachments (3)

get-comments-number-text.diff (1.4 KB) - added by nbachiyski 16 years ago.
10177.diff (1.3 KB) - added by kapeels 15 years ago.
10177.2.diff (1.9 KB) - added by wonderboymusic 11 years ago.

Download all attachments as: .zip

Change History (12)

#1 @ryan
16 years ago

  • Milestone changed from 2.9 to 3.0

#2 @nacin
15 years ago

  • Keywords early added
  • Milestone changed from 3.0 to 3.1
  • Type changed from defect (bug) to enhancement

#3 @nacin
15 years ago

  • Milestone changed from Awaiting Triage to 3.1

Instead of call_user_func_array() it would be cleaner to just pass the three arguments.

#4 @westi
15 years ago

  • Milestone changed from 3.1 to Future Release

Moving to Future Release as it is an enhancement.

#5 @kapeels
15 years ago

  • Cc kapeel.sable@… added
  • Keywords 2nd-opinion added

Would be great if you guys could give me points in gci for this bug-fix :)

@kapeels
15 years ago

#6 @nacin
11 years ago

  • Component changed from Template to Comments
  • Focuses template added

#7 @obenland
11 years ago

  • Keywords needs-refresh added; early 2nd-opinion removed

#8 @wonderboymusic
11 years ago

  • Keywords needs-refresh removed
  • Milestone changed from Future Release to 4.0

10177.2.diff does it.

#9 @wonderboymusic
11 years ago

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

In 28912:

Add a function, get_comments_number_text(), that returns instead of echoing. comments_number() wraps it.

Props kapeels, nbachiyski.
Fixes #10177.

Note: See TracTickets for help on using tickets.