Mailhandler borks at mailhandler_node_process_message_commands() if taxonomy.module is not enabled, albeit there is no dependencies[] = taxonomy in the .info file.
It would also be nice to be able to use the "type: nodetype" body command without the dependency for taxonomy.

Comments

bibo’s picture

bibo commented
Status: Active » Needs review

I also just noticed this. Got this while trying to retrieve new mails (via the interface link, not cron).

Fatal error: Call to undefined function taxonomy_get_vocabularies() in [path_to_modules]/mailhandler/mailhandler.module on line 575"

Adding this in "mailhandler_node_process_message_commands()"-function: 

if(function_exists('taxonomy_get_vocabularies')){
 ...

.. should do the trick without having to add taxonomy as a dependency. I don't want to manually patch mailhandler though, so I'll just activate the taxonomy-module for now.

Ian Ward’s picture

Ian Ward commented
Status: Needs review » Fixed

Thanks, committed a fix - checks whether taxonomy module is available and otherwise logs to watchdog.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.