search

LibreCat / Catmandu-MARC

Catmandu modules for working with MARC data
https://metacpan.org/release/Catmandu-MARC
Other
8 stars 10 forks source link

Crash when combining a range with a split #9

Closed eythian closed 8 years ago

eythian commented 10 years ago

The following fixer:

marc_map('008_/1-5','date-entered-on-file', -split => 1);

fails with:

Global symbol "$__0" requires explicit package name at (eval 120) line 1.
Global symbol "$__0" requires explicit package name at (eval 120) line 1.
Unmatched right curly bracket at (eval 120) line 1, at end of line
syntax error at (eval 120) line 1, near ";}"

I can't imagine combining these will be very common, but it something that has a possibility of coming up, in some crazy world somewhere. (I hit it because I'm autogenerating my fix rules from a database, and was just experimenting with adding the split to everything.)

phochste commented 10 years ago

As I understand in this case for fields 001,002,...009 and also the leader possibly you want to have the 'split' and 'join' options ignored?

eythian commented 10 years ago

Patrick Hochstenbach schreef op ma 09-06-2014 om 22:14 [-0700]:

As I understand in this case for fields 001,002,...009 and also the leader possibly you want to have the 'split' and 'join' options ignored?

There are probably really strange situations where someone might want them, so implementing them would probably be better, as well as being the avenue of least surprise. I guess it's meaningless on the leader, but I bet you someone out there has repeated control fields.

Robin.