Closed (fixed)
Project:
Taxonomy Term Reference Tree Widget
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 May 2012 at 12:50 UTC
Updated:
13 May 2019 at 16:15 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
FranckV commentedI had the same issue with version 7.x-1.9. The issue seems to come from a bad treatment when dealing with a non mandatory field where the "N/A" empty case generates a null.
Here is my fix, it should work for most of the case :
ORIGINAL CODE in modules/term_reference_tree/term_reference_tree.module
NEW CODE in modules/term_reference_tree/term_reference_tree.module
Comment #2
stongo commentedFranckV, thanks for the suggestion, it is a good starting point. The else statement is extraneous and should still check the first condition in the OR.
Here's a simpler patch against 7.x-1.x-dev that fixes the same issue.
Comment #3
stongo commentedActually, scrap the above patch, all it needs is an isset() in the original if condition.
Here's another patch.
Comment #4
dave reidComment #5
kbasarab commentedUpdates patch in #3 to apply to 7.x-1.10 and 7.x-1.x dev.
Comment #6
pifagorLook good
Comment #8
alex_optimComment #9
alex_optimComment #10
alex_optim