I think that the hook_taxonomy_vocabulary_load hook is missing either a return statement or referenced parameter. Without any of these the function is not doing its magic.

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Issue tags: +Novice

Yes, that should be by reference. Once in the function definition and once in the foreach, so that the single vocabs are by reference too. Looks like a nice Novice patch.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Closed (won't fix)

Actually, this is just fine.

$vocabularies is an array of vocabulary objects, and these are always by reference. It would not be possible to change the array (remove/add vocabs) but we don't want that anyway.

MiMe’s picture

MiMe commented

It would not be possible to change the array (remove/add vocabs) but we don't want that anyway.

I hope that I will be able to add my own stuff to the $vocabularies array?! If not, it's not a hook, then it's just a module function...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

See #471744: Removing terms