Default block cache mode

Howdy...

Just came across this today, so I thought I'd give you a heads up. It looks like the hook_block() call in the module doesn't set any block cache mode. So by default D6 assigns (BLOCK_CACHE_PER_ROLE).

However, since the autocomplete.js is added during the building of the block, if someone turns on block caching it isn't included. The block is returned properly from cache (of course :) ) but without the autocomplete.js it's not very exciting. So I guess the solution would be to either move that outside the block building process or set the blocks to BLOCK_NO_CACHE.

The only snag on doing BLOCK_NO_CACHE, is that Drupal currently doesn't update the existing cache mode of blocks in the block table when hook_block is run again. :/ Which sucks. But at least you'll know :)