I think you are right. That was a leftover from an earlier assumption. I'll
try and check it later tonight or tomorrow morning.
On Oct 6, 2013 6:37 PM, "Sean Linsley" <notifications@github.com> wrote:

> In lib/active_admin/filters/forms.rb:
>
> > @@ -24,7 +24,7 @@ def filter(method, options = {})
> >        def default_input_type(method, options = {})
> >          if method =~ /_(contains|starts_with|ends_with)\z/
> >            :string
> > -        elsif reflection_for(method) || polymorphic_foreign_type?(method)
> > +        elsif reflection_for(method) || polymorphic_foreign_type?(method) || through_association?(method)
>
> Are you sure this change is necessary? It seems like the
> reflection_for(method) earlier in the line would have already matched it.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gregbell/active_admin/pull/2541/files#r6788575>
> .
>