Pod v4 MAIN

.github/workflows/pod_main.yml

@@ -18,7 +18,7 @@ jobs:
         strategy:
             max-parallel: 2
             matrix:
-              python-version: ['3.8', '3.10']
+              python-version: ['3.10', '3.12']
         steps:
             - run: echo "🎉 The job was automatically triggered by a ${{ github.event_name }} event."
             - run: echo "🐧 This job is now running on a ${{ runner.os }} server hosted by GitHub!"

.gitignore

@@ -62,11 +62,13 @@ pod/static/
 .cache_ggshield
 htmlcov
 compile-model
+pod/activitypub/*.pub
 
 # IDE Files #
 #############
 .idea
 .vscode
+*.code-workspace
 
 # Certificates #
 ################

Makefile

@@ -13,15 +13,15 @@ help:
 # Démarre le serveur de test
 start:
 	(sleep 15 ; open http://pod.localhost:8000) &
-	python3 manage.py runserver pod.localhost:8000 --insecure
+	python3 -Wd manage.py runserver pod.localhost:8000 --insecure
 	# --insecure let serve static files even when DEBUG=False
 
 # Démarre le serveur de test en https auto-signé
 starts:
 	# nécessite les django-extensions
 	# cf https://timonweb.com/django/https-django-development-server-ssl-certificate/
 	(sleep 15 ; open https://pod.localhost:8000) &
-	python3 manage.py runserver_plus pod.localhost:8000 --cert-file cert.pem --key-file key.pem
+	python3 -Wd manage.py runserver_plus pod.localhost:8000 --cert-file cert.pem --key-file key.pem
 
 # Première installation de pod (BDD SQLite intégrée)
 install:
@@ -44,7 +44,7 @@ createDB:
 	find . -path "*/migrations/*.pyc" -delete
 	make updatedb
 	make migrate
-	python3 manage.py loaddata initial_data
+	python3 -Wd manage.py loaddata initial_data
 
 # Mise à jour des fichiers de langue
 lang:
@@ -82,8 +82,8 @@ statics:
 
 # Generate configuration docs in .MD format
 createconfigs:
-	python3 manage.py createconfiguration fr
-	python3 manage.py createconfiguration en
+	python3 -Wd manage.py createconfiguration fr
+	python3 -Wd manage.py createconfiguration en
 
 # -- Docker
 # Use for docker run and docker exec commands

pod/ai_enhancement/forms.py

@@ -2,8 +2,8 @@
 
 from django import forms
 from django.conf import settings
-from django.utils.translation import ugettext_lazy as _
-from tagging.fields import TagField
+from django.utils.translation import gettext_lazy as _
+from tagulous.models import TagField
 
 from pod.main.forms_utils import add_placeholder_and_asterisk
 from pod.video.models import Video, Discipline

pod/ai_enhancement/models.py

@@ -1,5 +1,5 @@
 from django.db import models
-from django.utils.translation import ugettext as _
+from django.utils.translation import gettext as _
 from django.dispatch import receiver
 from django.db.models.signals import post_save
 from pod.video.models import Video

pod/ai_enhancement/utils.py

@@ -7,7 +7,7 @@
 from django.conf import settings
 from requests import Response
 from django.urls import reverse
-from django.utils.translation import ugettext_lazy as _
+from django.utils.translation import gettext_lazy as _
 from pod.ai_enhancement.models import AIEnhancement
 from pod.main.utils import extract_json_from_str
 from pod.video.models import Discipline, Video
@@ -27,7 +27,7 @@
 
 AI_ENHANCEMENT_API_URL = getattr(settings, "AI_ENHANCEMENT_API_URL", "")
 AI_ENHANCEMENT_API_VERSION = getattr(settings, "AI_ENHANCEMENT_API_VERSION", "")
-USE_NOTIFICATIONS = getattr(settings, "USE_NOTIFICATIONS", True)
+USE_NOTIFICATIONS = getattr(settings, "USE_NOTIFICATIONS", False)
 EMAIL_ON_ENHANCEMENT_COMPLETION = getattr(
     settings, "EMAIL_ON_ENHANCEMENT_COMPLETION", True
 )

pod/ai_enhancement/views.py

@@ -9,7 +9,7 @@
 from django.http import HttpResponse, JsonResponse
 from django.shortcuts import get_object_or_404, redirect, render
 from django.urls import reverse
-from django.utils.translation import ugettext as _
+from django.utils.translation import gettext as _
 from django.views.decorators.csrf import csrf_protect, csrf_exempt
 from django.views.decorators.csrf import ensure_csrf_cookie
 from django.core.exceptions import PermissionDenied

pod/authentication/__init__.py

@@ -1 +0,0 @@
-default_app_config = "pod.authentication.apps.AuthConfig"

pod/authentication/admin.py

@@ -2,7 +2,7 @@
 from django.contrib import admin
 from django.contrib.auth.admin import UserAdmin as BaseUserAdmin
 from django.contrib.auth.models import User
-from django.utils.translation import ugettext_lazy as _
+from django.utils.translation import gettext_lazy as _
 from django.utils.html import format_html
 from django.contrib.sites.shortcuts import get_current_site
 from django.contrib.auth.models import Group
@@ -84,12 +84,11 @@ class Media:
 
 
 class UserAdmin(BaseUserAdmin):
+    @admin.display(description=_("Email"))
     def clickable_email(self, obj):
         email = obj.email
         return format_html('<a href="mailto:{}">{}</a>', email, email)
 
-    clickable_email.allow_tags = True
-    clickable_email.short_description = _("Email")
     list_display = (
         "username",
         "last_name",
@@ -130,11 +129,10 @@ def formfield_for_manytomany(self, db_field, request, **kwargs):
         kwargs["widget"] = widgets.FilteredSelectMultiple(db_field.verbose_name, False)
         return super().formfield_for_foreignkey(db_field, request, **kwargs)
 
+    @admin.display(description=_("Establishment"))
     def owner_establishment(self, obj) -> str:
         return "%s" % Owner.objects.get(user=obj).establishment
 
-    owner_establishment.short_description = _("Establishment")
-
     ordering = (
         "-is_superuser",
         "username",
@@ -188,6 +186,7 @@ def get_inline_instances(self, request, obj=None):
         return _inlines
 
 
+@admin.register(AccessGroup)
 class AccessGroupAdmin(admin.ModelAdmin):
     # form = AccessGroupAdminForm
     # search_fields = ["user__username__icontains", "user__email__icontains"]
@@ -200,6 +199,7 @@ class AccessGroupAdmin(admin.ModelAdmin):
     )
 
 
+@admin.register(Owner)
 class OwnerAdmin(admin.ModelAdmin):
     # form = AdminOwnerForm
     autocomplete_fields = ["user", "accessgroups"]
@@ -225,5 +225,3 @@ class Meta:
 # Register the new Group ModelAdmin instead of the original one.
 admin.site.unregister(Group)
 admin.site.register(Group, GroupAdmin)
-admin.site.register(Owner, OwnerAdmin)
-admin.site.register(AccessGroup, AccessGroupAdmin)

pod/authentication/forms.py

@@ -4,7 +4,7 @@
 from django.contrib.auth import get_user_model
 from django.contrib.admin.widgets import FilteredSelectMultiple
 from django.contrib.auth.models import Group
-from django.utils.translation import ugettext_lazy as _
+from django.utils.translation import gettext_lazy as _
 from django.contrib.sites.models import Site
 
 __FILEPICKER__ = False

pod/authentication/models.py

@@ -1,7 +1,7 @@
 """Esup-Pod authentication models."""
 
 from django.db import models
-from django.utils.translation import ugettext_lazy as _
+from django.utils.translation import gettext_lazy as _
 from django.contrib.auth.models import User, Permission, Group
 from django.conf import settings
 from django.dispatch import receiver

pod/authentication/tests/test_views.py

@@ -7,7 +7,7 @@
 from django.test import Client
 from django.contrib.auth.models import User
 from django.conf import settings
-from django.utils.translation import ugettext_lazy as _
+from django.utils.translation import gettext_lazy as _
 
 
 class authenticationViewsTestCase(TestCase):

pod/authentication/urls.py

@@ -2,26 +2,26 @@
 from .views import authentication_logout
 from .views import authentication_login_gateway
 
-from django.conf.urls import url
+from django.urls import path
 
 app_name = "authentication"
 
 urlpatterns = [
     # auth cas
-    url(
-        r"^login/$",
+    path(
+        "login/",
         authentication_login,
         name="authentication_login",
     ),
-    url(
-        r"^logout/$",
+    path(
+        "logout/",
         authentication_logout,
         name="authentication_logout",
     ),
-    # url(r"^login/$", authentication_login, name="login"),
-    # url(r"^logout/$", authentication_logout, name="logout"),
-    url(
-        r"^login_gateway/$",
+    # re_path(r"^login/$", authentication_login, name="login"),
+    # re_path(r"^logout/$", authentication_logout, name="logout"),
+    path(
+        "login_gateway/",
         authentication_login_gateway,
         name="authentication_login_gateway",
     ),

pod/authentication/views.py

@@ -6,7 +6,7 @@
 from django.urls import reverse
 from django.conf import settings
 from django.core.exceptions import SuspiciousOperation
-from django.utils.translation import ugettext_lazy as _
+from django.utils.translation import gettext_lazy as _
 from cas.decorators import gateway
 from django.contrib import auth
 

pod/chapter/admin.py

@@ -7,6 +7,7 @@
 from pod.video.models import Video
 
 
+@admin.register(Chapter)
 class ChapterAdmin(admin.ModelAdmin):
     """Chapter administration."""
 
@@ -33,9 +34,6 @@ def formfield_for_foreignkey(self, db_field, request, **kwargs):
         return super().formfield_for_foreignkey(db_field, request, **kwargs)
 
 
-admin.site.register(Chapter, ChapterAdmin)
-
-
 class ChapterInline(admin.TabularInline):
     model = Chapter
     extra = 0

pod/chapter/forms.py

@@ -3,7 +3,7 @@
 from django import forms
 from django.conf import settings
 from django.core.exceptions import ValidationError
-from django.utils.translation import ugettext as _
+from django.utils.translation import gettext as _
 
 from pod.chapter.models import Chapter
 from pod.chapter.utils import vtt_to_chapter

pod/chapter/models.py

@@ -5,7 +5,7 @@
 from django.core.exceptions import ValidationError
 from django.db import models
 from django.template.defaultfilters import slugify
-from django.utils.translation import ugettext as _
+from django.utils.translation import gettext as _
 
 from pod.video.models import Video
 from pod.video.utils import verify_field_length

pod/chapter/urls.py

@@ -1,10 +1,10 @@
-from django.conf.urls import url
+from django.urls import re_path
 from pod.chapter.views import video_chapter
 
 app_name = "chapter"
 
 urlpatterns = [
-    url(
+    re_path(
         r"^(?P<slug>[\-\d\w]+)/$",
         video_chapter,
         name="video_chapter",

pod/chapter/views.py

@@ -7,7 +7,7 @@
 from django.template.loader import render_to_string
 from django.contrib import messages
 from django.contrib.auth.decorators import login_required
-from django.utils.translation import ugettext as _
+from django.utils.translation import gettext as _
 from django.views.decorators.csrf import csrf_protect
 from pod.video.models import Video
 from pod.chapter.models import Chapter

pod/completion/admin.py

@@ -2,7 +2,7 @@
 
 from django.conf import settings
 from django.contrib import admin
-from django.utils.translation import ugettext_lazy as _
+from django.utils.translation import gettext_lazy as _
 from pod.completion.models import Contributor
 from pod.completion.models import Document
 from pod.completion.models import Overlay
@@ -45,6 +45,7 @@ def has_add_permission(self, request, obj=None):
         return False
 
 
+@admin.register(Contributor)
 class ContributorAdmin(admin.ModelAdmin):
     list_display = (
         "name",
@@ -71,9 +72,6 @@ def get_queryset(self, request):
         return qs
 
 
-admin.site.register(Contributor, ContributorAdmin)
-
-
 class DocumentInline(admin.TabularInline):
     model = Document
     readonly_fields = (
@@ -86,6 +84,7 @@ def has_add_permission(self, request, obj=None):
         return False
 
 
+@admin.register(Document)
 class DocumentAdmin(admin.ModelAdmin):
     if __FILEPICKER__:
         form = DocumentAdminForm
@@ -124,9 +123,6 @@ class Media:
         )
 
 
-admin.site.register(Document, DocumentAdmin)
-
-
 class TrackInline(admin.TabularInline):
     model = Track
     readonly_fields = (
@@ -142,6 +138,7 @@ def has_add_permission(self, request, obj=None):
         return False
 
 
+@admin.register(EnrichModelQueue)
 class EnrichModelQueueAdmin(admin.ModelAdmin):
     list_display = (
         "title",
@@ -150,9 +147,7 @@ class EnrichModelQueueAdmin(admin.ModelAdmin):
     list_filter = ("in_treatment",)
 
 
-admin.site.register(EnrichModelQueue, EnrichModelQueueAdmin)
-
-
+@admin.register(Track)
 class TrackAdmin(admin.ModelAdmin):
     def debug(text) -> None:
         if DEBUG:
@@ -311,9 +306,6 @@ class Media:
         )
 
 
-admin.site.register(Track, TrackAdmin)
-
-
 class OverlayInline(admin.TabularInline):
     model = Overlay
     readonly_fields = (
@@ -332,6 +324,7 @@ def has_add_permission(self, request, obj=None):
         return False
 
 
+@admin.register(Overlay)
 class OverlayAdmin(admin.ModelAdmin):
     list_display = (
         "title",
@@ -354,6 +347,3 @@ def formfield_for_foreignkey(self, db_field, request, **kwargs):
 
     # class Media:
     #     css = {"all": ("css/pod.css",)}
-
-
-admin.site.register(Overlay, OverlayAdmin)

pod/completion/forms.py

@@ -4,8 +4,6 @@
 from django.conf import settings
 from django.forms.widgets import HiddenInput
 
-# from django.utils.translation import ugettext_lazy as _
-
 from pod.completion.models import Contributor
 from pod.completion.models import Document
 from pod.completion.models import Track