Closed (fixed)
Project:
Taxonomy Manager
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
3 May 2013 at 20:32 UTC
Updated:
6 Jun 2020 at 01:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Anonymous (not verified) commentedpatch is attached
Comment #2
Anonymous (not verified) commentedComment #3
apotek commentedThis is a good point. A move operation currently doesn't allow any other module to hook into it. The two ways to deal with this as far as I can see, is to do what @SpaJenniOs is suggesting in her patch, which, by calling taxonomy_term_save(), will call all the following hooks:
which would allow other modules to interact with the move operation.
The other option is to modify the move function by making it create its own hooks that other modules can hook into, ie, put something like this in the function:
I think SpaJenniOs's solution is the preferable one since it relies on the taxonomy module's code and doesn't create a new hook, but if there is a reason taxonomy_term_save cannot be used, creating a hook for other module developers to use will work too.
@SpaJenniOs: If you roll this patch against the dev branch instead, I think you can get your patch into the testing bot.
Comment #4
Anonymous (not verified) commentedComment #5
Anonymous (not verified) commentedComment #6
mh86 commentedCommitted a slightly adapted version of #5. See http://drupalcode.org/project/taxonomy_manager.git/commitdiff/505c6eb
Thanks a lot for the patch.