If a feed provides a file ( i.e. in temp directory) which actually does not exists there will be an error thrown by FeedsEnclosure::getFile() called in filefield_feeds_set_target(). This error won't be caught until
FeedsNodeProcessor::process(). The result is that the feed-item with an invalid filename won't be imported at all.

Using feeds to migrate for example old websites its very likely that some referenced files aren't available any more. I'd prefer to import the item anyway but without the referenced file.

See also: #926616: FeedsEnclosure::getFile() should allow files to exist in the temp directory

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

osopolar’s picture

I would prefer proceed with importing the feed item but with leaving a message in watchdog by catching the error in filefield_feeds_set_target().

osopolar’s picture

Status: Active » Needs review
osopolar’s picture

Issue summary: View changes

Status: Needs review » Needs work

The last submitted patch, 1: feeds-invalid-enclosure-2237201-1.patch, failed testing.

osopolar’s picture

Status: Needs work » Needs review

failed testing seems not to be related to this patch

Status: Needs review » Needs work

The last submitted patch, 1: feeds-invalid-enclosure-2237201-1.patch, failed testing.

MegaChriz’s picture

Version: 6.x-1.0-beta13 » 6.x-1.x-dev

In most cases, patches should be tested against dev.

Status: Needs work » Needs review
twistor’s picture

Status: Needs review » Closed (outdated)