Ensure Django Model Classes Implement a __str__ Method

analytics/models.py

@@ -55,3 +55,8 @@ class PageView(models.Model):
             return model_class.objects.filter(pk__in=id_list).order_by(preserved)
         
         return model_class.objects.none()
+
+    def __str__(self):
+        model_name = self.__class__.__name__
+        fields_str = ", ".join((f"{field.name}={getattr(self, field.name)}" for field in self._meta.fields))
+        return f"{model_name}({fields_str})"

history_tracking/models.py

@@ -66,6 +66,11 @@ class TrackedModel(models.Model):
             ).order_by('-pgh_created_at')
         return self.__class__.objects.none()
 
+    def __str__(self):
+        model_name = self.__class__.__name__
+        fields_str = ", ".join((f"{field.name}={getattr(self, field.name)}" for field in self._meta.fields))
+        return f"{model_name}({fields_str})"
+
 class HistoricalSlug(models.Model):
     """Track historical slugs for models"""
     content_type = models.ForeignKey(ContentType, on_delete=models.CASCADE)

parks/models.py

@@ -165,13 +165,13 @@ class Park(TrackedModel):
                     print(f"Found park from historical slug: {park.name}")
                     return park, True
                 except cls.DoesNotExist:
-                    print("Park not found for historical slug record")
+                    print(f"Park not found for historical slug record")
                     pass
             else:
                 print("No historical slug record found")
             
             # Try pghistory events
-            print("Searching pghistory events")
+            print(f"Searching pghistory events")
             event_model = getattr(cls, 'event_model', None)
             if event_model:
                 historical_event = event_model.objects.filter(
@@ -185,7 +185,7 @@ class Park(TrackedModel):
                     print(f"Found park from pghistory: {park.name}")
                     return park, True
                 except cls.DoesNotExist:
-                    print("Park not found for pghistory event")
+                    print(f"Park not found for pghistory event")
                     pass
             else:
                 print("No pghistory event found")

parks/tests/test_models.py

@@ -94,7 +94,7 @@ class ParkModelTests(TestCase):
                 historical_records = event_model.objects.filter(
                     pgh_obj_id=park.id
                 ).order_by('-pgh_created_at')
-                print("\nPG History records:")
+                print(f"\nPG History records:")
                 for record in historical_records:
                     print(f"- Event ID: {record.pgh_id}")
                     print(f"  Name: {record.name}")
@@ -210,4 +210,4 @@ class ParkAreaModelTests(TestCase):
     def test_absolute_url(self):
         """Test get_absolute_url method"""
         expected_url = f"/parks/{self.park.slug}/areas/{self.area.slug}/"
-        self.assertEqual(self.area.get_absolute_url(), expected_url)
+        self.assertEqual(self.area.get_absolute_url(), expected_url)