Remove redundant code from forum.module

Stefan Nagtegaal - October 25, 2005 - 14:52
Project:Drupal
Version:x.y.z
Component:forum.module
Category:bug report
Priority:normal
Assigned:Stefan Nagtegaal
Status:closed
Description

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 KBIgnoredNoneNone

#1

sethcohn - October 25, 2005 - 15:13

+1

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

#2

moshe weitzman - October 25, 2005 - 21:05
Status:needs review» reviewed & tested by the community

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

#3

Dries - October 28, 2005 - 13:56
Status:reviewed & tested by the community» fixed

Committed to HEAD. Good catch.

#4

Anonymous - November 11, 2005 - 14:00
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.