[style] Format the code

manage.py

@@ -3,10 +3,7 @@ import os
 import sys
 
 if __name__ == "__main__":
-    os.environ.setdefault(
+    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "umap.settings")
-        "DJANGO_SETTINGS_MODULE",
-        "umap.settings"
-    )
 
     from django.core.management import execute_from_command_line
 

umap/admin.py

@@ -1,10 +1,14 @@
 from django.contrib.gis import admin
-from .models import Map, DataLayer, Pictogram, TileLayer, Licence
+
+from .models import DataLayer, Licence, Map, Pictogram, TileLayer
 
 
 class TileLayerAdmin(admin.ModelAdmin):
-    list_display = ('name', 'rank', )
+    list_display = (
-    list_editable = ('rank', )
+        "name",
+        "rank",
+    )
+    list_editable = ("rank",)
 
 
 class MapAdmin(admin.GISModelAdmin):

umap/autocomplete.py

@@ -1,11 +1,9 @@
+from agnocomplete.core import AgnocompleteModel
+from agnocomplete.register import register
 from django.conf import settings
 from django.contrib.auth import get_user_model
 
 
-from agnocomplete.register import register
-from agnocomplete.core import AgnocompleteModel
-
-
 @register
 class AutocompleteUser(AgnocompleteModel):
     model = get_user_model()
@@ -13,5 +11,5 @@ class AutocompleteUser(AgnocompleteModel):
 
     def item(self, current_item):
         data = super().item(current_item)
-        data['url'] = current_item.get_url()
+        data["url"] = current_item.get_url()
         return data

umap/bin/__init__.py

@@ -5,12 +5,9 @@ from django.core import management
 
 
 def main():
-    os.environ.setdefault(
+    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "umap.settings")
-        "DJANGO_SETTINGS_MODULE",
-        "umap.settings"
-    )
     management.execute_from_command_line()
 
 
-if __name__ == '__main__':
+if __name__ == "__main__":
     main()

umap/decorators.py

@@ -1,13 +1,12 @@
 from functools import wraps
 
-from django.urls import reverse_lazy
-from django.shortcuts import get_object_or_404
-from django.http import HttpResponseForbidden
 from django.conf import settings
+from django.http import HttpResponseForbidden
+from django.shortcuts import get_object_or_404
+from django.urls import reverse_lazy
 
-from .views import simple_json_response
 from .models import Map
-
+from .views import simple_json_response
 
 LOGIN_URL = getattr(settings, "LOGIN_URL", "login")
 LOGIN_URL = reverse_lazy(LOGIN_URL) if not LOGIN_URL.startswith("/") else LOGIN_URL

umap/forms.py

@@ -1,12 +1,12 @@
 from django import forms
-from django.contrib.gis.geos import Point
-from django.contrib.auth import get_user_model
-from django.utils.translation import gettext_lazy as _
-from django.template.defaultfilters import slugify
 from django.conf import settings
+from django.contrib.auth import get_user_model
+from django.contrib.gis.geos import Point
 from django.forms.utils import ErrorList
+from django.template.defaultfilters import slugify
+from django.utils.translation import gettext_lazy as _
 
-from .models import Map, DataLayer
+from .models import DataLayer, Map
 
 DEFAULT_LATITUDE = (
     settings.LEAFLET_LATITUDE if hasattr(settings, "LEAFLET_LATITUDE") else 51

umap/management/commands/generate_js_locale.py

@@ -1,24 +1,23 @@
 from pathlib import Path
 
-from django.core.management.base import BaseCommand
 from django.conf import settings
+from django.core.management.base import BaseCommand
 from django.template.loader import render_to_string
 from django.utils.translation import to_locale
 
-ROOT = Path(settings.PROJECT_DIR) / 'static/umap/locale/'
+ROOT = Path(settings.PROJECT_DIR) / "static/umap/locale/"
 
 
 class Command(BaseCommand):
-
     def handle(self, *args, **options):
-        self.verbosity = options['verbosity']
+        self.verbosity = options["verbosity"]
         for code, name in settings.LANGUAGES:
             code = to_locale(code)
             if self.verbosity > 0:
                 print("Processing", name)
-            path = ROOT / '{code}.json'.format(code=code)
+            path = ROOT / "{code}.json".format(code=code)
             if not path.exists():
-                print(path, 'does not exist.', 'Skipping')
+                print(path, "does not exist.", "Skipping")
             else:
                 with path.open(encoding="utf-8") as f:
                     if self.verbosity > 1:
@@ -26,12 +25,11 @@ class Command(BaseCommand):
                     self.render(code, f.read())
 
     def render(self, code, json):
-        path = ROOT / '{code}.js'.format(code=code)
+        path = ROOT / "{code}.js".format(code=code)
         with path.open("w", encoding="utf-8") as f:
-            content = render_to_string('umap/locale.js', {
+            content = render_to_string(
-                "locale": json,
+                "umap/locale.js", {"locale": json, "locale_code": code}
-                "locale_code": code
+            )
-            })
             if self.verbosity > 1:
                 print("Exporting to", path)
             f.write(content)

umap/management/commands/import_pictograms.py

@@ -59,7 +59,7 @@ class Command(BaseCommand):
                 else:
                     picto = Pictogram()
                     picto.name = name
-                if (path.name != self.path.name):  # Subfolders only
+                if path.name != self.path.name:  # Subfolders only
                     picto.category = path.name
                 picto.attribution = self.attribution
                 with (path / filename).open("rb") as f:

umap/managers.py

@@ -2,7 +2,9 @@ from django.db.models import Manager
 
 
 class PublicManager(Manager):
-
     def get_queryset(self):
-        return super(PublicManager, self).get_queryset().filter(
+        return (
-            share_status=self.model.PUBLIC)
+            super(PublicManager, self)
+            .get_queryset()
+            .filter(share_status=self.model.PUBLIC)
+        )

umap/middleware.py

@@ -5,13 +5,12 @@ from django.utils.translation import gettext as _
 
 
 def readonly_middleware(get_response):
-
     if not settings.UMAP_READONLY:
         raise MiddlewareNotUsed
 
     def middleware(request):
-        if request.method not in ['GET', 'OPTIONS']:
+        if request.method not in ["GET", "OPTIONS"]:
-            return HttpResponseForbidden(_('Site is readonly for maintenance'))
+            return HttpResponseForbidden(_("Site is readonly for maintenance"))
 
         return get_response(request)
 

umap/migrations/0001_initial.py

@@ -1,15 +1,15 @@
 # Generated by Django 2.0.5 on 2018-05-19 09:27
 
-from django.conf import settings
 import django.contrib.gis.db.models.fields
-from django.db import migrations, models
 import django.db.models.deletion
+from django.conf import settings
+from django.db import migrations, models
+
 import umap.fields
 import umap.models
 
 
 class Migration(migrations.Migration):
-
     initial = True
 
     dependencies = [
@@ -18,90 +18,238 @@ class Migration(migrations.Migration):
 
     operations = [
         migrations.CreateModel(
-            name='DataLayer',
+            name="DataLayer",
             fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('name', models.CharField(max_length=200, verbose_name='name')),
+                    "id",
-                ('description', models.TextField(blank=True, null=True, verbose_name='description')),
+                    models.AutoField(
-                ('geojson', models.FileField(blank=True, null=True, upload_to=umap.models.upload_to)),
+                        auto_created=True,
-                ('display_on_load', models.BooleanField(default=False, help_text='Display this layer on load.', verbose_name='display on load')),
+                        primary_key=True,
-                ('rank', models.SmallIntegerField(default=0)),
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                ("name", models.CharField(max_length=200, verbose_name="name")),
+                (
+                    "description",
+                    models.TextField(blank=True, null=True, verbose_name="description"),
+                ),
+                (
+                    "geojson",
+                    models.FileField(
+                        blank=True, null=True, upload_to=umap.models.upload_to
+                    ),
+                ),
+                (
+                    "display_on_load",
+                    models.BooleanField(
+                        default=False,
+                        help_text="Display this layer on load.",
+                        verbose_name="display on load",
+                    ),
+                ),
+                ("rank", models.SmallIntegerField(default=0)),
             ],
             options={
-                'ordering': ('rank',),
+                "ordering": ("rank",),
             },
         ),
         migrations.CreateModel(
-            name='Licence',
+            name="Licence",
             fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('name', models.CharField(max_length=200, verbose_name='name')),
+                    "id",
-                ('details', models.URLField(help_text='Link to a page where the licence is detailed.', verbose_name='details')),
+                    models.AutoField(
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                ("name", models.CharField(max_length=200, verbose_name="name")),
+                (
+                    "details",
+                    models.URLField(
+                        help_text="Link to a page where the licence is detailed.",
+                        verbose_name="details",
+                    ),
+                ),
             ],
             options={
-                'ordering': ('name',),
+                "ordering": ("name",),
-                'abstract': False,
+                "abstract": False,
             },
         ),
         migrations.CreateModel(
-            name='Map',
+            name="Map",
             fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('name', models.CharField(max_length=200, verbose_name='name')),
+                    "id",
-                ('slug', models.SlugField()),
+                    models.AutoField(
-                ('description', models.TextField(blank=True, null=True, verbose_name='description')),
+                        auto_created=True,
-                ('center', django.contrib.gis.db.models.fields.PointField(geography=True, srid=4326, verbose_name='center')),
+                        primary_key=True,
-                ('zoom', models.IntegerField(default=7, verbose_name='zoom')),
+                        serialize=False,
-                ('locate', models.BooleanField(default=False, help_text='Locate user on load?', verbose_name='locate')),
+                        verbose_name="ID",
-                ('modified_at', models.DateTimeField(auto_now=True)),
+                    ),
-                ('edit_status', models.SmallIntegerField(choices=[(1, 'Everyone can edit'), (2, 'Only editors can edit'), (3, 'Only owner can edit')], default=3, verbose_name='edit status')),
+                ),
-                ('share_status', models.SmallIntegerField(choices=[(1, 'everyone (public)'), (2, 'anyone with link'), (3, 'editors only')], default=1, verbose_name='share status')),
+                ("name", models.CharField(max_length=200, verbose_name="name")),
-                ('settings', umap.fields.DictField(blank=True, null=True, verbose_name='settings')),
+                ("slug", models.SlugField()),
-                ('editors', models.ManyToManyField(blank=True, to=settings.AUTH_USER_MODEL, verbose_name='editors')),
+                (
-                ('licence', models.ForeignKey(default=umap.models.get_default_licence, help_text='Choose the map licence.', on_delete=django.db.models.deletion.SET_DEFAULT, to='umap.Licence', verbose_name='licence')),
+                    "description",
-                ('owner', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, related_name='owned_maps', to=settings.AUTH_USER_MODEL, verbose_name='owner')),
+                    models.TextField(blank=True, null=True, verbose_name="description"),
+                ),
+                (
+                    "center",
+                    django.contrib.gis.db.models.fields.PointField(
+                        geography=True, srid=4326, verbose_name="center"
+                    ),
+                ),
+                ("zoom", models.IntegerField(default=7, verbose_name="zoom")),
+                (
+                    "locate",
+                    models.BooleanField(
+                        default=False,
+                        help_text="Locate user on load?",
+                        verbose_name="locate",
+                    ),
+                ),
+                ("modified_at", models.DateTimeField(auto_now=True)),
+                (
+                    "edit_status",
+                    models.SmallIntegerField(
+                        choices=[
+                            (1, "Everyone can edit"),
+                            (2, "Only editors can edit"),
+                            (3, "Only owner can edit"),
+                        ],
+                        default=3,
+                        verbose_name="edit status",
+                    ),
+                ),
+                (
+                    "share_status",
+                    models.SmallIntegerField(
+                        choices=[
+                            (1, "everyone (public)"),
+                            (2, "anyone with link"),
+                            (3, "editors only"),
+                        ],
+                        default=1,
+                        verbose_name="share status",
+                    ),
+                ),
+                (
+                    "settings",
+                    umap.fields.DictField(
+                        blank=True, null=True, verbose_name="settings"
+                    ),
+                ),
+                (
+                    "editors",
+                    models.ManyToManyField(
+                        blank=True, to=settings.AUTH_USER_MODEL, verbose_name="editors"
+                    ),
+                ),
+                (
+                    "licence",
+                    models.ForeignKey(
+                        default=umap.models.get_default_licence,
+                        help_text="Choose the map licence.",
+                        on_delete=django.db.models.deletion.SET_DEFAULT,
+                        to="umap.Licence",
+                        verbose_name="licence",
+                    ),
+                ),
+                (
+                    "owner",
+                    models.ForeignKey(
+                        blank=True,
+                        null=True,
+                        on_delete=django.db.models.deletion.PROTECT,
+                        related_name="owned_maps",
+                        to=settings.AUTH_USER_MODEL,
+                        verbose_name="owner",
+                    ),
+                ),
             ],
             options={
-                'ordering': ('name',),
+                "ordering": ("name",),
-                'abstract': False,
+                "abstract": False,
             },
         ),
         migrations.CreateModel(
-            name='Pictogram',
+            name="Pictogram",
             fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('name', models.CharField(max_length=200, verbose_name='name')),
+                    "id",
-                ('attribution', models.CharField(max_length=300)),
+                    models.AutoField(
-                ('pictogram', models.ImageField(upload_to='pictogram')),
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                ("name", models.CharField(max_length=200, verbose_name="name")),
+                ("attribution", models.CharField(max_length=300)),
+                ("pictogram", models.ImageField(upload_to="pictogram")),
             ],
             options={
-                'ordering': ('name',),
+                "ordering": ("name",),
-                'abstract': False,
+                "abstract": False,
             },
         ),
         migrations.CreateModel(
-            name='TileLayer',
+            name="TileLayer",
             fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('name', models.CharField(max_length=200, verbose_name='name')),
+                    "id",
-                ('url_template', models.CharField(help_text='URL template using OSM tile format', max_length=200)),
+                    models.AutoField(
-                ('minZoom', models.IntegerField(default=0)),
+                        auto_created=True,
-                ('maxZoom', models.IntegerField(default=18)),
+                        primary_key=True,
-                ('attribution', models.CharField(max_length=300)),
+                        serialize=False,
-                ('rank', models.SmallIntegerField(blank=True, help_text='Order of the tilelayers in the edit box', null=True)),
+                        verbose_name="ID",
+                    ),
+                ),
+                ("name", models.CharField(max_length=200, verbose_name="name")),
+                (
+                    "url_template",
+                    models.CharField(
+                        help_text="URL template using OSM tile format", max_length=200
+                    ),
+                ),
+                ("minZoom", models.IntegerField(default=0)),
+                ("maxZoom", models.IntegerField(default=18)),
+                ("attribution", models.CharField(max_length=300)),
+                (
+                    "rank",
+                    models.SmallIntegerField(
+                        blank=True,
+                        help_text="Order of the tilelayers in the edit box",
+                        null=True,
+                    ),
+                ),
             ],
             options={
-                'ordering': ('rank', 'name'),
+                "ordering": ("rank", "name"),
             },
         ),
         migrations.AddField(
-            model_name='map',
+            model_name="map",
-            name='tilelayer',
+            name="tilelayer",
-            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, related_name='maps', to='umap.TileLayer', verbose_name='background'),
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.PROTECT,
+                related_name="maps",
+                to="umap.TileLayer",
+                verbose_name="background",
+            ),
         ),
         migrations.AddField(
-            model_name='datalayer',
+            model_name="datalayer",
-            name='map',
+            name="map",
-            field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='umap.Map'),
+            field=models.ForeignKey(
+                on_delete=django.db.models.deletion.CASCADE, to="umap.Map"
+            ),
         ),
     ]

umap/migrations/0002_tilelayer_tms.py

@@ -4,15 +4,14 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0001_initial'),
+        ("umap", "0001_initial"),
     ]
 
     operations = [
         migrations.AddField(
-            model_name='tilelayer',
+            model_name="tilelayer",
-            name='tms',
+            name="tms",
             field=models.BooleanField(default=False),
         ),
     ]

umap/migrations/0003_add_tilelayer.py

@@ -5,22 +5,26 @@ from django.db import migrations
 
 
 def add_tilelayer(apps, *args):
-    TileLayer = apps.get_model('umap', 'TileLayer')
+    TileLayer = apps.get_model("umap", "TileLayer")
     if TileLayer.objects.count():
         return
     TileLayer(
-        name='Positron',
+        name="Positron",
-        url_template=('https://cartodb-basemaps-{s}.global.ssl.fastly.net/'
+        url_template=(
-                      'light_all/{z}/{x}/{y}.png'),
+            "https://cartodb-basemaps-{s}.global.ssl.fastly.net/"
-        attribution=('© [[http://www.openstreetmap.org/copyright|'
+            "light_all/{z}/{x}/{y}.png"
-                     'OpenStreetMap]] contributors, © '
+        ),
-                     '[[https://carto.com/attributions|CARTO]]')).save()
+        attribution=(
+            "© [[http://www.openstreetmap.org/copyright|"
+            "OpenStreetMap]] contributors, © "
+            "[[https://carto.com/attributions|CARTO]]"
+        ),
+    ).save()
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0002_tilelayer_tms'),
+        ("umap", "0002_tilelayer_tms"),
     ]
 
     operations = [

umap/migrations/0004_add_licence.py

@@ -5,18 +5,15 @@ from django.db import migrations
 
 
 def add_licence(apps, *args):
-    Licence = apps.get_model('umap', 'Licence')
+    Licence = apps.get_model("umap", "Licence")
     if Licence.objects.count():
         return
-    Licence(
+    Licence(name="ODbL", details="http://opendatacommons.org/licenses/odbl/").save()
-        name='ODbL',
-        details='http://opendatacommons.org/licenses/odbl/').save()
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0003_add_tilelayer'),
+        ("umap", "0003_add_tilelayer"),
     ]
 
     operations = [

umap/migrations/0005_remove_map_tilelayer.py

@@ -4,14 +4,13 @@ from django.db import migrations
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0004_add_licence'),
+        ("umap", "0004_add_licence"),
     ]
 
     operations = [
         migrations.RemoveField(
-            model_name='map',
+            model_name="map",
-            name='tilelayer',
+            name="tilelayer",
         ),
     ]

umap/migrations/0006_auto_20190407_0719.py

@@ -5,15 +5,16 @@ from django.db import migrations
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0005_remove_map_tilelayer'),
+        ("umap", "0005_remove_map_tilelayer"),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name='map',
+            model_name="map",
-            name='settings',
+            name="settings",
-            field=django.contrib.postgres.fields.jsonb.JSONField(blank=True, default=dict, null=True, verbose_name='settings'),
+            field=django.contrib.postgres.fields.jsonb.JSONField(
+                blank=True, default=dict, null=True, verbose_name="settings"
+            ),
         ),
     ]

umap/migrations/0007_auto_20190416_1757.py

@@ -4,15 +4,23 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0006_auto_20190407_0719'),
+        ("umap", "0006_auto_20190407_0719"),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name='map',
+            model_name="map",
-            name='share_status',
+            name="share_status",
-            field=models.SmallIntegerField(choices=[(1, 'everyone (public)'), (2, 'anyone with link'), (3, 'editors only'), (9, 'blocked')], default=1, verbose_name='share status'),
+            field=models.SmallIntegerField(
+                choices=[
+                    (1, "everyone (public)"),
+                    (2, "anyone with link"),
+                    (3, "editors only"),
+                    (9, "blocked"),
+                ],
+                default=1,
+                verbose_name="share status",
+            ),
         ),
     ]

umap/migrations/0008_alter_map_settings.py

@@ -4,7 +4,6 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
         ("umap", "0007_auto_20190416_1757"),
     ]

umap/migrations/0009_star.py

@@ -1,25 +1,44 @@
 # Generated by Django 4.1.7 on 2023-05-05 18:02
 
+import django.db.models.deletion
 from django.conf import settings
 from django.db import migrations, models
-import django.db.models.deletion
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
         migrations.swappable_dependency(settings.AUTH_USER_MODEL),
-        ('umap', '0008_alter_map_settings'),
+        ("umap", "0008_alter_map_settings"),
     ]
 
     operations = [
         migrations.CreateModel(
-            name='Star',
+            name="Star",
             fields=[
-                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                (
-                ('at', models.DateTimeField(auto_now=True)),
+                    "id",
-                ('by', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, related_name='stars', to=settings.AUTH_USER_MODEL)),
+                    models.AutoField(
-                ('map', models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='umap.map')),
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                ("at", models.DateTimeField(auto_now=True)),
+                (
+                    "by",
+                    models.ForeignKey(
+                        on_delete=django.db.models.deletion.CASCADE,
+                        related_name="stars",
+                        to=settings.AUTH_USER_MODEL,
+                    ),
+                ),
+                (
+                    "map",
+                    models.ForeignKey(
+                        on_delete=django.db.models.deletion.CASCADE, to="umap.map"
+                    ),
+                ),
             ],
         ),
     ]

umap/migrations/0010_alter_map_edit_status_alter_map_share_status.py

@@ -4,20 +4,32 @@ from django.db import migrations, models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0009_star'),
+        ("umap", "0009_star"),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name='map',
+            model_name="map",
-            name='edit_status',
+            name="edit_status",
-            field=models.SmallIntegerField(choices=[(1, 'Everyone'), (2, 'Editors only'), (3, 'Owner only')], default=3, verbose_name='edit status'),
+            field=models.SmallIntegerField(
+                choices=[(1, "Everyone"), (2, "Editors only"), (3, "Owner only")],
+                default=3,
+                verbose_name="edit status",
+            ),
         ),
         migrations.AlterField(
-            model_name='map',
+            model_name="map",
-            name='share_status',
+            name="share_status",
-            field=models.SmallIntegerField(choices=[(1, 'Everyone (public)'), (2, 'Anyone with link'), (3, 'Editors only'), (9, 'Blocked')], default=1, verbose_name='share status'),
+            field=models.SmallIntegerField(
+                choices=[
+                    (1, "Everyone (public)"),
+                    (2, "Anyone with link"),
+                    (3, "Editors only"),
+                    (9, "Blocked"),
+                ],
+                default=1,
+                verbose_name="share status",
+            ),
         ),
     ]

umap/migrations/0011_alter_map_edit_status_alter_map_share_status.py

@@ -1,24 +1,37 @@
 # Generated by Django 4.2.2 on 2023-08-07 06:07
 
 from django.db import migrations, models
+
 import umap.models
 
 
 class Migration(migrations.Migration):
-
     dependencies = [
-        ('umap', '0010_alter_map_edit_status_alter_map_share_status'),
+        ("umap", "0010_alter_map_edit_status_alter_map_share_status"),
     ]
 
     operations = [
         migrations.AlterField(
-            model_name='map',
+            model_name="map",
-            name='edit_status',
+            name="edit_status",
-            field=models.SmallIntegerField(choices=[(1, 'Everyone'), (2, 'Editors only'), (3, 'Owner only')], default=umap.models.get_default_edit_status, verbose_name='edit status'),
+            field=models.SmallIntegerField(
+                choices=[(1, "Everyone"), (2, "Editors only"), (3, "Owner only")],
+                default=umap.models.get_default_edit_status,
+                verbose_name="edit status",
+            ),
         ),
         migrations.AlterField(
-            model_name='map',
+            model_name="map",
-            name='share_status',
+            name="share_status",
-            field=models.SmallIntegerField(choices=[(1, 'Everyone (public)'), (2, 'Anyone with link'), (3, 'Editors only'), (9, 'Blocked')], default=umap.models.get_default_share_status, verbose_name='share status'),
+            field=models.SmallIntegerField(
+                choices=[
+                    (1, "Everyone (public)"),
+                    (2, "Anyone with link"),
+                    (3, "Editors only"),
+                    (9, "Blocked"),
+                ],
+                default=umap.models.get_default_share_status,
+                verbose_name="share status",
+            ),
         ),
     ]

umap/migrations/0014_map_created_at.py

@@ -1,7 +1,7 @@
 # Generated by Django 4.2.2 on 2023-09-27 08:50
 
-from django.db import migrations, models
 import django.utils.timezone
+from django.db import migrations, models
 
 
 class Migration(migrations.Migration):

umap/models.py

@@ -1,14 +1,14 @@
 import os
 import time
 
+from django.conf import settings
 from django.contrib.auth.models import User
 from django.contrib.gis.db import models
-from django.conf import settings
+from django.core.files.base import File
-from django.urls import reverse
-from django.utils.translation import gettext_lazy as _
 from django.core.signing import Signer
 from django.template.defaultfilters import slugify
-from django.core.files.base import File
+from django.urls import reverse
+from django.utils.translation import gettext_lazy as _
 
 from .managers import PublicManager
 
@@ -380,7 +380,7 @@ class DataLayer(NamedModel):
         }
         obj["id"] = self.pk
         obj["permissions"] = {"edit_status": self.edit_status}
-        obj["editMode"] = "advanced" if self.can_edit(user, request) else 'disabled'
+        obj["editMode"] = "advanced" if self.can_edit(user, request) else "disabled"
         return obj
 
     def clone(self, map_inst=None):
@@ -436,7 +436,7 @@ class DataLayer(NamedModel):
         root = self.storage_root()
         names = self.geojson.storage.listdir(root)[1]
         for name in names:
-            if name.startswith(f'{self.pk}_') and name.endswith(".gz"):
+            if name.startswith(f"{self.pk}_") and name.endswith(".gz"):
                 self.geojson.storage.delete(os.path.join(root, name))
 
     def can_edit(self, user=None, request=None):

umap/settings/base.py

@@ -4,7 +4,6 @@ from email.utils import parseaddr
 
 import environ
 from django.conf.locale import LANG_INFO
-from django.template.defaultfilters import slugify
 
 env = environ.Env()
 
@@ -244,9 +243,7 @@ UMAP_MAPS_PER_PAGE = 5
 UMAP_MAPS_PER_SEARCH = 25
 UMAP_MAPS_PER_PAGE_OWNER = 10
 UMAP_SEARCH_CONFIGURATION = "simple"
-UMAP_FEEDBACK_LINK = (
+UMAP_FEEDBACK_LINK = "https://wiki.openstreetmap.org/wiki/UMap#Feedback_and_help"
-    "https://wiki.openstreetmap.org/wiki/UMap#Feedback_and_help"  # noqa
-)
 USER_MAPS_URL = "user_maps"
 DATABASES = {"default": env.db(default="postgis://localhost:5432/umap")}
 UMAP_DEFAULT_SHARE_STATUS = None

umap/templates/auth/user_form.html

@@ -3,7 +3,7 @@
 {% block maincontent %}
   <div class="col wide">
     <h2 class="section tabs">
-      <a href="{% url 'user_dashboard' %}">{% trans "My Dashboard" %}</a> | {% trans "My Profile" %}
+      <a href="{% url "user_dashboard" %}">{% trans "My Dashboard" %}</a> | {% trans "My Profile" %}
     </h2>
   </div>
   <div class="wrapper">
@@ -16,7 +16,7 @@
       <form id="user_form" method="post">
         {% csrf_token %}
         {{ form }}
-        <input type="submit" value="{% trans 'Save' %}" />
+        <input type="submit" value="{% trans "Save" %}" />
       </form>
     </div>
     {% if backends.backends|length %}

umap/templates/base.html

@@ -16,9 +16,15 @@
     <meta name="viewport"
           content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no" />
     {# See https://evilmartians.com/chronicles/how-to-favicon-in-2021-six-files-that-fit-most-needs #}
-    <link rel="icon" href="{{ STATIC_URL }}umap/favicons/favicon.ico" sizes="32x32">
+    <link rel="icon"
-    <link rel="icon" href="{{ STATIC_URL }}umap/favicons/icon.svg" type="image/svg+xml">
+          href="{{ STATIC_URL }}umap/favicons/favicon.ico"
-    <link rel="apple-touch-icon" href="{{ STATIC_URL }}umap/favicons/apple-touch-icon.png"><!-- 180×180 -->
+          sizes="32x32">
+    <link rel="icon"
+          href="{{ STATIC_URL }}umap/favicons/icon.svg"
+          type="image/svg+xml">
+    <link rel="apple-touch-icon"
+          href="{{ STATIC_URL }}umap/favicons/apple-touch-icon.png">
+    <!-- 180×180 -->
     <link rel="manifest" href="/manifest.webmanifest">
   </head>
   <body class="{% block body_class %}{% endblock body_class %}">

umap/templatetags/umap_tags.py

@@ -10,53 +10,50 @@ from ..views import _urls_for_js
 register = template.Library()
 
 
-@register.inclusion_tag('umap/css.html')
+@register.inclusion_tag("umap/css.html")
 def umap_css():
-    return {
+    return {"STATIC_URL": settings.STATIC_URL}
-        "STATIC_URL": settings.STATIC_URL
-    }
 
 
-@register.inclusion_tag('umap/js.html')
+@register.inclusion_tag("umap/js.html")
 def umap_js(locale=None):
-    return {
+    return {"STATIC_URL": settings.STATIC_URL, "locale": locale}
-        "STATIC_URL": settings.STATIC_URL,
-        "locale": locale
-    }
 
 
-@register.inclusion_tag('umap/map_fragment.html')
+@register.inclusion_tag("umap/map_fragment.html")
 def map_fragment(map_instance, **kwargs):
     layers = DataLayer.objects.filter(map=map_instance)
     datalayer_data = [c.metadata() for c in layers]
     map_settings = map_instance.settings
     if "properties" not in map_settings:
-        map_settings['properties'] = {}
+        map_settings["properties"] = {}
-    map_settings['properties'].update({
+    map_settings["properties"].update(
-        'tilelayers': [TileLayer.get_default().json],
+        {
-        'datalayers': datalayer_data,
+            "tilelayers": [TileLayer.get_default().json],
-        'urls': _urls_for_js(),
+            "datalayers": datalayer_data,
-        'STATIC_URL': settings.STATIC_URL,
+            "urls": _urls_for_js(),
-        "editMode": 'disabled',
+            "STATIC_URL": settings.STATIC_URL,
-        'hash': False,
+            "editMode": "disabled",
-        'attributionControl': False,
+            "hash": False,
-        'scrollWheelZoom': False,
+            "attributionControl": False,
-        'umapAttributionControl': False,
+            "scrollWheelZoom": False,
-        'noControl': True,
+            "umapAttributionControl": False,
-        'umap_id': map_instance.pk,
+            "noControl": True,
-        'onLoadPanel': "none",
+            "umap_id": map_instance.pk,
-        'captionBar': False,
+            "onLoadPanel": "none",
-        'default_iconUrl': "%sumap/img/marker.png" % settings.STATIC_URL,
+            "captionBar": False,
-        'slideshow': {}
+            "default_iconUrl": "%sumap/img/marker.png" % settings.STATIC_URL,
-    })
+            "slideshow": {},
-    map_settings['properties'].update(kwargs)
+        }
-    prefix = kwargs.pop('prefix', None) or 'map'
+    )
-    page = kwargs.pop('page', None) or ''
+    map_settings["properties"].update(kwargs)
+    prefix = kwargs.pop("prefix", None) or "map"
+    page = kwargs.pop("page", None) or ""
     unique_id = prefix + str(page) + "_" + str(map_instance.pk)
     return {
         "map_settings": json.dumps(map_settings),
         "map": map_instance,
-        "unique_id": unique_id
+        "unique_id": unique_id,
     }
 
 
@@ -73,7 +70,7 @@ def tilelayer_preview(tilelayer):
 
 @register.filter
 def notag(s):
-    return s.replace('<', '&lt;')
+    return s.replace("<", "&lt;")
 
 
 @register.simple_tag(takes_context=True)
@@ -86,5 +83,6 @@ def paginate_querystring(context, page):
 @register.filter
 def ipdb(what):
     import ipdb
+
     ipdb.set_trace()
-    return ''
+    return ""

umap/tests/base.py

@@ -1,5 +1,5 @@
-import json
 import copy
+import json
 
 import factory
 from django.contrib.auth import get_user_model

umap/tests/conftest.py

@@ -77,4 +77,3 @@ def datalayer(map):
 @pytest.fixture
 def tilelayer():
     return TileLayerFactory()
-

umap/tests/integration/test_picto.py

@@ -1,8 +1,8 @@
 from pathlib import Path
 
 import pytest
-from playwright.sync_api import expect
 from django.core.files.base import ContentFile
+from playwright.sync_api import expect
 
 from umap.models import Map, Pictogram
 

umap/tests/integration/test_slideshow.py

@@ -1,8 +1,8 @@
 from pathlib import Path
 
 import pytest
-from playwright.sync_api import expect
 from django.core.files.base import ContentFile
+from playwright.sync_api import expect
 
 from umap.models import Map, Pictogram
 

umap/tests/test_datalayer.py

@@ -3,9 +3,10 @@ import os
 import pytest
 from django.core.files.base import ContentFile
 
-from .base import DataLayerFactory, MapFactory
 from umap.models import DataLayer, Map
 
+from .base import DataLayerFactory, MapFactory
+
 pytestmark = pytest.mark.django_db
 
 
@@ -63,13 +64,13 @@ def test_should_remove_old_versions_on_save(datalayer, map, settings):
     settings.UMAP_KEEP_VERSIONS = 3
     root = datalayer.storage_root()
     before = len(datalayer.geojson.storage.listdir(root)[1])
-    newer = f'{root}/{datalayer.pk}_1440924889.geojson'
+    newer = f"{root}/{datalayer.pk}_1440924889.geojson"
-    medium = f'{root}/{datalayer.pk}_1440923687.geojson'
+    medium = f"{root}/{datalayer.pk}_1440923687.geojson"
-    older = f'{root}/{datalayer.pk}_1440918637.geojson'
+    older = f"{root}/{datalayer.pk}_1440918637.geojson"
-    other = f'{root}/123456_1440918637.geojson'
+    other = f"{root}/123456_1440918637.geojson"
     for path in [medium, newer, older, other]:
         datalayer.geojson.storage.save(path, ContentFile("{}"))
-        datalayer.geojson.storage.save(path + '.gz', ContentFile("{}"))
+        datalayer.geojson.storage.save(path + ".gz", ContentFile("{}"))
     assert len(datalayer.geojson.storage.listdir(root)[1]) == 8 + before
     datalayer.save()
     files = datalayer.geojson.storage.listdir(root)[1]

umap/tests/test_map.py

@@ -50,11 +50,13 @@ def test_editors_can_edit_if_status_editors(map, user):
     assert map.can_edit(user)
 
 
-def test_logged_in_user_should_be_allowed_for_anonymous_map_with_anonymous_edit_status(map, user, rf):  # noqa
+def test_logged_in_user_should_be_allowed_for_anonymous_map_with_anonymous_edit_status(
+    map, user, rf
+):  # noqa
     map.owner = None
     map.edit_status = map.ANONYMOUS
     map.save()
-    url = reverse('map_update', kwargs={'map_id': map.pk})
+    url = reverse("map_update", kwargs={"map_id": map.pk})
     request = rf.get(url)
     request.user = user
     assert map.can_edit(user, request)
@@ -70,7 +72,7 @@ def test_anonymous_user_should_not_be_allowed_for_anonymous_map(map, user, rf):
 def test_clone_should_return_new_instance(map, user):
     clone = map.clone()
     assert map.pk != clone.pk
-    assert u"Clone of " + map.name == clone.name
+    assert "Clone of " + map.name == clone.name
     assert map.settings == clone.settings
     assert map.center == clone.center
     assert map.zoom == clone.zoom
@@ -108,8 +110,7 @@ def test_clone_should_clone_datalayers_and_features_too(map, user, datalayer):
 def test_publicmanager_should_get_only_public_maps(map, user, licence):
     map.share_status = map.PUBLIC
     open_map = MapFactory(owner=user, licence=licence, share_status=Map.OPEN)
-    private_map = MapFactory(owner=user, licence=licence,
+    private_map = MapFactory(owner=user, licence=licence, share_status=Map.PRIVATE)
-                             share_status=Map.PRIVATE)
     assert map in Map.public.all()
     assert open_map not in Map.public.all()
     assert private_map not in Map.public.all()

umap/tests/test_map_views.py

@@ -3,8 +3,8 @@ import json
 import pytest
 from django.contrib.auth import get_user_model
 from django.core import mail
-from django.urls import reverse
 from django.core.signing import Signer
+from django.urls import reverse
 
 from umap.models import DataLayer, Map, Star
 
@@ -275,9 +275,7 @@ def test_owner_cannot_access_map_with_share_status_blocked(client, map):
     assert response.status_code == 403
 
 
-def test_non_editor_cannot_access_map_if_share_status_private(
+def test_non_editor_cannot_access_map_if_share_status_private(client, map, user):  # noqa
-    client, map, user
-):  # noqa
     url = reverse("map", args=(map.slug, map.pk))
     map.share_status = map.PRIVATE
     map.save()
@@ -355,9 +353,7 @@ def test_anonymous_update_without_cookie_fails(client, anonymap, post_data):  #
 
 
 @pytest.mark.usefixtures("allow_anonymous")
-def test_anonymous_update_with_cookie_should_work(
+def test_anonymous_update_with_cookie_should_work(cookieclient, anonymap, post_data):  # noqa
-    cookieclient, anonymap, post_data
-):  # noqa
     url = reverse("map_update", kwargs={"map_id": anonymap.pk})
     # POST only mendatory fields
     name = "new map name"
@@ -438,9 +434,7 @@ def test_clone_anonymous_map_should_not_be_possible_if_user_is_not_allowed(
 
 
 @pytest.mark.usefixtures("allow_anonymous")
-def test_clone_map_should_be_possible_if_edit_status_is_anonymous(
+def test_clone_map_should_be_possible_if_edit_status_is_anonymous(client, anonymap):  # noqa
-    client, anonymap
-):  # noqa
     assert Map.objects.count() == 1
     url = reverse("map_clone", kwargs={"map_id": anonymap.pk})
     anonymap.edit_status = anonymap.ANONYMOUS

umap/tests/test_tilelayer.py

@@ -6,16 +6,22 @@ pytestmark = pytest.mark.django_db
 
 
 def test_tilelayer_json():
-    tilelayer = TileLayerFactory(attribution='Attribution', maxZoom=19,
+    tilelayer = TileLayerFactory(
-                                 minZoom=0, name='Name', rank=1, tms=True,
+        attribution="Attribution",
-                                 url_template='http://{s}.x.fr/{z}/{x}/{y}')
+        maxZoom=19,
+        minZoom=0,
+        name="Name",
+        rank=1,
+        tms=True,
+        url_template="http://{s}.x.fr/{z}/{x}/{y}",
+    )
     assert tilelayer.json == {
-        'attribution': 'Attribution',
+        "attribution": "Attribution",
-        'id': tilelayer.id,
+        "id": tilelayer.id,
-        'maxZoom': 19,
+        "maxZoom": 19,
-        'minZoom': 0,
+        "minZoom": 0,
-        'name': 'Name',
+        "name": "Name",
-        'rank': 1,
+        "rank": 1,
-        'tms': True,
+        "tms": True,
-        'url_template': 'http://{s}.x.fr/{z}/{x}/{y}'
+        "url_template": "http://{s}.x.fr/{z}/{x}/{y}",
     }

umap/urls.py

@@ -1,5 +1,4 @@
 from django.conf import settings
-from django.urls import include, path, re_path
 from django.conf.urls.i18n import i18n_patterns
 from django.conf.urls.static import static
 from django.contrib import admin
@@ -7,16 +6,17 @@ from django.contrib.auth import views as auth_views
 from django.contrib.auth.decorators import login_required
 from django.contrib.staticfiles.storage import staticfiles_storage
 from django.contrib.staticfiles.urls import staticfiles_urlpatterns
+from django.urls import include, path, re_path
 from django.views.decorators.cache import cache_control, cache_page, never_cache
 from django.views.decorators.csrf import ensure_csrf_cookie
 from django.views.generic.base import RedirectView
 
 from . import views
 from .decorators import (
-    jsonize_view,
-    login_required_if_not_anonymous_allowed,
     can_edit_map,
     can_view_map,
+    jsonize_view,
+    login_required_if_not_anonymous_allowed,
 )
 from .utils import decorated_patterns
 

umap/utils.py

@@ -1,8 +1,7 @@
 import gzip
 import os
 
-from django.urls import get_resolver
+from django.urls import URLPattern, URLResolver, get_resolver
-from django.urls import URLPattern, URLResolver
 
 
 def get_uri_template(urlname, args=None, prefix=""):

umap/wsgi.py

@@ -15,13 +15,13 @@ framework.
 """
 import os
 
-os.environ.setdefault("DJANGO_SETTINGS_MODULE",
+os.environ.setdefault("DJANGO_SETTINGS_MODULE", "umap.settings")
-                      "umap.settings")
 
 # This application object is used by any WSGI server configured to use this
 # file. This includes Django's development server, if the WSGI_APPLICATION
 # setting points here.
 from django.core.wsgi import get_wsgi_application
+
 application = get_wsgi_application()
 
 # Apply WSGI middleware here.