fix: use value from model_to_dict instead of getattr

This gives us a more meaningful represeantaion of the value
acdh-oeaw · Oct 16, 2023 · f07c7b7 · f07c7b7
1 parent 0b8b639
commit f07c7b7
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/apis_core/apis_entities/detail_generic.py b/apis_core/apis_entities/detail_generic.py
@@ -112,10 +112,8 @@ def get(self, request, *args, **kwargs):
             no_merge_labels = []
 
         relevant_fields = []
-        for field in model_to_dict(self.instance).keys():
-            relevant_fields.append(
-                (self.instance._meta.get_field(field), getattr(self.instance, field))
-            )
+        for field, value in model_to_dict(self.instance).items():
+            relevant_fields.append((self.instance._meta.get_field(field), value))
 
         return HttpResponse(
             template.render(