From 916d8b24ac5522f61714352f6ab70b59dc31847a Mon Sep 17 00:00:00 2001
From: Pascal Repond <pascal.repond@rero.ch>
Date: Thu, 29 Aug 2024 15:40:34 +0200
Subject: [PATCH] fix(public document view): allow harvested documents with
 physical items

* Closes #3738.

Co-Authored-by: Pascal Repond <pascal.repond@rero.ch>
---
 .../templates/rero_ils/detailed_view_documents.html    |  2 +-
 rero_ils/modules/documents/views.py                    | 10 ++++++++++
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/rero_ils/modules/documents/templates/rero_ils/detailed_view_documents.html b/rero_ils/modules/documents/templates/rero_ils/detailed_view_documents.html
index 42db633257..b8c0f3b8a7 100644
--- a/rero_ils/modules/documents/templates/rero_ils/detailed_view_documents.html
+++ b/rero_ils/modules/documents/templates/rero_ils/detailed_view_documents.html
@@ -363,7 +363,7 @@ <h3>
         {% if holdings_count > 0 %}
         {% include('rero_ils/_anonymous_button.html') %}
         {% endif %}
-        {% if record.harvested %}
+        {% if record.harvested and record|n_electronic_holdings %}
           {% include('rero_ils/_document_online.html') %}
         {% else %}
           <public-search-holdings
diff --git a/rero_ils/modules/documents/views.py b/rero_ils/modules/documents/views.py
index 52a552c079..bf615853ea 100644
--- a/rero_ils/modules/documents/views.py
+++ b/rero_ils/modules/documents/views.py
@@ -722,6 +722,16 @@ def get_articles(record):
     ]
 
 
+@blueprint.app_template_filter()
+def n_electronic_holdings(record):
+    """Get the number of electronic holdings.
+
+    :return: integer of number of electronic holdings
+    """
+
+    return Document.get_n_available_holdings(record.pid)[1]
+
+
 @blueprint.app_template_filter()
 def online_holdings(document_pid, viewcode="global"):
     """Find holdings by document pid and viewcode.