Posted by John Morahan on December 5, 2008 at 3:56pm
6 followers
Jump to:
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | taxonomy.module |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| taxonomy_get_tree-depth.patch | 2.7 KB | Idle | Passed: 7524 passes, 0 fails, 0 exceptions | View details |
Comments
#1
Good idea. I can confirm this patch also updates all occurences of taxonomy_get_tree() in core where needed.
#2
Much better.
#3
Committed to CVS HEAD. Thanks.
#4
This probably deserves a mention in http://drupal.org/update/modules
#5
Will do.
//Edit: done!
#6
Marking as fixed then :)
#7
Automatically closed -- issue fixed for two weeks with no activity.