Split model _meta into model_options and _model_meta

Author: davegaeddert

plain-admin/plain/admin/views/models.py

@@ -53,7 +53,7 @@ def get_title(self) -> str:
         if title := super().get_title():
             return title
 
-        return self.model._meta.model_name.capitalize() + "s"
+        return self.model.model_options.model_name.capitalize() + "s"
 
     @classmethod
     def get_nav_title(cls) -> str:
@@ -63,14 +63,14 @@ def get_nav_title(cls) -> str:
         if cls.title:
             return cls.title
 
-        return cls.model._meta.model_name.capitalize() + "s"
+        return cls.model.model_options.model_name.capitalize() + "s"
 
     @classmethod
     def get_path(cls) -> str:
         if path := super().get_path():
             return path
 
-        return f"{cls.model._meta.model_name}/"
+        return f"{cls.model.model_options.model_name}/"
 
     def get_template_context(self) -> dict[str, Any]:
         context = super().get_template_context()
@@ -167,20 +167,20 @@ def get_nav_title(cls) -> str:
         if cls.title:
             return cls.title
 
-        return cls.model._meta.model_name.capitalize()
+        return cls.model.model_options.model_name.capitalize()
 
     @classmethod
     def get_path(cls) -> str:
         if path := super().get_path():
             return path
 
-        return f"{cls.model._meta.model_name}/<int:id>/"
+        return f"{cls.model.model_options.model_name}/<int:id>/"
 
     def get_fields(self) -> list[str]:
         if fields := super().get_fields():
             return fields
 
-        return [f.name for f in self.object._meta.get_fields() if f.concrete]
+        return [f.name for f in self.object._model_meta.get_fields() if f.concrete]
 
     def get_field_value(self, obj: Any, field: str) -> Any:
         try:
@@ -204,14 +204,14 @@ def get_title(self) -> str:
         if title := super().get_title():
             return title
 
-        return f"New {self.model._meta.model_name}"
+        return f"New {self.model.model_options.model_name}"
 
     @classmethod
     def get_path(cls) -> str:
         if path := super().get_path():
             return path
 
-        return f"{cls.model._meta.model_name}/create/"
+        return f"{cls.model.model_options.model_name}/create/"
 
 
 class AdminModelUpdateView(AdminUpdateView):
@@ -230,7 +230,7 @@ def get_path(cls) -> str:
         if path := super().get_path():
             return path
 
-        return f"{cls.model._meta.model_name}/<int:id>/update/"
+        return f"{cls.model.model_options.model_name}/<int:id>/update/"
 
     def get_object(self) -> models.Model:
         return self.model.query.get(id=self.url_kwargs["id"])
@@ -247,7 +247,7 @@ def get_path(cls) -> str:
         if path := super().get_path():
             return path
 
-        return f"{cls.model._meta.model_name}/<int:id>/delete/"
+        return f"{cls.model.model_options.model_name}/<int:id>/delete/"
 
     def get_object(self) -> models.Model:
         return self.model.query.get(id=self.url_kwargs["id"])

plain-api/plain/api/README.md

@@ -221,7 +221,7 @@ class User(models.Model):
         required=False,
     )
 
-    _meta = models.Options(
+    model_options = models.Options(
         constraints=[
             models.UniqueConstraint(
                 fields=["api_key"],

plain-api/plain/api/models.py

@@ -23,7 +23,7 @@ class APIKey(models.Model):
 
     api_version = models.CharField(max_length=255, required=False)
 
-    _meta = models.Options(
+    model_options = models.Options(
         constraints=[
             models.UniqueConstraint(
                 fields=["uuid"], name="plainapi_apikey_unique_uuid"

plain-cache/plain/cache/models.py

@@ -27,7 +27,7 @@ class CachedItem(models.Model):
 
     query = CachedItemQuerySet()
 
-    _meta = models.Options(
+    model_options = models.Options(
         indexes=[
             models.Index(fields=["expires_at"]),
         ],

plain-flags/plain/flags/models.py

@@ -23,7 +23,7 @@ class FlagResult(models.Model):
     key = models.CharField(max_length=255)
     value = models.JSONField()
 
-    _meta = models.Options(
+    model_options = models.Options(
         constraints=[
             models.UniqueConstraint(
                 fields=["flag", "key"], name="plainflags_flagresult_unique_key"
@@ -51,7 +51,7 @@ class Flag(models.Model):
     # To provide an easier way to see if a flag is still being used
     used_at = models.DateTimeField(required=False, allow_null=True)
 
-    _meta = models.Options(
+    model_options = models.Options(
         constraints=[
             models.UniqueConstraint(
                 fields=["name"], name="plainflags_flag_unique_name"

plain-flags/plain/flags/utils.py

@@ -12,6 +12,8 @@ def coerce_key(key: Any) -> str:
         return key
 
     if isinstance(key, models.Model):
-        return f"{key._meta.package_label}.{key._meta.model_name}:{key.id}"
+        return (
+            f"{key.model_options.package_label}.{key.model_options.model_name}:{key.id}"
+        )
 
     return str(key)

plain-models/plain/models/README.md

@@ -200,7 +200,7 @@ class User(models.Model):
     username = models.CharField(max_length=150)
     age = models.IntegerField()
 
-    _meta = models.Options(
+    model_options = models.Options(
         indexes=[
             models.Index(fields=["email"]),
             models.Index(fields=["-created_at"], name="user_created_idx"),

plain-models/plain/models/backends/base/creation.py

@@ -98,7 +98,7 @@ def set_as_test_mirror(self, primary_settings_dict: dict[str, Any]) -> None:
     #                 and package_config.package_label in loader.migrated_packages
     #             ):
     #                 for model in package_config.get_models():
-    #                     if model._meta.can_migrate(
+    #                     if model.model_options.can_migrate(
     #                         self.connection
     #                     ) and router.allow_migrate_model(self.connection.alias, model):
     #                         queryset = model._base_manager.using(
@@ -127,7 +127,7 @@ def set_as_test_mirror(self, primary_settings_dict: dict[str, Any]) -> None:
     #                 "json", data, using=self.connection.alias
     #             ):
     #                 obj.save()
-    #                 table_names.add(obj.object.__class__._meta.db_table)
+    #                 table_names.add(obj.object.model_options.db_table)
     #         # Manually check for any invalid keys that might have been added,
     #         # because constraint checks were disabled.
     #         self.connection.check_constraints(table_names=table_names)

plain-models/plain/models/backends/base/introspection.py

@@ -94,7 +94,7 @@ def get_migratable_models(self) -> Generator[Any, None, None]:
             for model in models_registry.get_models(
                 package_label=package_config.package_label
             )
-            if model._meta.can_migrate(self.connection)
+            if model.model_options.can_migrate(self.connection)
         )
 
     def plain_table_names(
@@ -108,8 +108,10 @@ def plain_table_names(
         """
         tables = set()
         for model in self.get_migratable_models():
-            tables.add(model._meta.db_table)
-            tables.update(f.m2m_db_table() for f in model._meta.local_many_to_many)
+            tables.add(model.model_options.db_table)
+            tables.update(
+                f.m2m_db_table() for f in model._model_meta.local_many_to_many
+            )
         tables = list(tables)
         if only_existing:
             existing_tables = set(self.table_names(include_views=include_views))
@@ -128,7 +130,9 @@ def sequence_list(self) -> list[dict[str, Any]]:
             for model in self.get_migratable_models():
                 sequence_list.extend(
                     self.get_sequences(
-                        cursor, model._meta.db_table, model._meta.local_fields
+                        cursor,
+                        model.model_options.db_table,
+                        model._model_meta.local_fields,
                     )
                 )
         return sequence_list