[open-ils-commits] [GIT] Evergreen ILS branch rel_2_1 updated. 59d9d5ba24f013c4ea88866cf5fc383e42a15690

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_1 has been updated
       via  59d9d5ba24f013c4ea88866cf5fc383e42a15690 (commit)
      from  0559e324b511ec570feefbc03619eb1b7d121de3 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 59d9d5ba24f013c4ea88866cf5fc383e42a15690
Author: Jason Etheridge <jason at esilibrary.com>
Date:   Mon Jun 6 12:46:21 2011 -0400

    fix View Holds by preventing an empty IN list with cstore, and stubbing out part_holds
    
    Signed-off-by: Jason Etheridge <jason at esilibrary.com>
    Signed-off-by: Jason Stephenson <jstephenson at mvlc.org>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
index 85d8486..e31da8a 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
@@ -2750,7 +2750,7 @@ sub all_rec_holds {
     $args->{fulfillment_time} = undef; #  we don't want to see old fulfilled holds
 	$args->{cancel_time} = undef;
 
-	my $resp = { volume_holds => [], copy_holds => [], metarecord_holds => [], issuance_holds => [] };
+	my $resp = { volume_holds => [], copy_holds => [], metarecord_holds => [], part_holds => [], issuance_holds => [] };
 
     my $mr_map = $e->search_metabib_metarecord_source_map({source => $title_id})->[0];
     if($mr_map) {
@@ -2774,12 +2774,14 @@ sub all_rec_holds {
             record => $title_id
         }, {idlist=>1} );
 
-    $resp->{part_holds} = $e->search_action_hold_request(
-        {
-			hold_type => OILS_HOLD_TYPE_MONOPART,
-			target => $parts,
-			%$args
-        }, {idlist=>1} );
+    if (@$parts) {
+        $resp->{part_holds} = $e->search_action_hold_request(
+            {
+                hold_type => OILS_HOLD_TYPE_MONOPART,
+                target => $parts,
+                %$args
+            }, {idlist=>1} );
+    }
 
     my $subs = $e->search_serial_subscription(
         { record_entry => $title_id }, {idlist=>1});

-----------------------------------------------------------------------

Summary of changes:
 .../perlmods/lib/OpenILS/Application/Circ/Holds.pm |   16 +++++++++-------
 1 files changed, 9 insertions(+), 7 deletions(-)


hooks/post-receive
-- 
Evergreen ILS