If $depth is for "Internal use only" then why is it in the middle of the parameter list? If we're not supposed to use it, we shouldn't have to care about its default value either.

CommentFileSizeAuthor
taxonomy_get_tree-depth.patch2.7 KBJohn Morahan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

Good idea. I can confirm this patch also updates all occurences of taxonomy_get_tree() in core where needed.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Needs review » Reviewed & tested by the community

Much better.

Dries’s picture

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

Committed to CVS HEAD. Thanks.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Fixed » Needs work

This probably deserves a mention in http://drupal.org/update/modules

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

Will do.

//Edit: done!

swentel’s picture

swentel CreditAttribution: swentel commented
Status: Needs work » Fixed

Marking as fixed then :)

Status: Fixed » Closed (fixed)

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