udata 9.1.2.dev30355__py2.py3-none-any.whl → 9.1.2.dev30454__py2.py3-none-any.whl

udata/core/dataset/models.py

@@ -1,50 +1,70 @@
 import logging
-
 from datetime import datetime, timedelta
+from pydoc import locate
 from urllib.parse import urlparse
 
+import requests
 from blinker import signal
 from dateutil.parser import parse as parse_dt
 from flask import current_app
-from mongoengine import DynamicEmbeddedDocument, ValidationError as MongoEngineValidationError
-from mongoengine.signals import pre_save, post_save
+from mongoengine import DynamicEmbeddedDocument
+from mongoengine import ValidationError as MongoEngineValidationError
 from mongoengine.fields import DateTimeField
-from pydoc import locate
+from mongoengine.signals import post_save, pre_save
 from stringdist import rdlevenshtein
 from werkzeug.utils import cached_property
-import requests
 
 from udata.app import cache
 from udata.core import storages
+from udata.core.owned import Owned, OwnedQuerySet
 from udata.frontend.markdown import mdstrip
-from udata.models import db, WithMetrics, BadgeMixin, SpatialCoverage
-from udata.mongo.errors import FieldValidationError
 from udata.i18n import lazy_gettext as _
-from udata.utils import get_by, hash_url, to_naive_datetime
+from udata.models import BadgeMixin, SpatialCoverage, WithMetrics, db
+from udata.mongo.errors import FieldValidationError
 from udata.uris import ValidationError, endpoint_for
 from udata.uris import validate as validate_url
-from udata.core.owned import Owned, OwnedQuerySet
-from .constants import CHECKSUM_TYPES, CLOSED_FORMATS, DEFAULT_LICENSE, LEGACY_FREQUENCIES, MAX_DISTANCE, PIVOTAL_DATA, RESOURCE_FILETYPES, RESOURCE_TYPES, SCHEMA_CACHE_DURATION, UPDATE_FREQUENCIES
+from udata.utils import get_by, hash_url, to_naive_datetime
 
-from .preview import get_preview_url
+from .constants import (
+    CHECKSUM_TYPES,
+    CLOSED_FORMATS,
+    DEFAULT_LICENSE,
+    LEGACY_FREQUENCIES,
+    MAX_DISTANCE,
+    PIVOTAL_DATA,
+    RESOURCE_FILETYPES,
+    RESOURCE_TYPES,
+    SCHEMA_CACHE_DURATION,
+    UPDATE_FREQUENCIES,
+)
 from .exceptions import (
-    SchemasCatalogNotFoundException, SchemasCacheUnavailableException
+    SchemasCacheUnavailableException,
+    SchemasCatalogNotFoundException,
 )
+from .preview import get_preview_url
 
-__all__ = ('License', 'Resource', 'Schema', 'Dataset', 'Checksum', 'CommunityResource', 'ResourceSchema')
+__all__ = (
+    "License",
+    "Resource",
+    "Schema",
+    "Dataset",
+    "Checksum",
+    "CommunityResource",
+    "ResourceSchema",
+)
 
-NON_ASSIGNABLE_SCHEMA_TYPES = ['datapackage']
+NON_ASSIGNABLE_SCHEMA_TYPES = ["datapackage"]
 
 log = logging.getLogger(__name__)
 
 
 def get_json_ld_extra(key, value):
-    '''Serialize an extras key, value pair into JSON-LD'''
-    value = value.serialize() if hasattr(value, 'serialize') else value
+    """Serialize an extras key, value pair into JSON-LD"""
+    value = value.serialize() if hasattr(value, "serialize") else value
     return {
-        '@type': 'http://schema.org/PropertyValue',
-        'name': key,
-        'value': value,
+        "@type": "http://schema.org/PropertyValue",
+        "name": key,
+        "value": value,
     }
 
 
@@ -75,6 +95,7 @@ class Schema(db.EmbeddedDocument):
     - Known schema: url is not set, name is set, version is maybe set
     - Unknown schema: url is set, name and version are maybe set
     """
+
     url = db.URLField()
     name = db.StringField()
     version = db.StringField()
@@ -91,20 +112,23 @@ class Schema(db.EmbeddedDocument):
 
     def to_dict(self):
         return {
-            'url': self.url,
-            'name': self.name,
-            'version': self.version,
+            "url": self.url,
+            "name": self.name,
+            "version": self.version,
         }
 
     def clean(self, **kwargs):
         super().clean()
 
-        check_schema_in_catalog = kwargs.get('check_schema_in_catalog', False)
+        check_schema_in_catalog = kwargs.get("check_schema_in_catalog", False)
 
         if not self.url and not self.name:
             # There is no schema.
             if self.version:
-                raise FieldValidationError(_('A schema must contains a name or an URL when a version is provided.'), field='version')
+                raise FieldValidationError(
+                    _("A schema must contains a name or an URL when a version is provided."),
+                    field="version",
+                )
 
             return
 
@@ -115,11 +139,11 @@ class Schema(db.EmbeddedDocument):
                 self.url = None
                 self.name = info[0]
                 self.version = info[1]
-            
+
             # Nothing more to do since an URL can point to anywhere and have a random name/version
             return
 
-        # All the following checks are only run if there is 
+        # All the following checks are only run if there is
         # some schemas in the catalog. If there is no catalog
         # or no schema in the catalog we do not check the validity
         # of the name and version
@@ -128,31 +152,35 @@ class Schema(db.EmbeddedDocument):
             return
 
         # We know this schema so we can do some checks
-        existing_schema = next((schema for schema in catalog_schemas if schema['name'] == self.name), None)
+        existing_schema = next(
+            (schema for schema in catalog_schemas if schema["name"] == self.name), None
+        )
 
         if not existing_schema:
-            message = _('Schema name "{schema}" is not an allowed value. Allowed values: {values}').format(
+            message = _(
+                'Schema name "{schema}" is not an allowed value. Allowed values: {values}'
+            ).format(
                 schema=self.name,
-                values=', '.join(map(lambda schema: schema['name'], catalog_schemas))
+                values=", ".join(map(lambda schema: schema["name"], catalog_schemas)),
             )
             if check_schema_in_catalog:
-                raise FieldValidationError(message, field='name')
+                raise FieldValidationError(message, field="name")
             else:
                 log.warning(message)
                 return
 
         if self.version:
-            allowed_versions = list(map(lambda version: version['version_name'], existing_schema['versions']))
-            allowed_versions.append('latest')
+            allowed_versions = list(
+                map(lambda version: version["version_name"], existing_schema["versions"])
+            )
+            allowed_versions.append("latest")
 
             if self.version not in allowed_versions:
-                message = _('Version "{version}" is not an allowed value for the schema "{name}". Allowed versions: {values}').format(
-                    version=self.version,
-                    name=self.name,
-                    values=', '.join(allowed_versions)
-                )
+                message = _(
+                    'Version "{version}" is not an allowed value for the schema "{name}". Allowed versions: {values}'
+                ).format(version=self.version, name=self.name, values=", ".join(allowed_versions))
                 if check_schema_in_catalog:
-                    raise FieldValidationError(message, field='version')
+                    raise FieldValidationError(message, field="version")
                 else:
                     log.warning(message)
                     return
@@ -165,7 +193,7 @@ class License(db.Document):
     created_at = db.DateTimeField(default=datetime.utcnow, required=True)
     title = db.StringField(required=True)
     alternate_titles = db.ListField(db.StringField())
-    slug = db.SlugField(required=True, populate_from='title')
+    slug = db.SlugField(required=True, populate_from="title")
     url = db.URLField()
     alternate_urls = db.ListField(db.URLField())
     maintainer = db.StringField()
@@ -178,34 +206,36 @@ class License(db.Document):
 
     @classmethod
     def guess(cls, *strings, **kwargs):
-        '''
+        """
         Try to guess a license from a list of strings.
 
         Accept a `default` keyword argument which will be
         the default fallback license.
-        '''
+        """
         license = None
         for string in strings:
             license = cls.guess_one(string)
             if license:
                 break
-        return license or kwargs.get('default')
+        return license or kwargs.get("default")
 
     @classmethod
     def guess_one(cls, text):
-        '''
+        """
         Try to guess license from a string.
 
         Try to exact match on identifier then slugified title
         and fallback on edit distance ranking (after slugification)
-        '''
+        """
         if not text:
             return
         qs = cls.objects
         text = text.strip().lower()  # Stored identifiers are lower case
         slug = cls.slug.slugify(text)  # Use slug as it normalize string
         license = qs(
-            db.Q(id__iexact=text) | db.Q(slug=slug) | db.Q(url__iexact=text)
+            db.Q(id__iexact=text)
+            | db.Q(slug=slug)
+            | db.Q(url__iexact=text)
             | db.Q(alternate_urls__iexact=text)
         ).first()
 
@@ -218,9 +248,11 @@ class License(db.Document):
                 pass
             else:
                 parsed = urlparse(url)
-                path = parsed.path.rstrip('/')
-                query = f'{parsed.netloc}{path}'
-                license = qs(db.Q(url__icontains=query) | db.Q(alternate_urls__contains=query)).first()
+                path = parsed.path.rstrip("/")
+                query = f"{parsed.netloc}{path}"
+                license = qs(
+                    db.Q(url__icontains=query) | db.Q(alternate_urls__contains=query)
+                ).first()
 
         if license is None:
             # Try to single match `slug` with a low Damerau-Levenshtein distance
@@ -256,7 +288,7 @@ class License(db.Document):
 
     @classmethod
     def default(cls):
-        return cls.objects(id=DEFAULT_LICENSE['id']).first()
+        return cls.objects(id=DEFAULT_LICENSE["id"]).first()
 
 
 class DatasetQuerySet(OwnedQuerySet):
@@ -264,9 +296,7 @@ class DatasetQuerySet(OwnedQuerySet):
         return self(private__ne=True, deleted=None, archived=None)
 
     def hidden(self):
-        return self(db.Q(private=True) |
-                    db.Q(deleted__ne=None) |
-                    db.Q(archived__ne=None))
+        return self(db.Q(private=True) | db.Q(deleted__ne=None) | db.Q(archived__ne=None))
 
 
 class Checksum(db.EmbeddedDocument):
@@ -282,10 +312,8 @@ class ResourceMixin(object):
     id = db.AutoUUIDField(primary_key=True)
     title = db.StringField(verbose_name="Title", required=True)
     description = db.StringField()
-    filetype = db.StringField(
-        choices=list(RESOURCE_FILETYPES), default='file', required=True)
-    type = db.StringField(
-        choices=list(RESOURCE_TYPES), default='main', required=True)
+    filetype = db.StringField(choices=list(RESOURCE_FILETYPES), default="file", required=True)
+    type = db.StringField(choices=list(RESOURCE_TYPES), default="main", required=True)
     url = db.URLField(required=True)
     urlhash = db.StringField()
     checksum = db.EmbeddedDocumentField(Checksum)
@@ -296,7 +324,7 @@ class ResourceMixin(object):
     extras = db.ExtrasField()
     harvest = db.EmbeddedDocumentField(HarvestResourceMetadata)
     schema = db.EmbeddedDocumentField(Schema)
-    
+
     created_at_internal = db.DateTimeField(default=datetime.utcnow, required=True)
     last_modified_internal = db.DateTimeField(default=datetime.utcnow, required=True)
     deleted = db.DateTimeField()
@@ -304,25 +332,33 @@ class ResourceMixin(object):
     @property
     def internal(self):
         return {
-            'created_at_internal': self.created_at_internal,
-            'last_modified_internal': self.last_modified_internal
+            "created_at_internal": self.created_at_internal,
+            "last_modified_internal": self.last_modified_internal,
         }
 
     @property
     def created_at(self):
-        return self.harvest.created_at if self.harvest and self.harvest.created_at else self.created_at_internal
+        return (
+            self.harvest.created_at
+            if self.harvest and self.harvest.created_at
+            else self.created_at_internal
+        )
 
     @property
     def last_modified(self):
-        if self.harvest and self.harvest.modified_at and to_naive_datetime(self.harvest.modified_at) < datetime.utcnow():
+        if (
+            self.harvest
+            and self.harvest.modified_at
+            and to_naive_datetime(self.harvest.modified_at) < datetime.utcnow()
+        ):
             return to_naive_datetime(self.harvest.modified_at)
-        if self.filetype == 'remote' and self.extras.get('analysis:last-modified-at'):
-            return to_naive_datetime(self.extras.get('analysis:last-modified-at'))
+        if self.filetype == "remote" and self.extras.get("analysis:last-modified-at"):
+            return to_naive_datetime(self.extras.get("analysis:last-modified-at"))
         return self.last_modified_internal
 
     def clean(self):
         super(ResourceMixin, self).clean()
-        if not self.urlhash or 'url' in self._get_changed_fields():
+        if not self.urlhash or "url" in self._get_changed_fields():
             self.urlhash = hash_url(self.url)
 
     @cached_property  # Accessed at least 2 times in front rendering
@@ -338,40 +374,40 @@ class ResourceMixin(object):
         return not self.format or self.format.lower() in CLOSED_FORMATS
 
     def check_availability(self):
-        '''
+        """
         Return the check status from extras if any.
 
         NB: `unknown` will evaluate to True in the aggregate checks using
         `all([])` (dataset, organization, user).
-        '''
-        return self.extras.get('check:available', 'unknown')
+        """
+        return self.extras.get("check:available", "unknown")
 
     def need_check(self):
-        '''Does the resource needs to be checked against its linkchecker?
+        """Does the resource needs to be checked against its linkchecker?
 
         We check unavailable resources often, unless they go over the
         threshold. Available resources are checked less and less frequently
         based on their historical availability.
-        '''
+        """
         min_cache_duration, max_cache_duration, ko_threshold = [
-            current_app.config.get(k) for k in (
-                'LINKCHECKING_MIN_CACHE_DURATION',
-                'LINKCHECKING_MAX_CACHE_DURATION',
-                'LINKCHECKING_UNAVAILABLE_THRESHOLD',
+            current_app.config.get(k)
+            for k in (
+                "LINKCHECKING_MIN_CACHE_DURATION",
+                "LINKCHECKING_MAX_CACHE_DURATION",
+                "LINKCHECKING_UNAVAILABLE_THRESHOLD",
             )
         ]
-        count_availability = self.extras.get('check:count-availability', 1)
+        count_availability = self.extras.get("check:count-availability", 1)
         is_available = self.check_availability()
-        if is_available == 'unknown':
+        if is_available == "unknown":
             return True
         elif is_available or count_availability > ko_threshold:
-            delta = min(min_cache_duration * count_availability,
-                        max_cache_duration)
+            delta = min(min_cache_duration * count_availability, max_cache_duration)
         else:
             delta = min_cache_duration
-        if self.extras.get('check:date'):
+        if self.extras.get("check:date"):
             limit_date = datetime.utcnow() - timedelta(minutes=delta)
-            check_date = self.extras['check:date']
+            check_date = self.extras["check:date"]
             if not isinstance(check_date, datetime):
                 try:
                     check_date = parse_dt(check_date)
@@ -383,62 +419,63 @@ class ResourceMixin(object):
 
     @property
     def latest(self):
-        '''
+        """
         Permanent link to the latest version of this resource.
 
         If this resource is updated and `url` changes, this property won't.
-        '''
-        return endpoint_for('datasets.resource', 'api.resource_redirect', id=self.id, _external=True)
+        """
+        return endpoint_for(
+            "datasets.resource", "api.resource_redirect", id=self.id, _external=True
+        )
 
     @cached_property
     def json_ld(self):
-
         result = {
-            '@type': 'DataDownload',
-            '@id': str(self.id),
-            'url': self.latest,
-            'name': self.title or _('Nameless resource'),
-            'contentUrl': self.url,
-            'dateCreated': self.created_at.isoformat(),
-            'dateModified': self.last_modified.isoformat(),
-            'extras': [get_json_ld_extra(*item)
-                       for item in self.extras.items()],
+            "@type": "DataDownload",
+            "@id": str(self.id),
+            "url": self.latest,
+            "name": self.title or _("Nameless resource"),
+            "contentUrl": self.url,
+            "dateCreated": self.created_at.isoformat(),
+            "dateModified": self.last_modified.isoformat(),
+            "extras": [get_json_ld_extra(*item) for item in self.extras.items()],
         }
 
-        if 'views' in self.metrics:
-            result['interactionStatistic'] = {
-                '@type': 'InteractionCounter',
-                'interactionType': {
-                    '@type': 'DownloadAction',
+        if "views" in self.metrics:
+            result["interactionStatistic"] = {
+                "@type": "InteractionCounter",
+                "interactionType": {
+                    "@type": "DownloadAction",
                 },
-                'userInteractionCount': self.metrics['views']
+                "userInteractionCount": self.metrics["views"],
             }
 
         if self.format:
-            result['encodingFormat'] = self.format
+            result["encodingFormat"] = self.format
 
         if self.filesize:
-            result['contentSize'] = self.filesize
+            result["contentSize"] = self.filesize
 
         if self.mime:
-            result['fileFormat'] = self.mime
+            result["fileFormat"] = self.mime
 
         if self.description:
-            result['description'] = mdstrip(self.description)
+            result["description"] = mdstrip(self.description)
 
         return result
 
 
 class Resource(ResourceMixin, WithMetrics, db.EmbeddedDocument):
-    '''
+    """
     Local file, remote file or API provided by the original provider of the
     dataset
-    '''
-    on_added = signal('Resource.on_added')
-    on_deleted = signal('Resource.on_deleted')
+    """
+
+    on_added = signal("Resource.on_added")
+    on_deleted = signal("Resource.on_deleted")
 
     __metrics_keys__ = [
-        'views',
+        "views",
     ]
 
     @property
@@ -447,13 +484,15 @@ class Resource(ResourceMixin, WithMetrics, db.EmbeddedDocument):
             self._instance.id  # try to access attr from parent instance
             return self._instance
         except ReferenceError:  # weakly-referenced object no longer exists
-            log.warning('Weakly referenced object for resource.dataset no longer exists, '
-                        'using a poor performance query instead.')
+            log.warning(
+                "Weakly referenced object for resource.dataset no longer exists, "
+                "using a poor performance query instead."
+            )
             return Dataset.objects(resources__id=self.id).first()
 
     def save(self, *args, **kwargs):
         if not self.dataset:
-            raise RuntimeError('Impossible to save an orphan resource')
+            raise RuntimeError("Impossible to save an orphan resource")
         self.dataset.save(*args, **kwargs)
 
 
@@ -462,17 +501,18 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
     acronym = db.StringField(max_length=128)
     # /!\ do not set directly the slug when creating or updating a dataset
     # this will break the search indexation
-    slug = db.SlugField(max_length=255, required=True, populate_from='title',
-                        update=True, follow=True)
-    description = db.StringField(required=True, default='')
-    license = db.ReferenceField('License')
+    slug = db.SlugField(
+        max_length=255, required=True, populate_from="title", update=True, follow=True
+    )
+    description = db.StringField(required=True, default="")
+    license = db.ReferenceField("License")
 
     tags = db.TagListField()
     resources = db.ListField(db.EmbeddedDocumentField(Resource))
 
     private = db.BooleanField(default=False)
     frequency = db.StringField(choices=list(UPDATE_FREQUENCIES.keys()))
-    frequency_date = db.DateTimeField(verbose_name=_('Future date of update'))
+    frequency_date = db.DateTimeField(verbose_name=_("Future date of update"))
     temporal_coverage = db.EmbeddedDocumentField(db.DateRange)
     spatial = db.EmbeddedDocumentField(SpatialCoverage)
     schema = db.EmbeddedDocumentField(Schema)
@@ -483,60 +523,63 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
 
     featured = db.BooleanField(required=True, default=False)
 
-    contact_point = db.ReferenceField('ContactPoint', reverse_delete_rule=db.NULLIFY)
+    contact_point = db.ReferenceField("ContactPoint", reverse_delete_rule=db.NULLIFY)
 
-    created_at_internal = DateTimeField(verbose_name=_('Creation date'),
-                                        default=datetime.utcnow, required=True)
-    last_modified_internal = DateTimeField(verbose_name=_('Last modification date'),
-                                           default=datetime.utcnow, required=True)
+    created_at_internal = DateTimeField(
+        verbose_name=_("Creation date"), default=datetime.utcnow, required=True
+    )
+    last_modified_internal = DateTimeField(
+        verbose_name=_("Last modification date"), default=datetime.utcnow, required=True
+    )
     deleted = db.DateTimeField()
     archived = db.DateTimeField()
 
     def __str__(self):
-        return self.title or ''
+        return self.title or ""
 
     __badges__ = {
-        PIVOTAL_DATA: _('Pivotal data'),
+        PIVOTAL_DATA: _("Pivotal data"),
     }
 
     __metrics_keys__ = [
-        'discussions',
-        'reuses',
-        'followers',
-        'views',
-        'resources_downloads',
+        "discussions",
+        "reuses",
+        "followers",
+        "views",
+        "resources_downloads",
     ]
 
     meta = {
-        'indexes': [
-            '$title',
-            'created_at_internal',
-            'last_modified_internal',
-            'metrics.reuses',
-            'metrics.followers',
-            'metrics.views',
-            'slug',
-            'resources.id',
-            'resources.urlhash',
-        ] + Owned.meta['indexes'],
-        'ordering': ['-created_at_internal'],
-        'queryset_class': DatasetQuerySet,
-        'auto_create_index_on_save': True
+        "indexes": [
+            "$title",
+            "created_at_internal",
+            "last_modified_internal",
+            "metrics.reuses",
+            "metrics.followers",
+            "metrics.views",
+            "slug",
+            "resources.id",
+            "resources.urlhash",
+        ]
+        + Owned.meta["indexes"],
+        "ordering": ["-created_at_internal"],
+        "queryset_class": DatasetQuerySet,
+        "auto_create_index_on_save": True,
     }
 
-    before_save = signal('Dataset.before_save')
-    after_save = signal('Dataset.after_save')
-    on_create = signal('Dataset.on_create')
-    on_update = signal('Dataset.on_update')
-    before_delete = signal('Dataset.before_delete')
-    after_delete = signal('Dataset.after_delete')
-    on_delete = signal('Dataset.on_delete')
-    on_archive = signal('Dataset.on_archive')
-    on_resource_added = signal('Dataset.on_resource_added')
-    on_resource_updated = signal('Dataset.on_resource_updated')
-    on_resource_removed = signal('Dataset.on_resource_removed')
+    before_save = signal("Dataset.before_save")
+    after_save = signal("Dataset.after_save")
+    on_create = signal("Dataset.on_create")
+    on_update = signal("Dataset.on_update")
+    before_delete = signal("Dataset.before_delete")
+    after_delete = signal("Dataset.after_delete")
+    on_delete = signal("Dataset.on_delete")
+    on_archive = signal("Dataset.on_archive")
+    on_resource_added = signal("Dataset.on_resource_added")
+    on_resource_updated = signal("Dataset.on_resource_updated")
+    on_resource_removed = signal("Dataset.on_resource_removed")
 
-    verbose_name = _('dataset')
+    verbose_name = _("dataset")
 
     @classmethod
     def pre_save(cls, sender, document, **kwargs):
@@ -544,10 +587,10 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
 
     @classmethod
     def post_save(cls, sender, document, **kwargs):
-        if 'post_save' in kwargs.get('ignores', []):
+        if "post_save" in kwargs.get("ignores", []):
             return
         cls.after_save.send(document)
-        if kwargs.get('created'):
+        if kwargs.get("created"):
             cls.on_create.send(document)
         else:
             cls.on_update.send(document)
@@ -560,32 +603,36 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
             self.frequency = LEGACY_FREQUENCIES[self.frequency]
 
         for key, value in self.extras.items():
-            if not key.startswith('custom:'):
+            if not key.startswith("custom:"):
                 continue
             if not self.organization:
                 raise MongoEngineValidationError(
-                    'Custom metadatas are only accessible to dataset owned by on organization.')
-            custom_meta = key.split(':')[1]
-            org_custom = self.organization.extras.get('custom', [])
+                    "Custom metadatas are only accessible to dataset owned by on organization."
+                )
+            custom_meta = key.split(":")[1]
+            org_custom = self.organization.extras.get("custom", [])
             custom_present = False
             for custom in org_custom:
-                if custom['title'] != custom_meta:
+                if custom["title"] != custom_meta:
                     continue
                 custom_present = True
-                if custom['type'] == 'choice':
-                    if value not in custom['choices']:
+                if custom["type"] == "choice":
+                    if value not in custom["choices"]:
                         raise MongoEngineValidationError(
-                            'Custom metadata choice is not defined by organization.')
+                            "Custom metadata choice is not defined by organization."
+                        )
                 else:
-                    if not isinstance(value, locate(custom['type'])):
+                    if not isinstance(value, locate(custom["type"])):
                         raise MongoEngineValidationError(
-                            'Custom metadata is not of the right type.')
+                            "Custom metadata is not of the right type."
+                        )
             if not custom_present:
                 raise MongoEngineValidationError(
-                    'Dataset\'s organization did not define the requested custom metadata.')
+                    "Dataset's organization did not define the requested custom metadata."
+                )
 
     def url_for(self, *args, **kwargs):
-        return endpoint_for('datasets.show', 'api.dataset', dataset=self, *args, **kwargs)
+        return endpoint_for("datasets.show", "api.dataset", dataset=self, *args, **kwargs)
 
     display_url = property(url_for)
 
@@ -601,7 +648,7 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
     def full_title(self):
         if not self.acronym:
             return self.title
-        return '{title} ({acronym})'.format(**self._data)
+        return "{title} ({acronym})".format(**self._data)
 
     @property
     def external_url(self):
@@ -616,8 +663,7 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
 
     @property
     def frequency_label(self):
-        return UPDATE_FREQUENCIES.get(self.frequency or 'unknown',
-                                      UPDATE_FREQUENCIES['unknown'])
+        return UPDATE_FREQUENCIES.get(self.frequency or "unknown", UPDATE_FREQUENCIES["unknown"])
 
     def check_availability(self):
         """Check if resources from that dataset are available.
@@ -625,21 +671,28 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
         Return a list of (boolean or 'unknown')
         """
         # Only check remote resources.
-        remote_resources = [resource
-                            for resource in self.resources
-                            if resource.filetype == 'remote']
+        remote_resources = [
+            resource for resource in self.resources if resource.filetype == "remote"
+        ]
         if not remote_resources:
             return []
         return [resource.check_availability() for resource in remote_resources]
 
     @property
     def created_at(self):
-        return self.harvest.created_at if self.harvest and self.harvest.created_at else self.created_at_internal
+        return (
+            self.harvest.created_at
+            if self.harvest and self.harvest.created_at
+            else self.created_at_internal
+        )
 
     @property
     def last_modified(self):
-        if (self.harvest and self.harvest.modified_at and
-                to_naive_datetime(self.harvest.modified_at) < datetime.utcnow()):
+        if (
+            self.harvest
+            and self.harvest.modified_at
+            and to_naive_datetime(self.harvest.modified_at) < datetime.utcnow()
+        ):
             return to_naive_datetime(self.harvest.modified_at)
         return self.last_modified_internal
 
@@ -669,27 +722,27 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
         every 8 hours, but is maximum 24 hours later.
         """
         delta = None
-        if self.frequency == 'hourly':
+        if self.frequency == "hourly":
             delta = timedelta(hours=1)
-        elif self.frequency in ['fourTimesADay', 'threeTimesADay', 'semidaily', 'daily']:
+        elif self.frequency in ["fourTimesADay", "threeTimesADay", "semidaily", "daily"]:
             delta = timedelta(days=1)
-        elif self.frequency in ['fourTimesAWeek', 'threeTimesAWeek', 'semiweekly', 'weekly']:
+        elif self.frequency in ["fourTimesAWeek", "threeTimesAWeek", "semiweekly", "weekly"]:
             delta = timedelta(weeks=1)
-        elif self.frequency == 'biweekly':
+        elif self.frequency == "biweekly":
             delta = timedelta(weeks=2)
-        elif self.frequency in ['threeTimesAMonth', 'semimonthly', 'monthly']:
+        elif self.frequency in ["threeTimesAMonth", "semimonthly", "monthly"]:
             delta = timedelta(days=31)
-        elif self.frequency == 'bimonthly':
+        elif self.frequency == "bimonthly":
             delta = timedelta(days=31 * 2)
-        elif self.frequency == 'quarterly':
+        elif self.frequency == "quarterly":
             delta = timedelta(days=365 / 4)
-        elif self.frequency in ['threeTimesAYear', 'semiannual', 'annual']:
+        elif self.frequency in ["threeTimesAYear", "semiannual", "annual"]:
             delta = timedelta(days=365)
-        elif self.frequency == 'biennial':
+        elif self.frequency == "biennial":
             delta = timedelta(days=365 * 2)
-        elif self.frequency == 'triennial':
+        elif self.frequency == "triennial":
             delta = timedelta(days=365 * 3)
-        elif self.frequency == 'quinquennial':
+        elif self.frequency == "quinquennial":
             delta = timedelta(days=365 * 5)
         if delta is None:
             return
@@ -711,48 +764,49 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
             # Quality is only relevant on saved Datasets
             return result
 
-        result['license'] = True if self.license else False
-        result['temporal_coverage'] = True if self.temporal_coverage else False
-        result['spatial'] = True if self.spatial else False
+        result["license"] = True if self.license else False
+        result["temporal_coverage"] = True if self.temporal_coverage else False
+        result["spatial"] = True if self.spatial else False
 
-        result['update_frequency'] = self.frequency and self.frequency != 'unknown'
+        result["update_frequency"] = self.frequency and self.frequency != "unknown"
         if self.next_update:
             # Allow for being one day late on update.
             # We may have up to one day delay due to harvesting for example
-            result['update_fulfilled_in_time'] = (
+            result["update_fulfilled_in_time"] = (
                 True if (self.next_update - datetime.utcnow()).days >= -1 else False
             )
-        elif self.frequency in ['continuous', 'irregular', 'punctual']:
+        elif self.frequency in ["continuous", "irregular", "punctual"]:
             # For these frequencies, we don't expect regular updates or can't quantify them.
             # Thus we consider the update_fulfilled_in_time quality criterion to be true.
-            result['update_fulfilled_in_time'] = True
+            result["update_fulfilled_in_time"] = True
 
-        result['dataset_description_quality'] = (
-            True if len(self.description) > current_app.config.get('QUALITY_DESCRIPTION_LENGTH')
+        result["dataset_description_quality"] = (
+            True
+            if len(self.description) > current_app.config.get("QUALITY_DESCRIPTION_LENGTH")
             else False
         )
 
         if self.resources:
-            result['has_resources'] = True
-            result['has_open_format'] = not all(
-                resource.closed_or_no_format for resource in self.resources)
-            result['all_resources_available'] = all(self.check_availability())
+            result["has_resources"] = True
+            result["has_open_format"] = not all(
+                resource.closed_or_no_format for resource in self.resources
+            )
+            result["all_resources_available"] = all(self.check_availability())
             resource_doc = False
             resource_desc = False
             for resource in self.resources:
-                if resource.type == 'documentation':
+                if resource.type == "documentation":
                     resource_doc = True
                 if resource.description:
                     resource_desc = True
-            result['resources_documentation'] = resource_doc or resource_desc
+            result["resources_documentation"] = resource_doc or resource_desc
 
-        result['score'] = self.compute_quality_score(result)
+        result["score"] = self.compute_quality_score(result)
         return result
-    
+
     @property
     def downloads(self):
-        return sum(resource.metrics.get('views', 0) for resource in self.resources)
-
+        return sum(resource.metrics.get("views", 0) for resource in self.resources)
 
     @staticmethod
     def normalize_score(score):
@@ -772,25 +826,25 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
         """
         score = 0
         UNIT = 1
-        if quality['license']:
+        if quality["license"]:
             score += UNIT
-        if quality['temporal_coverage']:
+        if quality["temporal_coverage"]:
             score += UNIT
-        if quality['spatial']:
+        if quality["spatial"]:
             score += UNIT
-        if quality['update_frequency']:
+        if quality["update_frequency"]:
             score += UNIT
-        if 'update_fulfilled_in_time' in quality:
-            if quality['update_fulfilled_in_time']:
+        if "update_fulfilled_in_time" in quality:
+            if quality["update_fulfilled_in_time"]:
                 score += UNIT
-        if quality['dataset_description_quality']:
+        if quality["dataset_description_quality"]:
             score += UNIT
-        if 'has_resources' in quality:
-            if quality['has_open_format']:
+        if "has_resources" in quality:
+            if quality["has_open_format"]:
                 score += UNIT
-            if quality['all_resources_available']:
+            if quality["all_resources_available"]:
                 score += UNIT
-            if quality['resources_documentation']:
+            if quality["resources_documentation"]:
                 score += UNIT
         return self.normalize_score(score)
 
@@ -800,25 +854,18 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
         return obj or cls.objects.get_or_404(id=id_or_slug)
 
     def add_resource(self, resource):
-        '''Perform an atomic prepend for a new resource'''
+        """Perform an atomic prepend for a new resource"""
         resource.validate()
-        self.update(__raw__={
-            '$push': {
-                'resources': {
-                    '$each': [resource.to_mongo()],
-                    '$position': 0
-                }
-            }
-        })
+        self.update(
+            __raw__={"$push": {"resources": {"$each": [resource.to_mongo()], "$position": 0}}}
+        )
         self.reload()
         self.on_resource_added.send(self.__class__, document=self, resource_id=resource.id)
 
     def update_resource(self, resource):
-        '''Perform an atomic update for an existing resource'''
+        """Perform an atomic update for an existing resource"""
         index = self.resources.index(resource)
-        data = {
-            'resources__{index}'.format(index=index): resource
-        }
+        data = {"resources__{index}".format(index=index): resource}
         self.update(**data)
         self.reload()
         self.on_resource_updated.send(self.__class__, document=self, resource_id=resource.id)
@@ -838,29 +885,26 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
     @cached_property
     def json_ld(self):
         result = {
-            '@context': 'http://schema.org',
-            '@type': 'Dataset',
-            '@id': str(self.id),
-            'alternateName': self.slug,
-            'dateCreated': self.created_at.isoformat(),
-            'dateModified': self.last_modified.isoformat(),
-            'url': endpoint_for('datasets.show', 'api.dataset', dataset=self, _external=True),
-            'name': self.title,
-            'keywords': ','.join(self.tags),
-            'distribution': [resource.json_ld for resource in self.resources],
+            "@context": "http://schema.org",
+            "@type": "Dataset",
+            "@id": str(self.id),
+            "alternateName": self.slug,
+            "dateCreated": self.created_at.isoformat(),
+            "dateModified": self.last_modified.isoformat(),
+            "url": endpoint_for("datasets.show", "api.dataset", dataset=self, _external=True),
+            "name": self.title,
+            "keywords": ",".join(self.tags),
+            "distribution": [resource.json_ld for resource in self.resources],
             # Theses values are not standard
-            'contributedDistribution': [
-                resource.json_ld for resource in self.community_resources
-            ],
-            'extras': [get_json_ld_extra(*item)
-                       for item in self.extras.items()],
+            "contributedDistribution": [resource.json_ld for resource in self.community_resources],
+            "extras": [get_json_ld_extra(*item) for item in self.extras.items()],
         }
 
         if self.description:
-            result['description'] = mdstrip(self.description)
+            result["description"] = mdstrip(self.description)
 
         if self.license and self.license.url:
-            result['license'] = self.license.url
+            result["license"] = self.license.url
 
         if self.organization:
             author = self.organization.json_ld
@@ -870,34 +914,37 @@ class Dataset(WithMetrics, BadgeMixin, Owned, db.Document):
             author = None
 
         if author:
-            result['author'] = author
+            result["author"] = author
 
         return result
 
     @property
     def internal(self):
         return {
-            'created_at_internal': self.created_at_internal,
-            'last_modified_internal': self.last_modified_internal
+            "created_at_internal": self.created_at_internal,
+            "last_modified_internal": self.last_modified_internal,
         }
 
     @property
     def views_count(self):
-        return self.metrics.get('views', 0)
+        return self.metrics.get("views", 0)
 
     def count_discussions(self):
         from udata.models import Discussion
-        self.metrics['discussions'] = Discussion.objects(subject=self, closed=None).count()
+
+        self.metrics["discussions"] = Discussion.objects(subject=self, closed=None).count()
         self.save()
 
     def count_reuses(self):
         from udata.models import Reuse
-        self.metrics['reuses'] = Reuse.objects(datasets=self).visible().count()
+
+        self.metrics["reuses"] = Reuse.objects(datasets=self).visible().count()
         self.save()
 
     def count_followers(self):
         from udata.models import Follow
-        self.metrics['followers'] = Follow.objects(until=None).followers(self).count()
+
+        self.metrics["followers"] = Follow.objects(until=None).followers(self).count()
         self.save()
 
 
@@ -906,84 +953,93 @@ post_save.connect(Dataset.post_save, sender=Dataset)
 
 
 class CommunityResource(ResourceMixin, WithMetrics, Owned, db.Document):
-    '''
+    """
     Local file, remote file or API added by the community of the users to the
     original dataset
-    '''
+    """
+
     dataset = db.ReferenceField(Dataset, reverse_delete_rule=db.NULLIFY)
 
     __metrics_keys__ = [
-        'views',
+        "views",
     ]
 
     meta = {
-        'ordering': ['-created_at_internal'],
-        'queryset_class': OwnedQuerySet,
+        "ordering": ["-created_at_internal"],
+        "queryset_class": OwnedQuerySet,
     }
 
     @property
     def from_community(self):
         return True
 
+
 class ResourceSchema(object):
     @staticmethod
     @cache.memoize(timeout=SCHEMA_CACHE_DURATION)
     def all():
-        '''
+        """
         Get a list of schemas from a schema catalog endpoint.
 
         This has a double layer of cache:
         - @cache.cached decorator w/ short lived cache for normal operations
         - a long terme cache w/o timeout to be able to always render some content
-        '''
-        endpoint = current_app.config.get('SCHEMA_CATALOG_URL')
+        """
+        endpoint = current_app.config.get("SCHEMA_CATALOG_URL")
         if endpoint is None:
             return []
 
-        cache_key = 'schema-catalog-objects'
+        cache_key = "schema-catalog-objects"
         try:
             response = requests.get(endpoint, timeout=5)
             # do not cache 404 and forward status code
             if response.status_code == 404:
-                raise SchemasCatalogNotFoundException(f'Schemas catalog does not exist at {endpoint}')
+                raise SchemasCatalogNotFoundException(
+                    f"Schemas catalog does not exist at {endpoint}"
+                )
             response.raise_for_status()
         except requests.exceptions.RequestException as e:
-            log.exception(f'Error while getting schema catalog from {endpoint}')
+            log.exception(f"Error while getting schema catalog from {endpoint}")
             schemas = cache.get(cache_key)
         else:
-            schemas = response.json().get('schemas', [])
+            schemas = response.json().get("schemas", [])
             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')
-            raise SchemasCacheUnavailableException('No content in cache for schema catalog')
+            log.error(f"No content found inc. from cache for schema catalog")
+            raise SchemasCacheUnavailableException("No content in cache for schema catalog")
 
         return schemas
-    
+
     def assignable_schemas():
-        return [s for s in ResourceSchema.all() if s.get('schema_type') not in NON_ASSIGNABLE_SCHEMA_TYPES]
+        return [
+            s
+            for s in ResourceSchema.all()
+            if s.get("schema_type") not in NON_ASSIGNABLE_SCHEMA_TYPES
+        ]
 
     def get_existing_schema_info_by_url(url: str) -> tuple[str, str | None] | None:
-        '''
+        """
         Returns the name and the version if exists
-        '''
+        """
         for schema in ResourceSchema.all():
-            for version in schema['versions']:
-                if version['schema_url'] == url:
-                    return schema['name'], version['version_name']
+            for version in schema["versions"]:
+                if version["schema_url"] == url:
+                    return schema["name"], version["version_name"]
 
-            if schema['schema_url'] == url:
+            if schema["schema_url"] == url:
                 # The main schema URL is often the 'latest' version but
                 # not sure if it's mandatory everywhere so set the version to
                 # None here.
-                return schema['name'], None
+                return schema["name"], None
 
         return None
-    
+
+
 def get_resource(id):
-    '''Fetch a resource given its UUID'''
+    """Fetch a resource given its UUID"""
     dataset = Dataset.objects(resources__id=id).first()
     if dataset:
-        return get_by(dataset.resources, 'id', id)
+        return get_by(dataset.resources, "id", id)
     else:
         return CommunityResource.objects(id=id).first()