taxonomy_del_term is passing it's third arg as an object when it invokes hook taxonomy--this makes the delete case of hook_taxonomy inconsistent, which is reflected in forum_taxonomy--it won't work properly ATM. attached patch replaces the recently removed array cast in taxonomy_del_term, changes forum_taxonomy to use an array instead of an object, and removes a bit of extra whitespace. casting to an array makes the most sense here since all other cases of hook_taxonomy have their third arg as an array...

CommentFileSizeAuthor
taxo_forum.patch2.7 KBhunmonk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dopry’s picture

There are several other places where the cast as array has been done. Form API uses arrays, php post is an array... (why do we use objects at all?) On a visual inspection everything seems in order. Hunmonk says it works. I don't have forums setup anywhere to test right now.

+1.

Steven’s picture

Status: Active » Fixed

All other taxonomy ops use arrays. +1.

Committed to HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)