Ticket #5448 (closed defect (bug): fixed)

Opened 4 years ago

Last modified 4 years ago

Comments-template echoes strange comment link

Reported by: reinkim Owned by: westi
Priority: normal Milestone: 2.5
Component: Template Version: 2.5
Severity: normal Keywords: has-patch tested regression
Cc:

Description

After applying changeset [6368] comments_popup_link function in wp-includes/comments-template.php behalves differently from previous versions.

In [6368] comments_popup_link assembles $comments_popup_link variable, but in some execution path there are codes which echo the output, which should be appended to $comments_popup_link variable.

Problematic codes (all source lines from wp-includes/comments-template)

  • function comments_number echoes its result
  • function comments_link echoes its output

I made fix to the source (attached diff. file) which may need some tests.

Attachments

diff.txt Download (1.1 KB) - added by reinkim 4 years ago.
source diff

Change History

reinkim — 4 years ago

source diff

comment:1   pishmishy — 4 years ago

  • Keywords has-patch tested added
  • Version changed from 2.4 to 2.3.1
  • Milestone changed from 2.4 to 2.3.2

Confirmed. Tested patch, fixes issue. Needs to go into next release.

comment:2   westi — 4 years ago

  • Keywords regression added
  • Owner changed from anonymous to westi
  • Version changed from 2.3.1 to 2.4
  • Status changed from new to assigned
  • Milestone changed from 2.3.2 to 2.4

comment:3   ryan — 4 years ago

comments_number() can't be changed to always return since that could break some templates. We could hijack $deprecated and turn it into an $echo boolean, however.

comment:4   matt — 4 years ago

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

(In [6372]) Revert [6368]. Fixes #5448, fixes #5247 by adding a new filter to hook in specifically to add attributes, rather than filtering the whole thing. Minor formatting cleanup of comments_popup_link.

Note: See TracTickets for help on using tickets.