Open-ILS/src/perlmods/lib/OpenILS/Utils/MFHD.pm

   1 package MFHD;
   2 use strict;
   3 use warnings;
   4 use integer;
   5 use Carp;
   6 use DateTime::Format::Strptime;
   7 use Data::Dumper;
   8
   9 # for inherited methods to work properly, we need to force a
  10 # MARC::Record version greater than 2.0.0
  11 use MARC::Record "2.0.1";
  12 use base 'MARC::Record';
  13
  14 use OpenILS::Utils::MFHD::Caption;
  15 use OpenILS::Utils::MFHD::Holding;
  16
  17 sub new {
  18     my $proto = shift;
  19     my $class = ref($proto) || $proto;
  20     my $self  = shift;
  21
  22     $self->{_strp_date} = new DateTime::Format::Strptime(pattern => '%F');
  23
  24     $self->{_mfhd_CAPTIONS} = {};
  25     $self->{_mfhd_COMPRESSIBLE} = (substr($self->leader, 17, 1) =~ /[45]/);
  26
  27     foreach my $field ('853', '854', '855') {
  28         my $captions = {};
  29         foreach my $caption ($self->field($field)) {
  30             my $cap_id;
  31
  32             $cap_id = $caption->subfield('8') || '0';
  33
  34             if (exists $captions->{$cap_id}) {
  35                 carp "Multiple MFHD captions with label '$cap_id'";
  36             }
  37
  38             $captions->{$cap_id} = new MFHD::Caption($caption);
  39             if ($self->{_mfhd_COMPRESSIBLE}) {
  40                 $self->{_mfhd_COMPRESSIBLE} &&=
  41                   $captions->{$cap_id}->compressible;
  42             }
  43         }
  44         $self->{_mfhd_CAPTIONS}->{$field} = $captions;
  45     }
  46
  47     foreach my $field ('863', '864', '865') {
  48         my $holdings = {};
  49         my $cap_field;
  50
  51         ($cap_field = $field) =~ s/6/5/;
  52
  53         foreach my $hfield ($self->field($field)) {
  54             my ($linkage, $link_id, $seqno);
  55             my $holding;
  56
  57             $linkage = $hfield->subfield('8');
  58             ($link_id, $seqno) = split(/\./, $linkage);
  59
  60             if (!exists $holdings->{$link_id}) {
  61                 $holdings->{$link_id} = {};
  62             }
  63             $holding =
  64               new MFHD::Holding($seqno, $hfield,
  65                 $self->{_mfhd_CAPTIONS}->{$cap_field}->{$link_id});
  66             $holdings->{$link_id}->{$seqno} = $holding;
  67
  68             if ($self->{_mfhd_COMPRESSIBLE}) {
  69                 $self->{_mfhd_COMPRESSIBLE} &&= $holding->validate;
  70             }
  71         }
  72         $self->{_mfhd_HOLDINGS}->{$field} = $holdings;
  73     }
  74
  75     bless($self, $class);
  76     return $self;
  77 }
  78
  79 sub compressible {
  80     my $self = shift;
  81
  82     return $self->{_mfhd_COMPRESSIBLE};
  83 }
  84
  85 sub caption_link_ids {
  86     my $self  = shift;
  87     my $field = shift;
  88
  89     return sort keys %{$self->{_mfhd_CAPTIONS}->{$field}};
  90 }
  91
  92 # optional argument to get back a 'hashref' or an 'array' (default)
  93 sub captions {
  94     my $self  = shift;
  95     my $tag = shift;
  96     my $return_type = shift;
  97
  98     # TODO: add support for caption types as argument? (base, index, supplement)
  99     my @sorted_ids = $self->caption_link_ids($tag);
 100
 101     if (defined($return_type) and $return_type eq 'hashref') {
 102         my %captions;
 103         foreach my $link_id (@sorted_ids) {
 104             $captions{$link_id} = $self->{_mfhd_CAPTIONS}{$tag}{$link_id};
 105         }
 106         return \%captions;
 107     } else {
 108         my @captions;
 109         foreach my $link_id (@sorted_ids) {
 110             push(@captions, $self->{_mfhd_CAPTIONS}{$tag}{$link_id});
 111         }
 112         return @captions;
 113     }
 114 }
 115
 116 sub append_fields {
 117     my $self = shift;
 118
 119     my $field_count = $self->SUPER::append_fields(@_);
 120     if ($field_count) {
 121         foreach my $field (@_) {
 122             $self->_avoid_link_collision($field);
 123             my $field_type = ref $field;
 124             if ($field_type eq 'MFHD::Holding') {
 125                 $self->{_mfhd_HOLDINGS}{$field->tag}{$field->caption->link_id}{$field->seqno} = $field;
 126             } elsif ($field_type eq 'MFHD::Caption') {
 127                 $self->{_mfhd_CAPTIONS}{$field->tag}{$field->link_id} = $field;
 128             }
 129         }
 130         return $field_count;
 131     } else {
 132         return;
 133     }
 134 }
 135
 136 sub delete_field {
 137     my $self = shift;
 138     my $field = shift;
 139
 140     my $field_count = $self->SUPER::delete_field($field);
 141     if ($field_count) {
 142         my $field_type = ref($field);
 143         if ($field_type eq 'MFHD::Holding') {
 144             delete($self->{_mfhd_HOLDINGS}{$field->tag}{$field->caption->link_id}{$field->seqno});
 145         } elsif ($field_type eq 'MFHD::Caption') {
 146             delete($self->{_mfhd_CAPTIONS}{$field->tag}{$field->link_id});
 147         }
 148         return $field_count;
 149     } else {
 150         return;
 151     }
 152 }
 153
 154 sub insert_fields_before {
 155     my $self = shift;
 156     my $before = shift;
 157
 158     my $field_count = $self->SUPER::insert_fields_before($before, @_);
 159     if ($field_count) {
 160         foreach my $field (@_) {
 161             $self->_avoid_link_collision($field);
 162             my $field_type = ref $field;
 163             if ($field_type eq 'MFHD::Holding') {
 164                 $self->{_mfhd_HOLDINGS}{$field->tag}{$field->caption->link_id}{$field->seqno} = $field;
 165             } elsif ($field_type eq 'MFHD::Caption') {
 166                 $self->{_mfhd_CAPTIONS}{$field->tag}{$field->link_id} = $field;
 167             }
 168         }
 169         return $field_count;
 170     } else {
 171         return;
 172     }
 173 }
 174
 175 sub insert_fields_after {
 176     my $self = shift;
 177     my $after = shift;
 178
 179     my $field_count = $self->SUPER::insert_fields_after($after, @_);
 180     if ($field_count) {
 181         foreach my $field (@_) {
 182             $self->_avoid_link_collision($field);
 183             my $field_type = ref $field;
 184             if ($field_type eq 'MFHD::Holding') {
 185                 $self->{_mfhd_HOLDINGS}{$field->tag}{$field->caption->link_id}{$field->seqno} = $field;
 186             } elsif ($field_type eq 'MFHD::Caption') {
 187                 $self->{_mfhd_CAPTIONS}{$field->tag}{$field->link_id} = $field;
 188             }
 189         }
 190         return $field_count;
 191     } else {
 192         return;
 193     }
 194 }
 195
 196 sub _avoid_link_collision {
 197     my $self = shift;
 198     my $field = shift;
 199
 200     my $fieldref = ref($field);
 201     if ($fieldref eq 'MFHD::Holding') {
 202         my $seqno = $field->seqno;
 203         my $changed_seqno = 0;
 204         if (exists($self->{_mfhd_HOLDINGS}{$field->tag}{$field->caption->link_id}{$seqno})) {
 205             $changed_seqno = 1;
 206             do {
 207                 $seqno++;
 208             } while (exists($self->{_mfhd_HOLDINGS}{$field->tag}{$field->caption->link_id}{$seqno}));
 209         }
 210         $field->seqno($seqno) if $changed_seqno;
 211     } elsif ($fieldref eq 'MFHD::Caption') {
 212         my $link_id = $field->link_id;
 213         my $changed_link_id = 0;
 214         if (exists($self->{_mfhd_CAPTIONS}{$field->tag}{$link_id})) {
 215             $link_id++;
 216             $changed_link_id = 1;
 217             do {
 218                 $link_id++;
 219             } while (exists($self->{_mfhd_CAPTIONS}{$field->tag}{$link_id}));
 220         }
 221         $field->link_id($link_id) if $changed_link_id;
 222     }
 223 }
 224
 225 sub active_captions {
 226     my $self  = shift;
 227     my $tag = shift;
 228
 229     # TODO: add support for caption types as argument? (basic, index, supplement)
 230     my @captions;
 231     my @active_captions;
 232
 233     @captions = $self->captions($tag);
 234
 235     # TODO: for now, we will assume the last 85X field is active
 236     # and the rest are historical.  The standard is hazy about
 237     # how multiple active patterns of the same 85X type should be
 238     # handled.  We will, however, return as an array for future
 239     # use.
 240     push(@active_captions, $captions[-1]);
 241
 242     return @active_captions;
 243 }
 244
 245 sub holdings {
 246     my $self  = shift;
 247     my $field = shift;
 248     my $capid = shift;
 249
 250     return
 251       sort { $a->seqno <=> $b->seqno }
 252       values %{$self->{_mfhd_HOLDINGS}->{$field}->{$capid}};
 253 }
 254
 255 sub holdings_by_caption {
 256     my $self  = shift;
 257     my $caption = shift;
 258
 259     my $htag    = $caption->tag;
 260     my $link_id = $caption->link_id;
 261     $htag =~ s/^85/86/;
 262     return $self->holdings($htag, $link_id);
 263 }
 264
 265 sub _holding_date {
 266     my $self = shift;
 267     my $holding = shift;
 268
 269     return $self->{_strp_date}->parse_datetime($holding->chron_to_date);
 270 }
 271
 272 #
 273 # generate_predictions()
 274 # Accepts a hash ref of options initially defined as:
 275 # base_holding : reference to the holding field to predict from
 276 # num_to_predict : the number of issues you wish to predict
 277 # OR
 278 # end_holding : holding field ref, keep predicting until you meet or exceed it
 279 # OR
 280 # end_date : keep predicting until you exceed this
 281 #
 282 # The basic method is to first convert to a single holding if compressed, then
 283 # increment the holding and save the resulting values to @predictions.
 284 #
 285 # returns @predictions, an array of holding field refs (including end_holding
 286 # if applicable but NOT base_holding)
 287 #
 288 sub generate_predictions {
 289     my ($self, $options) = @_;
 290
 291     my $base_holding   = $options->{base_holding};
 292     my $num_to_predict = $options->{num_to_predict};
 293     my $end_holding    = $options->{end_holding};
 294     my $end_date       = $options->{end_date};
 295     my $max_to_predict = $options->{max_to_predict} || 10000; # fail-safe
 296
 297     if (!defined($base_holding)) {
 298         carp("Base holding not defined in generate_predictions, returning empty set");
 299         return ();
 300     }
 301     if ($base_holding->is_compressed) {
 302         carp("Ambiguous compressed base holding in generate_predictions, returning empty set");
 303         return ();
 304     }
 305     my $curr_holding = $base_holding->clone; # prevent side-effects
 306
 307     my @predictions;
 308
 309     if ($num_to_predict) {
 310         for (my $i = 0; $i < $num_to_predict; $i++) {
 311             push(@predictions, $curr_holding->increment->clone);
 312         }
 313     } elsif (defined($end_holding)) {
 314         $end_holding = $end_holding->clone; # prevent side-effects
 315         my $next_holding = $curr_holding->increment->clone;
 316         my $num_predicted = 0;
 317         while ($next_holding le $end_holding) {
 318             push(@predictions, $next_holding);
 319             $num_predicted++;
 320             if ($num_predicted >= $max_to_predict) {
 321                 carp("Maximum prediction count exceeded");
 322                 last;
 323             }
 324             $next_holding = $curr_holding->increment->clone;
 325         }
 326     } elsif (defined($end_date)) {
 327         my $next_holding = $curr_holding->increment->clone;
 328         my $num_predicted = 0;
 329         while ($self->_holding_date($next_holding) <= $end_date) {
 330             push(@predictions, $next_holding);
 331             $num_predicted++;
 332             if ($num_predicted >= $max_to_predict) {
 333                 carp("Maximum prediction count exceeded");
 334                 last;
 335             }
 336             $next_holding = $curr_holding->increment->clone;
 337         }
 338     }
 339
 340     return @predictions;
 341 }
 342
 343 #
 344 # create an array of compressed holdings from all holdings for a given caption,
 345 # compressing as needed
 346 #
 347 # Optionally you can skip sorting, but the resulting compression will be compromised
 348 # if the current holdings are out of order
 349 #
 350 # TODO: gap marking, gap preservation
 351 #
 352 # TODO: some of this could be moved to the Caption object to allow for
 353 # decompression in the absense of an overarching MFHD object
 354 #
 355 sub get_compressed_holdings {
 356     my $self = shift;
 357     my $caption = shift;
 358     my $opts = shift;
 359     my $skip_sort = $opts->{'skip_sort'};
 360
 361     # make sure none are compressed (except for open-ended)
 362     my @decomp_holdings;
 363     if ($skip_sort) {
 364         @decomp_holdings = $self->get_decompressed_holdings($caption, {'skip_sort' => 1, 'passthru_open_ended' => 1});
 365     } else {
 366         # sort for best algorithm
 367         @decomp_holdings = $self->get_decompressed_holdings($caption, {'dedupe' => 1, 'passthru_open_ended' => 1});
 368     }
 369
 370     return () if !@decomp_holdings;
 371
 372     # if first holding is open-ended, it 'includes' all the rest, so return
 373     if ($decomp_holdings[0]->is_open_ended) {
 374         return ($decomp_holdings[0]);
 375     }
 376
 377     my $runner = $decomp_holdings[0]->clone->increment;
 378     my $curr_holding = shift(@decomp_holdings);
 379     $curr_holding = $curr_holding->clone;
 380     my $seqno = 1;
 381     $curr_holding->seqno($seqno);
 382     my @comp_holdings;
 383     foreach my $holding (@decomp_holdings) {
 384         if ($runner eq $holding) {
 385             $curr_holding->extend;
 386             $runner->increment;
 387         } elsif ($runner gt $holding) { # should not happen unless holding is not in series
 388             carp("Found unexpected holding, skipping");
 389         } elsif ($holding->is_open_ended) { # special case, as it will always be the last
 390             if ($runner eq $holding->clone->compressed_to_first) {
 391                 $curr_holding->compressed_end();
 392             } else {
 393                 push(@comp_holdings, $curr_holding);
 394                 $curr_holding = $holding->clone;
 395                 $seqno++;
 396                 $curr_holding->seqno($seqno);
 397             }
 398             last;
 399         } else {
 400             push(@comp_holdings, $curr_holding);
 401             while ($runner le $holding) {
 402                 $runner->increment;
 403             }
 404             $curr_holding = $holding->clone;
 405             $seqno++;
 406             $curr_holding->seqno($seqno);
 407         }
 408     }
 409     push(@comp_holdings, $curr_holding);
 410
 411     return @comp_holdings;
 412 }
 413
 414 #
 415 # create an array of single holdings from all holdings for a given caption,
 416 # decompressing as needed
 417 #
 418 # optional arguments:
 419 #    skip_sort: do not sort the returned holdings
 420 #    dedupe: remove any duplicate holdings from the set
 421 #    passthru_open_ended: open-ended compressed holdings cannot be logically
 422 #    decompressed (they are infinite); if set to true these holdings are passed
 423 #    thru rather than skipped
 424 # TODO: some of this could be moved to the Caption (and/or Holding) object to
 425 # allow for decompression in the absense of an overarching MFHD object
 426 #
 427 sub get_decompressed_holdings {
 428     my $self = shift;
 429     my $caption = shift;
 430     my $opts = shift;
 431     my $skip_sort = $opts->{'skip_sort'};
 432     my $dedupe = $opts->{'dedupe'};
 433     my $passthru_open_ended = $opts->{'passthru_open_ended'};
 434
 435     if ($dedupe and $skip_sort) {
 436         carp("Attempted deduplication without sorting, failure likely");
 437     }
 438
 439     my @holdings = $self->holdings_by_caption($caption);
 440
 441     return () if !@holdings;
 442
 443     my @decomp_holdings;
 444
 445     foreach my $holding (@holdings) {
 446         if (!$holding->is_compressed) {
 447             push(@decomp_holdings, $holding->clone);
 448         } elsif ($holding->is_open_ended) {
 449             if ($passthru_open_ended) {
 450                 push(@decomp_holdings, $holding->clone);
 451             } else {
 452                 carp("Open-ended holdings cannot be decompressed, skipping");
 453             }
 454         } else {
 455             my $base_holding = $holding->clone->compressed_to_first;
 456             my @new_holdings = $self->generate_predictions(
 457                 {'base_holding' => $base_holding,
 458                  'end_holding' => $holding->clone->compressed_to_last});
 459             push(@decomp_holdings, $base_holding, @new_holdings);
 460         }
 461     }
 462
 463     unless ($skip_sort) {
 464         my @temp_holdings = sort {$a cmp $b} @decomp_holdings;
 465         @decomp_holdings = @temp_holdings;
 466     }
 467
 468     my @return_holdings = (shift(@decomp_holdings));
 469     $return_holdings[0]->seqno(1);
 470     my $seqno = 2;
 471     foreach my $holding (@decomp_holdings) { # renumber sequence
 472         if ($holding eq $return_holdings[-1] and $dedupe) {
 473             carp("Found duplicate holding in decompression set, discarding");
 474             next;
 475         }
 476         $holding->seqno($seqno);
 477         $seqno++;
 478         push(@return_holdings, $holding);
 479     }
 480
 481     return @return_holdings;
 482 }
 483
 484 ##
 485 ## close any open-ended holdings which are followed by another holding by
 486 ## combining them
 487 ##
 488 ## This needs more thought about concerning usability (e.g. should it be a
 489 ## mutator?), commenting out for now
 490 #sub _get_truncated_holdings {
 491 #    my $self = shift;
 492 #    my $caption = shift;
 493 #
 494 #    my @holdings = $self->holdings_by_caption($caption);
 495 #
 496 #    return () if !@holdings;
 497 #
 498 #    @holdings = sort {$a cmp $b} @holdings;
 499 #
 500 #    my $current_open_holding;
 501 #    my @truncated_holdings;
 502 #    foreach my $holding (@holdings) {
 503 #        if ($current_open_holding) {
 504 #            if ($holding->is_open_ended) {
 505 #                next; # consecutive open holdings are meaningless, as they are contained by the previous
 506 #            } elsif ($holding->is_compressed) {
 507 #                $current_open_holding->compressed_end($holding->compressed_to_last);
 508 #            } else {
 509 #                $current_open_holding->compressed_end($holding);
 510 #            }
 511 #            push(@truncated_holdings, $current_open_holding);
 512 #            $current_open_holding = undef;
 513 #        } elsif ($holding->is_open_ended) {
 514 #            $current_open_holding = $holding;
 515 #        } else {
 516 #            push(@truncated_holdings, $holding);
 517 #        }
 518 #    }
 519 #
 520 #    # catch possible open holding at end
 521 #    push(@truncated_holdings, $current_open_holding) if $current_open_holding;
 522 #
 523 #    my $seqno = 1;
 524 #    foreach my $holding (@truncated_holdings) { # renumber sequence
 525 #        $holding->seqno($seqno);
 526 #        $seqno++;
 527 #    }
 528 #
 529 #    return @truncated_holdings;
 530 #}
 531
 532 #
 533 # format_holdings(): Generate textual display of all holdings in record
 534 # for given type of caption (853--855) taking into account all the
 535 # captions, holdings statements, and textual
 536 # holdings.
 537 #
 538 # returns string formatted holdings as one very long line.
 539 # Caller must provide any label (such as "library has:" and insert
 540 # line breaks as appropriate.
 541
 542 # Translate caption field labels to the corresponding textual holdings
 543 # statement labels. That is, convert 853 "Basic bib unit" caption to
 544 # 866 "basic bib unit" text holdings label.
 545
 546 my %cap_to_txt = (
 547                   '853' => '866',
 548                   '854' => '867',
 549                   '855' => '868',
 550                  );
 551
 552 sub format_holdings {
 553     my $self = shift;
 554     my $field = shift;
 555     my $holdings_field;
 556     my @txt_holdings;
 557     my %txt_link_ids;
 558     my $holdings_stmt = '';
 559     my ($l, $start);
 560
 561     # convert caption field id to holdings field id
 562     ($holdings_field = $field) =~ s/5/6/;
 563
 564     # Textual holdings statements complicate the basic algorithm for
 565     # formatting the holdings: If there's a textual holdings statement
 566     # with the subfield "$80", then that overrides ALL the MFHD holdings
 567     # information and is all that is displayed. Otherwise, the textual
 568     # holdings statements will either replace some of the MFHD holdings
 569     # information, or supplement it, depending on the value of the
 570     # $8 linkage subfield.
 571
 572     if (defined $self->field($cap_to_txt{$field})) {
 573         @txt_holdings = $self->field($cap_to_txt{$field});
 574
 575         foreach my $txt (@txt_holdings) {
 576
 577             # if there's a $80 subfield, then we're done, it's
 578             # all the formatted holdings
 579             if ($txt->subfield('8') eq '0') {
 580                 # textual holdings statement that completely
 581                 # replaces MFHD holdings in 853/863, etc.
 582                 $holdings_stmt = $txt->subfield('a');
 583
 584                 if (defined $txt->subfield('z')) {
 585                     $holdings_stmt .= ' -- ' . $txt->subfield('z');
 586                 }
 587
 588                 printf("# format_holdings() returning %s txt holdings\n",
 589                        $cap_to_txt{$field});
 590                 return $holdings_stmt;
 591             }
 592
 593             # If there are non-$80 subfields in the textual holdings
 594             # then we need to keep track of the subfields, so we can
 595             # intersperse the textual holdings in with the the calculated
 596             # holdings from the 853/863 fields.
 597             foreach my $linkid ($txt->subfield('8')) {
 598                 $txt_link_ids{$linkid} = $txt;
 599             }
 600         }
 601     }
 602
 603     # Now loop through all the captions, finding the corresponding
 604     # holdings statements (either MFHD or textual), and build up the
 605     # complete formatted holdings statement. The textual holdings statements
 606     # have either the same link id field as a caption, which means that
 607     # the text holdings win, or they have ids that are interfiled with
 608     # the captions, which mean they go into the middle.
 609
 610     my @ids = sort($self->caption_link_ids($field), keys %txt_link_ids);
 611     foreach my $cap_id (@ids) {
 612         my $last_txt = undef;
 613
 614         if (exists $txt_link_ids{$cap_id}) {
 615             # there's a textual holding statement with this caption ID,
 616             # so just use that. This covers both the "replaces" and
 617             # the "supplements" holdings information options.
 618
 619             # a single textual holdings statement can replace multiple
 620             # captions. If the _last_ caption we saw had a textual
 621             # holdings statement, and this caption has the same one, then
 622             # we don't add the holdings again.
 623             if (!defined $last_txt || ($last_txt != $txt_link_ids{$cap_id})) {
 624                 my $txt = $txt_link_ids{$cap_id};
 625                 $holdings_stmt .= ',' if $holdings_stmt;
 626                 $holdings_stmt .= $txt->subfield('a');
 627                 if (defined $txt->subfield('z')) {
 628                     $holdings_stmt .= ' -- ' . $txt->subfield('z');
 629                 }
 630
 631                 $last_txt = $txt;
 632             }
 633             next;
 634         }
 635
 636         # We found a caption that doesn't have a corresponding textual
 637         # holdings statement, so reset $last_txt to undef.
 638         $last_txt = undef;
 639
 640         my @holdings = $self->holdings($holdings_field, $cap_id);
 641
 642         next unless scalar @holdings;
 643
 644         # XXX Need to format compressed holdings. see code in test.pl
 645         # for example. Try to do it without indexing?
 646         $holdings_stmt .= ',' if $holdings_stmt;
 647
 648         if ($self->compressible) {
 649             $start = $l = shift @holdings;
 650             $holdings_stmt .= $l->format;
 651
 652             while (my $h = shift @holdings) {
 653                 if (!$h->matches($l->next)) {
 654                     # this item is not part of the current run,
 655                     # close out the run and record this item
 656                     if ($l != $start) {
 657                         $holdings_stmt .= '-' . $l->format;
 658                     }
 659
 660                     $holdings_stmt .= ',' . $h->format;
 661                     $start = $h
 662                 } elsif (!scalar(@holdings) || defined($h->subfield('z'))) {
 663                     # This is the end of the holdings for this caption
 664                     # or this item has a public note that we want
 665                     # to display
 666                     $holdings_stmt .= '-' . $h->format;
 667                 }
 668
 669                 if (defined $h->subfield('z')) {
 670                     $holdings_stmt .= ' -- ' . $h->subfield('z');
 671                 }
 672
 673                 $l = $h;
 674             }
 675         } else {
 676             $holdings_stmt .= ',' if $holdings_stmt;
 677             $holdings_stmt .= (shift @holdings)->format;
 678             foreach my $h (@holdings) {
 679                 $holdings_stmt .= ',' . $h->format;
 680                 if (defined $h->subfield('z')) {
 681                     $holdings_stmt .= ' -- ' . $h->subfield('z');
 682                 }
 683             }
 684         }
 685     }
 686
 687     return $holdings_stmt;
 688 }
 689
 690 1;