Hi, I'm trying to use autocomplete filter configuration with an email field and it never finds anything. Does it work with email fields? Thanks!

CommentFileSizeAuthor
#6 views_autocomplete_filters-fixes_email_field-2715639-6.patch2.03 KBjmdeleon
#4 views_autocomplete_filters-fixes_email_field-2715639-4.patch2.26 KBjmdeleon
#2 fixes_email_field.patch5.04 KBbryden
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JordiTR created an issue. See original summary.

bryden’s picture

bryden CreditAttribution: bryden as a volunteer commented
FileSize
fixes_email_field.patch5.04 KB

I was trying the same thing and it wasn't working on email fields for me either. I believe this should fix the problem for email fields.

bryden’s picture

bryden CreditAttribution: bryden as a volunteer commented
Status: Active » Needs review
jmdeleon’s picture

jmdeleon CreditAttribution: jmdeleon commented
FileSize
views_autocomplete_filters-fixes_email_field-2715639-4.patch2.26 KB

Re-roll of the patch in #2, producing diffs against the release version rather than the dev version, incorporating only the changes that apply to this issue.

jmdeleon’s picture

jmdeleon CreditAttribution: jmdeleon commented
jmdeleon’s picture

jmdeleon CreditAttribution: jmdeleon commented
FileSize
views_autocomplete_filters-fixes_email_field-2715639-6.patch2.03 KB

Patch in #6 with diff paths adjusted.

jfurnas’s picture

jfurnas CreditAttribution: jfurnas commented

I can confirm that the patch in #6 worked for me

jenlampton’s picture

jenlampton
she / her
CreditAttribution: jenlampton at Jeneration Web Development for Metropolitan Transportation Commission commented
Status: Needs review » Reviewed & tested by the community

As per #7, marking RTBC.

jenlampton’s picture

jenlampton
she / her
CreditAttribution: jenlampton at Jeneration Web Development for Ivar Jacobsen International commented
Title: Does it work with email field? » make it work with other field types (like email)
Category: Support request » Feature request
Status: Reviewed & tested by the community » Needs work

The patch in #6 is going to need a re-roll against the latest dev, but I don't think hard-coding support for email fields is the right way to go. Many field types could have columns with names other than value so it would be best if we could write this solution in a way that would work for all of them, or at least, in a way that might prevent the duplication of code for each option as it's added.