getTotal() Method of FeedsBatch class, sometimes called (i.e. from default FeedsNodeProcessor processor) before setTotal with appropriate $stage.

In that cases next string cause notices:
return $this->total[$stage];

I propose add isset() statement before return value of $this->total property.

CommentFileSizeAuthor
#4 feeds-1087622-4.patch523 bytesdanepowell

Comments

IRuslan’s picture

Status: Active » Closed (fixed)

Sorry, wrong project.

danepowell’s picture

Project: SOAP Client » Feeds
Version: 6.x-1.0-beta2 » 6.x-1.0-beta12
Status: Closed (fixed) » Active

This is still a problem with the latest Feeds release.

danepowell’s picture

Priority: Normal » Major

... unfortunately also major since it breaks automated testing, much like #1450156: Fatal error: Class 'FeedsMapperTestCase' not found in

danepowell’s picture

Category: support » bug
Status: Active » Needs review
StatusFileSize
new523 bytes

The attached patch fixes this error.

twistor’s picture

Status: Needs review » Fixed
twistor’s picture

Status: Fixed » Needs review

Wrong issue.

twistor’s picture

Version: 6.x-1.0-beta12 » 6.x-1.x-dev
Priority: Major » Minor
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.