Needs work
Project:
Internationalization
Version:
7.x-1.10
Component:
Taxonomy
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Sep 2013 at 21:22 UTC
Updated:
12 Dec 2013 at 17:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dmaciej commentedComment #2
dmaciej commentedTests with i18n taxonomy disabled and some fix...
Comment #3
jose reyero commentedThe idea looks good though the patch has some issues, may be improved:
- It is assuming you want to regenerate the path for every language every time, it may not be so.
- Instead of invoking one module from the other, can't we just use generic hook_taxonomy_update() kind of hooks so we don't need that many 'if module_exists' ?
Btw, set the patch to 'Needs review' when ready because otherwise the testbot -and possibly me too- will ignore it.
Comment #4
dmaciej commented@Jose Reyero:
1. I will think about it.
2. Do you suggest to change
to a form hook in pathauto module - it will be one `module_exists` still.
I don't see a way to do these things from the taxonomy hook. Could you give more explaination here?
Comment #5
dmaciej commentedComment #6
dmaciej commented