Bug 11940 - ./tools/export.pl die if too many items in a record
Summary: ./tools/export.pl die if too many items in a record
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Command-line Utilities (show other bugs)
Version: 3.14
Hardware: All All
: P5 - low normal (vote)
Assignee: Robin Sheat
QA Contact: Testopia
URL:
Keywords:
Depends on: 11096
Blocks:
  Show dependency treegraph
 
Reported: 2014-03-14 10:55 UTC by Stéphane Delaune
Modified: 2014-12-07 20:03 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stéphane Delaune 2014-03-14 10:55:33 UTC
./tools/export.pl die if too many items in a record because eval is on GetMarcBiblio($recordid) but not on  C4::Biblio::EmbedItemsInMarcBiblio
Comment 1 Stéphane Delaune 2014-03-14 11:52:28 UTC
bug is :

~/src/tools$ ./export.pl --filename=/home/koha/dumps/exports/`date +%F`-notices_total.mrc --clean
Undefined subroutine &MARC::Record::carp called at /usr/share/perl5/MARC/Record.pm line 565.


in fact the koha-community marc-record package have this bug (corrected in newer versions)
Comment 2 Stéphane Delaune 2014-03-14 14:13:20 UTC
in community repository, libmarc-record-perl is in 2.0.2-1 ; a newer version is 2.0.3-1
Comment 3 Stéphane Delaune 2014-03-14 16:07:02 UTC
(In reply to stephane.delaune from comment #2)
> in community repository, libmarc-record-perl is in 2.0.2-1 ; a newer version
> is 2.0.3-1

this is a mistake, libmarc-record-perl is not into community repository ; it's a dependancy.

I propose to set in C4/Installer/PerlDependencies.pm the min_ver to 2.0.3 instead of 2 for MARC::Record

I propose to add the 2.0.3 version deb package into community repository for all squeeze users
Comment 4 Robin Sheat 2014-03-17 07:11:50 UTC
MARC::Record 2.0.6 has been uploaded to the repo now. Note that this won't fix the underlying issue, i.e. there are too many items for USMARC to handle, but it'll stop the bad error.
Comment 5 Fridolin Somers 2014-03-19 07:58:55 UTC
(In reply to Robin Sheat from comment #4)
> MARC::Record 2.0.6 has been uploaded to the repo now. Note that this won't
> fix the underlying issue, i.e. there are too many items for USMARC to
> handle, but it'll stop the bad error.
Do you agree to change the min version in C4::Installer::PerlDependencies ?
Comment 6 Robin Sheat 2014-03-19 21:35:18 UTC
(In reply to Fridolin SOMERS from comment #5)
> Do you agree to change the min version in C4::Installer::PerlDependencies ?

It's probably best, given it does cause actual breakage.
Comment 7 Fridolin Somers 2014-05-16 07:54:49 UTC
Bug 11096 has set the dependency to 2.0.6.