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.

Files: 
CommentFileSizeAuthor
#12 372416-12.patch856 bytesjibran
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]
#9 372416-9.patch1.34 KBnaxoc
PASSED: [[SimpleTest]]: [MySQL] 39,559 pass(es).
[ View ]
#6 372416-6.patch1.35 KBnaxoc
PASSED: [[SimpleTest]]: [MySQL] 42,382 pass(es).
[ View ]
#2 372416-3.patch1.34 KBnaxoc
PASSED: [[SimpleTest]]: [MySQL] 39,570 pass(es).
[ View ]

Comments

Title:comment_wrapper.tpl.php documentation references a non-existent functioncomment-wrapper.tpl.php documentation references a non-existent function

Version:6.9» 7.x-dev
Status:Active» Needs review
StatusFileSize
new1.34 KB
PASSED: [[SimpleTest]]: [MySQL] 39,570 pass(es).
[ View ]

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

Here is a patch.

Component:comment.module» documentation

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.

Issue tags:+Novice

adding Novice tag

Status:Needs work» Needs review
StatusFileSize
new1.35 KB
PASSED: [[SimpleTest]]: [MySQL] 42,382 pass(es).
[ View ]

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

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.

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.

Status:Patch (to be ported)» Needs review
StatusFileSize
new1.34 KB
PASSED: [[SimpleTest]]: [MySQL] 39,559 pass(es).
[ View ]

Here is the 7.x patch from #2

Assigned:Unassigned» jhodgdon
Status:Needs review» Reviewed & tested by the community

Thanks! I'll get that committed shortly.

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.

Status:Patch (to be ported)» Needs review
StatusFileSize
new856 bytes
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]

D6 patch

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.