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     # basic check for necessary pattern information
 362     if (!scalar keys %{$caption->pattern}) {
 363         carp "Cannot compress without pattern data, returning original holdings";
 364         return $self->holdings_by_caption($caption);
 365     }
 366
 367     # make sure none are compressed (except for open-ended)
 368     my @decomp_holdings;
 369     if ($skip_sort) {
 370         @decomp_holdings = $self->get_decompressed_holdings($caption, {'skip_sort' => 1, 'passthru_open_ended' => 1});
 371     } else {
 372         # sort for best algorithm
 373         @decomp_holdings = $self->get_decompressed_holdings($caption, {'dedupe' => 1, 'passthru_open_ended' => 1});
 374     }
 375
 376     return () if !@decomp_holdings;
 377
 378     # if first holding is open-ended, it 'includes' all the rest, so return
 379     if ($decomp_holdings[0]->is_open_ended) {
 380         return ($decomp_holdings[0]);
 381     }
 382
 383     my $runner = $decomp_holdings[0]->clone->increment;
 384     my $curr_holding = shift(@decomp_holdings);
 385     $curr_holding = $curr_holding->clone;
 386     my $seqno = 1;
 387     $curr_holding->seqno($seqno);
 388     my @comp_holdings;
 389     foreach my $holding (@decomp_holdings) {
 390         if ($runner eq $holding) {
 391             $curr_holding->extend;
 392             $runner->increment;
 393         } elsif ($runner gt $holding) { # should not happen unless holding is not in series
 394             carp("Found unexpected holding, skipping");
 395         } elsif ($holding->is_open_ended) { # special case, as it will always be the last
 396             if ($runner eq $holding->clone->compressed_to_first) {
 397                 $curr_holding->compressed_end();
 398             } else {
 399                 push(@comp_holdings, $curr_holding);
 400                 $curr_holding = $holding->clone;
 401                 $seqno++;
 402                 $curr_holding->seqno($seqno);
 403             }
 404             last;
 405         } else {
 406             push(@comp_holdings, $curr_holding);
 407             while ($runner le $holding) {
 408                 $runner->increment;
 409             }
 410             $curr_holding = $holding->clone;
 411             $seqno++;
 412             $curr_holding->seqno($seqno);
 413         }
 414     }
 415     push(@comp_holdings, $curr_holding);
 416
 417     return @comp_holdings;
 418 }
 419
 420 #
 421 # create an array of single holdings from all holdings for a given caption,
 422 # decompressing as needed
 423 #
 424 # optional arguments:
 425 #    skip_sort: do not sort the returned holdings
 426 #    dedupe: remove any duplicate holdings from the set
 427 #    passthru_open_ended: open-ended compressed holdings cannot be logically
 428 #    decompressed (they are infinite); if set to true these holdings are passed
 429 #    thru rather than skipped
 430 # TODO: some of this could be moved to the Caption (and/or Holding) object to
 431 # allow for decompression in the absense of an overarching MFHD object
 432 #
 433 sub get_decompressed_holdings {
 434     my $self = shift;
 435     my $caption = shift;
 436     my $opts = shift;
 437     my $skip_sort = $opts->{'skip_sort'};
 438     my $dedupe = $opts->{'dedupe'};
 439     my $passthru_open_ended = $opts->{'passthru_open_ended'};
 440
 441     if ($dedupe and $skip_sort) {
 442         carp("Attempted deduplication without sorting, failure likely");
 443     }
 444
 445     my @holdings = $self->holdings_by_caption($caption);
 446
 447     return () if !@holdings;
 448
 449     my @decomp_holdings;
 450
 451     foreach my $holding (@holdings) {
 452         if (!$holding->is_compressed) {
 453             push(@decomp_holdings, $holding->clone);
 454         } elsif ($holding->is_open_ended) {
 455             if ($passthru_open_ended) {
 456                 push(@decomp_holdings, $holding->clone);
 457             } else {
 458                 carp("Open-ended holdings cannot be decompressed, skipping");
 459             }
 460         } else {
 461             my $base_holding = $holding->clone->compressed_to_first;
 462             my @new_holdings = $self->generate_predictions(
 463                 {'base_holding' => $base_holding,
 464                  'end_holding' => $holding->clone->compressed_to_last});
 465             push(@decomp_holdings, $base_holding, @new_holdings);
 466         }
 467     }
 468
 469     unless ($skip_sort) {
 470         my @temp_holdings = sort {$a cmp $b} @decomp_holdings;
 471         @decomp_holdings = @temp_holdings;
 472     }
 473
 474     my @return_holdings = (shift(@decomp_holdings));
 475     $return_holdings[0]->seqno(1);
 476     my $seqno = 2;
 477     foreach my $holding (@decomp_holdings) { # renumber sequence
 478         if ($holding eq $return_holdings[-1] and $dedupe) {
 479             carp("Found duplicate holding in decompression set, discarding");
 480             next;
 481         }
 482         $holding->seqno($seqno);
 483         $seqno++;
 484         push(@return_holdings, $holding);
 485     }
 486
 487     return @return_holdings;
 488 }
 489
 490 ##
 491 ## close any open-ended holdings which are followed by another holding by
 492 ## combining them
 493 ##
 494 ## This needs more thought about concerning usability (e.g. should it be a
 495 ## mutator?), commenting out for now
 496 #sub _get_truncated_holdings {
 497 #    my $self = shift;
 498 #    my $caption = shift;
 499 #
 500 #    my @holdings = $self->holdings_by_caption($caption);
 501 #
 502 #    return () if !@holdings;
 503 #
 504 #    @holdings = sort {$a cmp $b} @holdings;
 505 #
 506 #    my $current_open_holding;
 507 #    my @truncated_holdings;
 508 #    foreach my $holding (@holdings) {
 509 #        if ($current_open_holding) {
 510 #            if ($holding->is_open_ended) {
 511 #                next; # consecutive open holdings are meaningless, as they are contained by the previous
 512 #            } elsif ($holding->is_compressed) {
 513 #                $current_open_holding->compressed_end($holding->compressed_to_last);
 514 #            } else {
 515 #                $current_open_holding->compressed_end($holding);
 516 #            }
 517 #            push(@truncated_holdings, $current_open_holding);
 518 #            $current_open_holding = undef;
 519 #        } elsif ($holding->is_open_ended) {
 520 #            $current_open_holding = $holding;
 521 #        } else {
 522 #            push(@truncated_holdings, $holding);
 523 #        }
 524 #    }
 525 #
 526 #    # catch possible open holding at end
 527 #    push(@truncated_holdings, $current_open_holding) if $current_open_holding;
 528 #
 529 #    my $seqno = 1;
 530 #    foreach my $holding (@truncated_holdings) { # renumber sequence
 531 #        $holding->seqno($seqno);
 532 #        $seqno++;
 533 #    }
 534 #
 535 #    return @truncated_holdings;
 536 #}
 537
 538 #
 539 # format_holdings(): Generate textual display of all holdings in record
 540 # for given type of caption (853--855) taking into account all the
 541 # captions, holdings statements, and textual
 542 # holdings.
 543 #
 544 # returns string formatted holdings as one very long line.
 545 # Caller must provide any label (such as "library has:" and insert
 546 # line breaks as appropriate.
 547
 548 # Translate caption field labels to the corresponding textual holdings
 549 # statement labels. That is, convert 853 "Basic bib unit" caption to
 550 # 866 "basic bib unit" text holdings label.
 551
 552 my %cap_to_txt = (
 553                   '853' => '866',
 554                   '854' => '867',
 555                   '855' => '868',
 556                  );
 557
 558 sub format_holdings {
 559     my $self = shift;
 560     my $field = shift;
 561     my $holdings_field;
 562     my @txt_holdings;
 563     my %txt_link_ids;
 564     my $holdings_stmt = '';
 565     my ($l, $start);
 566
 567     # convert caption field id to holdings field id
 568     ($holdings_field = $field) =~ s/5/6/;
 569
 570     # Textual holdings statements complicate the basic algorithm for
 571     # formatting the holdings: If there's a textual holdings statement
 572     # with the subfield "$80", then that overrides ALL the MFHD holdings
 573     # information and is all that is displayed. Otherwise, the textual
 574     # holdings statements will either replace some of the MFHD holdings
 575     # information, or supplement it, depending on the value of the
 576     # $8 linkage subfield.
 577
 578     if (defined $self->field($cap_to_txt{$field})) {
 579         @txt_holdings = $self->field($cap_to_txt{$field});
 580
 581         foreach my $txt (@txt_holdings) {
 582
 583             # if there's a $80 subfield, then we're done, it's
 584             # all the formatted holdings
 585             if ($txt->subfield('8') eq '0') {
 586                 # textual holdings statement that completely
 587                 # replaces MFHD holdings in 853/863, etc.
 588                 $holdings_stmt = $txt->subfield('a');
 589
 590                 if (defined $txt->subfield('z')) {
 591                     $holdings_stmt .= ' -- ' . $txt->subfield('z');
 592                 }
 593
 594                 printf("# format_holdings() returning %s txt holdings\n",
 595                        $cap_to_txt{$field});
 596                 return $holdings_stmt;
 597             }
 598
 599             # If there are non-$80 subfields in the textual holdings
 600             # then we need to keep track of the subfields, so we can
 601             # intersperse the textual holdings in with the the calculated
 602             # holdings from the 853/863 fields.
 603             foreach my $linkid ($txt->subfield('8')) {
 604                 $txt_link_ids{$linkid} = $txt;
 605             }
 606         }
 607     }
 608
 609     # Now loop through all the captions, finding the corresponding
 610     # holdings statements (either MFHD or textual), and build up the
 611     # complete formatted holdings statement. The textual holdings statements
 612     # have either the same link id field as a caption, which means that
 613     # the text holdings win, or they have ids that are interfiled with
 614     # the captions, which mean they go into the middle.
 615
 616     my @ids = sort($self->caption_link_ids($field), keys %txt_link_ids);
 617     foreach my $cap_id (@ids) {
 618         my $last_txt = undef;
 619
 620         if (exists $txt_link_ids{$cap_id}) {
 621             # there's a textual holding statement with this caption ID,
 622             # so just use that. This covers both the "replaces" and
 623             # the "supplements" holdings information options.
 624
 625             # a single textual holdings statement can replace multiple
 626             # captions. If the _last_ caption we saw had a textual
 627             # holdings statement, and this caption has the same one, then
 628             # we don't add the holdings again.
 629             if (!defined $last_txt || ($last_txt != $txt_link_ids{$cap_id})) {
 630                 my $txt = $txt_link_ids{$cap_id};
 631                 $holdings_stmt .= ',' if $holdings_stmt;
 632                 $holdings_stmt .= $txt->subfield('a');
 633                 if (defined $txt->subfield('z')) {
 634                     $holdings_stmt .= ' -- ' . $txt->subfield('z');
 635                 }
 636
 637                 $last_txt = $txt;
 638             }
 639             next;
 640         }
 641
 642         # We found a caption that doesn't have a corresponding textual
 643         # holdings statement, so reset $last_txt to undef.
 644         $last_txt = undef;
 645
 646         my @holdings = $self->holdings($holdings_field, $cap_id);
 647
 648         next unless scalar @holdings;
 649
 650         # XXX Need to format compressed holdings. see code in test.pl
 651         # for example. Try to do it without indexing?
 652         $holdings_stmt .= ',' if $holdings_stmt;
 653
 654         if ($self->compressible) {
 655             $start = $l = shift @holdings;
 656             $holdings_stmt .= $l->format;
 657
 658             while (my $h = shift @holdings) {
 659                 if (!$h->matches($l->next)) {
 660                     # this item is not part of the current run,
 661                     # close out the run and record this item
 662                     if ($l != $start) {
 663                         $holdings_stmt .= '-' . $l->format;
 664                     }
 665
 666                     $holdings_stmt .= ',' . $h->format;
 667                     $start = $h
 668                 } elsif (!scalar(@holdings) || defined($h->subfield('z'))) {
 669                     # This is the end of the holdings for this caption
 670                     # or this item has a public note that we want
 671                     # to display
 672                     $holdings_stmt .= '-' . $h->format;
 673                 }
 674
 675                 if (defined $h->subfield('z')) {
 676                     $holdings_stmt .= ' -- ' . $h->subfield('z');
 677                 }
 678
 679                 $l = $h;
 680             }
 681         } else {
 682             $holdings_stmt .= ',' if $holdings_stmt;
 683             $holdings_stmt .= (shift @holdings)->format;
 684             foreach my $h (@holdings) {
 685                 $holdings_stmt .= ',' . $h->format;
 686                 if (defined $h->subfield('z')) {
 687                     $holdings_stmt .= ' -- ' . $h->subfield('z');
 688                 }
 689             }
 690         }
 691     }
 692
 693     return $holdings_stmt;
 694 }
 695
 696 1;