C4/HoldsQueue.pm

   1 package C4::HoldsQueue;
   2
   3 # Copyright 2011 Catalyst IT
   4 #
   5 # This file is part of Koha.
   6 #
   7 # Koha is free software; you can redistribute it and/or modify it
   8 # under the terms of the GNU General Public License as published by
   9 # the Free Software Foundation; either version 3 of the License, or
  10 # (at your option) any later version.
  11 #
  12 # Koha is distributed in the hope that it will be useful, but
  13 # WITHOUT ANY WARRANTY; without even the implied warranty of
  14 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  15 # GNU General Public License for more details.
  16 #
  17 # You should have received a copy of the GNU General Public License
  18 # along with Koha; if not, see <http://www.gnu.org/licenses>.
  19
  20 # FIXME: expand perldoc, explain intended logic
  21
  22 use strict;
  23 use warnings;
  24
  25 use C4::Context;
  26 use C4::Search;
  27 use C4::Items;
  28 use C4::Circulation;
  29 use C4::Members;
  30 use C4::Biblio;
  31 use Koha::DateUtils;
  32 use Koha::Items;
  33 use Koha::Patrons;
  34
  35 use List::Util qw(shuffle);
  36 use List::MoreUtils qw(any);
  37 use Data::Dumper;
  38
  39 use vars qw(@ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
  40 BEGIN {
  41     require Exporter;
  42     @ISA = qw(Exporter);
  43     @EXPORT_OK = qw(
  44         &CreateQueue
  45         &GetHoldsQueueItems
  46
  47         &TransportCostMatrix
  48         &UpdateTransportCostMatrix
  49      );
  50 }
  51
  52
  53 =head1 FUNCTIONS
  54
  55 =head2 TransportCostMatrix
  56
  57   TransportCostMatrix();
  58
  59 Returns Transport Cost Matrix as a hashref <to branch code> => <from branch code> => cost
  60
  61 =cut
  62
  63 sub TransportCostMatrix {
  64     my $dbh   = C4::Context->dbh;
  65     my $transport_costs = $dbh->selectall_arrayref("SELECT * FROM transport_cost",{ Slice => {} });
  66
  67     my $today = dt_from_string();
  68     my $calendars;
  69     my %transport_cost_matrix;
  70     foreach (@$transport_costs) {
  71         my $from     = $_->{frombranch};
  72         my $to       = $_->{tobranch};
  73         my $cost     = $_->{cost};
  74         my $disabled = $_->{disable_transfer};
  75         $transport_cost_matrix{$to}{$from} = {
  76             cost             => $cost,
  77             disable_transfer => $disabled
  78         };
  79
  80         if ( C4::Context->preference("HoldsQueueSkipClosed") ) {
  81             $calendars->{$from} ||= Koha::Calendar->new( branchcode => $from );
  82             $transport_cost_matrix{$to}{$from}{disable_transfer} ||=
  83               $calendars->{$from}->is_holiday( $today );
  84         }
  85
  86     }
  87     return \%transport_cost_matrix;
  88 }
  89
  90 =head2 UpdateTransportCostMatrix
  91
  92   UpdateTransportCostMatrix($records);
  93
  94 Updates full Transport Cost Matrix table. $records is an arrayref of records.
  95 Records: { frombranch => <code>, tobranch => <code>, cost => <figure>, disable_transfer => <0,1> }
  96
  97 =cut
  98
  99 sub UpdateTransportCostMatrix {
 100     my ($records) = @_;
 101     my $dbh   = C4::Context->dbh;
 102
 103     my $sth = $dbh->prepare("INSERT INTO transport_cost (frombranch, tobranch, cost, disable_transfer) VALUES (?, ?, ?, ?)");
 104
 105     $dbh->do("DELETE FROM transport_cost");
 106     foreach (@$records) {
 107         my $cost = $_->{cost};
 108         my $from = $_->{frombranch};
 109         my $to = $_->{tobranch};
 110         if ($_->{disable_transfer}) {
 111             $cost ||= 0;
 112         }
 113         elsif ( !defined ($cost) || ($cost !~ m/(0|[1-9][0-9]*)(\.[0-9]*)?/o) ) {
 114             warn  "Invalid $from -> $to cost $cost - must be a number >= 0, disabling";
 115             $cost = 0;
 116             $_->{disable_transfer} = 1;
 117         }
 118         $sth->execute( $from, $to, $cost, $_->{disable_transfer} ? 1 : 0 );
 119     }
 120 }
 121
 122 =head2 GetHoldsQueueItems
 123
 124   GetHoldsQueueItems($branch);
 125
 126 Returns hold queue for a holding branch. If branch is omitted, then whole queue is returned
 127
 128 =cut
 129
 130 sub GetHoldsQueueItems {
 131     my ($branchlimit) = @_;
 132     my $dbh   = C4::Context->dbh;
 133
 134     my @bind_params = ();
 135     my $query = q/SELECT tmp_holdsqueue.*, biblio.author, items.ccode, items.itype, biblioitems.itemtype, items.location,
 136                          items.enumchron, items.cn_sort, biblioitems.publishercode,
 137                          biblio.copyrightdate, biblio.subtitle, biblio.medium,
 138                          biblio.part_number, biblio.part_name,
 139                          biblioitems.publicationyear, biblioitems.pages, biblioitems.size,
 140                          biblioitems.isbn, items.copynumber
 141                   FROM tmp_holdsqueue
 142                        JOIN biblio      USING (biblionumber)
 143                   LEFT JOIN biblioitems USING (biblionumber)
 144                   LEFT JOIN items       USING (  itemnumber)
 145                 /;
 146     if ($branchlimit) {
 147         $query .=" WHERE tmp_holdsqueue.holdingbranch = ?";
 148         push @bind_params, $branchlimit;
 149     }
 150     $query .= " ORDER BY ccode, location, cn_sort, author, title, pickbranch, reservedate";
 151     my $sth = $dbh->prepare($query);
 152     $sth->execute(@bind_params);
 153     my $items = [];
 154     while ( my $row = $sth->fetchrow_hashref ){
 155         # return the bib-level or item-level itype per syspref
 156         if (!C4::Context->preference('item-level_itypes')) {
 157             $row->{itype} = $row->{itemtype};
 158         }
 159         delete $row->{itemtype};
 160
 161         push @$items, $row;
 162     }
 163     return $items;
 164 }
 165
 166 =head2 CreateQueue
 167
 168   CreateQueue();
 169
 170 Top level function that turns reserves into tmp_holdsqueue and hold_fill_targets.
 171
 172 =cut
 173
 174 sub CreateQueue {
 175     my $dbh   = C4::Context->dbh;
 176
 177     $dbh->do("DELETE FROM tmp_holdsqueue");  # clear the old table for new info
 178     $dbh->do("DELETE FROM hold_fill_targets");
 179
 180     my $total_bibs            = 0;
 181     my $total_requests        = 0;
 182     my $total_available_items = 0;
 183     my $num_items_mapped      = 0;
 184
 185     my $branches_to_use;
 186     my $transport_cost_matrix;
 187     my $use_transport_cost_matrix = C4::Context->preference("UseTransportCostMatrix");
 188     if ($use_transport_cost_matrix) {
 189         $transport_cost_matrix = TransportCostMatrix();
 190         unless (keys %$transport_cost_matrix) {
 191             warn "UseTransportCostMatrix set to yes, but matrix not populated";
 192             undef $transport_cost_matrix;
 193         }
 194     }
 195     unless ($transport_cost_matrix) {
 196         $branches_to_use = load_branches_to_pull_from();
 197     }
 198
 199     my $bibs_with_pending_requests = GetBibsWithPendingHoldRequests();
 200
 201     foreach my $biblionumber (@$bibs_with_pending_requests) {
 202         $total_bibs++;
 203         my $hold_requests   = GetPendingHoldRequestsForBib($biblionumber);
 204         my $available_items = GetItemsAvailableToFillHoldRequestsForBib($biblionumber, $branches_to_use);
 205         $total_requests        += scalar(@$hold_requests);
 206         $total_available_items += scalar(@$available_items);
 207
 208         my $item_map = MapItemsToHoldRequests($hold_requests, $available_items, $branches_to_use, $transport_cost_matrix);
 209         $item_map  or next;
 210         my $item_map_size = scalar(keys %$item_map)
 211           or next;
 212
 213         $num_items_mapped += $item_map_size;
 214         CreatePicklistFromItemMap($item_map);
 215         AddToHoldTargetMap($item_map);
 216         if (($item_map_size < scalar(@$hold_requests  )) and
 217             ($item_map_size < scalar(@$available_items))) {
 218             # DOUBLE CHECK, but this is probably OK - unfilled item-level requests
 219             # FIXME
 220             #warn "unfilled requests for $biblionumber";
 221             #warn Dumper($hold_requests), Dumper($available_items), Dumper($item_map);
 222         }
 223     }
 224 }
 225
 226 =head2 GetBibsWithPendingHoldRequests
 227
 228   my $biblionumber_aref = GetBibsWithPendingHoldRequests();
 229
 230 Return an arrayref of the biblionumbers of all bibs
 231 that have one or more unfilled hold requests.
 232
 233 =cut
 234
 235 sub GetBibsWithPendingHoldRequests {
 236     my $dbh = C4::Context->dbh;
 237
 238     my $bib_query = "SELECT DISTINCT biblionumber
 239                      FROM reserves
 240                      WHERE found IS NULL
 241                      AND priority > 0
 242                      AND reservedate <= CURRENT_DATE()
 243                      AND suspend = 0
 244                      ";
 245     my $sth = $dbh->prepare($bib_query);
 246
 247     $sth->execute();
 248     my $biblionumbers = $sth->fetchall_arrayref();
 249
 250     return [ map { $_->[0] } @$biblionumbers ];
 251 }
 252
 253 =head2 GetPendingHoldRequestsForBib
 254
 255   my $requests = GetPendingHoldRequestsForBib($biblionumber);
 256
 257 Returns an arrayref of hashrefs to pending, unfilled hold requests
 258 on the bib identified by $biblionumber.  The following keys
 259 are present in each hashref:
 260
 261     biblionumber
 262     borrowernumber
 263     itemnumber
 264     priority
 265     branchcode
 266     reservedate
 267     reservenotes
 268     borrowerbranch
 269
 270 The arrayref is sorted in order of increasing priority.
 271
 272 =cut
 273
 274 sub GetPendingHoldRequestsForBib {
 275     my $biblionumber = shift;
 276
 277     my $dbh = C4::Context->dbh;
 278
 279     my $request_query = "SELECT biblionumber, borrowernumber, itemnumber, priority, reserves.branchcode,
 280                                 reservedate, reservenotes, borrowers.branchcode AS borrowerbranch, itemtype
 281                          FROM reserves
 282                          JOIN borrowers USING (borrowernumber)
 283                          WHERE biblionumber = ?
 284                          AND found IS NULL
 285                          AND priority > 0
 286                          AND reservedate <= CURRENT_DATE()
 287                          AND suspend = 0
 288                          ORDER BY priority";
 289     my $sth = $dbh->prepare($request_query);
 290     $sth->execute($biblionumber);
 291
 292     my $requests = $sth->fetchall_arrayref({});
 293     return $requests;
 294
 295 }
 296
 297 =head2 GetItemsAvailableToFillHoldRequestsForBib
 298
 299   my $available_items = GetItemsAvailableToFillHoldRequestsForBib($biblionumber, $branches_ar);
 300
 301 Returns an arrayref of items available to fill hold requests
 302 for the bib identified by C<$biblionumber>.  An item is available
 303 to fill a hold request if and only if:
 304
 305     * it is not on loan
 306     * it is not withdrawn
 307     * it is not marked notforloan
 308     * it is not currently in transit
 309     * it is not lost
 310     * it is not sitting on the hold shelf
 311     * it is not damaged (unless AllowHoldsOnDamagedItems is on)
 312
 313 =cut
 314
 315 sub GetItemsAvailableToFillHoldRequestsForBib {
 316     my ($biblionumber, $branches_to_use) = @_;
 317
 318     my $dbh = C4::Context->dbh;
 319     my $items_query = "SELECT itemnumber, homebranch, holdingbranch, itemtypes.itemtype AS itype
 320                        FROM items ";
 321
 322     if (C4::Context->preference('item-level_itypes')) {
 323         $items_query .=   "LEFT JOIN itemtypes ON (itemtypes.itemtype = items.itype) ";
 324     } else {
 325         $items_query .=   "JOIN biblioitems USING (biblioitemnumber)
 326                            LEFT JOIN itemtypes USING (itemtype) ";
 327     }
 328     $items_query .=   "WHERE items.notforloan = 0
 329                        AND holdingbranch IS NOT NULL
 330                        AND itemlost = 0
 331                        AND withdrawn = 0";
 332     $items_query .= "  AND damaged = 0" unless C4::Context->preference('AllowHoldsOnDamagedItems');
 333     $items_query .= "  AND items.onloan IS NULL
 334                        AND (itemtypes.notforloan IS NULL OR itemtypes.notforloan = 0)
 335                        AND itemnumber NOT IN (
 336                            SELECT itemnumber
 337                            FROM reserves
 338                            WHERE biblionumber = ?
 339                            AND itemnumber IS NOT NULL
 340                            AND (found IS NOT NULL OR priority = 0)
 341                         )
 342                        AND items.biblionumber = ?";
 343
 344     my @params = ($biblionumber, $biblionumber);
 345     if ($branches_to_use && @$branches_to_use) {
 346         $items_query .= " AND holdingbranch IN (" . join (",", map { "?" } @$branches_to_use) . ")";
 347         push @params, @$branches_to_use;
 348     }
 349     my $sth = $dbh->prepare($items_query);
 350     $sth->execute(@params);
 351
 352     my $itm = $sth->fetchall_arrayref({});
 353     my @items = grep { ! scalar GetTransfers($_->{itemnumber}) } @$itm;
 354     return [ grep {
 355         my $rule = GetBranchItemRule($_->{homebranch}, $_->{itype});
 356         $_->{holdallowed} = $rule->{holdallowed};
 357         $_->{hold_fulfillment_policy} = $rule->{hold_fulfillment_policy};
 358     } @items ];
 359 }
 360
 361 =head2 MapItemsToHoldRequests
 362
 363   MapItemsToHoldRequests($hold_requests, $available_items, $branches, $transport_cost_matrix)
 364
 365 =cut
 366
 367 sub MapItemsToHoldRequests {
 368     my ($hold_requests, $available_items, $branches_to_use, $transport_cost_matrix) = @_;
 369
 370     # handle trival cases
 371     return unless scalar(@$hold_requests) > 0;
 372     return unless scalar(@$available_items) > 0;
 373
 374     # identify item-level requests
 375     my %specific_items_requested = map { $_->{itemnumber} => 1 }
 376                                    grep { defined($_->{itemnumber}) }
 377                                    @$hold_requests;
 378
 379     map { $_->{_object} = Koha::Items->find( $_->{itemnumber} ) } @$available_items;
 380     my $libraries = {};
 381     map { $libraries->{$_->id} = $_ } Koha::Libraries->search();
 382
 383     # group available items by itemnumber
 384     my %items_by_itemnumber = map { $_->{itemnumber} => $_ } @$available_items;
 385
 386     # items already allocated
 387     my %allocated_items = ();
 388
 389     # map of items to hold requests
 390     my %item_map = ();
 391
 392     # figure out which item-level requests can be filled
 393     my $num_items_remaining = scalar(@$available_items);
 394
 395     # Look for Local Holds Priority matches first
 396     if ( C4::Context->preference('LocalHoldsPriority') ) {
 397         my $LocalHoldsPriorityPatronControl =
 398           C4::Context->preference('LocalHoldsPriorityPatronControl');
 399         my $LocalHoldsPriorityItemControl =
 400           C4::Context->preference('LocalHoldsPriorityItemControl');
 401
 402         foreach my $request (@$hold_requests) {
 403             last if $num_items_remaining == 0;
 404
 405             my $local_hold_match;
 406             foreach my $item (@$available_items) {
 407                 next
 408                   if ( !$item->{holdallowed} )
 409                   || ( $item->{holdallowed} == 1
 410                     && $item->{homebranch} ne $request->{borrowerbranch} );
 411
 412                 next unless $item->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } );
 413
 414                 my $local_holds_priority_item_branchcode =
 415                   $item->{$LocalHoldsPriorityItemControl};
 416
 417                 my $local_holds_priority_patron_branchcode =
 418                   ( $LocalHoldsPriorityPatronControl eq 'PickupLibrary' )
 419                   ? $request->{branchcode}
 420                   : ( $LocalHoldsPriorityPatronControl eq 'HomeLibrary' )
 421                   ? $request->{borrowerbranch}
 422                   : undef;
 423
 424                 $local_hold_match =
 425                   $local_holds_priority_item_branchcode eq
 426                   $local_holds_priority_patron_branchcode;
 427
 428                 if ($local_hold_match) {
 429                     if ( exists $items_by_itemnumber{ $item->{itemnumber} }
 430                         and not exists $allocated_items{ $item->{itemnumber} }
 431                         and not $request->{allocated})
 432                     {
 433                         $item_map{ $item->{itemnumber} } = {
 434                             borrowernumber => $request->{borrowernumber},
 435                             biblionumber   => $request->{biblionumber},
 436                             holdingbranch  => $item->{holdingbranch},
 437                             pickup_branch  => $request->{branchcode}
 438                               || $request->{borrowerbranch},
 439                             item_level   => 0,
 440                             reservedate  => $request->{reservedate},
 441                             reservenotes => $request->{reservenotes},
 442                         };
 443                         $allocated_items{ $item->{itemnumber} }++;
 444                         $request->{allocated} = 1;
 445                         $num_items_remaining--;
 446                     }
 447                 }
 448             }
 449         }
 450     }
 451
 452     foreach my $request (@$hold_requests) {
 453         last if $num_items_remaining == 0;
 454         next if $request->{allocated};
 455
 456         # is this an item-level request?
 457         if (defined($request->{itemnumber})) {
 458             # fill it if possible; if not skip it
 459             if (
 460                     exists $items_by_itemnumber{ $request->{itemnumber} }
 461                 and not exists $allocated_items{ $request->{itemnumber} }
 462                 and ( # Don't fill item level holds that contravene the hold pickup policy at this time
 463                     ( $items_by_itemnumber{ $request->{itemnumber} }->{hold_fulfillment_policy} eq 'any' )
 464                     || ( $request->{branchcode} eq $items_by_itemnumber{ $request->{itemnumber} }->{ $items_by_itemnumber{ $request->{itemnumber} }->{hold_fulfillment_policy} }  )
 465                 and ( !$request->{itemtype} # If hold itemtype is set, item's itemtype must match
 466                     || $items_by_itemnumber{ $request->{itemnumber} }->{itype} eq $request->{itemtype} )
 467                 )
 468                 and $items_by_itemnumber{ $request->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } )
 469
 470               )
 471             {
 472
 473                 $item_map{ $request->{itemnumber} } = {
 474                     borrowernumber => $request->{borrowernumber},
 475                     biblionumber   => $request->{biblionumber},
 476                     holdingbranch  => $items_by_itemnumber{ $request->{itemnumber} }->{holdingbranch},
 477                     pickup_branch  => $request->{branchcode} || $request->{borrowerbranch},
 478                     item_level     => 1,
 479                     reservedate    => $request->{reservedate},
 480                     reservenotes   => $request->{reservenotes},
 481                 };
 482                 $allocated_items{ $request->{itemnumber} }++;
 483                 $num_items_remaining--;
 484             }
 485         } else {
 486             # it's title-level request that will take up one item
 487             $num_items_remaining--;
 488         }
 489     }
 490
 491     # group available items by branch
 492     my %items_by_branch = ();
 493     foreach my $item (@$available_items) {
 494         next unless $item->{holdallowed};
 495
 496         push @{ $items_by_branch{ $item->{holdingbranch} } }, $item
 497           unless exists $allocated_items{ $item->{itemnumber} };
 498     }
 499     return \%item_map unless keys %items_by_branch;
 500
 501     # now handle the title-level requests
 502     $num_items_remaining = scalar(@$available_items) - scalar(keys %allocated_items);
 503     my $pull_branches;
 504     foreach my $request (@$hold_requests) {
 505         last if $num_items_remaining == 0;
 506         next if $request->{allocated};
 507         next if defined($request->{itemnumber}); # already handled these
 508
 509         # look for local match first
 510         my $pickup_branch = $request->{branchcode} || $request->{borrowerbranch};
 511         my ($itemnumber, $holdingbranch);
 512
 513         my $holding_branch_items = $items_by_branch{$pickup_branch};
 514         if ( $holding_branch_items ) {
 515             foreach my $item (@$holding_branch_items) {
 516                 next unless $items_by_itemnumber{ $item->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } );
 517
 518                 if (
 519                     $request->{borrowerbranch} eq $item->{homebranch}
 520                     && ( ( $item->{hold_fulfillment_policy} eq 'any' ) # Don't fill item level holds that contravene the hold pickup policy at this time
 521                         || $request->{branchcode} eq $item->{ $item->{hold_fulfillment_policy} } )
 522                     && ( !$request->{itemtype} # If hold itemtype is set, item's itemtype must match
 523                         || $items_by_itemnumber{ $request->{itemnumber} }->{itype} eq $request->{itemtype} )
 524                   )
 525                 {
 526                     $itemnumber = $item->{itemnumber};
 527                     last;
 528                 }
 529             }
 530             $holdingbranch = $pickup_branch;
 531         }
 532         elsif ($transport_cost_matrix) {
 533             $pull_branches = [keys %items_by_branch];
 534             $holdingbranch = least_cost_branch( $pickup_branch, $pull_branches, $transport_cost_matrix );
 535             if ( $holdingbranch ) {
 536
 537                 my $holding_branch_items = $items_by_branch{$holdingbranch};
 538                 foreach my $item (@$holding_branch_items) {
 539                     next if $request->{borrowerbranch} ne $item->{homebranch};
 540                     next unless $items_by_itemnumber{ $item->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } );
 541
 542                     # Don't fill item level holds that contravene the hold pickup policy at this time
 543                     next unless $item->{hold_fulfillment_policy} eq 'any'
 544                         || $request->{branchcode} eq $item->{ $item->{hold_fulfillment_policy} };
 545
 546                     # If hold itemtype is set, item's itemtype must match
 547                     next unless ( !$request->{itemtype}
 548                         || $item->{itype} eq $request->{itemtype} );
 549
 550                     $itemnumber = $item->{itemnumber};
 551                     last;
 552                 }
 553             }
 554             else {
 555                 next;
 556             }
 557         }
 558
 559         unless ($itemnumber) {
 560             # not found yet, fall back to basics
 561             if ($branches_to_use) {
 562                 $pull_branches = $branches_to_use;
 563             } else {
 564                 $pull_branches = [keys %items_by_branch];
 565             }
 566
 567             # Try picking items where the home and pickup branch match first
 568             PULL_BRANCHES:
 569             foreach my $branch (@$pull_branches) {
 570                 my $holding_branch_items = $items_by_branch{$branch}
 571                   or next;
 572
 573                 $holdingbranch ||= $branch;
 574                 foreach my $item (@$holding_branch_items) {
 575                     next if $pickup_branch ne $item->{homebranch};
 576                     next if ( $item->{holdallowed} == 1 && $item->{homebranch} ne $request->{borrowerbranch} );
 577                     next unless $items_by_itemnumber{ $item->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } );
 578
 579                     # Don't fill item level holds that contravene the hold pickup policy at this time
 580                     next unless $item->{hold_fulfillment_policy} eq 'any'
 581                         || $request->{branchcode} eq $item->{ $item->{hold_fulfillment_policy} };
 582
 583                     # If hold itemtype is set, item's itemtype must match
 584                     next unless ( !$request->{itemtype}
 585                         || $item->{itype} eq $request->{itemtype} );
 586
 587                     $itemnumber = $item->{itemnumber};
 588                     $holdingbranch = $branch;
 589                     last PULL_BRANCHES;
 590                 }
 591             }
 592
 593             # Now try items from the least cost branch based on the transport cost matrix or StaticHoldsQueueWeight
 594             unless ( $itemnumber ) {
 595                 foreach my $current_item ( @{ $items_by_branch{$holdingbranch} } ) {
 596                     if ( $holdingbranch && ( $current_item->{holdallowed} == 2 || $request->{borrowerbranch} eq $current_item->{homebranch} ) ) {
 597
 598                         # Don't fill item level holds that contravene the hold pickup policy at this time
 599                         next unless $current_item->{hold_fulfillment_policy} eq 'any'
 600                             || $request->{branchcode} eq $current_item->{ $current_item->{hold_fulfillment_policy} };
 601
 602                         # If hold itemtype is set, item's itemtype must match
 603                         next unless ( !$request->{itemtype}
 604                             || $current_item->{itype} eq $request->{itemtype} );
 605
 606                         next unless $items_by_itemnumber{ $current_item->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } );
 607
 608                         $itemnumber = $current_item->{itemnumber};
 609                         last; # quit this loop as soon as we have a suitable item
 610                     }
 611                 }
 612             }
 613
 614             # Now try for items for any item that can fill this hold
 615             unless ( $itemnumber ) {
 616                 PULL_BRANCHES2:
 617                 foreach my $branch (@$pull_branches) {
 618                     my $holding_branch_items = $items_by_branch{$branch}
 619                       or next;
 620
 621                     foreach my $item (@$holding_branch_items) {
 622                         next if ( $item->{holdallowed} == 1 && $item->{homebranch} ne $request->{borrowerbranch} );
 623
 624                         # Don't fill item level holds that contravene the hold pickup policy at this time
 625                         next unless $item->{hold_fulfillment_policy} eq 'any'
 626                             || $request->{branchcode} eq $item->{ $item->{hold_fulfillment_policy} };
 627
 628                         # If hold itemtype is set, item's itemtype must match
 629                         next unless ( !$request->{itemtype}
 630                             || $item->{itype} eq $request->{itemtype} );
 631
 632                         next unless $items_by_itemnumber{ $item->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } );
 633
 634                         $itemnumber = $item->{itemnumber};
 635                         $holdingbranch = $branch;
 636                         last PULL_BRANCHES2;
 637                     }
 638                 }
 639             }
 640         }
 641
 642         if ($itemnumber) {
 643             my $holding_branch_items = $items_by_branch{$holdingbranch}
 644               or die "Have $itemnumber, $holdingbranch, but no items!";
 645             @$holding_branch_items = grep { $_->{itemnumber} != $itemnumber } @$holding_branch_items;
 646             delete $items_by_branch{$holdingbranch} unless @$holding_branch_items;
 647
 648             $item_map{$itemnumber} = {
 649                 borrowernumber => $request->{borrowernumber},
 650                 biblionumber => $request->{biblionumber},
 651                 holdingbranch => $holdingbranch,
 652                 pickup_branch => $pickup_branch,
 653                 item_level => 0,
 654                 reservedate => $request->{reservedate},
 655                 reservenotes => $request->{reservenotes},
 656             };
 657             $num_items_remaining--;
 658         }
 659     }
 660     return \%item_map;
 661 }
 662
 663 =head2 CreatePickListFromItemMap
 664
 665 =cut
 666
 667 sub CreatePicklistFromItemMap {
 668     my $item_map = shift;
 669
 670     my $dbh = C4::Context->dbh;
 671
 672     my $sth_load=$dbh->prepare("
 673         INSERT INTO tmp_holdsqueue (biblionumber,itemnumber,barcode,surname,firstname,phone,borrowernumber,
 674                                     cardnumber,reservedate,title, itemcallnumber,
 675                                     holdingbranch,pickbranch,notes, item_level_request)
 676         VALUES (?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)
 677     ");
 678
 679     foreach my $itemnumber  (sort keys %$item_map) {
 680         my $mapped_item = $item_map->{$itemnumber};
 681         my $biblionumber = $mapped_item->{biblionumber};
 682         my $borrowernumber = $mapped_item->{borrowernumber};
 683         my $pickbranch = $mapped_item->{pickup_branch};
 684         my $holdingbranch = $mapped_item->{holdingbranch};
 685         my $reservedate = $mapped_item->{reservedate};
 686         my $reservenotes = $mapped_item->{reservenotes};
 687         my $item_level = $mapped_item->{item_level};
 688
 689         my $item = Koha::Items->find($itemnumber);
 690         my $barcode = $item->barcode;
 691         my $itemcallnumber = $item->itemcallnumber;
 692
 693         my $patron = Koha::Patrons->find( $borrowernumber );
 694         my $cardnumber = $patron->cardnumber;
 695         my $surname = $patron->surname;
 696         my $firstname = $patron->firstname;
 697         my $phone = $patron->phone;
 698
 699         my $biblio = Koha::Biblios->find( $biblionumber );
 700         my $title = $biblio->title;
 701
 702         $sth_load->execute($biblionumber, $itemnumber, $barcode, $surname, $firstname, $phone, $borrowernumber,
 703                            $cardnumber, $reservedate, $title, $itemcallnumber,
 704                            $holdingbranch, $pickbranch, $reservenotes, $item_level);
 705     }
 706 }
 707
 708 =head2 AddToHoldTargetMap
 709
 710 =cut
 711
 712 sub AddToHoldTargetMap {
 713     my $item_map = shift;
 714
 715     my $dbh = C4::Context->dbh;
 716
 717     my $insert_sql = q(
 718         INSERT INTO hold_fill_targets (borrowernumber, biblionumber, itemnumber, source_branchcode, item_level_request)
 719                                VALUES (?, ?, ?, ?, ?)
 720     );
 721     my $sth_insert = $dbh->prepare($insert_sql);
 722
 723     foreach my $itemnumber (keys %$item_map) {
 724         my $mapped_item = $item_map->{$itemnumber};
 725         $sth_insert->execute($mapped_item->{borrowernumber}, $mapped_item->{biblionumber}, $itemnumber,
 726                              $mapped_item->{holdingbranch}, $mapped_item->{item_level});
 727     }
 728 }
 729
 730 # Helper functions, not part of any interface
 731
 732 sub _trim {
 733     return $_[0] unless $_[0];
 734     $_[0] =~ s/^\s+//;
 735     $_[0] =~ s/\s+$//;
 736     $_[0];
 737 }
 738
 739 sub load_branches_to_pull_from {
 740     my @branches_to_use;
 741
 742     my $static_branch_list = C4::Context->preference("StaticHoldsQueueWeight");
 743     @branches_to_use = map { _trim($_) } split( /,/, $static_branch_list )
 744       if $static_branch_list;
 745
 746     @branches_to_use =
 747       Koha::Database->new()->schema()->resultset('Branch')
 748       ->get_column('branchcode')->all()
 749       unless (@branches_to_use);
 750
 751     @branches_to_use = shuffle(@branches_to_use)
 752       if C4::Context->preference("RandomizeHoldsQueueWeight");
 753
 754     my $today = dt_from_string();
 755     if ( C4::Context->preference('HoldsQueueSkipClosed') ) {
 756         @branches_to_use = grep {
 757             !Koha::Calendar->new( branchcode => $_ )
 758               ->is_holiday( $today )
 759         } @branches_to_use;
 760     }
 761
 762     return \@branches_to_use;
 763 }
 764
 765 sub least_cost_branch {
 766
 767     #$from - arrayref
 768     my ($to, $from, $transport_cost_matrix) = @_;
 769
 770     # Nothing really spectacular: supply to branch, a list of potential from branches
 771     # and find the minimum from - to value from the transport_cost_matrix
 772     return $from->[0] if ( @$from == 1 && $transport_cost_matrix->{$to}{$from->[0]}->{disable_transfer} != 1 );
 773
 774     # If the pickup library is in the list of libraries to pull from,
 775     # return that library right away, it is obviously the least costly
 776     return ($to) if any { $_ eq $to } @$from;
 777
 778     my ($least_cost, @branch);
 779     foreach (@$from) {
 780         my $cell = $transport_cost_matrix->{$to}{$_};
 781         next if $cell->{disable_transfer};
 782
 783         my $cost = $cell->{cost};
 784         next unless defined $cost; # XXX should this be reported?
 785
 786         unless (defined $least_cost) {
 787             $least_cost = $cost;
 788             push @branch, $_;
 789             next;
 790         }
 791
 792         next if $cost > $least_cost;
 793
 794         if ($cost == $least_cost) {
 795             push @branch, $_;
 796             next;
 797         }
 798
 799         @branch = ($_);
 800         $least_cost = $cost;
 801     }
 802
 803     return $branch[0];
 804
 805     # XXX return a random @branch with minimum cost instead of the first one;
 806     # return $branch[0] if @branch == 1;
 807 }
 808
 809
 810 1;