Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.We use required field a lot.
When the field is absent, the row of csv / node is not created what is what we want.
But it fails silently, there is no feedback whatsoever on how many rows, which rows where skipped or why.
This would be very useful if feeds is being used by regular users to upload stuff.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | feeds_tamper-required-message-2492159-4.patch | 1.13 KB | twistor |
| #1 | feeds_tamper-required-message-2492159-1.patch | 1.13 KB | twistor |
Comments
Comment #1
twistor commentedI haven't tested it, but something like this should work.
Comment #2
ikeigenwijs commentedThis looks splendid, i will test this thusday
thx twistor!
Comment #3
ikeigenwijs commentedOk , later than planned.
The configuration field is available but it does not work.
There is no difference between enabled and disabled.
cleared cache after patching and enabling to be sure.
found this in watchdog:
Warning: Illegal offset type in drupal_set_message() (line 1789 of /var/www/d7_rcollectie/includes/bootstrap.inc).
Warning: Invalid argument supplied for foreach() in entityreference_feeds_feeds_processor_targets_alter() (line 120 of xxxx/sites/all/modules/entityreference_feeds/entityreference_feeds.module).
our mandatory field commes before a entityreference look up.
The basic workings are sill intact, node is not created, but no feedback.
Comment #4
twistor commentedOops, yeah.e
Comment #5
ikeigenwijs commentedOk, this patch works !!
Is it possible to add the line number of the skipped entry in the feedback message?
Otherwise RTBC, cant go much wrong here ,-)
Comment #6
twistor commentedIt's not possible to add the line number, but we could be able to add the item number. For a CSV, it will be close. For others, it might be useful.
Comment #8
twistor commentedComment #9
ikeigenwijs commentedWe are indeed using csv imports an item number would be nice ;-)
thx for the commit
Comment #10
ikeigenwijs commented