LP#963341: Unrelated YAOUS breaks MFHD management
authorMike Rylander <mrylander@gmail.com>
Mon, 7 Dec 2015 16:57:11 +0000 (11:57 -0500)
committerGalen Charlton <gmc@esilibrary.com>
Mon, 14 Mar 2016 21:47:38 +0000 (17:47 -0400)
Part of LP bug 963341 is about how the setting opac.fully_compressed_serial_holdings
breaks staff's ability to manage MFHD attached to serials records.  This
commit unbreaks that by fetching the relevant MFHD regardless of that setting
but should not otherwise change user-visible behavior.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Record.pm

index 418f884..5f3d51c 100644 (file)
@@ -150,6 +150,9 @@ sub load_record {
         return Apache2::Const::HTTP_NOT_FOUND;
     }
 
+    $ctx->{mfhd_summaries} =
+        $self->get_mfhd_summaries($rec_id, $org, $copy_depth);
+
     if (
         $ctx->{get_org_setting}->
             ($org, "opac.fully_compressed_serial_holdings")
@@ -158,9 +161,6 @@ sub load_record {
         # *are* going to display something in the "issues" expandy?
         $self->load_serial_holding_summaries($rec_id, $org, $copy_depth);
     } else {
-        $ctx->{mfhd_summaries} =
-            $self->get_mfhd_summaries($rec_id, $org, $copy_depth);
-
         if ($ctx->{mfhd_summaries} && scalar(@{$ctx->{mfhd_summaries}})
         ) {
             $ctx->{have_mfhd_to_show} = 1;