It would be great if taxonomy.module used nodeapi 'load' operation to load taxonomy terms for given node.
Right now, if module saves nodes (like blogapi for example), it has to load terms by hand. It's not too clean to use "alien" functions in own module (ie. using functions from other modules) - it creates dependancies which are very likely to be broken at next release...
At first i thought that taxonomy.module doesn't load terms at 'load' call to not to add too much db calls and kb of data, but it seems that terms are loaded anyway (taxonomy/term/X page, on node view page, and probably others), so why not load it clean way?

I can make a patch for taxonomy.module and all core modules if there's chance of it to be accepted :)

Comments

moshe weitzman’s picture

Status: Active » Fixed
ahwayakchih’s picture

Status: Fixed » Active

I'm reopening it: http://cvs.drupal.org/viewcvs/drupal/drupal/modules/taxonomy.module?view...
Look at nodeapi hook - it still doesn't load terms for node. If some other module (like aggregator2) loads nodes and then saves them, it has to manipulate terms now. It would be great if taxonomy module would load terms automatically at node load time.

moshe weitzman’s picture

Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)