Download & Extend

Remove redundant code from forum.module

Project:Drupal core
Version:x.y.z
Component:forum.module
Category:bug report
Priority:normal
Assigned:Stefan Nagtegaal
Status:closed (fixed)

Issue Summary

forum.module uses _forum_depth(), which is depricated. The same function is present in the taxonomy.module, and does exactly the same thing.. Here it's called, _taxonomy_depth()..

Attached small patch removes the function _forum_depth() and all it's references..

Please review and apply...

AttachmentSizeStatusTest resultOperations
forum-module_1.patch1.88 KBIgnored: Check issue status.NoneNone

Comments

#1

+1

Forums already require taxonomy.module, so no new dependancy is added, code is simplified.

#2

Status:needs review» reviewed & tested by the community

i read through this simple patch and it is ready for commit.

#3

Status:reviewed & tested by the community» fixed

Committed to HEAD. Good catch.

#4

Status:fixed» closed (fixed)
nobody click here