API page: http://api.drupal.org/api/drupal/modules%21forum%21forums.tpl.php/7

There is an @see to theme_forums() in forums.tpl.php. This function does not exist, so it should be removed.

#1 drupal8_core-remove_theme_forums_function_from_forums_template_file.patch383 bytessgtsaughter
PASSED: [[SimpleTest]]: [MySQL] 37,109 pass(es). View


sgtsaughter’s picture

383 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,109 pass(es). View
jhodgdon’s picture

Status: Active » Reviewed & tested by the community

The patch looks good to me! When uploading a patch, you need to set the issue status to "needs review", to alert both humans and test-bots that there is a patch to look at. (In this case, I'm going directly to "reviewed and tested" though, and I'll get it committed soon.)


sgtsaughter’s picture

No problem, this was my first time doing a patch so it was a learning experience and this was an easy enough issue for me to learn with. The pleasures all mine.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Excellent -- welcome to the project! Hopefully you found

Anyway, this patch has now been committed to 8.x and 7.x -- thanks again, and I hope your next patch goes well too!

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