When importing files (batch or not) the log messages contain no reference to the actual file name, which, in case of a batch, makes things hard to debug in case of a problem. This patch intends to add that information in the log.

CommentFileSizeAuthor
#2 1045818-feeds_log_filename.patch823 bytesmakara
#1 feeds_log_filename.patch930 bytesmakara
feeds_log_filename.patch904 byteshunvreus

Comments

makara’s picture

makara commented
Status: Active » Needs review
StatusFileSize
new feeds_log_filename.patch930 bytes

There's a mistake in the patch - wrong file name for a batch import.

Here's an updated patch, built on GIT branch 7.x-2.x.

makara’s picture

makara commented
StatusFileSize
new 1045818-feeds_log_filename.patch823 bytes

The patch in #1 only works for Git (or patch -p1).

Here's a more general patch.

twistor’s picture

twistor commented
Status: Needs review » Needs work

Wouldn't this mean that FeedsHTTPFetcher also needs to log what url its downloading, etc?

dianacastillo’s picture

dianacastillo commented
Issue summary: View changes

is this incorporated into 7.x-2.0-beta1 ?

dianacastillo’s picture

dianacastillo commented

This is a great help, please add this to the next version of feeds. Thanks

bluegeek9’s picture

bluegeek9 commented
Status: Needs work » Closed (outdated)
//www.flaticon.com/free-icons/thank-you Thank you for your contribution!

Unfortunately, Drupal 7 is End of Life and no longer supported. We strongly encourage you to upgrade to a supported version of Drupal.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.