Opened 2 years ago

Closed 19 months ago

#17655 closed defect (bug) (invalid)

Fix deprecated v1.2 pattern in comments_popup_link()

Reported by: miqrogroove Owned by:
Priority: normal Milestone:
Component: General Version: 2.0
Severity: normal Keywords: has-patch
Cc:

Description

In v2.0, someone added a ridiculous if block in comments_popup_link() that duplicated the logic already contained in get_comments_link().

The patch is simple.

Attachments (1)

comments-link-fix-by-miqrogroove.patch​ (635 bytes) - added by miqrogroove 2 years ago.

Download all attachments as: .zip

Change History (3)

miqrogroove — 2 years ago

comment:1   SergeyBiryukov — 21 months ago

  • Keywords has-patch added

comment:2   nacin — 19 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

With comments, comments_link()/get_comments_link() would correctly provide permalink#comments. However, with no comments, comments_popup_link() links to #respond.

It's silly and should have been standardized on introduction, but I'm gonna have to close this.

One of these days we will kill off comments_popup_link() for being a pain, especially when it comes to internationalization.

Note: See TracTickets for help on using tickets.