i18n: Fix untranslated strings in facets

invenio_records_resources/records/systemfields/entity_reference.py

@@ -8,8 +8,7 @@
 
 """Systemfield for managing referenced entities in request."""
 
-from functools import partial
-
+from invenio_i18n import gettext as _
 from invenio_records.systemfields import SystemField
 
 from ...references.entity_resolvers import EntityProxy
@@ -43,7 +42,9 @@ def set_obj(self, instance, obj):
 
         # check if the reference is allowed
         if not self._check_reference(instance, obj):
-            raise ValueError(f"Invalid reference for '{self.key}': {obj}")
+            raise ValueError(
+                _("Invalid reference for '{key}': {obj}").format(key=self.key, obj=obj)
+            )
 
         # set dictionary key and reset the cache
         self.set_dictkey(instance, obj)

invenio_records_resources/resources/errors.py

@@ -9,11 +9,13 @@
 # details.
 
 """Common Errors handling for Resources."""
+
 from json import JSONDecodeError
 
 import marshmallow as ma
 from flask import jsonify, make_response, request, url_for
 from flask_resources import HTTPJSONException, create_error_handler
+from invenio_i18n import gettext
 from invenio_i18n import lazy_gettext as _
 from invenio_pidstore.errors import (
     PIDAlreadyExists,
@@ -116,56 +118,59 @@ class ErrorHandlersMixin:
         QuerystringValidationError: create_error_handler(
             HTTPJSONException(
                 code=400,
-                description="Invalid querystring parameters.",
+                description=_("Invalid querystring parameters."),
             )
         ),
         PermissionDeniedError: create_error_handler(
             HTTPJSONException(
                 code=403,
-                description="Permission denied.",
+                description=_("Permission denied."),
             )
         ),
         RecordPermissionDeniedError: create_error_handler(
             HTTPJSONException(
                 code=403,
-                description="Permission denied.",
+                description=_("Permission denied."),
             )
         ),
         PIDDeletedError: create_error_handler(
             HTTPJSONException(
                 code=410,
-                description="The record has been deleted.",
+                description=_("The record has been deleted."),
             )
         ),
         PIDAlreadyExists: create_error_handler(
             HTTPJSONException(
                 code=400,
-                description="The persistent identifier is already registered.",
+                description=_("The persistent identifier is already registered."),
             )
         ),
         PIDDoesNotExistError: create_error_handler(
             HTTPJSONException(
                 code=404,
-                description="The persistent identifier does not exist.",
+                description=_("The persistent identifier does not exist."),
             )
         ),
         PIDUnregistered: create_error_handler(
             HTTPJSONException(
                 code=404,
-                description="The persistent identifier is not registered.",
+                description=_("The persistent identifier is not registered."),
             )
         ),
         PIDRedirectedError: create_pid_redirected_error_handler(),
         NoResultFound: create_error_handler(
             HTTPJSONException(
                 code=404,
-                description="Not found.",
+                description=_("Not found."),
             )
         ),
         FacetNotFoundError: create_error_handler(
             lambda e: HTTPJSONException(
                 code=404,
-                description=f"Facet {e.vocabulary_id} not found.",
+                # using gettext here in order to be able to call the format method
+                description=gettext("Facet {vocabulary_id} not found.").format(
+                    vocabulary_id=e.vocabulary_id
+                ),
             )
         ),
         FileKeyNotFoundError: create_error_handler(
@@ -177,19 +182,19 @@ class ErrorHandlersMixin:
         JSONDecodeError: create_error_handler(
             HTTPJSONException(
                 code=400,
-                description="Unable to decode JSON data in request body.",
+                description=_("Unable to decode JSON data in request body."),
             )
         ),
         InvalidRelationValue: create_error_handler(
             HTTPJSONException(
                 code=400,
-                description="Not a valid value.",
+                description=_("Not a valid value."),
             )
         ),
         InvalidCheckValue: create_error_handler(
             HTTPJSONException(
                 code=400,
-                description="Not a valid value.",
+                description=_("Not a valid value."),
             )
         ),
         search.exceptions.RequestError: create_error_handler(
@@ -198,13 +203,15 @@ class ErrorHandlersMixin:
         FailedFileUploadException: create_error_handler(
             HTTPJSONException(
                 code=400,
-                description="The file upload transfer failed, please try again.",
+                description=_("The file upload transfer failed, please try again."),
             )
         ),
         FilesCountExceededException: create_error_handler(
             HTTPJSONException(
                 code=400,
-                description="Uploading selected files will result in exceeding the max amount per record.",
+                description=_(
+                    "Uploading selected files will result in exceeding the max amount per record."
+                ),
             )
         ),
     }

invenio_records_resources/services/custom_fields/errors.py

@@ -7,9 +7,10 @@
 
 """Custom Fields for InvenioRDM."""
 
-
 from abc import abstractmethod
 
+from invenio_i18n import gettext as _
+
 
 class CustomFieldsException(Exception):
     """Base class for custom fields exceptions."""
@@ -32,10 +33,9 @@ def __init__(self, field_name, given_namespace):
     @property
     def description(self):
         """Exception's description."""
-        return (
-            f"Namespace {self.given_namespace} is not valid for custom field "
-            f"{self.field_name}."
-        )
+        return _(
+            "Namespace {given_namespace} is not valid for custom field " "{field_name}."
+        ).format(given_namespace=self.given_namespace, field_name=self.field_name)
 
 
 class CustomFieldsNotConfigured(CustomFieldsException):
@@ -48,7 +48,9 @@ def __init__(self, field_names):
     @property
     def description(self):
         """Exception's description."""
-        return f"Custom fields {self.field_names} are not configured."
+        return _("Custom fields {field_names} are not configured.").format(
+            field_names=self.field_names
+        )
 
 
 class CustomFieldsInvalidArgument(CustomFieldsException):
@@ -61,6 +63,6 @@ def __init__(self, arg_name):
     @property
     def description(self):
         """Exception's description."""
-        return (
-            f"Invalid argument {self.arg_name} passed when initializing custom field."
-        )
+        return _(
+            "Invalid argument {arg_name} passed when initializing custom field."
+        ).format(arg_name=self.arg_name)

invenio_records_resources/services/errors.py

@@ -30,7 +30,10 @@ def __init__(self, action_name=None, record=None, *args, **kwargs):
 class PermissionDeniedError(PermissionDenied):
     """Permission denied error."""
 
-    description = "Permission denied."
+    @property
+    def description(self):
+        """Description."""
+        return _("Permission denied.")
 
 
 class RevisionIdMismatchError(Exception):
@@ -44,9 +47,12 @@ def __init__(self, record_revision_id, expected_revision_id):
     @property
     def description(self):
         """Exception's description."""
-        return (
-            f"Revision id provided({self.expected_revision_id}) doesn't match "
-            f"record's one({self.record_revision_id})"
+        return _(
+            "Revision id provided({expected_revision_id}) doesn't match "
+            "record's one({record_revision_id})"
+        ).format(
+            expected_revision_id=self.expected_revision_id,
+            record_revision_id=self.record_revision_id,
         )
 
 

invenio_records_resources/services/files/service.py

@@ -10,6 +10,7 @@
 """File Service API."""
 
 from flask import current_app
+from invenio_i18n import gettext as _
 
 from ..base import LinksTemplate, Service
 from ..errors import FailedFileUploadException, FileKeyNotFoundError
@@ -257,10 +258,10 @@ def set_file_content(
 
         except FailedFileUploadException as e:
             file = e.file
-            current_app.logger.exception(f"File upload transfer failed.")
+            current_app.logger.exception("File upload transfer failed.")
             # we gracefully fail so that uow can commit the cleanup operation in
             # FileContentComponent
-            errors = "File upload transfer failed."
+            errors = _("File upload transfer failed.")
 
         return self.file_result_item(
             self,

invenio_records_resources/services/records/params/pagination.py

@@ -10,6 +10,8 @@
 
 from copy import deepcopy
 
+from invenio_i18n import gettext as _
+
 from ....pagination import Pagination
 from ...errors import QuerystringValidationError
 from .base import ParamInterpreter
@@ -34,6 +36,6 @@ def apply(self, identity, search, params):
         )
 
         if not p.valid():
-            raise QuerystringValidationError("Invalid pagination parameters.")
+            raise QuerystringValidationError(_("Invalid pagination parameters."))
 
         return search[p.from_idx : p.to_idx]

invenio_records_resources/services/records/params/querystr.py

@@ -8,9 +8,10 @@
 
 """Query parameter interpreter API."""
 
-from ...errors import QuerystringValidationError
-
 # Here for backward compatibility
+from invenio_i18n import gettext as _
+
+from ...errors import QuerystringValidationError
 from ..queryparser import QueryParser, SuggestQueryParser  # noqa
 from .base import ParamInterpreter
 
@@ -25,8 +26,10 @@ def apply(self, identity, search, params):
 
         if q_str and suggest_str:
             raise QuerystringValidationError(
-                "You cannot specify both 'q' and 'suggest' parameters at the "
-                "same time."
+                _(
+                    "You cannot specify both 'q' and 'suggest' parameters at the "
+                    "same time."
+                )
             )
 
         query_str = q_str
@@ -35,7 +38,7 @@ def apply(self, identity, search, params):
             query_str = suggest_str
             parser_cls = self.config.suggest_parser_cls
             if parser_cls is None:
-                raise QuerystringValidationError("Invalid 'suggest' parameter.")
+                raise QuerystringValidationError(_("Invalid 'suggest' parameter."))
 
         if query_str:
             query = parser_cls(identity).parse(query_str)

invenio_records_resources/services/records/params/sort.py

@@ -10,6 +10,7 @@
 
 from copy import deepcopy
 
+from invenio_i18n import gettext as _
 from marshmallow import ValidationError
 
 from .base import ParamInterpreter
@@ -57,5 +58,9 @@ def _compute_sort_fields(self, params):
 
         sort = options.get(params["sort"])
         if sort is None:
-            raise ValidationError(f"Invalid sort option '{params['sort']}'.")
+            raise ValidationError(
+                _("Invalid sort option '{sort_option}'.").format(
+                    sort_option=params["sort"]
+                )
+            )
         return sort["fields"]