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

Comments

xano’s picture

xano
he/they
Primary language English
Location Southampton
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
commented
Status: Needs review » Reviewed & tested by the community

Much better.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

catch’s picture

catch
he/him
Primary language English
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
commented

Will do.

//Edit: done!

swentel’s picture

swentel commented
Status: Needs work » Fixed

Marking as fixed then :)

Status: Fixed » Closed (fixed)

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