Would you consider a patch for this? I'm not sure what the use case is for exposing, but this will make it optional.

Thanks

CommentFileSizeAuthor
#5 tax_depth_20110208.patch1.57 KBchawl
tax_depth.patch1.73 KBawolfey

Comments

chawl’s picture

Status: Active » Reviewed & tested by the community

You saved my life awolfey. I don't have a dropdown with several-thousand children terms anymore.

This simple, neat and perfectly working patch should be committed ASAP (if this module is alive anyway :/) .

asb’s picture

Status: Reviewed & tested by the community » Needs work

Sorry, this patch doesn't work for me when appying it from within ./sites/all/modules/tax_depth_lim_filter against 6.x-1.1, neither with or without the -p0 option:

# patch -p0 < tax_depth_0.patch 
(Stripping trailing CRs from patch.)
can't find file to patch at input line 7
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|# This patch file was generated by NetBeans IDE
|# This patch can be applied using context Tools: Apply Diff Patch action on respective folder.
|# It uses platform neutral UTF-8 encoding.
|# Above lines and this line are ignored by the patching process.
|--- C:\wamp\www\automobius\sites\all\modules\tax_depth_lim_filter\views_handler_filter_term_node_tid_depth_limited_1.inc
|+++ C:\wamp\www\automobius\sites\all\modules\tax_depth_lim_filter\views_handler_filter_term_node_tid_depth_limited.inc
--------------------------
File to patch: ^C
awolfey’s picture

I ended up not using this module after all. Maybe chawl can reroll the patch. Or, it's only a few lines, could easily be done by hand.

asb’s picture

Thank Aaron, I'm still interested to get this option into the module's code.

May I ask what alternative you have found to this module?

Greetings, -asb

chawl’s picture

StatusFileSize
new1.57 KB

Ok, try this from module directory.

asb’s picture

Status: Needs work » Reviewed & tested by the community

Thanks chawl, this patch applies cleanly and works perfectly!

mstrelan’s picture

+1

MBroberg’s picture

This is very needed, thank you! Works great for me, although terminology is a little confusing so perhaps an example would help.

hanskuiters’s picture

Thanks for the patch.