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...

CommentFileSizeAuthor
forum-module_1.patch1.88 KBStefan Nagtegaal
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sethcohn’s picture

sethcohn CreditAttribution: sethcohn commented

+1

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

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

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

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Good catch.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)