It seems that the following lines:

// Add the pager variable to the title link if it needs it.
$fragment = 'comment-' . $vars['comment']->cid;
$query = '';
if (!empty($page)) {
$query = 'page='. $page;
}

Should be placed before, and not after:

// If comment subjects are disabled, don't display them.
if (variable_get('comment_subject_field_' . $vars['node']->type, 1) == 0) {
$vars['title'] = '';
}
else {
$vars['title'] = l($vars['comment']->subject, 'node/'. $vars['node']->nid, array('query' => $query, 'fragment' => $fragment));
}

Since current status creates a report about missing $fragment and $query variables. A bug not found in earlier versions where the to blocks had the opposite placement.

Comments

designerbrent’s picture

Status: Active » Fixed

Fixed in the current Dev version.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.