Override processing at admin/content/taxonomy to trigger lineage_update_all()
xjm - October 24, 2009 - 09:00
| Project: | Taxonomy Lineage |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
Currently, if one uses the "Reset to alphabetical" option at admin/content/taxonomy/#, the lineages are not updated and therefore end up being entirely wrong. This action needs to trigger lineage_update_all().
I'm not quite sure how to go about fixing this bug, as "Reset to alphabetical" doesn't trigger hook_taxonomy().
#1
Perhaps with
lineage_form_taxonomy_overview_terms_alter().#2
Here's a patch to override the form action following the "Reset to alphabetical" confirmation so that we can run
lineage_update_all()after the vocabulary weights have been set. I also added an optional vid tolineage_update_all()so we can act only on the affected rows.Tested and seems to work fine on my sandbox.
#3
Here's a better patch. The original was fewer lines, but interrupted the normal flow of form processing, whereas this one simply overrides the whole submit handler. It will be easier to maintain and also allows lineage_update_all to be called just once after the reordering form has been processed (to ensure the correct lineages for all terms).
Applied and tested; works fine.
#4
Committed
#5
Automatically closed -- issue fixed for 2 weeks with no activity.