entities: public view for local and remote

* Changes english date format to en_GB (DD/MM/YYYY). * Fixes `reasons_not_to_delete` and `get_links_to_me` methods for `Entity` resource. * Adds `alternative_names` as alias for autocomplete search. Co-Authored-by: Bertrand Zuchuat <[email protected]> Co-Authored-by: Renaud Michotte <[email protected]>
Garfield-fr · Aug 21, 2023 · 7aa2d64 · 7aa2d64
1 parent fc56524
commit 7aa2d64
Show file tree

Hide file tree

Showing 39 changed files with 1,012 additions and 635 deletions.
diff --git a/pyproject.toml b/pyproject.toml
@@ -186,7 +186,7 @@ rero-ils = "rero_ils.modules.ext:REROILSAPP"
 [tool.poetry.plugins."invenio_base.blueprints"]
 circ_policies = "rero_ils.modules.circ_policies.views:blueprint"
 collections = "rero_ils.modules.collections.views:blueprint"
-remote_entities = "rero_ils.modules.entities.remote_entities.views:blueprint"
+entities = "rero_ils.modules.entities.views:blueprint"
 documents = "rero_ils.modules.documents.views:blueprint"
 holdings = "rero_ils.modules.holdings.views:blueprint"
 ill_requests = "rero_ils.modules.ill_requests.views:blueprint"

diff --git a/rero_ils/config.py b/rero_ils/config.py
@@ -3060,6 +3060,7 @@ def _(x):
 
 #: Entities
 RERO_ILS_AGENTS_SOURCES = ['idref', 'gnd', 'rero']
+RERO_ILS_AGENTS_SOURCES_EXCLUDE_LINK = ['rero']
 RERO_ILS_AGENTS_LABEL_ORDER = {
     'fallback': 'fr',
     'fr': ['idref', 'rero', 'gnd'],

diff --git a/rero_ils/filter.py b/rero_ils/filter.py
@@ -25,6 +25,7 @@
 import dateparser
 from babel.dates import format_date, format_datetime, format_time
 from flask import current_app, render_template
+from flask_babelex import gettext as _
 from invenio_i18n.ext import current_i18n
 from jinja2 import TemplateNotFound
 from markupsafe import Markup
@@ -102,6 +103,10 @@ def format_date_filter(
     if not locale:
         locale = current_i18n.locale.language
 
+    # Date formatting in GB English (DD/MM/YYYY)
+    if locale == 'en':
+        locale += '_GB'
+
     if timezone:
         tzinfo = timezone
     else:
@@ -184,3 +189,19 @@ def message_filter(key):
     :return: none or a json (check structure into the class Message).
     """
     return Message.get(key)
+
+
+def translate(data, prefix='', separator=', '):
+    """Translate data.
+
+    :param data: the data to translate
+    :param prefix: A prefix as a character string
+    :param separator: A character string separator.
+    :return: The translated string
+    """
+    if data:
+        if isinstance(data, list):
+            translated = [_(f'{prefix}{item}') for item in data]
+            return separator.join(translated)
+        elif isinstance(data, str):
+            return _(f'{prefix}{data}')
diff --git a/rero_ils/modules/documents/api.py b/rero_ils/modules/documents/api.py
@@ -73,6 +73,30 @@ class Meta:
 
         default_filter = None
 
+    def by_entity(self, entity, subjects=True, imported_subjects=True,
+                  genre_forms=True):
+        """Build a search to get hits related to an entity.
+
+        :param entity: the entity record to search.
+        :param subjects: search on `subject` field.
+        :param imported_subjects: search on `imported_subject` field.
+        :param genre_forms: search on `genre_forms` field.
+        :returns: An ElasticSearch query to get hits related the entity.
+        :rtype: `elasticsearch_dsl.Search`
+        """
+        field = f'contribution.entity.pids.{entity.resource_type}'
+        filters = Q('term', **{field: entity.pid})
+        if subjects:
+            field = f'subjects.entity.pids.{entity.resource_type}'
+            filters |= Q('term', **{field: entity.pid})
+        if imported_subjects:
+            field = f'subjects_imported.pids.{entity.resource_type}'
+            filters |= Q('term', **{field: entity.pid})
+        if genre_forms:
+            field = f'genreForm.entity.pids.{entity.resource_type}'
+            filters |= Q('term', **{field: entity.pid})
+        return self.filter(filters)
+
 
 class Document(IlsRecord):
     """Document class."""

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
@@ -2,6 +2,7 @@
 
   RERO ILS
   Copyright (C) 2019-2023 RERO
+  Copyright (C) 2019-2023 UCLouvain
 
   This program is free software: you can redistribute it and/or modify
   it under the terms of the GNU Affero General Public License as published by
@@ -62,14 +63,24 @@
       </ul>
       {% if 'genreForm' in record %}
         <ul class="list-unstyled mb-0">
-           {% for genreForm in record.genreForm %}
-             {% if 'entity' in genreForm %}
+          {% for genreForm in record.genreForm %}
+            {% if 'entity' in genreForm %}
+              {% set type, value, label = genreForm.entity | doc_entity_label(language=current_i18n.language) %}
+              {% if 'textual' == type %}
+                {% set query = 'genreForm.entity.authorized_access_point_' ~ current_i18n.language ~ ':"' ~ value ~ '"' %}
+              {% elif type in ['local', 'remote'] %}
+                {% set query = 'genreForm.entity.pids.' ~ type ~ ':' ~ value %}
+              {% endif %}
                <li class="figure-caption">
-                 <i class="fa fa-tag mr-1"></i>
-                 {{ genreForm.entity | doc_entity_label(language=current_i18n.language) }}
+                  <i class="fa fa-tag mr-1"></i>
+                  {% if query %}
+                  <a href="{{ url_for('rero_ils.search', viewcode=viewcode, recordType='documents', q=query, simple=0) }}">{{ label }}</a>
+                  {% else %}
+                    {{ label }}
+                  {% endif %}
                </li>
-             {% endif %}
-           {% endfor %}
+            {% endif %}
+          {% endfor %}
         </ul>
       {% endif %}
     </figcaption>
@@ -230,7 +241,18 @@ <h3>
       {% for subject in record.subjects %}
         <span class="badge badge-secondary my-0" title="{{ _(subject.type) }}">
           {% if 'entity' in subject %}
-            <i class="fa fa-tag"></i> {{ subject.entity | doc_entity_label(language=current_i18n.language) }}
+            {% set type, value, label = subject.entity | doc_entity_label(language=current_i18n.language) %}
+            {% if 'textual' == type %}
+              {% set query = 'subjects.entity.authorized_access_point_' ~ current_i18n.language ~ ':"' ~ value ~ '"' %}
+            {% elif type in ['local', 'remote'] %}
+              {% set query = 'subjects.entity.pids.' ~ type ~ ':' ~ value %}
+            {% endif %}
+            <i class="fa fa-tag"></i>
+            {% if query %}
+              <a class="text-white" href="{{ url_for('rero_ils.search', viewcode=viewcode, recordType='documents', q=query, simple=0) }}">{{ label }}</a>
+            {% else %}
+              {{ label }}
+            {% endif %}
           {% endif %}
         </span>
       {% endfor %}

diff --git a/rero_ils/modules/documents/views.py b/rero_ils/modules/documents/views.py
@@ -24,23 +24,17 @@
 
 import click
 from elasticsearch_dsl.query import Q
-from flask import Blueprint, abort, current_app, jsonify, render_template, \
-    url_for
+from flask import Blueprint, abort, current_app, jsonify, render_template
 from flask import request as flask_request
+from flask import url_for
 from flask_babelex import gettext as _
 from flask_login import current_user
 from invenio_records_ui.signals import record_viewed
 
-from .api import Document, DocumentsSearch
-from .extensions import EditionStatementExtension, \
-    ProvisionActivitiesExtension, SeriesStatementExtension, TitleExtension
-from .utils import display_alternate_graphic_first, get_remote_cover, \
-    title_format_text, title_format_text_alternate_graphic, \
-    title_variant_format_text
 from rero_ils.modules.collections.api import CollectionsSearch
-from rero_ils.modules.entities.remote_entities.api import RemoteEntity
-from rero_ils.modules.entities.models import EntityType
+from rero_ils.modules.entities.api import Entity
 from rero_ils.modules.entities.helpers import get_entity_record_from_data
+from rero_ils.modules.entities.models import EntityType
 from rero_ils.modules.holdings.models import HoldingNoteTypes
 from rero_ils.modules.items.models import ItemCirculationAction
 from rero_ils.modules.libraries.api import Library
@@ -49,6 +43,13 @@
 from rero_ils.modules.patrons.api import current_patrons
 from rero_ils.modules.utils import cached, extracted_data_from_ref
 
+from .api import Document, DocumentsSearch
+from .extensions import EditionStatementExtension, \
+    ProvisionActivitiesExtension, SeriesStatementExtension, TitleExtension
+from .utils import display_alternate_graphic_first, get_remote_cover, \
+    title_format_text, title_format_text_alternate_graphic, \
+    title_variant_format_text
+
 
 def doc_item_view_method(pid, record, template=None, **kwargs):
     """Display default view.
@@ -274,21 +275,27 @@ def contribution_format(contributions, language, viewcode, with_roles=False):
                      f'{entity.pid}',
                 'simple': 0
             }
-            url = url_for('rero_ils.search', **args)
-            label = f'<a href="{url}">{text}</a>'
         else:
             default_key = 'authorized_access_point'
-            localized_key = f'{default_key}_{language}'
-            label = contrib['entity'].get(localized_key) or \
+            localized_key = f'authorized_access_point_{language}'
+            text = contrib['entity'].get(localized_key) or \
                 contrib['entity'].get(default_key)
+            args = {
+                'viewcode': viewcode,
+                'recordType': 'documents',
+                'q': f'contribution.entity.{localized_key}:"{text}"',
+                'simple': 0
+            }
+        url = url_for('rero_ils.search', **args)
+        label = f'<a href="{url}">{text}</a>'
 
         if with_roles:
             if roles := [_(role) for role in contrib.get('role', [])]:
                 roles_str = ', '.join(roles)
                 label += f'<span class="text-secondary"> ({roles_str})</span>'
         output.append(label)
 
-    return '&#8239;; '.join(output)
+    return ' ; '.join(output)
 
 
 @blueprint.app_template_filter()
@@ -301,20 +308,28 @@ def doc_entity_label(entity, language=None, part_separator=' - ') -> str:
     :returns: the best possible label to display.
     """
     parts = []
-    if 'pid' in entity:
-        entity = RemoteEntity.get_record_by_pid(entity['pid'])
-        parts.append(entity.get_authorized_access_point(language=language))
+    if '$ref' in entity:
+        # Local or remote entity
+        if entity := Entity.get_record_by_ref(entity['$ref']):
+            entity_type = entity.resource_type
+            value = entity.pid
+            parts.append(entity.get_authorized_access_point(language=language))
     else:
+        # Textual entity
+        entity_type = 'textual'
         default_key = 'authorized_access_point'
         localized_key = f'{default_key}_{language}'
-        parts.append(entity.get(localized_key) or entity.get(default_key))
+        value = entity.get(localized_key) or entity.get(default_key)
+        parts.append(value)
 
+    # Subdivisions (only for textual entity)
     for subdivision in entity.get('subdivisions', []):
         if sub_entity := subdivision.get('entity'):
-            parts.append(
-                doc_entity_label(sub_entity, language, part_separator))
+            _, _, label = doc_entity_label(
+                sub_entity, language, part_separator)
+            parts.append(label)
 
-    return part_separator.join(filter(None, parts))
+    return entity_type, value, part_separator.join(filter(None, parts))
 
 
 @blueprint.app_template_filter()