set fetch attributes to false by default

Author: Archmonger

src/django_idom/hooks.py

@@ -17,8 +17,8 @@
 from django_idom.types import (
     IdomWebsocket,
     Mutation,
-    QueryOptions,
     Query,
+    QueryOptions,
     _Params,
     _Result,
 )
@@ -203,7 +203,7 @@ def reset() -> None:
 def _postprocess_django_query(fetch_cls: QueryOptions) -> None:
     """Recursively fetch all fields within a `Model` or `QuerySet` to ensure they are not performed lazily.
 
-    Some behaviors can be modified through `OrmFetch` attributes."""
+    Some behaviors can be modified through `query_options` attributes."""
     data = fetch_cls._data
 
     # `QuerySet`, which is effectively a list of `Model` instances

src/django_idom/types.py

@@ -57,6 +57,9 @@ class ViewComponentIframe:
 class QueryOptions:
     """A Django ORM query function, alongside some configuration values."""
 
+    def __call__(self, *args, **kwargs):
+        return self.func(*args, **kwargs)
+
     func: Callable
     """Callable that fetches ORM object(s)."""
 
@@ -65,10 +68,10 @@ class QueryOptions:
     This is only intended to be set automatically by the `use_query` hook."""
 
     postprocessor_options: dict[str, Any] = field(
-        default_factory=lambda: {"many_to_many": True, "many_to_one": True}
+        default_factory=lambda: {"many_to_many": False, "many_to_one": False}
     )
     """Configuration values usable by the `postprocessor`."""
 
     postprocessor: Callable[[QueryOptions], None] | None = None
-    """A post processing callable that can read/modify the `OrmFetch` object. If unset, the default fetch
+    """A post processing callable that can read/modify the `QueryOptions` object. If unset, the default fetch
     handler is used to prevent lazy loading of Django fields."""

tests/test_app/components.py

@@ -166,10 +166,12 @@ def create_relational_parent() -> RelationalParent:
     return parent
 
 
+@query_options(many_to_many=True, many_to_one=True)
 def get_relational_parent_query():
     return RelationalParent.objects.first() or create_relational_parent()
 
 
+@query_options(many_to_many=True, many_to_one=True)
 def get_foriegn_child_query():
     child = ForiegnChild.objects.first()
     if not child:
@@ -207,41 +209,6 @@ def relational_query():
     )
 
 
-@query_options(many_to_many=False, many_to_one=False)
-def get_relational_parent_query_fail():
-    return RelationalParent.objects.first() or create_relational_parent()
-
-
-@component
-def relational_query_fail_mtm():
-    relational_parent = use_query(get_relational_parent_query_fail)
-
-    if not relational_parent.data:
-        return
-
-    return html.div(
-        {"id": "relational-query-fail-mtm"},
-        html.div(
-            f"Relational Parent Many To Many Fail: {relational_parent.data.many_to_many.all()}"
-        ),
-    )
-
-
-@component
-def relational_query_fail_mto():
-    relational_parent = use_query(get_relational_parent_query_fail)
-
-    if not relational_parent.data:
-        return
-
-    return html.div(
-        {"id": "relational-query-fail-mto"},
-        html.div(
-            f"Relational Parent Many to One: {relational_parent.data.foriegnchild_set.all()}"
-        ),
-    )
-
-
 def get_todo_query():
     return TodoItem.objects.all()
 

tests/test_app/templates/base.html

@@ -32,8 +32,6 @@ <h1>IDOM Test Page</h1>
   <div>{% component "test_app.components.django_js" %}</div>
   <div>{% component "test_app.components.unauthorized_user" %}</div>
   <div>{% component "test_app.components.authorized_user" %}</div>
-  <div>{% component "test_app.components.relational_query_fail_mtm" %}</div>
-  <div>{% component "test_app.components.relational_query_fail_mto" %}</div>
   <div>{% component "test_app.components.relational_query" %}</div>
   <div>{% component "test_app.components.todo_list" %}</div>
   <div>{% component "test_app.components.view_to_component_sync_func" %}</div>