Bug 4123 – Harvesting MODS archives produces "Invalid argument supplied for foreach()" error

Bug 4123 - Harvesting MODS archives produces "Invalid argument supplied for foreach()" error

Summary:

Harvesting MODS archives produces "Invalid argument supplied for foreach()" e...

Status:	RESOLVED FIXED

Product:	OHS
Classification:	Unclassified
Component:	Indexing
Version:	2.3.1
Platform:	PC Windows XP

Importance:	P5 normal
Assigned To:	Alec Smecher

URL:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2009-03-11 14:05 PDT by Mark Jordan
Modified:	2009-03-11 18:49 PDT (History)
CC List:	1 user (show)

See Also:
Version Reported In:
Also Affects:

Attachments
Patch against Harvester pre-2.3.1 CVS (616 bytes, patch) 2009-03-11 18:48 PDT, Alec Smecher	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Mark Jordan 2009-03-11 14:05:31 PDT

Harvesting MODS archives using harvest.php produces the following error:

"PHP Warning:  Invalid argument supplied for foreach() in /mnt/u0/var/www/html/chodarr-test/lib/pkp/classes/xml/XMLNode.inc.php on line 138"

Records are added but they are empty (more specifically, nothing is showing up in the browse list but a link to view record, and the record itself is empty.

The error message does not appear when harvesting via the web interface, but the result is the same -- empty records.

Harvesting DC and ETD-MS archives works as expected. The MODS archives (they are actually static repositories at http://harvesters.sfu.ca/static_repositories/) that are producing the error/not populating records posed no problems in Harvester 2.0.1.0 Harvester. The DC and ETD-MS archives I tested were not static repositories.

Comment 1 Alec Smecher 2009-03-11 18:48:32 PDT

Created attachment 1568 [details]
Patch against Harvester pre-2.3.1 CVS