]> git.evergreen-ils.org Git - Evergreen.git/blob - Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
adding capture time recheck to targeter
[Evergreen.git] / Open-ILS / src / perlmods / OpenILS / Application / Storage / Publisher / action.pm
1 package OpenILS::Application::Storage::Publisher::action;
2 use base qw/OpenILS::Application::Storage::Publisher/;
3 use OpenSRF::Utils::Logger qw/:level/;
4 use OpenSRF::Utils qw/:datetime/;
5 use OpenSRF::AppSession;
6 use OpenSRF::EX qw/:try/;
7 use OpenILS::Utils::Fieldmapper;
8 use OpenILS::Utils::PermitHold;
9 use DateTime;
10 use DateTime::Format::ISO8601;
11
12 sub isTrue {
13         my $v = shift;
14         return 1 if ($v == 1);
15         return 1 if ($v =~ /^t/io);
16         return 1 if ($v =~ /^y/io);
17         return 0;
18 }
19
20 my $parser = DateTime::Format::ISO8601->new;
21 my $log = 'OpenSRF::Utils::Logger';
22
23 sub open_noncat_circs {
24         my $self = shift;
25         my $client = shift;
26         my $user = shift;
27
28         my $a = action::non_cataloged_circulation->table;
29         my $c = config::non_cataloged_type->table;
30
31         my $sql = <<"   SQL";
32                 SELECT  a.id
33                   FROM  $a a
34                         JOIN $c c ON (a.item_type = c.id)
35                   WHERE a.circ_time + c.circ_duration > current_timestamp
36                         AND a.patron = ?
37         SQL
38
39         return action::non_cataloged_circulation->db_Main->selectcol_arrayref($sql, {}, $user);
40 }
41 __PACKAGE__->register_method(
42         api_name        => 'open-ils.storage.action.open_non_cataloged_circulation.user',
43         api_level       => 1,
44         argc            => 1,
45         method          => 'open_noncat_circs',
46 );
47
48
49 sub ou_hold_requests {
50         my $self = shift;
51         my $client = shift;
52         my $ou = shift;
53
54         my $h_table = action::hold_request->table;
55         my $c_table = asset::copy->table;
56         my $o_table = actor::org_unit->table;
57
58         my $SQL = <<"   SQL";
59                 SELECT  h.id
60                   FROM  $h_table h
61                         JOIN $c_table cp ON (cp.id = h.current_copy)
62                         JOIN $o_table ou ON (ou.id = cp.circ_lib)
63                   WHERE ou.id = ?
64                         AND h.capture_time IS NULL
65                         AND h.cancel_time IS NULL
66                   ORDER BY h.request_time
67         SQL
68
69         my $sth = action::hold_request->db_Main->prepare_cached($SQL);
70         $sth->execute($ou);
71
72         $client->respond($_) for (
73                 map {
74                         $self
75                                 ->method_lookup('open-ils.storage.direct.action.hold_request.retrieve')
76                                 ->run($_)
77                 } map {
78                         $_->[0]
79                 } @{ $sth->fetchall_arrayref }
80         );
81         return undef;
82 }
83 __PACKAGE__->register_method(
84         api_name        => 'open-ils.storage.action.targeted_hold_request.org_unit',
85         api_level       => 1,
86         argc            => 1,
87         stream          => 1,
88         method          => 'ou_hold_requests',
89 );
90
91
92 sub overdue_circs {
93         my $grace = shift;
94
95         my $c_t = action::circulation->table;
96
97         $grace = " - ($grace * (fine_interval))" if ($grace);
98
99         my $sql = <<"   SQL";
100                 SELECT  *
101                   FROM  $c_t
102                   WHERE stop_fines IS NULL
103                         AND due_date < ( CURRENT_TIMESTAMP $grace)
104         SQL
105
106         my $sth = action::circulation->db_Main->prepare_cached($sql);
107         $sth->execute;
108
109         return ( map { action::circulation->construct($_) } $sth->fetchall_hash );
110
111 }
112
113 sub complete_reshelving {
114         my $self = shift;
115         my $client = shift;
116         my $window = shift;
117
118         local $OpenILS::Application::Storage::WRITE = 1;
119
120         throw OpenSRF::EX::InvalidArg ("I need an interval of more than 0 seconds!")
121                 unless (interval_to_seconds( $window ));
122
123         my $circ = action::circulation->table;
124         my $cp = asset::copy->table;
125
126         my $sql = <<"   SQL";
127                 UPDATE  $cp
128                   SET   status = 0
129                   WHERE id IN
130                         ( SELECT id FROM (
131                                 SELECT  cp.id, MAX(circ.checkin_time)
132                                   FROM  $cp cp
133                                         JOIN $circ circ ON (circ.target_copy = cp.id)
134                                   WHERE circ.checkin_time IS NOT NULL
135                                         AND cp.status = 7
136                                   GROUP BY 1
137                                         HAVING MAX(circ.checkin_time) < NOW() - CAST(? AS INTERVAL)
138                         ) AS foo
139                 )
140         SQL
141
142         my $sth = action::circulation->db_Main->prepare_cached($sql);
143         $sth->execute($window);
144
145         return $sth->rows;
146
147 }
148 __PACKAGE__->register_method(
149         api_name        => 'open-ils.storage.action.circulation.reshelving.complete',
150         api_level       => 1,
151         stream          => 1,
152         argc            => 1,
153         method          => 'complete_reshelving',
154 );
155
156 sub grab_overdue {
157         my $self = shift;
158         my $client = shift;
159         my $grace = shift || '';
160
161         $client->respond( $_->to_fieldmapper ) for ( overdue_circs($grace) );
162
163         return undef;
164
165 }
166 __PACKAGE__->register_method(
167         api_name        => 'open-ils.storage.action.circulation.overdue',
168         api_level       => 1,
169         stream          => 1,
170         method          => 'grab_overdue',
171 );
172
173 sub nearest_hold {
174         my $self = shift;
175         my $client = shift;
176         my $pl = shift;
177         my $cp = shift;
178         my $limit = int(shift()) || 10;
179
180         my ($id) = action::hold_request->db_Main->selectrow_array(<<"   SQL", {}, $cp, $pl);
181                 SELECT  h.id
182                   FROM  action.hold_request h
183                         JOIN action.hold_copy_map hm ON (hm.hold = h.id)
184                         JOIN actor.org_unit_proximity p ON (p.from_org = h.pickup_lib)
185                   WHERE hm.target_copy = ?
186                         AND p.to_org = ?
187                         AND h.capture_time IS NULL
188                         AND h.cancel_time IS NULL
189                 ORDER BY
190                         p.prox,
191                         h.selection_depth DESC,
192                         h.request_time
193                 LIMIT $limit
194         SQL
195         return $id;
196 }
197 __PACKAGE__->register_method(
198         api_name        => 'open-ils.storage.action.hold_request.nearest_hold',
199         api_level       => 1,
200         stream          => 1,
201         method          => 'nearest_hold',
202 );
203
204 sub next_resp_group_id {
205         my $self = shift;
206         my $client = shift;
207
208         # XXX This is not replication safe!!!
209
210         my ($id) = action::survey->db_Main->selectrow_array(<<" SQL");
211                 SELECT NEXTVAL('action.survey_response_group_id_seq'::TEXT)
212         SQL
213         return $id;
214 }
215 __PACKAGE__->register_method(
216         api_name        => 'open-ils.storage.action.survey_response.next_group_id',
217         api_level       => 1,
218         method          => 'next_resp_group_id',
219 );
220
221 sub patron_circ_summary {
222         my $self = shift;
223         my $client = shift;
224         my $id = ''.shift();
225
226         return undef unless ($id);
227         my $c_table = action::circulation->table;
228         my $b_table = money::billing->table;
229
230         $log->debug("Retrieving patron summary for id $id", DEBUG);
231
232         my $select = <<"        SQL";
233                 SELECT  COUNT(DISTINCT c.id), SUM( COALESCE(b.amount,0) )
234                   FROM  $c_table c
235                         LEFT OUTER JOIN $b_table b ON (c.id = b.xact AND b.voided = FALSE)
236                   WHERE c.usr = ?
237                         AND c.xact_finish IS NULL
238                         AND (
239                                 c.stop_fines NOT IN ('CLAIMSRETURNED','LOST')
240                                 OR c.stop_fines IS NULL
241                         )
242         SQL
243
244         return action::survey->db_Main->selectrow_arrayref($select, {}, $id);
245 }
246 __PACKAGE__->register_method(
247         api_name        => 'open-ils.storage.action.circulation.patron_summary',
248         api_level       => 1,
249         method          => 'patron_circ_summary',
250 );
251
252 #XXX Fix stored proc calls
253 sub find_local_surveys {
254         my $self = shift;
255         my $client = shift;
256         my $ou = ''.shift();
257
258         return undef unless ($ou);
259         my $s_table = action::survey->table;
260
261         my $select = <<"        SQL";
262                 SELECT  s.*
263                   FROM  $s_table s
264                         JOIN actor.org_unit_full_path(?) p ON (p.id = s.owner)
265                   WHERE CURRENT_TIMESTAMP BETWEEN s.start_date AND s.end_date
266         SQL
267
268         my $sth = action::survey->db_Main->prepare_cached($select);
269         $sth->execute($ou);
270
271         $client->respond( $_->to_fieldmapper ) for ( map { action::survey->construct($_) } $sth->fetchall_hash );
272
273         return undef;
274 }
275 __PACKAGE__->register_method(
276         api_name        => 'open-ils.storage.action.survey.all',
277         api_level       => 1,
278         stream          => 1,
279         method          => 'find_local_surveys',
280 );
281
282 #XXX Fix stored proc calls
283 sub find_opac_surveys {
284         my $self = shift;
285         my $client = shift;
286         my $ou = ''.shift();
287
288         return undef unless ($ou);
289         my $s_table = action::survey->table;
290
291         my $select = <<"        SQL";
292                 SELECT  s.*
293                   FROM  $s_table s
294                         JOIN actor.org_unit_full_path(?) p ON (p.id = s.owner)
295                   WHERE CURRENT_TIMESTAMP BETWEEN s.start_date AND s.end_date
296                         AND s.opac IS TRUE;
297         SQL
298
299         my $sth = action::survey->db_Main->prepare_cached($select);
300         $sth->execute($ou);
301
302         $client->respond( $_->to_fieldmapper ) for ( map { action::survey->construct($_) } $sth->fetchall_hash );
303
304         return undef;
305 }
306 __PACKAGE__->register_method(
307         api_name        => 'open-ils.storage.action.survey.opac',
308         api_level       => 1,
309         stream          => 1,
310         method          => 'find_opac_surveys',
311 );
312
313 sub hold_pull_list {
314         my $self = shift;
315         my $client = shift;
316         my $ou = shift;
317         my $limit = shift || 10;
318         my $offset = shift || 0;
319
320         return undef unless ($ou);
321         my $h_table = action::hold_request->table;
322         my $a_table = asset::copy->table;
323
324         my $idlist = 1 if ($self->api_name =~/id_list/o);
325
326         my $select = <<"        SQL";
327                 SELECT  h.*
328                   FROM  $h_table h
329                         JOIN $a_table a ON (h.current_copy = a.id)
330                   WHERE a.circ_lib = ?
331                         AND h.capture_time IS NULL
332                         AND h.cancel_time IS NULL
333                   ORDER BY h.request_time ASC
334                   LIMIT $limit
335                   OFFSET $offset
336         SQL
337
338         my $sth = action::survey->db_Main->prepare_cached($select);
339         $sth->execute($ou);
340
341         if ($idlist) {
342                 $client->respond( $_->{id} ) for ( $sth->fetchall_hash );
343         } else {
344                 $client->respond( $_->to_fieldmapper ) for ( map { action::hold_request->construct($_) } $sth->fetchall_hash );
345         }
346
347         return undef;
348 }
349 __PACKAGE__->register_method(
350         api_name        => 'open-ils.storage.direct.action.hold_request.pull_list.id_list.current_copy_circ_lib',
351         api_level       => 1,
352         stream          => 1,
353         signature       => [
354                 "Returns the hold ids for a specific library's pull list.",
355                 [ [org_unit => "The library's org id", "number"],
356                   [limit => 'An optional page size, defaults to 10', 'number'],
357                   [offset => 'Offset for paging, defaults to 0, 0 based', 'number'],
358                 ],
359                 ['A list of holds for the stated library to pull for', 'array']
360         ],
361         method          => 'hold_pull_list',
362 );
363 __PACKAGE__->register_method(
364         api_name        => 'open-ils.storage.direct.action.hold_request.pull_list.search.current_copy_circ_lib',
365         api_level       => 1,
366         stream          => 1,
367         signature       => [
368                 "Returns the holds for a specific library's pull list.",
369                 [ [org_unit => "The library's org id", "number"],
370                   [limit => 'An optional page size, defaults to 10', 'number'],
371                   [offset => 'Offset for paging, defaults to 0, 0 based', 'number'],
372                 ],
373                 ['A list of holds for the stated library to pull for', 'array']
374         ],
375         method          => 'hold_pull_list',
376 );
377
378 sub find_optional_surveys {
379         my $self = shift;
380         my $client = shift;
381         my $ou = ''.shift();
382
383         return undef unless ($ou);
384         my $s_table = action::survey->table;
385
386         my $select = <<"        SQL";
387                 SELECT  s.*
388                   FROM  $s_table s
389                         JOIN actor.org_unit_full_path(?) p ON (p.id = s.owner)
390                   WHERE CURRENT_TIMESTAMP BETWEEN s.start_date AND s.end_date
391                         AND s.required IS FALSE;
392         SQL
393
394         my $sth = action::survey->db_Main->prepare_cached($select);
395         $sth->execute($ou);
396
397         $client->respond( $_->to_fieldmapper ) for ( map { action::survey->construct($_) } $sth->fetchall_hash );
398
399         return undef;
400 }
401 __PACKAGE__->register_method(
402         api_name        => 'open-ils.storage.action.survey.optional',
403         api_level       => 1,
404         stream          => 1,
405         method          => 'find_optional_surveys',
406 );
407
408 sub find_required_surveys {
409         my $self = shift;
410         my $client = shift;
411         my $ou = ''.shift();
412
413         return undef unless ($ou);
414         my $s_table = action::survey->table;
415
416         my $select = <<"        SQL";
417                 SELECT  s.*
418                   FROM  $s_table s
419                         JOIN actor.org_unit_full_path(?) p ON (p.id = s.owner)
420                   WHERE CURRENT_TIMESTAMP BETWEEN s.start_date AND s.end_date
421                         AND s.required IS TRUE;
422         SQL
423
424         my $sth = action::survey->db_Main->prepare_cached($select);
425         $sth->execute($ou);
426
427         $client->respond( $_->to_fieldmapper ) for ( map { action::survey->construct($_) } $sth->fetchall_hash );
428
429         return undef;
430 }
431 __PACKAGE__->register_method(
432         api_name        => 'open-ils.storage.action.survey.required',
433         api_level       => 1,
434         stream          => 1,
435         method          => 'find_required_surveys',
436 );
437
438 sub find_usr_summary_surveys {
439         my $self = shift;
440         my $client = shift;
441         my $ou = ''.shift();
442
443         return undef unless ($ou);
444         my $s_table = action::survey->table;
445
446         my $select = <<"        SQL";
447                 SELECT  s.*
448                   FROM  $s_table s
449                         JOIN actor.org_unit_full_path(?) p ON (p.id = s.owner)
450                   WHERE CURRENT_TIMESTAMP BETWEEN s.start_date AND s.end_date
451                         AND s.usr_summary IS TRUE;
452         SQL
453
454         my $sth = action::survey->db_Main->prepare_cached($select);
455         $sth->execute($ou);
456
457         $client->respond( $_->to_fieldmapper ) for ( map { action::survey->construct($_) } $sth->fetchall_hash );
458
459         return undef;
460 }
461 __PACKAGE__->register_method(
462         api_name        => 'open-ils.storage.action.survey.usr_summary',
463         api_level       => 1,
464         stream          => 1,
465         method          => 'find_usr_summary_surveys',
466 );
467
468
469 sub generate_fines {
470         my $self = shift;
471         my $client = shift;
472         my $grace = shift;
473         my $circ = shift;
474         my $overbill = shift;
475
476         local $OpenILS::Application::Storage::WRITE = 1;
477
478         my @circs;
479         if ($circ) {
480                 push @circs, action::circulation->search_where( { id => $circ, stop_fines => undef } );
481         } else {
482                 push @circs, overdue_circs($grace);
483         }
484
485         my %hoo = map { ( $_->id => $_ ) } actor::org_unit::hours_of_operation->retrieve_all;
486
487         my $penalty = OpenSRF::AppSession->create('open-ils.penalty');
488         for my $c (@circs) {
489         
490                 try {
491                         if ($self->method_lookup('open-ils.storage.transaction.current')->run) {
492                                 $log->debug("Cleaning up after previous transaction\n");
493                                 $self->method_lookup('open-ils.storage.transaction.rollback')->run;
494                         }
495                         $self->method_lookup('open-ils.storage.transaction.begin')->run( $client );
496                         $log->info("Processing circ ".$c->id."...\n");
497
498
499                         my $due_dt = $parser->parse_datetime( clense_ISO8601( $c->due_date ) );
500         
501                         my $due = $due_dt->epoch;
502                         my $now = time;
503                         my $fine_interval = interval_to_seconds( $c->fine_interval );
504         
505                         if ( interval_to_seconds( $c->fine_interval ) >= interval_to_seconds('1d') ) {  
506                                 my $tz_offset_s = 0;
507                                 if ($due_dt->strftime('%z') =~ /(-|\+)(\d{2}):?(\d{2})/) {
508                                         $tz_offset_s = $1 . interval_to_seconds( "${2}h ${3}m"); 
509                                 }
510         
511                                 $due -= ($due % $fine_interval) + $tz_offset_s;
512                                 $now -= ($now % $fine_interval) + $tz_offset_s;
513                         }
514         
515                         $client->respond(
516                                 "ARG! Overdue circulation ".$c->id.
517                                 " for item ".$c->target_copy.
518                                 " (user ".$c->usr.").\n".
519                                 "\tItem was due on or before: ".localtime($due)."\n");
520         
521                         my @fines = money::billing->search_where(
522                                 { xact => $c->id, billing_type => 'Overdue materials' },
523                                 { order_by => 'billing_ts DESC'}
524                         );
525
526                         my $f_idx = 0;
527                         my $fine = $fines[$f_idx] if (@fines);
528                         if ($overbill) {
529                                 $fine = $fines[++$f_idx] while ($fine and $fine->voided);
530                         }
531
532                         my $current_fine_total = 0;
533                         $current_fine_total += int($_->amount * 100) for (grep { $_ and !$_->voided } @fines);
534         
535                         my $last_fine;
536                         if ($fine) {
537                                 $client->respond( "Last billing time: ".$fine->billing_ts." (clensed fromat: ".clense_ISO8601( $fine->billing_ts ).")");
538                                 $last_fine = $parser->parse_datetime( clense_ISO8601( $fine->billing_ts ) )->epoch;
539                         } else {
540                                 $log->info( "Potential first billing for circ ".$c->id );
541                                 $last_fine = $due;
542
543                                 if (0) {
544                                         if (my $h = $hoo{$c->circ_lib}) { 
545
546                                                 $log->info( "Circ lib has an hours-of-operation entry" );
547                                                 # find the day after the due date...
548                                                 $due_dt = $due_dt->add( days => 1 );
549
550                                                 # get the day of the week for that day...
551                                                 my $dow = $due_dt->day_of_week_0;
552                                                 my $dow_open = "dow_${dow}_open";
553                                                 my $dow_close = "dow_${dow}_close";
554
555                                                 my $count = 0;
556                                                 while ( $h->$dow_open eq '00:00:00' and $h->$dow_close eq '00:00:00' ) {
557                                                         # if the circ lib is closed, add a day to the grace period...
558
559                                                         $grace++;
560                                                         $log->info( "Grace period for circ ".$c->id." extended to $grace intervals" );
561                                                         $log->info( "Day of week $dow open $dow_open, close $dow_close" );
562
563                                                         $due_dt = $due_dt->add( days => 1 );
564                                                         $dow = $due_dt->day_of_week_0;
565                                                         $dow_open = "dow_${dow}_open";
566                                                         $dow_close = "dow_${dow}_close";
567
568                                                         $count++;
569
570                                                         # and check for up to a week
571                                                         last if ($count > 6);
572                                                 }
573                                         }
574                                 }
575                         }
576
577
578                         my $pending_fine_count = int( ($now - $last_fine) / $fine_interval ); 
579                         if ($pending_fine_count < 1 + $grace) {
580                                 $client->respond( "\tNo fines to create.  " );
581                                 if ($grace && $now < $due + $fine_interval * $grace) {
582                                         $client->respond( "Still inside grace period of: ". seconds_to_interval( $fine_interval * $grace)."\n" );
583                                         $log->info( "Circ ".$c->id." is still inside grace period of: $grace [". seconds_to_interval( $fine_interval * $grace).']' );
584                                 } else {
585                                         $client->respond( "Last fine generated for: ".localtime($last_fine)."\n" );
586                                 }
587                                 next;
588                         }
589         
590                         $client->respond( "\t$pending_fine_count pending fine(s)\n" );
591
592                         my $recuring_fine = int($c->recuring_fine * 100);
593                         my $max_fine = int($c->max_fine * 100);
594
595                         my ($latest_billing_ts, $latest_amount) = ('',0);
596                         for (my $bill = 1; $bill <= $pending_fine_count; $bill++) {
597         
598                                 if ($current_fine_total >= $max_fine) {
599                                         $c->update({stop_fines => 'MAXFINES', stop_fines_time => 'now'});
600                                         $client->respond(
601                                                 "\tMaximum fine level of ".$c->max_fine.
602                                                 " reached for this circulation.\n".
603                                                 "\tNo more fines will be generated.\n" );
604                                         last;
605                                 }
606                                 
607                                 my $billing_ts = DateTime->from_epoch( epoch => $last_fine + $fine_interval * $bill );
608
609                                 my $dow = $billing_ts->day_of_week_0();
610                                 my $dow_open = "dow_${dow}_open";
611                                 my $dow_close = "dow_${dow}_close";
612
613                                 if (my $h = $hoo{$c->circ_lib}) {
614                                         next if ( $h->$dow_open eq '00:00:00' and $h->$dow_close eq '00:00:00');
615                                 }
616
617                                 my $timestamptz = $billing_ts->strftime('%FT%T%z');
618                                 my @cl = actor::org_unit::closed_date->search_where(
619                                                 { close_start   => { '<=' => $timestamptz },
620                                                   close_end     => { '>=' => $timestamptz },
621                                                   org_unit      => $c->circ_lib }
622                                 );
623                                 next if (@cl);
624         
625                                 $current_fine_total += $recuring_fine;
626                                 $latest_amount += $recuring_fine;
627                                 $latest_billing_ts = $timestamptz;
628
629                                 money::billing->create(
630                                         { xact          => ''.$c->id,
631                                           note          => "System Generated Overdue Fine",
632                                           billing_type  => "Overdue materials",
633                                           amount        => sprintf('%0.2f', $recuring_fine/100),
634                                           billing_ts    => $timestamptz,
635                                         }
636                                 );
637
638                         }
639
640                         $client->respond( "\t\tAdding fines totaling $latest_amount for overdue up to $latest_billing_ts\n" )
641                                 if ($latest_billing_ts and $latest_amount);
642
643                         $self->method_lookup('open-ils.storage.transaction.commit')->run;
644
645                         $penalty->request(
646                                 'open-ils.penalty.patron_penalty.calculate',
647                                 { patron        => $c->usr->to_fieldmapper,
648                                   update        => 1,
649                                   background    => 1,
650                                 }
651                         )->gather(1);
652
653                 } catch Error with {
654                         my $e = shift;
655                         $client->respond( "Error processing overdue circulation [".$c->id."]:\n\n$e\n" );
656                         $log->error("Error processing overdue circulation [".$c->id."]:\n$e\n");
657                         $self->method_lookup('open-ils.storage.transaction.rollback')->run;
658                         throw $e ifif ($e =~ /IS NOT CONNECTED TO THE NETWORK/o);
659                 };
660         }
661 }
662 __PACKAGE__->register_method(
663         api_name        => 'open-ils.storage.action.circulation.overdue.generate_fines',
664         api_level       => 1,
665         stream          => 1,
666         method          => 'generate_fines',
667 );
668
669
670
671 sub new_hold_copy_targeter {
672         my $self = shift;
673         my $client = shift;
674         my $check_expire = shift;
675         my $one_hold = shift;
676
677         local $OpenILS::Application::Storage::WRITE = 1;
678
679         my $holds;
680
681         try {
682                 if ($one_hold) {
683                         $holds = [ action::hold_request->search_where( { id => $one_hold, fulfillment_time => undef, cancel_time => undef } ) ];
684                 } elsif ( $check_expire ) {
685
686                         # what's the retarget time threashold?
687                         my $time = time;
688                         $check_expire ||= '12h';
689                         $check_expire = interval_to_seconds( $check_expire );
690
691                         my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) = gmtime(time() - $check_expire);
692                         $year += 1900;
693                         $mon += 1;
694                         my $expire_threshold = sprintf(
695                                 '%s-%0.2d-%0.2dT%0.2d:%0.2d:%0.2d-00',
696                                 $year, $mon, $mday, $hour, $min, $sec
697                         );
698
699                         # find all the holds holds needing retargeting
700                         $holds = [ action::hold_request->search_where(
701                                                         { capture_time => undef,
702                                                           fulfillment_time => undef,
703                                                           cancel_time => undef,
704                                                           prev_check_time => { '<=' => $expire_threshold },
705                                                         },
706                                                         { order_by => 'selection_depth DESC, request_time,prev_check_time' } ) ];
707
708                         # find all the holds holds needing first time targeting
709                         push @$holds, action::hold_request->search(
710                                                         capture_time => undef,
711                                                         fulfillment_time => undef,
712                                                         prev_check_time => undef,
713                                                         cancel_time => undef,
714                                                         { order_by => 'selection_depth DESC, request_time' } );
715                 } else {
716
717                         # find all the holds holds needing first time targeting ONLY
718                         $holds = [ action::hold_request->search(
719                                                         capture_time => undef,
720                                                         fulfillment_time => undef,
721                                                         prev_check_time => undef,
722                                                         cancel_time => undef,
723                                                         { order_by => 'selection_depth DESC, request_time' } ) ];
724                 }
725         } catch Error with {
726                 my $e = shift;
727                 die "Could not retrieve uncaptured hold requests:\n\n$e\n";
728         };
729
730         my @closed = actor::org_unit::closed_date->search_where(
731                 { close_start => { '<=', 'today' },
732                   close_end => { '>=', 'today' } }
733         );
734
735
736         my @successes;
737
738         for my $hold (@$holds) {
739                 try {
740                         #first, re-fetch the hold, to make sure it's not captured already
741                         $hold = action::hold_request->retrieve( $hold->id );
742                         die "OK\n" if ($hold->capture_time);
743
744                         #start a transaction if needed
745                         if ($self->method_lookup('open-ils.storage.transaction.current')->run) {
746                                 $log->debug("Cleaning up after previous transaction\n");
747                                 $self->method_lookup('open-ils.storage.transaction.rollback')->run;
748                         }
749                         $self->method_lookup('open-ils.storage.transaction.begin')->run( $client );
750                         $log->info("Processing hold ".$hold->id."...\n");
751
752                         # remove old auto-targeting maps
753                         my @oldmaps = action::hold_copy_map->search( hold => $hold->id );
754                         $_->delete for (@oldmaps);
755
756         
757                         my $all_copies = [];
758
759                         # find filters for MR holds
760                         my ($types, $formats, $lang) = split '-', $hold->holdable_formats;
761
762                         # find all the potential copies
763                         if ($hold->hold_type eq 'M') {
764                                 for my $r ( map
765                                                 {$_->record}
766                                                 metabib::record_descriptor
767                                                         ->search(
768                                                                 record => [ map { $_->id } metabib::metarecord->retrieve($hold->target)->source_records ],
769                                                                 ( $types   ? (item_type => [split '', $types])   : () ),
770                                                                 ( $formats ? (item_form => [split '', $formats]) : () ),
771                                                                 ( $lang    ? (item_lang => $lang)                : () ),
772                                                         )
773                                 ) {
774                                         my ($rtree) = $self
775                                                 ->method_lookup( 'open-ils.storage.biblio.record_entry.ranged_tree')
776                                                 ->run( $r->id, $hold->selection_ou, $hold->selection_depth );
777
778                                         for my $cn ( @{ $rtree->call_numbers } ) {
779                                                 push @$all_copies,
780                                                         asset::copy->search( id => [map {$_->id} @{ $cn->copies }] );
781                                         }
782                                 }
783                         } elsif ($hold->hold_type eq 'T') {
784                                 my ($rtree) = $self
785                                         ->method_lookup( 'open-ils.storage.biblio.record_entry.ranged_tree')
786                                         ->run( $hold->target, $hold->selection_ou, $hold->selection_depth );
787
788                                 unless ($rtree) {
789                                         push @successes, { hold => $hold->id, eligible_copies => 0, error => 'NO_RECORD' };
790                                         die "OK\n";
791                                 }
792
793                                 for my $cn ( @{ $rtree->call_numbers } ) {
794                                         push @$all_copies,
795                                                 asset::copy->search( id => [map {$_->id} @{ $cn->copies }] );
796                                 }
797                         } elsif ($hold->hold_type eq 'V') {
798                                 my ($vtree) = $self
799                                         ->method_lookup( 'open-ils.storage.asset.call_number.ranged_tree')
800                                         ->run( $hold->target, $hold->selection_ou, $hold->selection_depth );
801
802                                 push @$all_copies,
803                                         asset::copy->search( id => [map {$_->id} @{ $vtree->copies }] );
804                                         
805                         } elsif  ($hold->hold_type eq 'C') {
806
807                                 $all_copies = [asset::copy->retrieve($hold->target)];
808                         }
809
810                         # trim unholdables
811                         @$all_copies = grep {   isTrue($_->status->holdable) && 
812                                                 isTrue($_->location->holdable) && 
813                                                 isTrue($_->holdable)
814                                         } @$all_copies;
815
816                         # let 'em know we're still working
817                         $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
818                         
819                         # if we have no copies ...
820                         if (!ref $all_copies || !@$all_copies) {
821                                 $log->info("\tNo copies available for targeting at all!\n");
822                                 push @successes, { hold => $hold->id, eligible_copies => 0, error => 'NO_COPIES' };
823
824                                 $hold->update( { prev_check_time => 'today' } );
825                                 $self->method_lookup('open-ils.storage.transaction.commit')->run;
826                                 die "OK\n";
827                         }
828
829                         my $copy_count = @$all_copies;
830
831                         # map the potentials, so that we can pick up checkins
832                         $log->debug( "\tMapping ".scalar(@$all_copies)." potential copies for hold ".$hold->id);
833                         action::hold_copy_map->create( { hold => $hold->id, target_copy => $_->id } ) for (@$all_copies);
834
835                         $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
836
837                         my @good_copies;
838                         for my $c (@$all_copies) {
839                                 # current target
840                                 next if ($c->id == $hold->current_copy);
841
842                                 # circ lib is closed
843                                 next if ( grep { ''.$_->org_unit == ''.$c->circ_lib } @closed );
844
845                                 # target of another hold
846                                 next if (action::hold_request
847                                                 ->search_where(
848                                                         { current_copy => $c->id,
849                                                           fulfillment_time => undef,
850                                                           cancel_time => undef,
851                                                         }
852                                                 )
853                                 );
854
855                                 # we passed all three, keep it
856                                 push @good_copies, $c if ($c);
857                                 $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
858                         }
859
860                         $log->debug("\t".scalar(@good_copies)." (non-current) copies available for targeting...");
861
862                         my $old_best = $hold->current_copy;
863                         $hold->update({ current_copy => undef }) if ($old_best);
864         
865                         if (!scalar(@good_copies)) {
866                                 $log->info("\tNo (non-current) copies eligible to fill the hold.");
867                                 if (
868                                   $old_best &&
869                                   grep { $old_best eq $_ } @$all_copies &&
870                                   !action::hold_request->search_where({ current_copy => $old_best->id, capture_time => undef, cancel_time => undef })
871                                 ) {
872                                         # the old copy is still available
873                                         $log->debug("\tPushing current_copy back onto the targeting list");
874                                         push @good_copies, $old_best;
875                                 } else {
876                                         # oops, old copy is not available
877                                         $log->debug("\tcurrent_copy is no longer available for targeting... NEXT HOLD, PLEASE!");
878                                         $hold->update( { prev_check_time => 'today' } );
879                                         $self->method_lookup('open-ils.storage.transaction.commit')->run;
880                                         push @successes, { hold => $hold->id, eligible_copies => 0, error => 'NO_TARGETS' };
881                                         die "OK\n";
882                                 }
883                         }
884
885                         $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
886                         my $prox_list = [];
887                         $$prox_list[0] =
888                         [
889                                 grep {
890                                         $_->circ_lib == $hold->pickup_lib
891                                 } @good_copies
892                         ];
893
894                         $all_copies = [grep {$_->circ_lib != $hold->pickup_lib } @good_copies];
895
896                         $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
897                         my $best = choose_nearest_copy($hold, $prox_list);
898                         $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
899
900                         if (!$best) {
901                                 $log->debug("\tNothing at the pickup lib, looking elsewhere among ".scalar(@$all_copies)." copies");
902                                 $prox_list = create_prox_list( $self, $hold->pickup_lib, $all_copies );
903
904                                 $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
905
906                                 $best = choose_nearest_copy($hold, $prox_list);
907                         }
908
909                         $client->status( new OpenSRF::DomainObject::oilsContinueStatus );
910                         if ($old_best) {
911                                 # hold wasn't fulfilled, record the fact
912                         
913                                 $log->info("\tHold was not (but should have been) fulfilled by ".$old_best->id);
914                                 action::unfulfilled_hold_list->create(
915                                                 { hold => ''.$hold->id,
916                                                   current_copy => ''.$old_best->id,
917                                                   circ_lib => ''.$old_best->circ_lib,
918                                                 });
919                         }
920
921                         if ($best) {
922                                 $hold->update( { current_copy => ''.$best->id, prev_check_time => 'now' } );
923                                 $log->debug("\tUpdating hold [".$hold->id."] with new 'current_copy' [".$best->id."] for hold fulfillment.");
924                         } else {
925                                 $hold->update( { prev_check_time => 'now' } );
926                                 $log->info( "\tThere were no targetable copies for the hold" );
927                         }
928
929                         $self->method_lookup('open-ils.storage.transaction.commit')->run;
930                         $log->info("\tProcessing of hold ".$hold->id." complete.");
931
932                         push @successes,
933                                 { hold => $hold->id,
934                                   old_target => ($old_best ? $old_best->id : undef),
935                                   eligible_copies => $copy_count,
936                                   target => ($best ? $best->id : undef) };
937
938                 } otherwise {
939                         my $e = shift;
940                         if ($e !~ /^OK/o) {
941                                 $log->error("Processing of hold failed:  $e");
942                                 $self->method_lookup('open-ils.storage.transaction.rollback')->run;
943                                 throw $e if ($e =~ /IS NOT CONNECTED TO THE NETWORK/o);
944                         }
945                 };
946         }
947
948         return \@successes;
949 }
950 __PACKAGE__->register_method(
951         api_name        => 'open-ils.storage.action.hold_request.copy_targeter',
952         api_level       => 1,
953         method          => 'new_hold_copy_targeter',
954 );
955
956 my $locations;
957 my $statuses;
958 my %cache = (titles => {}, cns => {});
959 sub hold_copy_targeter {
960         my $self = shift;
961         my $client = shift;
962         my $check_expire = shift;
963         my $one_hold = shift;
964
965         $self->{user_filter} = OpenSRF::AppSession->create('open-ils.circ');
966         $self->{user_filter}->connect;
967         $self->{client} = $client;
968
969         my $time = time;
970         $check_expire ||= '12h';
971         $check_expire = interval_to_seconds( $check_expire );
972
973         my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) = gmtime(time() - $check_expire);
974         $year += 1900;
975         $mon += 1;
976         my $expire_threshold = sprintf(
977                 '%s-%0.2d-%0.2dT%0.2d:%0.2d:%0.2d-00',
978                 $year, $mon, $mday, $hour, $min, $sec
979         );
980
981
982         $statuses ||= [ config::copy_status->search(holdable => 't') ];
983
984         $locations ||= [ asset::copy_location->search(holdable => 't') ];
985
986         my $holds;
987
988         %cache = (titles => {}, cns => {});
989
990         try {
991                 if ($one_hold) {
992                         $holds = [ action::hold_request->search(id => $one_hold) ];
993                 } else {
994                         $holds = [ action::hold_request->search_where(
995                                                         { capture_time => undef,
996                                                           prev_check_time => { '<=' => $expire_threshold },
997                                                         },
998                                                         { order_by => 'request_time,prev_check_time' } ) ];
999                         push @$holds, action::hold_request->search_where(
1000                                                         { capture_time => undef,
1001                                                           prev_check_time => undef,
1002                                                         },
1003                                                         { order_by => 'request_time' } );
1004                 }
1005         } catch Error with {
1006                 my $e = shift;
1007                 die "Could not retrieve uncaptured hold requests:\n\n$e\n";
1008         };
1009
1010         for my $hold (@$holds) {
1011                 try {
1012                         #action::hold_request->db_Main->begin_work;
1013                         if ($self->method_lookup('open-ils.storage.transaction.current')->run) {
1014                                 $client->respond("Cleaning up after previous transaction\n");
1015                                 $self->method_lookup('open-ils.storage.transaction.rollback')->run;
1016                         }
1017                         $self->method_lookup('open-ils.storage.transaction.begin')->run( $client );
1018                         $client->respond("Processing hold ".$hold->id."...\n");
1019
1020                         my $copies;
1021
1022                         $copies = $self->metarecord_hold_capture($hold) if ($hold->hold_type eq 'M');
1023                         $self->{client}->status( new OpenSRF::DomainObject::oilsContinueStatus );
1024
1025                         $copies = $self->title_hold_capture($hold) if ($hold->hold_type eq 'T');
1026                         $self->{client}->status( new OpenSRF::DomainObject::oilsContinueStatus );
1027                         
1028                         $copies = $self->volume_hold_capture($hold) if ($hold->hold_type eq 'V');
1029                         $self->{client}->status( new OpenSRF::DomainObject::oilsContinueStatus );
1030                         
1031                         $copies = $self->copy_hold_capture($hold) if ($hold->hold_type eq 'C');
1032
1033                         unless (ref $copies || !@$copies) {
1034                                 $client->respond("\tNo copies available for targeting at all!\n");
1035                         }
1036
1037                         my @good_copies;
1038                         for my $c (@$copies) {
1039                                 next if ( grep {$c->id == $hold->current_copy} @good_copies);
1040                                 push @good_copies, $c if ($c);
1041                         }
1042
1043                         $client->respond("\t".scalar(@good_copies)." (non-current) copies available for targeting...\n");
1044
1045                         my $old_best = $hold->current_copy;
1046                         $hold->update({ current_copy => undef });
1047         
1048                         if (!scalar(@good_copies)) {
1049                                 $client->respond("\tNo (non-current) copies available to fill the hold.\n");
1050                                 if ( $old_best && grep {$c->id == $hold->current_copy} @$copies ) {
1051                                         $client->respond("\tPushing current_copy back onto the targeting list\n");
1052                                         push @good_copies, asset::copy->retrieve( $old_best );
1053                                 } else {
1054                                         $client->respond("\tcurrent_copy is no longer available for targeting... NEXT HOLD, PLEASE!\n");
1055                                         next;
1056                                 }
1057                         }
1058
1059                         my $prox_list;
1060                         $$prox_list[0] = [grep {$_->circ_lib == $hold->pickup_lib } @good_copies];
1061                         $copies = [grep {$_->circ_lib != $hold->pickup_lib } @good_copies];
1062
1063                         my $best = choose_nearest_copy($hold, $prox_list);
1064
1065                         if (!$best) {
1066                                 $prox_list = create_prox_list( $self, $hold->pickup_lib, $copies );
1067                                 $best = choose_nearest_copy($hold, $prox_list);
1068                         }
1069
1070                         if ($old_best) {
1071                                 # hold wasn't fulfilled, record the fact
1072                         
1073                                 $client->respond("\tHold was not (but should have been) fulfilled by ".$old_best->id.".\n");
1074                                 action::unfulfilled_hold_list->create(
1075                                                 { hold => ''.$hold->id,
1076                                                   current_copy => ''.$old_best->id,
1077                                                   circ_lib => ''.$old_best->circ_lib,
1078                                                 });
1079                         }
1080
1081                         if ($best) {
1082                                 $hold->update( { current_copy => ''.$best->id } );
1083                                 $client->respond("\tTargeting copy ".$best->id." for hold fulfillment.\n");
1084                         }
1085
1086                         $hold->update( { prev_check_time => 'now' } );
1087                         $client->respond("\tUpdating hold ".$hold->id." with new 'current_copy' for hold fulfillment.\n");
1088
1089                         $client->respond("\tProcessing of hold ".$hold->id." complete.\n");
1090                         $self->method_lookup('open-ils.storage.transaction.commit')->run;
1091
1092                         #action::hold_request->dbi_commit;
1093
1094                 } otherwise {
1095                         my $e = shift;
1096                         $log->error("Processing of hold failed:  $e");
1097                         $client->respond("\tProcessing of hold failed!.\n\t\t$e\n");
1098                         $self->method_lookup('open-ils.storage.transaction.rollback')->run;
1099                         #action::hold_request->dbi_rollback;
1100                 };
1101         }
1102
1103         $self->{user_filter}->disconnect;
1104         $self->{user_filter}->finish;
1105         delete $$self{user_filter};
1106         return undef;
1107 }
1108 __PACKAGE__->register_method(
1109         api_name        => 'open-ils.storage.action.hold_request.copy_targeter',
1110         api_level       => 0,
1111         stream          => 1,
1112         method          => 'hold_copy_targeter',
1113 );
1114
1115
1116 sub copy_hold_capture {
1117         my $self = shift;
1118         my $hold = shift;
1119         my $cps = shift;
1120
1121         if (!defined($cps)) {
1122                 try {
1123                         $cps = [ asset::copy->search( id => $hold->target ) ];
1124                 } catch Error with {
1125                         my $e = shift;
1126                         die "Could not retrieve initial volume list:\n\n$e\n";
1127                 };
1128         }
1129
1130         my @copies = grep { $_->holdable } @$cps;
1131
1132         for (my $i = 0; $i < @$cps; $i++) {
1133                 next unless $$cps[$i];
1134                 
1135                 my $cn = $cache{cns}{$copies[$i]->call_number};
1136                 my $rec = $cache{titles}{$cn->record};
1137                 $copies[$i] = undef if ($copies[$i] && !grep{ $copies[$i]->status eq $_->id}@$statuses);
1138                 $copies[$i] = undef if ($copies[$i] && !grep{ $copies[$i]->location eq $_->id}@$locations);
1139                 $copies[$i] = undef if (
1140                         !$copies[$i] ||
1141                         !$self->{user_filter}->request(
1142                                 'open-ils.circ.permit_hold',
1143                                 $hold->to_fieldmapper, do {
1144                                         my $cp_fm = $copies[$i]->to_fieldmapper;
1145                                         $cp_fm->circ_lib( $copies[$i]->circ_lib->to_fieldmapper );
1146                                         $cp_fm->location( $copies[$i]->location->to_fieldmapper );
1147                                         $cp_fm->status( $copies[$i]->status->to_fieldmapper );
1148                                         $cp_fm;
1149                                 },
1150                                 { title => $rec->to_fieldmapper,
1151                                   usr => actor::user->retrieve($hold->usr)->to_fieldmapper,
1152                                   requestor => actor::user->retrieve($hold->requestor)->to_fieldmapper,
1153                                 })->gather(1)
1154                 );
1155                 $self->{client}->status( new OpenSRF::DomainObject::oilsContinueStatus );
1156         }
1157
1158         @copies = grep { $_ } @copies;
1159
1160         my $count = @copies;
1161
1162         return unless ($count);
1163         
1164         action::hold_copy_map->search( hold => $hold->id )->delete_all;
1165         
1166         my @maps;
1167         $self->{client}->respond( "\tMapping ".scalar(@copies)." eligable copies for hold ".$hold->id."\n");
1168         for my $c (@copies) {
1169                 push @maps, action::hold_copy_map->create( { hold => $hold->id, target_copy => $c->id } );
1170         }
1171         $self->{client}->respond( "\tA total of ".scalar(@maps)." mapping were created for hold ".$hold->id."\n");
1172
1173         return \@copies;
1174 }
1175
1176
1177 sub choose_nearest_copy {
1178         my $hold = shift;
1179         my $prox_list = shift;
1180
1181         for my $p ( 0 .. int( scalar(@$prox_list) - 1) ) {
1182                 next unless (ref $$prox_list[$p]);
1183
1184                 my @capturable = grep { $_->status == 0 || $_->status == 7 } @{ $$prox_list[$p] };
1185                 next unless (@capturable);
1186
1187                 my $rand = int(rand(scalar(@capturable)));
1188                 while (my ($c) = splice(@capturable,$rand)) {
1189                         unless ( OpenILS::Utils::PermitHold::permit_copy_hold(
1190                                 { title => $c->call_number->record->to_fieldmapper,
1191                                   title_descriptor => $c->call_number->record->record_descriptor->next->to_fieldmapper,
1192                                   patron => $hold->usr->to_fieldmapper,
1193                                   copy => $c->to_fieldmapper,
1194                                   requestor => $hold->requestor->to_fieldmapper,
1195                                   request_lib => $hold->request_lib->to_fieldmapper,
1196                                 }
1197                         )) {
1198                                 last unless(@capturable);
1199                                 $rand = int(rand(scalar(@capturable)));
1200                                 next;
1201                         }
1202                         return $c;
1203                 }
1204         }
1205 }
1206
1207 sub create_prox_list {
1208         my $self = shift;
1209         my $lib = shift;
1210         my $copies = shift;
1211
1212         my @prox_list;
1213         for my $cp (@$copies) {
1214                 my ($prox) = $self->method_lookup('open-ils.storage.asset.copy.proximity')->run( $cp, $lib );
1215                 next unless (defined($prox));
1216                 $prox_list[$prox] = [] unless defined($prox_list[$prox]);
1217                 push @{$prox_list[$prox]}, $cp;
1218         }
1219         return \@prox_list;
1220 }
1221
1222 sub volume_hold_capture {
1223         my $self = shift;
1224         my $hold = shift;
1225         my $vols = shift;
1226
1227         if (!defined($vols)) {
1228                 try {
1229                         $vols = [ asset::call_number->search( id => $hold->target ) ];
1230                         $cache{cns}{$_->id} = $_ for (@$vols);
1231                 } catch Error with {
1232                         my $e = shift;
1233                         die "Could not retrieve initial volume list:\n\n$e\n";
1234                 };
1235         }
1236
1237         my @v_ids = map { $_->id } @$vols;
1238
1239         my $cp_list;
1240         try {
1241                 $cp_list = [ asset::copy->search( call_number => \@v_ids ) ];
1242         
1243         } catch Error with {
1244                 my $e = shift;
1245                 warn "Could not retrieve copy list:\n\n$e\n";
1246         };
1247
1248         $self->copy_hold_capture($hold,$cp_list) if (ref $cp_list and @$cp_list);
1249 }
1250
1251 sub title_hold_capture {
1252         my $self = shift;
1253         my $hold = shift;
1254         my $titles = shift;
1255
1256         if (!defined($titles)) {
1257                 try {
1258                         $titles = [ biblio::record_entry->search( id => $hold->target ) ];
1259                         $cache{titles}{$_->id} = $_ for (@$titles);
1260                 } catch Error with {
1261                         my $e = shift;
1262                         die "Could not retrieve initial title list:\n\n$e\n";
1263                 };
1264         }
1265
1266         my @t_ids = map { $_->id } @$titles;
1267         my $cn_list;
1268         try {
1269                 ($cn_list) = $self->method_lookup('open-ils.storage.direct.asset.call_number.search.record.atomic')->run( \@t_ids );
1270         
1271         } catch Error with {
1272                 my $e = shift;
1273                 warn "Could not retrieve volume list:\n\n$e\n";
1274         };
1275
1276         $cache{cns}{$_->id} = $_ for (@$cn_list);
1277
1278         $self->volume_hold_capture($hold,$cn_list) if (ref $cn_list and @$cn_list);
1279 }
1280
1281 sub metarecord_hold_capture {
1282         my $self = shift;
1283         my $hold = shift;
1284
1285         my $titles;
1286         try {
1287                 $titles = [ metabib::metarecord_source_map->search( metarecord => $hold->target) ];
1288         
1289         } catch Error with {
1290                 my $e = shift;
1291                 die "Could not retrieve initial title list:\n\n$e\n";
1292         };
1293
1294         try {
1295                 my @recs = map {$_->record} metabib::record_descriptor->search( record => $titles, item_type => [split '', $hold->holdable_formats] ); 
1296
1297                 $titles = [ biblio::record_entry->search( id => \@recs ) ];
1298         
1299         } catch Error with {
1300                 my $e = shift;
1301                 die "Could not retrieve format-pruned title list:\n\n$e\n";
1302         };
1303
1304
1305         $cache{titles}{$_->id} = $_ for (@$titles);
1306         $self->title_hold_capture($hold,$titles) if (ref $titles and @$titles);
1307 }
1308
1309 1;