Replace related_name with explicit reverse descriptors

Author: davegaeddert

plain-api/plain/api/README.md

@@ -33,7 +33,7 @@ class BaseAPIView(APIView, APIKeyView):
     def use_api_key(self):
         super().use_api_key()
 
-        if user := self.api_key.users.first():
+        if user := User.query.filter(api_key=self.api_key).first():
             set_request_user(self.request, user)
         else:
             raise ResponseException(
@@ -106,11 +106,12 @@ Handling authentication in the API is pretty straightforward. If you use [API ke
 ```python
 class BaseAPIView(APIView, APIKeyView):
     def use_api_key(self):
-        from plain.auth import set_request_user
+        from plain.auth import get_request_user, set_request_user
+        from app.users.models import User
 
         super().use_api_key()
 
-        if user := self.api_key.users.first():
+        if user := User.query.filter(api_key=self.api_key).first():
             set_request_user(self.request, user)
         else:
             raise ResponseException(
@@ -216,7 +217,6 @@ class User(models.Model):
     api_key = models.ForeignKey(
         APIKey,
         on_delete=models.CASCADE,
-        related_name="users",
         allow_null=True,
         required=False,
     )
@@ -246,13 +246,16 @@ To use API keys in your views, you can inherit from `APIKeyView` and customize t
 # app/api/views.py
 from plain.api.views import APIKeyView, APIView
 from plain.auth import set_request_user
+from plain.views.exceptions import ResponseException
+
+from app.users.models import User
 
 
 class BaseAPIView(APIView, APIKeyView):
     def use_api_key(self):
         super().use_api_key()
 
-        if user := self.api_key.users.first():
+        if user := User.query.filter(api_key=self.api_key).first():
             set_request_user(self.request, user)
         else:
             raise ResponseException(

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

@@ -60,7 +60,6 @@ def _all_related_fields(model: type[Model]) -> list[Any]:
         model._model_meta._get_fields(
             forward=False,
             reverse=True,
-            include_hidden=True,
         ),
         key=operator.attrgetter("name"),
     )

plain-models/plain/models/deletion.py

@@ -95,7 +95,7 @@ def get_candidate_relations_to_delete(meta: Meta) -> Generator[Any, None, None]:
     # N-N  (i.e., many-to-many) relations aren't candidates for deletion.
     return (
         f
-        for f in meta.get_fields(include_hidden=True)
+        for f in meta.get_fields(include_reverse=True)
         if f.auto_created and not f.concrete and f.one_to_many
     )
 

plain-models/plain/models/fields/__init__.py

@@ -147,14 +147,11 @@ class Field(RegisterLookupMixin, Generic[T]):
         "limit_choices_to",
         # Database-level options are not supported, see #21961.
         "on_delete",
-        "related_name",
         "related_query_name",
         "validators",
     )
 
     # Field flags
-    hidden = False
-
     many_to_many = None
     many_to_one = None
     one_to_many = None