Attached is a small patch (against trunk) which fixes the GetOptions
in marc2sre.xml. It fixes a small bug with the "idsubfield" parameter,
and changes "idfield" to accept a string instead of an int (to match
the behaviour of marc2bre.pl).

Cheers,
  Warren
Index: Open-ILS/src/extras/import/marc2sre.pl
===================================================================
--- Open-ILS/src/extras/import/marc2sre.pl	(revision 15151)
+++ Open-ILS/src/extras/import/marc2sre.pl	(working copy)
@@ -29,8 +29,8 @@
 	('001', 1, 'admin', 'open-ils', '/openils/conf/opensrf_core.xml', 'USMARC');
 
 GetOptions(
-	'idfield=i'	=> \$idfield,
-	'idsubfield=a'	=> \$idsubfield,
+	'idfield=s'	=> \$idfield,
+	'idsubfield=s'	=> \$idsubfield,
 	'startid=i'	=> \$count,
 	'user=s'	=> \$user,
 	'password=s'	=> \$password,

Reply via email to