At first glance I thought this did nothing; the theme function sets messages however.

This could really do with a comment, as calling theme without printing or assigning looks very odd.

      if ($node->moderate) { //cut this?
        theme('modr8_message', FALSE, $node->type, $op);
      }