ありがとうございます．
ソースを見ると一発ですね．分離／可視グループに関わらずグループに属していなければメール送信はされない．それに加えて site:accessallgroups が設定されていないこと．学生はそうですね．しかし，”編集権限のない教師”（コースで変更可），”教師”，”マネージャー”は権限を持っています．したがってグループに属していない教師もこの if文の条件を満たさないのでメール送信されてしまう，ということですね．
かといって教師に全グループへのアクセス権を与えない訳には行きません．確かに通常の使用方法ではグループごとへどのようなフォーラム投稿を行ったかコースに属する教師は把握しておきたいので，このコードは正しいですね．ううむ，では１０グループ中の８グループに同じ内容の投稿を行った場合，教師には同じ内容のメールが８通，届いてしまう訳ですね…（同じ内容とは限らないのですから）．
投稿先の選択に「教師を除く（全学生対象）」を追加してここに if文を追加するか，投稿先の選択には手を加えず新たに「教師を除く」のチェックボックスを設けてif文を追加するなどの方法で強引に回避できますが，あまり需要はなさそうですね．機能ばかり増えると利用者を混乱させるだけですので．