Can't imagine why there would even be code in SN module to touch the term tables; but...
whenever i modify a "newsletter" settings at url like: admin/content/simplenews/types/edit/155
and hit submit... SN modifies the structure of my taxonomy..
in case it matters, structure is like
Travel
- Air
- Marine
if i edit newsletter for Marine, and submit, i end up with this:
Marine
Travel
- Air
and i do get a msg saying "Updating term Marine"
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | keep_heirarchy.patch | 778 bytes | liquidcms |
Comments
Comment #1
liquidcms commentedahh... now i see.. in the submit function for that form there is code for:
// Create or update taxonomy term for this newsletter.
// When creating a term $form_state['values']['tid'] is NULL.
i think you are trying to let that form set the terms Description, weight, etc.. but not doing it correctly so it messes up tax structure.
i'll see about patch.. or maybe just comment all that out.. since i dont need it.
Comment #2
liquidcms commentedand here is the patch to fix.
Comment #3
liquidcms commentedComment #4
simon georges commentedI don't think newsletters have been thought to have a hierarchy, that's probably the reason, but good catch anyway, and thanks for the patch !
Commited, to be ported to 7.x-1.x.
Comment #5
simon georges commentedI just tested, there's no need for the patch in 7.x-1.x.