I needed to validate an email in feeds and not import if it wasn't valid.
I based this on the required field filter.

CommentFileSizeAuthor
#1 feeds_tamper-filter-on-valid-email-2514490-1.patch1.68 KBT-lo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

T-lo’s picture

T-lo
Primary language English
Location Bristol, UK
CreditAttribution: T-lo at Investis Digital commented
Status: Active » Needs review
FileSize
feeds_tamper-filter-on-valid-email-2514490-1.patch1.68 KB
T-lo’s picture

T-lo
Primary language English
Location Bristol, UK
CreditAttribution: T-lo at Investis Digital commented
Component: Code » Plugins
T-lo’s picture

T-lo
Primary language English
Location Bristol, UK
CreditAttribution: T-lo at Investis Digital commented
Assigned: T-lo » Unassigned
Patrick Danielsson’s picture

Patrick Danielsson CreditAttribution: Patrick Danielsson commented

Great, thanks!!

Sophie.SK’s picture

Sophie.SK
she/her
Primary language English
CreditAttribution: Sophie.SK commented
Status: Needs review » Reviewed & tested by the community

Have been using this for the last few weeks on a client site and can definitely see further use cases for it in the future. Not sure why it wasn't a default filter, very useful.

twistor’s picture

twistor CreditAttribution: twistor as a volunteer commented
Status: Reviewed & tested by the community » Needs work

Can we get a simple test for this? There should be enough examples for how to create the test.

T-lo’s picture

T-lo
Primary language English
Location Bristol, UK
CreditAttribution: T-lo at Investis Digital commented
Assigned: Unassigned » T-lo

sure, I should be able to get that done today

T-lo’s picture

T-lo
Primary language English
Location Bristol, UK
CreditAttribution: T-lo at Investis Digital commented
Assigned: T-lo » Unassigned

sorry, I'm unlikely to get to this this week. will take back if I have time

mattyy21’s picture

mattyy21
Primary language Italian
CreditAttribution: mattyy21 commented

Same Problem. How can I solve it?

T-lo’s picture

T-lo
Primary language English
Location Bristol, UK
CreditAttribution: T-lo at Investis Digital commented

Hi mattyy21, the patch above will work