udata 9.1.2.dev30754__py2.py3-none-any.whl → 9.1.4.dev30983__py2.py3-none-any.whl

udata/core/dataset/actions.py

@@ -4,12 +4,12 @@ from datetime import datetime
 from flask import current_app, render_template
 
 from udata import i18n
-from udata.models import Discussion, Message
+from udata.models import Dataset, Discussion, Message
 
 log = logging.getLogger(__name__)
 
 
-def archive(dataset, comment=False):
+def archive(dataset: Dataset, comment=False) -> None:
     """Archive a dataset"""
     if dataset.archived:
         log.warning("Dataset %s already archived, bumping date", dataset)

udata/core/dataset/forms.py

@@ -1,5 +1,3 @@
-from mongoengine import ValidationError
-
 from udata.core.spatial.forms import SpatialCoverageField
 from udata.core.storages import resources
 from udata.forms import ModelForm, fields, validators

udata/core/dataset/models.py

@@ -256,8 +256,8 @@ class License(db.Document):
 
         if license is None:
             # Try to single match `slug` with a low Damerau-Levenshtein distance
-            computed = ((l, rdlevenshtein(l.slug, slug)) for l in cls.objects)
-            candidates = [l for l, d in computed if d <= MAX_DISTANCE]
+            computed = ((license_, rdlevenshtein(license_.slug, slug)) for license_ in cls.objects)
+            candidates = [license_ for license_, d in computed if d <= MAX_DISTANCE]
             # If there is more that one match, we cannot determinate
             # which one is closer to safely choose between candidates
             if len(candidates) == 1:
@@ -265,8 +265,10 @@ class License(db.Document):
 
         if license is None:
             # Try to match `title` with a low Damerau-Levenshtein distance
-            computed = ((l, rdlevenshtein(l.title.lower(), text)) for l in cls.objects)
-            candidates = [l for l, d in computed if d <= MAX_DISTANCE]
+            computed = (
+                (license_, rdlevenshtein(license_.title.lower(), text)) for license_ in cls.objects
+            )
+            candidates = [license_ for license_, d in computed if d <= MAX_DISTANCE]
             # If there is more that one match, we cannot determinate
             # which one is closer to safely choose between candidates
             if len(candidates) == 1:
@@ -275,11 +277,11 @@ class License(db.Document):
         if license is None:
             # Try to single match `alternate_titles` with a low Damerau-Levenshtein distance
             computed = (
-                (l, rdlevenshtein(cls.slug.slugify(t), slug))
-                for l in cls.objects
-                for t in l.alternate_titles
+                (license_, rdlevenshtein(cls.slug.slugify(title_), slug))
+                for license_ in cls.objects
+                for title_ in license_.alternate_titles
             )
-            candidates = [l for l, d in computed if d <= MAX_DISTANCE]
+            candidates = [license_ for license_, distance_ in computed if distance_ <= MAX_DISTANCE]
             # If there is more that one license matching, we cannot determinate
             # which one is closer to safely choose between candidates
             if len(set(candidates)) == 1:
@@ -998,7 +1000,7 @@ class ResourceSchema(object):
                     f"Schemas catalog does not exist at {endpoint}"
                 )
             response.raise_for_status()
-        except requests.exceptions.RequestException as e:
+        except requests.exceptions.RequestException:
             log.exception(f"Error while getting schema catalog from {endpoint}")
             schemas = cache.get(cache_key)
         else:
@@ -1006,7 +1008,7 @@ class ResourceSchema(object):
             cache.set(cache_key, schemas)
         # no cached version or no content
         if not schemas:
-            log.error(f"No content found inc. from cache for schema catalog")
+            log.error("No content found inc. from cache for schema catalog")
             raise SchemasCacheUnavailableException("No content in cache for schema catalog")
 
         return schemas

udata/core/dataset/rdf.py

@@ -387,7 +387,7 @@ def spatial_from_rdf(graph):
             continue
 
     if not polygons:
-        log.warning(f"No supported types found in the GeoJSON data.")
+        log.warning("No supported types found in the GeoJSON data.")
         return None
 
     spatial_coverage = SpatialCoverage(

udata/core/discussions/api.py

@@ -152,7 +152,7 @@ class DiscussionAPI(API):
 
 @ns.route("/<id>/comments/<int:cidx>/spam", endpoint="discussion_comment_spam")
 @ns.doc(delete={"id": "unspam"})
-class DiscussionSpamAPI(SpamAPIMixin):
+class DiscussionCommentSpamAPI(SpamAPIMixin):
     def get_model(self, id, cidx):
         discussion = Discussion.objects.get_or_404(id=id_or_404(id))
         if len(discussion.discussion) <= cidx:

udata/core/discussions/models.py

@@ -20,7 +20,7 @@ class Message(SpamMixin, db.EmbeddedDocument):
         return [self.content]
 
     def spam_report_message(self, breadcrumb):
-        message = f"Spam potentiel dans le message"
+        message = "Spam potentiel dans le message"
         if self.posted_by:
             message += f" de [{self.posted_by.fullname}]({self.posted_by.external_url})"
 
@@ -34,7 +34,7 @@ class Message(SpamMixin, db.EmbeddedDocument):
         discussion = breadcrumb[0]
         if not isinstance(discussion, Discussion):
             log.warning(
-                f"`spam_report_message` called on message with a breadcrumb not containing a Discussion at index 0.",
+                "`spam_report_message` called on message with a breadcrumb not containing a Discussion at index 0.",
                 extra={"breadcrumb": breadcrumb},
             )
             return message

udata/core/discussions/tasks.py

@@ -5,7 +5,7 @@ from udata.core.reuse.models import Reuse
 from udata.i18n import lazy_gettext as _
 from udata.tasks import connect, get_logger
 
-from .models import Discussion, Message
+from .models import Discussion
 from .signals import on_discussion_closed, on_new_discussion, on_new_discussion_comment
 
 log = get_logger(__name__)

udata/core/organization/api.py

@@ -8,6 +8,8 @@ from udata.api.parsers import ModelApiParser
 from udata.auth import admin_permission, current_user
 from udata.core.badges import api as badges_api
 from udata.core.badges.fields import badge_fields
+from udata.core.contact_point.api import ContactPointApiParser
+from udata.core.contact_point.api_fields import contact_point_page_fields
 from udata.core.dataset.api import DatasetApiParser
 from udata.core.dataset.api_fields import dataset_page_fields
 from udata.core.dataset.models import Dataset
@@ -21,6 +23,7 @@ from udata.core.storages.api import (
     parse_uploaded_image,
     uploaded_image_fields,
 )
+from udata.models import ContactPoint
 from udata.rdf import RDF_EXTENSIONS, graph_response, negociate_content
 from udata.utils import multi_to_dict
 
@@ -207,10 +210,6 @@ class OrganizationBadgeAPI(API):
         return badges_api.remove(org, badge_kind)
 
 
-from udata.core.contact_point.api import ContactPointApiParser
-from udata.core.contact_point.api_fields import contact_point_page_fields
-from udata.models import ContactPoint
-
 contact_point_parser = ContactPointApiParser()
 
 

udata/core/organization/api_fields.py

@@ -41,7 +41,8 @@ org_ref_fields = api.inherit(
     },
 )
 
-from udata.core.user.api_fields import user_ref_fields  # noqa: required
+# This import is not at the top of the file to avoid circular imports
+from udata.core.user.api_fields import user_ref_fields  # noqa
 
 
 def check_can_access_email():

udata/core/organization/apiv2.py

@@ -7,7 +7,7 @@ from udata.core.contact_point.api_fields import contact_point_fields
 from udata.utils import multi_to_dict
 
 from .api_fields import member_fields, org_fields, org_page_fields
-from .permissions import EditOrganizationPermission, OrganizationPrivatePermission
+from .permissions import EditOrganizationPermission
 from .search import OrganizationSearch
 
 apiv2.inherit("OrganizationPage", org_page_fields)

udata/core/reports/api.py

@@ -4,6 +4,7 @@ from flask_login import current_user
 
 from udata.api import API, api, fields
 from udata.api_fields import patch
+from udata.auth import admin_permission
 
 from .constants import reports_reasons_translations
 from .models import Report
@@ -16,6 +17,7 @@ class ReportsAPI(API):
     @api.doc("list_reports")
     @api.expect(Report.__index_parser__)
     @api.marshal_with(Report.__page_fields__)
+    @api.secure(admin_permission)
     def get(self):
         query = Report.objects
 
@@ -37,6 +39,15 @@ class ReportsAPI(API):
         return report, 201
 
 
+@ns.route("/<report:report>/", endpoint="report")
+class ReportAPI(API):
+    @api.doc("get_report")
+    @api.marshal_with(Report.__read_fields__)
+    @api.secure(admin_permission)
+    def get(self, report):
+        return report
+
+
 @ns.route("/reasons/", endpoint="reports_reasons")
 class ReportsReasonsAPI(API):
     @api.doc("list_reports_reasons")

udata/core/reports/models.py

@@ -3,10 +3,11 @@ from datetime import datetime
 from bson import DBRef
 from mongoengine import DO_NOTHING, NULLIFY, signals
 
-from udata.api_fields import field, generate_fields
+from udata.api_fields import field, function_field, generate_fields
 from udata.core.user.api_fields import user_ref_fields
 from udata.core.user.models import User
 from udata.mongo import db
+from udata.uris import endpoint_for
 
 from .constants import REPORT_REASONS_CHOICES, REPORTABLE_MODELS
 
@@ -45,6 +46,10 @@ class Report(db.Document):
         readonly=True,
     )
 
+    @function_field(description="Link to the API endpoint for this report")
+    def self_api_url(self):
+        return endpoint_for("api.report", report=self, _external=True)
+
     @classmethod
     def mark_as_deleted_soft_delete(cls, sender, document, **kwargs):
         """

udata/core/reuse/metrics.py

@@ -3,5 +3,5 @@ from udata.models import Reuse
 
 @Reuse.on_create.connect
 @Reuse.on_update.connect
-def update_reuses_dataset_metric(reuse, **kwargs):
+def update_reuses_dataset_metric(reuse: Reuse, **kwargs) -> None:
     reuse.count_datasets()

udata/core/reuse/permissions.py

@@ -3,10 +3,11 @@ from udata.core.organization.permissions import (
     OrganizationAdminNeed,
     OrganizationEditorNeed,
 )
+from udata.core.reuse.models import Reuse
 
 
 class ReuseEditPermission(Permission):
-    def __init__(self, reuse):
+    def __init__(self, reuse: Reuse) -> None:
         needs = []
 
         if reuse.organization:

udata/core/reuse/search.py

@@ -38,7 +38,7 @@ class ReuseSearch(ModelSearchAdapter):
     }
 
     @classmethod
-    def is_indexable(cls, reuse):
+    def is_indexable(cls, reuse: Reuse) -> bool:
         return reuse.deleted is None and len(reuse.datasets) > 0 and not reuse.private
 
     @classmethod
@@ -55,7 +55,7 @@ class ReuseSearch(ModelSearchAdapter):
         return reuses.order_by(sort).skip(offset).limit(args["page_size"]), reuses.count()
 
     @classmethod
-    def serialize(cls, reuse):
+    def serialize(cls, reuse: Reuse) -> dict:
         organization = None
         owner = None
         if reuse.organization:

udata/core/reuse/tasks.py

@@ -10,7 +10,7 @@ log = get_logger(__name__)
 
 
 @job("purge-reuses")
-def purge_reuses(self):
+def purge_reuses(self) -> None:
     for reuse in Reuse.objects(deleted__ne=None):
         log.info(f"Purging reuse {reuse}")
         # Remove followers
@@ -32,7 +32,7 @@ def purge_reuses(self):
 
 
 @task
-def notify_new_reuse(reuse_id):
+def notify_new_reuse(reuse_id: int) -> None:
     reuse = Reuse.objects.get(pk=reuse_id)
     for dataset in reuse.datasets:
         if dataset.organization:

udata/core/spatial/commands.py

@@ -177,8 +177,8 @@ def migrate():
 
     level_summary = "\n".join(
         [
-            " - {0}: {1}".format(l.id, counter[l.id])
-            for l in GeoLevel.objects.order_by("admin_level")
+            " - {0}: {1}".format(geolevel.id, counter[geolevel.id])
+            for geolevel in GeoLevel.objects.order_by("admin_level")
         ]
     )
     summary = "\n".join(
@@ -188,7 +188,7 @@ def migrate():
     Summary
     =======
     Processed {zones} zones in {datasets} datasets:\
-    """.format(level_summary, **counter)
+    """.format(**counter)
             ),
             level_summary,
         ]

udata/core/spatial/factories.py

@@ -2,7 +2,7 @@ import factory
 from faker.providers import BaseProvider
 from geojson.utils import generate_random
 
-from udata.factories import DateRangeFactory, ModelFactory
+from udata.factories import ModelFactory
 from udata.utils import faker_provider
 
 from . import geoids

udata/core/spatial/forms.py

@@ -62,7 +62,7 @@ class GeomField(Field):
                     self.data = geojson.loads(value)
                 else:
                     self.data = geojson.GeoJSON.to_instance(value)
-            except:
+            except Exception:
                 self.data = None
                 log.exception("Unable to parse GeoJSON")
                 raise ValueError(self.gettext("Not a valid GeoJSON"))

udata/core/spatial/models.py

@@ -120,7 +120,7 @@ class GeoZone(WithMetrics, db.Document):
 @cache.memoize()
 def get_spatial_granularities(lang):
     with language(lang):
-        return [(l.id, _(l.name)) for l in GeoLevel.objects] + [
+        return [(geolevel.id, _(geolevel.name)) for geolevel in GeoLevel.objects] + [
             (id, str(label)) for id, label in BASE_GRANULARITIES
         ]
 
@@ -130,7 +130,7 @@ spatial_granularities = LocalProxy(lambda: get_spatial_granularities(get_locale(
 
 @cache.cached(timeout=50, key_prefix="admin_levels")
 def get_spatial_admin_levels():
-    return dict((l.id, l.admin_level) for l in GeoLevel.objects)
+    return dict((geolevel.id, geolevel.admin_level) for geolevel in GeoLevel.objects)
 
 
 admin_levels = LocalProxy(get_spatial_admin_levels)

udata/core/spatial/tests/test_models.py

@@ -2,7 +2,7 @@ from datetime import timedelta
 
 from udata.tests import DBTestMixin, TestCase
 
-from ..factories import GeoLevelFactory, GeoZoneFactory
+from ..factories import GeoZoneFactory
 from ..models import GeoZone, SpatialCoverage
 
 A_YEAR = timedelta(days=365)

udata/core/spatial/translations.py

@@ -1,5 +1,7 @@
 # Here to force translations by gettext
-_ = lambda s: s  # noqa: force translations
+def _(s):
+    return s  # noqa: force translations
+
 
 TRANSLATIONS = (
     _("French region"),

udata/core/user/api.py

@@ -4,7 +4,6 @@ from slugify import slugify
 from udata.api import API, api
 from udata.api.parsers import ModelApiParser
 from udata.auth import admin_permission
-from udata.core import storages
 from udata.core.dataset.api_fields import community_resource_fields, dataset_fields
 from udata.core.discussions.actions import discussions_for
 from udata.core.discussions.api import discussion_fields
@@ -321,9 +320,10 @@ class UserAPI(API):
         return "", 204
 
 
-from udata.core.contact_point.api import ContactPointApiParser
-from udata.core.contact_point.api_fields import contact_point_page_fields
-from udata.models import ContactPoint
+# These imports are not at the top of the file to avoid circular imports
+from udata.core.contact_point.api import ContactPointApiParser  # noqa
+from udata.core.contact_point.api_fields import contact_point_page_fields  # noqa
+from udata.models import ContactPoint  # noqa
 
 contact_point_parser = ContactPointApiParser()
 

udata/core/user/metrics.py

@@ -1,6 +1,6 @@
 from udata.core.followers.signals import on_follow, on_unfollow
 from udata.core.owned import Owned
-from udata.models import Dataset, Reuse, User, db
+from udata.models import Dataset, Reuse, User
 
 
 @Dataset.on_create.connect

udata/frontend/__init__.py

@@ -23,7 +23,7 @@ _template_hooks = {}
 
 
 @hook.app_template_global()
-def package_version(name):
+def package_version(name: str) -> str:
     return pkg_resources.get_distribution(name).version
 
 

udata/harvest/actions.py

@@ -177,7 +177,7 @@ def purge_jobs():
         bucket = current_app.config.get("HARVEST_GRAPHS_S3_BUCKET")
         if bucket is None:
             log.error(
-                f"Bucket isn't configured anymore, but jobs still exist with external filenames. Could not delete them."
+                "Bucket isn't configured anymore, but jobs still exist with external filenames. Could not delete them."
             )
             break
 

udata/harvest/backends/__init__.py

@@ -14,4 +14,4 @@ def get_all(app):
     return get_enabled("udata.harvesters", app)
 
 
-from .base import BaseBackend, HarvestFeature, HarvestFilter  # flake8: noqa
+from .base import BaseBackend, HarvestFeature, HarvestFilter  # noqa

udata/harvest/tasks.py

@@ -1,4 +1,3 @@
-from celery import chord
 from flask import current_app
 
 from udata.tasks import get_logger, job, task

udata/harvest/tests/factories.py

@@ -3,8 +3,6 @@ import pytest
 from factory.fuzzy import FuzzyChoice
 from flask.signals import Namespace
 
-from udata.core.dataset.factories import DatasetFactory
-from udata.core.dataset.models import Dataset
 from udata.factories import ModelFactory
 
 from .. import backends

udata/harvest/tests/test_base_backend.py

@@ -2,7 +2,6 @@ from datetime import datetime, timedelta
 from urllib.parse import urlparse
 
 import pytest
-from dateutil.parser import parse
 from voluptuous import Schema
 
 from udata.core.dataset import tasks

udata/harvest/tests/test_dcat_backend.py

@@ -4,7 +4,6 @@ import re
 import xml.etree.ElementTree as ET
 from datetime import date
 
-import boto3
 import pytest
 from flask import current_app
 
@@ -245,7 +244,7 @@ class DcatBackendTest:
 
         datasets = {d.harvest.dct_identifier: d for d in Dataset.objects}
 
-        assert datasets["1"].schema == None
+        assert datasets["1"].schema is None
         resources_by_title = {resource["title"]: resource for resource in datasets["1"].resources}
 
         # Schema with wrong version are considered as external. Maybe we could change this in the future
@@ -253,8 +252,8 @@ class DcatBackendTest:
             resources_by_title["Resource 1-2"].schema.url
             == "https://schema.data.gouv.fr/schemas/etalab/schema-irve-statique/1337.42.0/schema-statique.json"
         )
-        assert resources_by_title["Resource 1-2"].schema.name == None
-        assert resources_by_title["Resource 1-2"].schema.version == None
+        assert resources_by_title["Resource 1-2"].schema.name is None
+        assert resources_by_title["Resource 1-2"].schema.version is None
 
         assert datasets["2"].schema.name == "RGF93 / Lambert-93 (EPSG:2154)"
         assert (
@@ -266,17 +265,17 @@ class DcatBackendTest:
         # Unknown schema are kept as they were provided
         assert resources_by_title["Resource 2-1"].schema.name == "Example Schema"
         assert resources_by_title["Resource 2-1"].schema.url == "https://example.org/schema.json"
-        assert resources_by_title["Resource 2-1"].schema.version == None
+        assert resources_by_title["Resource 2-1"].schema.version is None
 
-        assert resources_by_title["Resource 2-2"].schema == None
+        assert resources_by_title["Resource 2-2"].schema is None
 
-        assert datasets["3"].schema == None
+        assert datasets["3"].schema is None
         resources_by_title = {resource["title"]: resource for resource in datasets["3"].resources}
 
         # If there is just the URL, and it matches a known schema inside the catalog, only set the name and the version
         # (discard the URL)
         assert resources_by_title["Resource 3-1"].schema.name == "etalab/schema-irve-statique"
-        assert resources_by_title["Resource 3-1"].schema.url == None
+        assert resources_by_title["Resource 3-1"].schema.url is None
         assert resources_by_title["Resource 3-1"].schema.version == "2.2.0"
 
         job = HarvestJob.objects.order_by("-id").first()
@@ -321,7 +320,7 @@ class DcatBackendTest:
 
         datasets = {d.harvest.dct_identifier: d for d in Dataset.objects}
 
-        assert datasets["1"].spatial == None
+        assert datasets["1"].spatial is None
         assert datasets["2"].spatial.geom == {
             "type": "MultiPolygon",
             "coordinates": [
@@ -330,7 +329,7 @@ class DcatBackendTest:
                 [[[159, -25.0], [159, -11], [212, -11], [212, -25.0], [159, -25.0]]],
             ],
         }
-        assert datasets["3"].spatial == None
+        assert datasets["3"].spatial is None
 
     @pytest.mark.options(SCHEMA_CATALOG_URL="https://example.com/schemas")
     def test_harvest_schemas(self, rmock):
@@ -345,7 +344,7 @@ class DcatBackendTest:
 
         datasets = {d.harvest.dct_identifier: d for d in Dataset.objects}
 
-        assert datasets["1"].schema == None
+        assert datasets["1"].schema is None
         resources_by_title = {resource["title"]: resource for resource in datasets["1"].resources}
 
         # Schema with wrong version are considered as external. Maybe we could change this in the future
@@ -353,8 +352,8 @@ class DcatBackendTest:
             resources_by_title["Resource 1-2"].schema.url
             == "https://schema.data.gouv.fr/schemas/etalab/schema-irve-statique/1337.42.0/schema-statique.json"
         )
-        assert resources_by_title["Resource 1-2"].schema.name == None
-        assert resources_by_title["Resource 1-2"].schema.version == None
+        assert resources_by_title["Resource 1-2"].schema.name is None
+        assert resources_by_title["Resource 1-2"].schema.version is None
 
         assert datasets["2"].schema.name == "RGF93 / Lambert-93 (EPSG:2154)"
         assert (
@@ -366,17 +365,17 @@ class DcatBackendTest:
         # Unknown schema are kept as they were provided
         assert resources_by_title["Resource 2-1"].schema.name == "Example Schema"
         assert resources_by_title["Resource 2-1"].schema.url == "https://example.org/schema.json"
-        assert resources_by_title["Resource 2-1"].schema.version == None
+        assert resources_by_title["Resource 2-1"].schema.version is None
 
-        assert resources_by_title["Resource 2-2"].schema == None
+        assert resources_by_title["Resource 2-2"].schema is None
 
-        assert datasets["3"].schema == None
+        assert datasets["3"].schema is None
         resources_by_title = {resource["title"]: resource for resource in datasets["3"].resources}
 
         # If there is just the URL, and it matches a known schema inside the catalog, only set the name and the version
         # (discard the URL)
         assert resources_by_title["Resource 3-1"].schema.name == "etalab/schema-irve-statique"
-        assert resources_by_title["Resource 3-1"].schema.url == None
+        assert resources_by_title["Resource 3-1"].schema.url is None
         assert resources_by_title["Resource 3-1"].schema.version == "2.2.0"
 
     def test_simple_nested_attributes(self, rmock):

udata/migrations/2020-07-24-remove-s-from-scope-oauth.py

@@ -18,7 +18,7 @@ def migrate(db):
     oauth_clients = db.oauth2_client
     oauth_clients.update_many({}, {"$rename": {"scopes": "scope"}})
     for client in oauth_clients.find():
-        if type(client["scope"]) == list:
+        if type(client["scope"]) is list:
             scope_str = " ".join(client["scope"])
             client["scope"] = scope_str
             oauth_clients.save(client)

udata/migrations/2021-07-05-remove-unused-badges.py

@@ -4,7 +4,6 @@ The purpose here is to update every resource's metadata 'schema' name.
 
 import logging
 
-from udata.core.reuse.models import Reuse
 from udata.models import Dataset, Reuse
 
 log = logging.getLogger(__name__)

udata/migrations/2023-02-08-rename-internal-dates.py

@@ -6,8 +6,6 @@ import logging
 
 from mongoengine.connection import get_db
 
-from udata.models import Dataset
-
 log = logging.getLogger(__name__)
 
 

udata/migrations/2024-06-11-fix-reuse-datasets-references.py

@@ -4,7 +4,6 @@ Add a default topic to all reuses in db
 
 import logging
 
-import mongoengine
 from bson import DBRef
 
 from udata.models import Reuse

udata/models/__init__.py

@@ -22,6 +22,7 @@ from udata.core.jobs.models import *  # noqa
 from udata.core.tags.models import *  # noqa
 from udata.core.spam.models import *  # noqa
 from udata.core.reports.models import *  # noqa
+from udata.core.dataservices.models import *  # noqa
 
 from udata.features.transfer.models import *  # noqa
 from udata.features.territories.models import *  # noqa

udata/routing.py

@@ -153,6 +153,10 @@ class ContactPointConverter(ModelConverter):
     model = models.ContactPoint
 
 
+class ReportConverter(ModelConverter):
+    model = models.Report
+
+
 class TerritoryConverter(PathConverter):
     DEFAULT_PREFIX = "fr"  # TODO: make it a setting parameter
 
@@ -231,3 +235,4 @@ def init_app(app):
     app.url_map.converters["post"] = PostConverter
     app.url_map.converters["territory"] = TerritoryConverter
     app.url_map.converters["contact_point"] = ContactPointConverter
+    app.url_map.converters["report"] = ReportConverter

udata/search/commands.py

@@ -93,7 +93,7 @@ def finalize_reindex(models, start):
         r = requests.post(url, json=payload)
         r.raise_for_status()
     except Exception:
-        log.exception(f"Unable to set alias for index")
+        log.exception("Unable to set alias for index")
 
     modified_since_reindex = 0
     for adapter in iter_adapters():