[open-ils-commits] [GIT] Evergreen ILS branch master updated. 989579ab508ede9747b9e60a42c103b77f8eb016

[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, master has been updated
       via  989579ab508ede9747b9e60a42c103b77f8eb016 (commit)
      from  ea27fde5470d45bdbd73bf4ff8b0af2dfa0d8516 (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 989579ab508ede9747b9e60a42c103b77f8eb016
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 ddfd0ad..1be6838 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