Should we have an internal version of `reflect_on_all_associations`?  Internal code should never call this with `has_and_belongs_to_many`, so we're doing a wasteful comparison.  Not to mention, if someone passes `nil`, then they'll get the values for our internal reflections.