The documentation in comment_wrapper.tpl.php contains a reference to the nonexistence function theme_comment_wrapper() on line 31:

 * @see template_preprocess_comment_wrapper()
 * @see theme_comment_wrapper()
 */
?>

The last line should be removed, as that theme function was replaced with a template file in Drupal 6.x.

CommentFileSizeAuthor
#12 372416-12.patch856 bytesjibran
#9 372416-9.patch1.34 KBnaxoc
#6 372416-6.patch1.35 KBnaxoc
#2 372416-3.patch1.34 KBnaxoc

Comments

todd nienkerk’s picture

todd nienkerk commented
Title: comment_wrapper.tpl.php documentation references a non-existent function » comment-wrapper.tpl.php documentation references a non-existent function
naxoc’s picture

naxoc commented
Version: 6.9 » 7.x-dev
Status: Active » Needs review
StatusFileSize
new 372416-3.patch1.34 KB

This has gotten me confused a couple of times too. Problem still is in D7.

Here is a patch.

naxoc’s picture

naxoc commented
Component: comment.module » documentation
jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 7.x-dev » 8.x-dev
Status: Needs review » Needs work
Issue tags: +Needs backport to D6, +Needs backport to D7

Thanks! This needs to be fixed in Drupal 8.x first, and it needs to be fixed in both the modules/comment and themes/bartik versions of comment-wrapper.tpl.php. Then we can backport that fix to Drupal 7 and Drupal 6.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue tags: +Novice

adding Novice tag

naxoc’s picture

naxoc commented
Status: Needs work » Needs review
StatusFileSize
new 372416-6.patch1.35 KB

Woops. Sorry. I thought I had looked for them in D8. New D8 patch.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: Unassigned » jhodgdon
Status: Needs review » Reviewed & tested by the community

Thanks! That looks like the right patch for D8 -- I'll get it committed shortly.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks again -- committed to 8.x.

It looks like the same 3 files need to be patched for d7 but they're in different locations.

naxoc’s picture

naxoc commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 372416-9.patch1.34 KB

Here is the 7.x patch from #2

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: Unassigned » jhodgdon
Status: Needs review » Reviewed & tested by the community

Thanks! I'll get that committed shortly.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 7.x-dev » 6.x-dev
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks! This is in 7.x now too. Still needs a backport to 6.x.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 372416-12.patch856 bytes

D6 patch

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Fixed

Thanks! this one is committed to 6.x.

Status: Fixed » Closed (fixed)
Issue tags: -Needs backport to D6, -Novice, -Needs backport to D7

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

Issue tags: +Needs backport to D6, +Novice, +Needs backport to D7

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.