django-pfx 1.7.2.dev4__tar.gz → 1.7.2.dev6__tar.gz

{django_pfx-1.7.2.dev4 → django_pfx-1.7.2.dev6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-pfx
-Version: 1.7.2.dev4
+Version: 1.7.2.dev6
 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.2.dev4 → django_pfx-1.7.2.dev6}/django_pfx.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-pfx
-Version: 1.7.2.dev4
+Version: 1.7.2.dev6
 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.2.dev4 → django_pfx-1.7.2.dev6}/pfx/pfxcore/models/ordered_model_mixin.py

@@ -25,10 +25,12 @@ class OrderedModelMixin(models.Model):
         super().clean_fields(exclude=exclude)
 
     def save(self, *args, **kwargs):
+        created = bool(self.pk)
         if not self.seq:
             self.seq = self._order_default
         super().save(*args, **kwargs)
-        if self.__previous_ordered_by_values != self._ordered_by_values:
+        if (not created and
+                self.__previous_ordered_by_values != self._ordered_by_values):
             if self.ordered_default == 'first':
                 self.top()
             else:

{django_pfx-1.7.2.dev4 → django_pfx-1.7.2.dev6}/tests/tests/test_ordered_rest_view_mixin.py

@@ -316,6 +316,15 @@ class TestOrderedRestViewMixin(TestAssertMixin, TestCase):
             order(OrderedModel),
             [self.o3.pk, self.o1.pk, self.o2.pk])
 
+        response = self.client.post(
+            '/api/ordered?move=top', {'name': "Test"})
+        self.assertRC(response, 200)
+        o4_pk = self.get_val(response, 'pk')
+
+        self.assertEqual(
+            order(OrderedModel),
+            [o4_pk, self.o3.pk, self.o1.pk, self.o2.pk])
+
     def test_top_by_name(self):
         response = self.client.put(
             f'/api/ordered-by-name/{self.obn3.pk}?move=top', {})
@@ -370,6 +379,15 @@ class TestOrderedRestViewMixin(TestAssertMixin, TestCase):
             order(OrderedModel),
             [self.o2.pk, self.o3.pk, self.o1.pk])
 
+        response = self.client.post(
+            '/api/ordered?move=bottom', {'name': "Test"})
+        self.assertRC(response, 200)
+        o4_pk = self.get_val(response, 'pk')
+
+        self.assertEqual(
+            order(OrderedModel),
+            [self.o2.pk, self.o3.pk, self.o1.pk, o4_pk])
+
     def test_bottom_by_name(self):
         response = self.client.put(
             f'/api/ordered-by-name/{self.obn2.pk}?move=bottom', {})