Make loan type display configurable.

vufind-org · Jan 15, 2025 · 0284356 · 0284356
1 parent 7dfa676
commit 0284356
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 2 deletions.
diff --git a/config/vufind/config.ini b/config/vufind/config.ini
@@ -459,6 +459,9 @@ title_level_holds_mode = "disabled"
 ; can extend/override the VuFind\View\Helper\Root\Holdings::holdingIsVisible method.
 ;display_items_without_barcodes = true
 
+; Whether to display the loan type on items in the Holdings display.
+;display_loan_type_in_holdings = true
+
 ; This section controls features related to user accounts
 [Account]
 ; Allow the user to set a home library through the Profile screen, which will

diff --git a/module/VuFind/src/VuFind/View/Helper/Root/Config.php b/module/VuFind/src/VuFind/View/Helper/Root/Config.php
@@ -194,4 +194,15 @@ public function dateTimeFormat($separator = ' '): string
     {
         return $this->dateFormat() . $separator . $this->timeFormat();
     }
+
+    /**
+     * Check if the loan type should be displayed in holdings
+     *
+     * @return bool
+     */
+    public function displayLoanType(): bool
+    {
+        return (bool)($this->get('config')->Catalog
+            ->display_loan_type_in_holdings ?? false);
+    }
 }
diff --git a/themes/bootstrap3/templates/RecordTab/holdingsils/standard.phtml b/themes/bootstrap3/templates/RecordTab/holdingsils/standard.phtml
@@ -30,7 +30,7 @@
       </span>
       <?php if ($availabilityStatus->isAvailable()): ?>
         <?php /* Begin Available Items (Holds) */ ?>
-        <?php if (!empty($holding['loan_type_name'])): ?>
+        <?php if ($this->config()->displayLoanType() ?? false && !empty($holding['loan_type_name'])): ?>
           <span class="loan-type"><?=$this->transEsc('Loan Type')?>: <?=$holding['loan_type_name']?>&nbsp;</span>
         <?php endif; ?>
         <?php if ($holding['link'] ?? false): ?>

diff --git a/themes/bootstrap5/templates/RecordTab/holdingsils/standard.phtml b/themes/bootstrap5/templates/RecordTab/holdingsils/standard.phtml
@@ -30,7 +30,7 @@
       </span>
       <?php if ($availabilityStatus->isAvailable()): ?>
         <?php /* Begin Available Items (Holds) */ ?>
-        <?php if (!empty($holding['loan_type_name'])): ?>
+        <?php if ($this->config()->displayLoanType() ?? false && !empty($holding['loan_type_name'])): ?>
           <span class="loan-type"><?=$this->transEsc('Loan Type')?>: <?=$holding['loan_type_name']?>&nbsp;</span>
         <?php endif; ?>
         <?php if ($holding['link'] ?? false): ?>