I was trying to use the Parent Term ID limit, I had no chance to get it working, I guess it is due to this:

$element['#parent_tid'] = $settings['parent_term_id'] || $field['settings']['allowed_values'][0]['parent'];

shoud be

$element['#parent_tid'] = $settings['parent_term_id'] ? $settings['parent_term_id'] : $field['settings']['allowed_values'][0]['parent'];

As the first is always returning 1

After patching the feature is working correctly.
Thanks for this very useful mudule.

CommentFileSizeAuthor
term_reference_tree.widget.inc_.patch893 bytesmarco.giaco

Comments

enekid’s picture

enekid commented

The patch works!

Thank you. It shows correctly the subtree.

But testing the functionality I got a question.

Supose we have this taxonomy

A
-B
--C
X

And have as "Parent Term ID limit" 'A' and the "Select parents automatically" checked. If we check 'C' now we get also 'B' added, but not 'A'. ¿Shouldn't 'A' also be added?

hephaestus’s picture

hephaestus commented

Thanks for the patch! Works for us.

OnkelTem’s picture

OnkelTem commented

Thanks for the patch!

p.s. Why it is not committed yet?

duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented

The answer to #1 is "No" (I just made the test)

I think we may open a new issue to relate this problem.

inky@inky3d.com’s picture

inky@inky3d.com commented

Patch worked for me too.

duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented
Status: Active » Reviewed & tested by the community

Let's mark it as RTBC :)

duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented

Separate issue for the automatic selection of parent terms in a filtered tree :
#1679438: Allow parents automatic selection on filtered widgets

Taxoman’s picture

Taxoman commented

#1/#4: Created separate issue here:
#1679470: Option to include all parents

sokrplare’s picture

sokrplare commented

The patch worked perfect for me with a fairly sizable list - thanks!

natuk’s picture

natuk
Primary language English
Location Cymru
commented

Works over here as well. Thanks.

Zuzuesque’s picture

Zuzuesque commented

Tested and worked for us. Thanks for the patch!

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

I can also confirm that the patch just works.

askibinski’s picture

askibinski commented

I can also confirm this patch works.

jaydub’s picture

jaydub commented

Confirmed that patch fixed issue for me as well.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x. Thanks everyone!

http://drupalcode.org/project/term_reference_tree.git/commit/b561a42

Status: Fixed » Closed (fixed)

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