Merge pull request #29 from PnX-SI/fix/from_dict_del_pk_key

Fix/from dict del pk key
PnX-SI · Jun 6, 2023 · bff8af8 · bff8af8
2 parents 4a1d299 + 8f279e3
commit bff8af8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/docs/changelog.rst b/docs/changelog.rst
@@ -2,6 +2,14 @@
 CHANGELOG
 =========
 
+0.3.4 (2023-06-06)
+------------------
+
+**🐛 Corrections**
+
+* Dé-sérialiseurs : `from_dict`
+  * ajout d'un test sur l’existence d'une valeur de clé primaire nulle dans les données avant de la supprimer.
+
 0.3.3 (2023-04-11)
 ------------------
 
@@ -114,14 +122,14 @@ CHANGELOG
 
 **🐛 Corrections**
 
-* Correction des régressions de performance sur la sérialisation 
+* Correction des régressions de performance sur la sérialisation
 
 0.2.2 (2021-06-22)
 ------------------
 
 **🐛 Corrections**
 
-* Support des propriétés hybrides des modèles 
+* Support des propriétés hybrides des modèles
 * Ajout de tests sur les modèles polymorphiques
 
 0.2.1 (2021-06-03)

diff --git a/src/utils_flask_sqla/serializers.py b/src/utils_flask_sqla/serializers.py
@@ -408,12 +408,8 @@ def populatefn(self, dict_in, recursif=False):
                 v_obj = []
 
                 for data in values:
-                    id_value = data.get(id_field_name)
 
-                    # si id_value est null
-                    # creation -> on supprime id_value
-                    if not id_value:
-                        data.pop(id_field_name)
+                    id_value = data.pop(id_field_name, None)
 
                     res = (
                         # si on a une id -> on recupère dans la liste preload_res_with_ids