Needless class_eval

This was doing nothing dynamic, and the eval is wastefully repeated if
there are multiple calls to `acts_as_taggable_on` with a context.

The helper for these methods is already a regular instance method not
used anywhere else, so the `tag_types.empty?` guard wasn't exactly a
huge win in saving namespace pollution.