invenio-app-ils 1.0.0a54__py2.py3-none-any.whl → 6.1.0__py2.py3-none-any.whl

invenio_app_ils/circulation/loaders/schemas/json/loan_request.py

@@ -21,6 +21,8 @@ from marshmallow import (
     validates_schema,
 )
 
+from invenio_app_ils.proxies import current_app_ils
+
 from .base import LoanBaseSchemaV1
 
 
@@ -31,14 +33,54 @@ def request_start_date_default():
 
 def request_expire_date_default():
     """Set default value for request_expire_date field."""
-    duration_days = current_app.config[
-        "ILS_CIRCULATION_LOAN_REQUEST_DURATION_DAYS"
-    ]
+    duration_days = current_app.config["ILS_CIRCULATION_LOAN_REQUEST_DURATION_DAYS"]
     duration = timedelta(days=duration_days)
     now = arrow.get().utcnow()
     return (now + duration).date().isoformat()
 
 
+def get_offset_duration(request_start_date, location_pid, working_days_to_offset):
+    location = current_app_ils.location_record_cls.get_record_by_pid(location_pid)
+
+    weekdays = location["opening_weekdays"]
+    exceptions = location["opening_exceptions"]
+    disabled = set()
+
+    date = request_start_date
+    max_date = request_start_date + timedelta(
+        days=current_app.config["ILS_CIRCULATION_LOAN_REQUEST_DURATION_DAYS"]
+    )
+    # Calculate all disabled days that need to be skipped
+    # Used in calculating non-closure days to offset in post-load checks
+    while date <= max_date:
+        date_iso = date.date().isoformat()
+        is_open = weekdays[date.weekday()]["is_open"]
+
+        for exception in exceptions:
+            start_date = arrow.get(exception["start_date"])
+            end_date = arrow.get(exception["end_date"])
+
+            if start_date <= arrow.get(date.date()) <= end_date:
+                is_open = exception["is_open"]
+
+        if not is_open:
+            disabled.add(date_iso)
+
+        date += timedelta(days=1)
+
+    total_offset_days = 0
+    date = request_start_date
+    while working_days_to_offset > 0:
+        date_iso = date.date().isoformat()
+        if date_iso not in disabled:
+            disabled.add(date_iso)
+            working_days_to_offset -= 1
+        date += timedelta(days=1)
+        total_offset_days += 1
+
+    return timedelta(days=total_offset_days)
+
+
 class LoanRequestDeliverySchemaV1(Schema):
     """Loan common delivery Schema."""
 
@@ -65,12 +107,12 @@ class LoanRequestSchemaV1(LoanBaseSchemaV1):
     """Loan request schema."""
 
     delivery = fields.Nested(LoanRequestDeliverySchemaV1)
-    request_expire_date = DateString(missing=request_expire_date_default)
-    request_start_date = DateString(missing=request_start_date_default)
+    request_expire_date = DateString(load_default=request_expire_date_default)
+    request_start_date = DateString(load_default=request_start_date_default)
 
     @validates_schema()
     def validates_schema(self, data, **kwargs):
-        """Validate schema delivery field."""
+        """Validate schema fields."""
         delivery = data.get("delivery")
         # if delivery methods is configured, it has to be a mandatory field
         if (
@@ -79,29 +121,44 @@ class LoanRequestSchemaV1(LoanBaseSchemaV1):
         ):
             raise ValidationError("Delivery is required.", "delivery")
 
+        if current_app.config["ILS_CIRCULATION_LOAN_REQUEST_OFFSET"] < 0:
+            raise ValidationError(
+                {
+                    "ILS_CIRCULATION_LOAN_REQUEST_OFFSET": [
+                        "The minimum days after which a loan can start cannot be negative."
+                    ]
+                }
+            )
+
     @post_load()
     def postload_checks(self, data, **kwargs):
         """Validate dates values."""
         start = arrow.get(data["request_start_date"]).date()
         end = arrow.get(data["request_expire_date"]).date()
-        duration_days = current_app.config[
-            "ILS_CIRCULATION_LOAN_REQUEST_DURATION_DAYS"
-        ]
+        duration_days = current_app.config["ILS_CIRCULATION_LOAN_REQUEST_DURATION_DAYS"]
         duration = timedelta(days=duration_days)
 
+        loan_request_offset = current_app.config["ILS_CIRCULATION_LOAN_REQUEST_OFFSET"]
+        offset_with_closures = get_offset_duration(
+            arrow.get(data["request_start_date"]),
+            data["transaction_location_pid"],
+            loan_request_offset,
+        )
+
         if end < start:
             raise ValidationError(
                 {
-                    "request_start_date": "The request start date "
-                    "cannot be after the end date.",
-                    "request_end_date": "The request end date "
-                    "cannot be before the start date.",
+                    "request_start_date": (
+                        "The request start date cannot be after the end date."
+                    ),
+                    "request_end_date": (
+                        "The request end date cannot be before the start date."
+                    ),
                 }
             )
         elif end - start > duration:
-            message = (
-                "The request duration "
-                + "cannot be longer than {} days.".format(duration_days)
+            message = "The request duration cannot be longer than {} days.".format(
+                duration_days
             )
             raise ValidationError(
                 {
@@ -109,4 +166,13 @@ class LoanRequestSchemaV1(LoanBaseSchemaV1):
                     "request_expire_date": [message],
                 }
             )
+        elif end - start < offset_with_closures:
+            message = "The requested start date must be at least {} non-closure days from today.".format(
+                loan_request_offset
+            )
+            raise ValidationError(
+                {
+                    "request_start_date": [message],
+                }
+            )
         return data

invenio_app_ils/circulation/loaders/schemas/json/loan_self_checkout.py

@@ -0,0 +1,19 @@
+# -*- coding: utf-8 -*-
+#
+# Copyright (C) 2019 CERN.
+#
+# invenio-app-ils is free software; you can redistribute it and/or modify
+# it under the terms of the MIT License; see LICENSE file for more details.
+
+"""Invenio App ILS circulation Loan Checkout loader JSON schema."""
+
+from invenio_circulation.records.loaders.schemas.json import LoanItemPIDSchemaV1
+from marshmallow import fields
+
+from .base import LoanBaseSchemaV1
+
+
+class LoanSelfCheckoutSchemaV1(LoanBaseSchemaV1):
+    """Loan self-checkout schema."""
+
+    item_pid = fields.Nested(LoanItemPIDSchemaV1, required=True)

invenio_app_ils/circulation/notifications/api.py

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright (C) 2021 CERN.
+# Copyright (C) 2021-2025 CERN.
 #
 # invenio-app-ils is free software; you can redistribute it and/or modify it
 # under the terms of the MIT License; see LICENSE file for more details.
@@ -10,10 +10,7 @@
 from flask import current_app
 from invenio_records_rest.utils import obj_or_import_string
 
-from invenio_app_ils.notifications.api import (
-    build_common_msg_ctx,
-    send_notification,
-)
+from invenio_app_ils.notifications.api import build_common_msg_ctx, send_notification
 from invenio_app_ils.patrons.api import Patron
 
 
@@ -36,9 +33,7 @@ def send_loan_notification(loan, action, msg_extra_ctx=None, **kwargs):
     msg_ctx.update(build_common_msg_ctx(loan))
     msg_ctx.update(_build_circulation_msg_ctx())
 
-    func_or_path = current_app.config[
-        "ILS_CIRCULATION_NOTIFICATIONS_MSG_BUILDER"
-    ]
+    func_or_path = current_app.config["ILS_CIRCULATION_NOTIFICATIONS_MSG_BUILDER"]
     builder = obj_or_import_string(func_or_path)
     msg = builder(loan, action, msg_ctx, **kwargs)
 
@@ -69,12 +64,22 @@ def send_expiring_loan_reminder_notification(loan, expiring_in_days):
     )
 
 
-def send_bulk_extend_notification(extended_loans,
-                                  not_extended_loans,
-                                  patron_pid,
-                                  action="bulk_extend",
-                                  msg_extra_ctx=None,
-                                  **kwargs):
+def send_dates_updated_notification(loan):
+    """Send reminder notification."""
+    send_loan_notification(
+        action="update_dates",
+        loan=loan,
+    )
+
+
+def send_bulk_extend_notification(
+    extended_loans,
+    not_extended_loans,
+    patron_pid,
+    action="bulk_extend",
+    msg_extra_ctx=None,
+    **kwargs
+):
     """Send notification to patron about bulk extend action."""
     _filter = current_app.config["ILS_CIRCULATION_NOTIFICATIONS_FILTER"]
     if _filter and _filter({}, action, **kwargs) is False:
@@ -92,13 +97,12 @@ def send_bulk_extend_notification(extended_loans,
         context.update(build_common_msg_ctx(loan))
         not_extended_loans_ctx.append(context)
 
-    msg_ctx.update(extended_loans=extended_loans_ctx,
-                   not_extended_loans_ctx=not_extended_loans_ctx)
+    msg_ctx.update(
+        extended_loans=extended_loans_ctx, not_extended_loans_ctx=not_extended_loans_ctx
+    )
     msg_ctx.update(_build_circulation_msg_ctx())
 
-    func_or_path = current_app.config[
-        "ILS_CIRCULATION_NOTIFICATIONS_MSG_BUILDER"
-    ]
+    func_or_path = current_app.config["ILS_CIRCULATION_NOTIFICATIONS_MSG_BUILDER"]
     builder = obj_or_import_string(func_or_path)
     msg = builder({}, action, msg_ctx, **kwargs)
 

invenio_app_ils/circulation/notifications/messages.py

@@ -1,6 +1,6 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright (C) 2021 CERN.
+# Copyright (C) 2021-2025 CERN.
 #
 # invenio-app-ils is free software; you can redistribute it and/or modify it
 # under the terms of the MIT License; see LICENSE file for more details.
@@ -25,12 +25,14 @@ class NotificationLoanMsg(NotificationMsg):
         request="request.html",
         request_no_items="request_no_items.html",
         checkout="checkout.html",
+        self_checkout="self_checkout.html",
         checkin="checkin.html",
         extend="extend.html",
         cancel="cancel.html",
+        update_dates="update_dates.html",
         overdue_reminder="overdue_reminder.html",
         expiring_reminder="will_expire_in_reminder.html",
-        bulk_extend="bulk_extend.html"
+        bulk_extend="bulk_extend.html",
     )
 
     def __init__(self, loan, action, msg_ctx, **kwargs):
@@ -67,9 +69,7 @@ class NotificationLoanMsg(NotificationMsg):
         """Get the template filename based on the loan action."""
         new_state = self.loan["state"]
         document_pid = self.loan.get("document_pid")
-        is_request = (
-            new_state in current_app.config["CIRCULATION_STATES_LOAN_REQUEST"]
-        )
+        is_request = new_state in current_app.config["CIRCULATION_STATES_LOAN_REQUEST"]
         if (
             is_request
             and document_pid
@@ -112,6 +112,6 @@ class NotificationBulkExtendLoanMsg(NotificationLoanMsg):
 
 def notification_loan_msg_builder(loan, action, msg_ctx, **kwargs):
     """Factory builder to create a notification msg."""
-    if action == 'bulk_extend':
+    if action == "bulk_extend":
         return NotificationBulkExtendLoanMsg(action, msg_ctx, **kwargs)
     return NotificationLoanMsg(loan, action, msg_ctx, **kwargs)

invenio_app_ils/circulation/notifications/tasks.py

@@ -27,8 +27,7 @@ celery_logger = get_task_logger(__name__)
 @shared_task
 def send_overdue_loans_notification_reminder():
     """Send notification message for loans that are overdue every X days."""
-    days = current_app.config[
-        "ILS_CIRCULATION_NOTIFICATION_OVERDUE_REMINDER_INTERVAL"]
+    days = current_app.config["ILS_CIRCULATION_NOTIFICATION_OVERDUE_REMINDER_INTERVAL"]
     for hit in get_all_overdue_loans().scan():
         loan = hit.to_dict()
         days_ago = circulation_overdue_loan_days(loan)

invenio_app_ils/circulation/receivers.py

@@ -10,9 +10,7 @@
 from invenio_circulation.proxies import current_circulation
 from invenio_circulation.signals import loan_replace_item, loan_state_changed
 
-from invenio_app_ils.circulation.notifications.api import (
-    send_loan_notification,
-)
+from invenio_app_ils.circulation.notifications.api import send_loan_notification
 from invenio_app_ils.circulation.utils import resolve_item_from_loan
 from invenio_app_ils.ill.api import BORROWING_REQUEST_PID_TYPE
 from invenio_app_ils.ill.proxies import current_ils_ill
@@ -53,8 +51,7 @@ def index_after_loan_replace_item(_, old_item_pid, new_item_pid):
         indexer.index(rec)
 
 
-def send_notification_after_loan_change(_, initial_loan, loan, trigger,
-                                        **kwargs):
+def send_notification_after_loan_change(_, initial_loan, loan, trigger, **kwargs):
     """Send email notification when the loan changes."""
     send_notification = kwargs.get("send_notification", True)
     if send_notification:

invenio_app_ils/circulation/search.py

@@ -9,10 +9,10 @@
 
 from datetime import datetime, timedelta
 
-from elasticsearch_dsl import A, Q
 from flask import current_app
 from invenio_circulation.proxies import current_circulation
 from invenio_circulation.search.api import search_by_pid
+from invenio_search.engine import dsl
 
 from invenio_app_ils.items.api import ITEM_PID_TYPE
 
@@ -21,9 +21,7 @@ def get_loan_next_available_date(document_pid):
     """Return active loans on document sorted by the earliest end date."""
     return search_by_pid(
         document_pid=document_pid,
-        filter_states=current_app.config.get(
-            "CIRCULATION_STATES_LOAN_ACTIVE", []
-        ),
+        filter_states=current_app.config.get("CIRCULATION_STATES_LOAN_ACTIVE", []),
         sort_by_field="end_date",
     )
 
@@ -32,9 +30,7 @@ def get_active_loan_by_item_pid(item_pid):
     """Return any active loans for the given item."""
     return search_by_pid(
         item_pid=item_pid,
-        filter_states=current_app.config.get(
-            "CIRCULATION_STATES_LOAN_ACTIVE", []
-        ),
+        filter_states=current_app.config.get("CIRCULATION_STATES_LOAN_ACTIVE", []),
     )
 
 
@@ -69,12 +65,19 @@ def get_all_range_expiring_loans_by_patron_pid(expiring_in_days, patron_pid):
     )
 
 
-def get_all_range_expiring_ils_loans_by_patron_pid(expiring_in_days,
-                                                   patron_pid):
-    """Return ILS loans expiring in the next <days> days for given patron."""
-    return get_all_range_expiring_loans_by_patron_pid(
-        expiring_in_days, patron_pid)\
+def get_all_expiring_or_overdue_loans_by_patron_pid(patron_pid):
+    """Return ILS loans expiring or overdue for given patron."""
+    days = current_app.config["ILS_CIRCULATION_LOAN_WILL_EXPIRE_DAYS"]
+    future = (datetime.today() + timedelta(days=days)).date()
+    states = current_app.config["CIRCULATION_STATES_LOAN_ACTIVE"]
+    search_cls = current_circulation.loan_search_cls
+    return (
+        search_cls()
+        .filter("term", patron_pid=patron_pid)
         .filter("term", item_pid__type=ITEM_PID_TYPE)
+        .filter("range", end_date=dict(lte="{}||/d".format(future)))
+        .filter("terms", state=states)
+    )
 
 
 def get_all_expired_loans():
@@ -103,9 +106,9 @@ def get_all_overdue_loans():
 def get_overdue_loans_by_doc_pid(document_pid):
     """Return any overdue loans for the given document."""
     states = current_app.config["CIRCULATION_STATES_LOAN_ACTIVE"]
-    return search_by_pid(
-        document_pid=document_pid, filter_states=states
-    ).filter("range", end_date=dict(lt="now/d"))
+    return search_by_pid(document_pid=document_pid, filter_states=states).filter(
+        "range", end_date=dict(lt="now/d")
+    )
 
 
 def get_most_loaned_documents(from_date, to_date, bucket_size):
@@ -124,13 +127,13 @@ def get_most_loaned_documents(from_date, to_date, bucket_size):
     search = search_cls().query(
         "bool",
         must=[
-            Q("terms", state=states),
-            Q("range", start_date=dict(gte=from_date, lte=to_date)),
+            dsl.Q("terms", state=states),
+            dsl.Q("range", start_date=dict(gte=from_date, lte=to_date)),
         ],
     )
 
     # Aggregation with sub-aggregation to calculate the extension count sum
-    aggs = A("terms", field="document_pid", size=bucket_size)
+    aggs = dsl.A("terms", field="document_pid", size=bucket_size)
     aggs = aggs.metric("extensions", "sum", field="extension_count")
     search.aggs.bucket("most_loaned_documents", aggs)
 
@@ -151,7 +154,7 @@ def get_loans_aggregated_by_states(document_pid, states, patron_pid=None):
     if patron_pid:
         search = search.filter("term", patron_pid=patron_pid)
     # Aggregation
-    aggs = A("terms", field="state")
+    aggs = dsl.A("terms", field="state")
     search.aggs.bucket("states", aggs)
 
     return search
@@ -166,9 +169,7 @@ def get_loans_by_patron_pid(patron_pid):
 def get_active_loans_by_patron_pid(patron_pid):
     """Returns a search for all the active loans of a given patron."""
     active_states = current_app.config["CIRCULATION_STATES_LOAN_ACTIVE"]
-    search = get_loans_by_patron_pid(patron_pid).filter(
-        "terms", state=active_states
-    )
+    search = get_loans_by_patron_pid(patron_pid).filter("terms", state=active_states)
     return search
 
 

invenio_app_ils/circulation/serializers/__init__.py

@@ -6,6 +6,7 @@
 # under the terms of the MIT License; see LICENSE file for more details.
 
 """Loan serializers."""
+
 from invenio_records_rest.serializers.response import search_responsify
 
 from invenio_app_ils.records.schemas.json import ILSRecordSchemaJSONV1
@@ -23,5 +24,4 @@ json_v1 = LoanJSONSerializer(ILSRecordSchemaJSONV1, replace_refs=True)
 json_v1_response = record_responsify_no_etag(json_v1, "application/json")
 json_v1_search = search_responsify(json_v1, "application/json")
 
-bulk_extend_v1_response = bulk_loan_extend_responsify(json_v1,
-                                                      'application/json')
+bulk_extend_v1_response = bulk_loan_extend_responsify(json_v1, "application/json")

invenio_app_ils/circulation/serializers/csv.py

@@ -31,9 +31,7 @@ class LoanCSVSerializer(CSVSerializer):
         field_transaction_user(loan["metadata"])
         return loan
 
-    def transform_search_hit(
-        self, pid, record_hit, links_factory=None, **kwargs
-    ):
+    def transform_search_hit(self, pid, record_hit, links_factory=None, **kwargs):
         """Transform search result hit into an intermediate representation."""
         hit = super().transform_search_hit(
             pid, record_hit, links_factory=links_factory, **kwargs

invenio_app_ils/circulation/serializers/custom_fields.py

@@ -11,6 +11,7 @@ from flask import current_app
 from invenio_pidstore.errors import PIDDeletedError, PIDDoesNotExistError
 
 from invenio_app_ils.circulation.utils import circulation_overdue_loan_days
+from invenio_app_ils.errors import PatronNotFoundError
 from invenio_app_ils.proxies import current_app_ils
 from invenio_app_ils.records.jsonresolvers.api import pick
 
@@ -19,8 +20,7 @@ def field_is_overdue(metadata):
     """Calculate if the loan is overdue."""
     metadata["is_overdue"] = False
     is_loan_active = (
-        metadata["state"]
-        in current_app.config["CIRCULATION_STATES_LOAN_ACTIVE"]
+        metadata["state"] in current_app.config["CIRCULATION_STATES_LOAN_ACTIVE"]
     )
     if is_loan_active and "end_date" in metadata:
         metadata["is_overdue"] = circulation_overdue_loan_days(metadata) > 0
@@ -50,13 +50,9 @@ def field_transaction_location(metadata):
         return
     Location = current_app_ils.location_record_cls
     try:
-        transaction_location = Location.get_record_by_pid(
-            transaction_location_pid
-        )
+        transaction_location = Location.get_record_by_pid(transaction_location_pid)
     except PIDDeletedError:
-        metadata["transaction_location"] = {
-            "name": "This location was deleted."
-        }
+        metadata["transaction_location"] = {"name": "This location was deleted."}
         return
     except PIDDoesNotExistError:
         metadata["transaction_location"] = {"name": "Location PID invalid."}
@@ -72,6 +68,6 @@ def field_transaction_user(metadata):
     Patron = current_app_ils.patron_cls
     try:
         transaction_user = Patron.get_patron(transaction_user_pid)
-    except PIDDeletedError:
+    except (PIDDeletedError, PatronNotFoundError):
         return
     metadata["transaction_user"] = pick(transaction_user, "name")

invenio_app_ils/circulation/serializers/json.py

@@ -9,9 +9,7 @@
 
 from invenio_records_rest.serializers.json import JSONSerializer
 
-from invenio_app_ils.literature.serializers.custom_fields import (
-    field_cover_metadata,
-)
+from invenio_app_ils.literature.serializers.custom_fields import field_cover_metadata
 
 from .custom_fields import (
     field_is_overdue,
@@ -36,9 +34,7 @@ class LoanJSONSerializer(JSONSerializer):
         field_cover_metadata(loan["metadata"].get("document", {}))
         return loan
 
-    def transform_search_hit(
-        self, pid, record_hit, links_factory=None, **kwargs
-    ):
+    def transform_search_hit(self, pid, record_hit, links_factory=None, **kwargs):
         """Transform search result hit into an intermediate representation."""
         hit = super().transform_search_hit(
             pid, record_hit, links_factory=links_factory, **kwargs

invenio_app_ils/circulation/serializers/response.py

@@ -6,34 +6,34 @@
 # under the terms of the MIT License; see LICENSE file for more details.
 
 """Response serializers for circulation module."""
+
 import json
 
 from flask import current_app
 
 
-def bulk_loan_extend_responsify(serializer,
-                                mimetype):
+def bulk_loan_extend_responsify(serializer, mimetype):
     """Bulk loan extend serializer.
 
     :param schema_class: Serializer instance.
     :param mimetype: MIME type of response.
     """
 
-    def view(extended_loans, not_extended_loans,
-             code=200, headers=None, **kwargs):
+    def view(extended_loans, not_extended_loans, code=200, headers=None, **kwargs):
         """Generate the response object."""
-        response_data = {"extended_loans": [],
-                         "not_extended_loans": []}
+        response_data = {"extended_loans": [], "not_extended_loans": []}
         for loan in extended_loans:
-            response_data["extended_loans"]\
-                .append(serializer.serialize(loan["pid"], loan))
+            response_data["extended_loans"].append(
+                serializer.serialize(loan["pid"], loan)
+            )
         for loan in not_extended_loans:
-            response_data["not_extended_loans"]\
-                .append(serializer.serialize(loan["pid"], loan))
+            response_data["not_extended_loans"].append(
+                serializer.serialize(loan["pid"], loan)
+            )
 
         response = current_app.response_class(
-            json.dumps(response_data),
-            mimetype=mimetype)
+            json.dumps(response_data), mimetype=mimetype
+        )
         response.status_code = code
 
         if headers is not None:

invenio_app_ils/circulation/stats/views.py

@@ -16,10 +16,7 @@ from invenio_rest import ContentNegotiatedMethodView
 
 from invenio_app_ils.circulation.stats.api import fetch_most_loaned_documents
 from invenio_app_ils.config import RECORDS_REST_MAX_RESULT_WINDOW
-from invenio_app_ils.documents.api import (
-    DOCUMENT_PID_FETCHER,
-    DOCUMENT_PID_TYPE,
-)
+from invenio_app_ils.documents.api import DOCUMENT_PID_FETCHER, DOCUMENT_PID_TYPE
 from invenio_app_ils.errors import InvalidParameterError
 from invenio_app_ils.permissions import need_permissions
 
@@ -29,13 +26,10 @@ def create_most_loaned_documents_view(blueprint, app):
     endpoints = app.config.get("RECORDS_REST_ENDPOINTS", [])
     document_endpoint = endpoints.get(DOCUMENT_PID_TYPE, {})
     default_media_type = document_endpoint.get("default_media_type", "")
-    search_serializers_aliases = document_endpoint.get(
-        "search_serializers_aliases", ""
-    )
+    search_serializers_aliases = document_endpoint.get("search_serializers_aliases", "")
     search_serializers = document_endpoint.get("search_serializers")
     serializers = {
-        mime: obj_or_import_string(func)
-        for mime, func in search_serializers.items()
+        mime: obj_or_import_string(func) for mime, func in search_serializers.items()
     }
 
     view_class = MostLoanedDocumentsResource
@@ -54,9 +48,7 @@ def create_most_loaned_documents_view(blueprint, app):
 
 def create_circulation_stats_blueprint(app):
     """Add statistics views to the blueprint."""
-    blueprint = Blueprint(
-        "invenio_app_ils_circulation_stats", __name__, url_prefix=""
-    )
+    blueprint = Blueprint("invenio_app_ils_circulation_stats", __name__, url_prefix="")
 
     create_most_loaned_documents_view(blueprint, app)
 
@@ -134,9 +126,7 @@ class MostLoanedDocumentsResource(ContentNegotiatedMethodView):
         """
         size = self._validate_bucket_size()
         from_date, to_date = self._validate_start_date_range()
-        most_loaned_documents = fetch_most_loaned_documents(
-            from_date, to_date, size
-        )
+        most_loaned_documents = fetch_most_loaned_documents(from_date, to_date, size)
         return self.make_response(
             pid_fetcher=current_pidstore.fetchers[DOCUMENT_PID_FETCHER],
             search_result=most_loaned_documents,

invenio_app_ils/circulation/tasks.py

@@ -29,9 +29,7 @@ def cancel_expired_loan_requests():
 
     for hit in get_all_expired_loans().scan():
         loan = Loan.get_record_by_pid(hit.pid)
-        duration_days = current_app.config[
-            "ILS_CIRCULATION_LOAN_REQUEST_DURATION_DAYS"
-        ]
+        duration_days = current_app.config["ILS_CIRCULATION_LOAN_REQUEST_DURATION_DAYS"]
         params = deepcopy(loan)
         params.update(
             dict(
@@ -40,9 +38,7 @@ def cancel_expired_loan_requests():
                 transaction_user_pid=system_agent_id,
             )
         )
-        current_circulation.circulation.trigger(
-            loan, **dict(params, trigger="cancel")
-        )
+        current_circulation.circulation.trigger(loan, **dict(params, trigger="cancel"))
 
         loan.commit()
         db.session.commit()

invenio_app_ils/circulation/templates/invenio_app_ils_circulation/notifications/self_checkout.html

@@ -0,0 +1,19 @@
+{% block title %}
+InvenioILS: loan started for "{{ document.title|safe }}"
+{% endblock %}
+
+{% block body_plain %}
+Dear {{ patron.name }},
+
+your self-checkout loan for "{{ document.full_title }}" <{{ spa_routes.HOST }}{{ spa_routes.PATHS['literature']|format(pid=document.pid) }}> has started.
+
+The due date is {{ loan.end_date }}.
+{% endblock %}
+
+{% block body_html %}
+Dear {{ patron.name }}, <br/><br/>
+
+your self-checkout loan for <a href="{{ spa_routes.HOST }}{{ spa_routes.PATHS['literature']|format(pid=document.pid) }}">"{{ document.full_title }}"</a> has <b>started</b>. <br/><br/>
+
+<b>The due date is {{ loan.end_date }}</b>.<br/>
+{% endblock %}