in the following:

if (!empty($node->nid)) $nids[] = $node->nid;

_biblio_pm_import() has no way of knowing the nid since in the case of a batch API, nodes are actually created after they have been put into the cache, and effectively after _biblio_pm_import() has finished.

I'm sure what the best way to fix this is, but for the sake of returning the right amount of nodes which are due to be created, we could record a zero value nid for unkown nid. At least the count will be correct.

Comments

scor’s picture

Status: Active » Needs review
StatusFileSize
new2.81 KB
scor’s picture

StatusFileSize
new862 bytes

patch above has one to many hunk.

rjerome’s picture

There is actually another (more general) way around this which I implemented in the 6.x-2.x branch but forgot to port over to 7.x. I'll do that later today. Again your patch is specific to PM but this issue affects all import formats.

Cheers,

Ron.

rjerome’s picture

I just checked in the fix for this... http://drupal.org/cvs?commit=435106

rjerome’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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