[Koha-bugs] [Bug 11046] Better handling of uncertain years for publicationyear/copyrightdate
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Mon Sep 20 16:08:43 CEST 2021
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11046
--- Comment #20 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
Just a late observation about the last changes:
- } elsif( $retval =~ m/(\d)[.Xx?]{3}|(\d\d)[.Xx?]{2}|(\d{3})[.Xx?]/ ) {
- my $digits = $1 || $2 || $3;
- $retval = $digits * ( 10 ** ( 4 - length($digits) ));
- } elsif( $retval =~ m/(\d)[-]{3}\?|(\d\d)[-]{2}\?|(\d{3})[-]\?/ ) {
- # the form 198-? occurred in Dutch ISBD rules
- my $digits = $1 || $2 || $3;
+ } elsif( $retval =~ m/
+ (?<year>\d)[-]?[.Xx?]{3}
+ |(?<year>\d{2})[.Xx?]{2}
+ |(?<year>\d{3})[.Xx?]
+ |(?<year>\d)[-]{3}\?
+ |(?<year>\d\d)[-]{2}\?
+ |(?<year>\d{3})[-]\?
+ /xms ) { # the form 198-? occurred in Dutch ISBD rules
This part of the regex made me wonder:
(?<year>\d)[-]?[.Xx?]{3}
What does the minus do here after the first digit? Do we allow 1-234 or so?
Looking at the above, this must have been a typo (copy/paste) after all..
Continued on bug 24674
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list