django-pfx 1.7.dev4__tar.gz → 1.8.dev2__tar.gz

{django_pfx-1.7.dev4 → django_pfx-1.8.dev2}/.gitlab-ci.yml

@@ -27,7 +27,7 @@ stages:
 
 quality check:
   variables:
-    DJANGO_VERSION: 'django==4.2.*'  # LTS
+    DJANGO_VERSION: 'django==5.2.*'  # LTS
   script:
     - apt-get update
     - apt-get install -y gettext
@@ -110,7 +110,7 @@ package:
     - job: quality check
     - job: test
   rules:
-    - if: $CI_COMMIT_TAG =~ /^\d+\.\d+$/
+    - if: $CI_COMMIT_TAG =~ /^\d+\.\d+(\.\d+)?$/
       when: on_success
     - if: $CI_COMMIT_BRANCH == "master"
       when: on_success

{django_pfx-1.7.dev4 → django_pfx-1.8.dev2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-pfx
-Version: 1.7.dev4
+Version: 1.8.dev2
 Summary: Django PFX is a toolkit designed to streamline the development of RESTful APIs using the Django framework.
 Author: Hervé Martinet
 Author-email: herve.martinet@gmail.com

{django_pfx-1.7.dev4 → django_pfx-1.8.dev2}/django_pfx.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-pfx
-Version: 1.7.dev4
+Version: 1.8.dev2
 Summary: Django PFX is a toolkit designed to streamline the development of RESTful APIs using the Django framework.
 Author: Hervé Martinet
 Author-email: herve.martinet@gmail.com

{django_pfx-1.7.dev4 → django_pfx-1.8.dev2}/django_pfx.egg-info/SOURCES.txt

@@ -67,6 +67,7 @@ pfx/pfxcore/middleware/profiling.py
 pfx/pfxcore/migrations/0001_initial.py
 pfx/pfxcore/migrations/0002_pfxpermissionsuser.py
 pfx/pfxcore/migrations/0003_delete_pfxpermissionsuser.py
+pfx/pfxcore/migrations/0004_alter_loginban_failed_counter_and_more.py
 pfx/pfxcore/migrations/__init__.py
 pfx/pfxcore/migrations/operations/__init__.py
 pfx/pfxcore/migrations/operations/permissions.py

{django_pfx-1.7.dev4 → django_pfx-1.8.dev2}/pfx/pfxcore/locale/fr/LC_MESSAGES/django.po

@@ -7,7 +7,7 @@ msgid ""
 msgstr ""
 "Project-Id-Version: \n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2026-03-09 15:11+0100\n"
+"POT-Creation-Date: 2026-03-11 15:09+0100\n"
 "PO-Revision-Date: 2021-06-22 23:31+0200\n"
 "Last-Translator: \n"
 "Language-Team: \n"
@@ -67,23 +67,23 @@ msgstr ""
 msgid "users"
 msgstr ""
 
-#: models/login_ban.py:45
+#: models/login_ban.py:54
 msgid "Username"
 msgstr "Nom d’utilisateur"
 
-#: models/login_ban.py:46
+#: models/login_ban.py:55
 msgid "Failed counter"
 msgstr "Compteur d’échec"
 
-#: models/login_ban.py:47
+#: models/login_ban.py:56
 msgid "Last failed"
 msgstr "Dernier échec"
 
-#: models/login_ban.py:52
+#: models/login_ban.py:61
 msgid "Login ban"
 msgstr "Login banni"
 
-#: models/login_ban.py:53
+#: models/login_ban.py:62
 msgid "Login bans"
 msgstr "Login bannis"
 
@@ -302,7 +302,7 @@ msgstr "{model} {obj} créé."
 msgid "{model} {obj} updated."
 msgstr "{model} {obj} modifié."
 
-#: views/rest_views.py:1276
+#: views/rest_views.py:1282
 #, python-brace-format
 msgid "{model} {obj} deleted."
 msgstr "{model} {obj} supprimé."

django_pfx-1.8.dev2/pfx/pfxcore/migrations/0004_alter_loginban_failed_counter_and_more.py

@@ -0,0 +1,24 @@
+# Generated by Django 4.2.25 on 2026-03-11 09:19
+# flake8: noqa
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('pfxcore', '0003_delete_pfxpermissionsuser'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='loginban',
+            name='failed_counter',
+            field=models.IntegerField(default=0, verbose_name='Failed counter'),
+        ),
+        migrations.AlterField(
+            model_name='loginban',
+            name='last_failed',
+            field=models.DateTimeField(verbose_name='Last failed'),
+        ),
+    ]

{django_pfx-1.7.dev4 → django_pfx-1.8.dev2}/pfx/pfxcore/models/login_ban.py

@@ -1,7 +1,8 @@
 from datetime import timedelta
 
-from django.db import models
-from django.utils import timezone
+from django.db import IntegrityError, models
+from django.db.models import F
+from django.utils import timezone as tz
 from django.utils.translation import gettext_lazy as _
 
 from pfx.pfxcore.settings import settings
@@ -21,19 +22,27 @@ class LoginBanQuerySet(models.QuerySet):
                     ban.failed_counter //
                     settings.PFX_LOGIN_BAN_FAILED_NUMBER - 1))
             ban_time = ban.last_failed + timedelta(seconds=seconds)
-            now = timezone.now()
+            now = tz.now()
             if now < ban_time:
                 return ban_time - now
         return False
 
+    def _increment(self, username, last_failed):
+        return self.filter(username=username).update(
+            failed_counter=F("failed_counter") + 1,
+            last_failed=last_failed)
+
     def ban(self, username):
         if not username:
             return
-        try:
-            ban = self.get(username=username)
-            ban.save()
-        except LoginBan.DoesNotExist:
-            LoginBan.objects.create(username=username)
+        now = tz.now()
+        updated = self._increment(username, now)
+        if not updated:
+            try:
+                self.create(
+                    username=username, failed_counter=1, last_failed=now)
+            except IntegrityError:
+                self._increment(username, now)
 
     def unban(self, username):
         if not username:
@@ -43,8 +52,8 @@ class LoginBanQuerySet(models.QuerySet):
 
 class LoginBan(models.Model):
     username = models.CharField(_("Username"), max_length=150, unique=True)
-    failed_counter = models.IntegerField(_("Failed counter"))
-    last_failed = models.DateTimeField(_("Last failed"), auto_now=True)
+    failed_counter = models.IntegerField(_("Failed counter"), default=0)
+    last_failed = models.DateTimeField(_("Last failed"))
 
     objects = LoginBanQuerySet.as_manager()
 
@@ -54,7 +63,3 @@ class LoginBan(models.Model):
 
     def __str__(self):
         return self.username
-
-    def save(self, *args, **kwargs):
-        self.failed_counter = (self.failed_counter or 0) + 1
-        return super().save(*args, **kwargs)

{django_pfx-1.7.dev4 → django_pfx-1.8.dev2}/pfx/pfxcore/views/rest_views.py

@@ -561,14 +561,20 @@ class ModelBodyMixin(BodyMixin, ModelMixin):
                 mk, mv = field.to_model_value(v, self.get_related_queryset)
                 if field.field_type == FT.ModelObjectList:
                     if field.related_fields:
+                        fk_name = field.model_field.field.name
+                        fk_attname = field.model_field.field.attname
                         for i, rv in enumerate(v):
+                            rel_obj = mv[i]
+                            m2o_pk = getattr(rel_obj, fk_attname)
+                            if m2o_pk and m2o_pk != obj.pk:
+                                # Force a copy if the existing related
+                                # object is linked to another parent.
+                                rel_obj.pk = None
                             for rk, rf in field.related_fields.items():
                                 rmk, rmv = rf.to_model_value(
                                     rv.get(rk), self.get_related_queryset)
-                                setattr(mv[i], rmk, rmv)
-                            setattr(
-                                mv[i], '_save_related',
-                                field.model_field.field.name)
+                                setattr(rel_obj, rmk, rmv)
+                            setattr(rel_obj, '_save_related', fk_name)
                     res_rel[mk] = mv
                 else:
                     res[mk] = mv