Add get_request_handler so htmx can re-use response logic

Author: davegaeddert

bolt-htmx/bolt/htmx/views.py

@@ -22,19 +22,18 @@ def get_template_response(self, context=None) -> HttpResponse:
 
         return super().get_template_response(context=context)
 
-    def get_response(self):
+    def get_request_handler(self):
         if self.is_htmx_request:
             # You can use an htmx_{method} method on views
             # (or htmx_{method}_{action} for specific actions)
             method = f"htmx_{self.request.method.lower()}"
             if self.htmx_action_name:
                 method += f"_{self.htmx_action_name}"
 
-            handler = getattr(self, method, None)
-            if handler:
-                return handler()
+            if handler := getattr(self, method, None):
+                return handler
 
-        return super().get_response()
+        return super().get_request_handler()
 
     def get_template_names(self):
         # TODO is this part necessary anymore?? can I replace those with fragments now?

bolt/views/base.py

@@ -53,7 +53,8 @@ def view(request, *args, **kwargs):
 
         return view
 
-    def get_response(self) -> HttpResponseBase:
+    def get_request_handler(self) -> callable:
+        """Return the handler for the current request method."""
         if not self.request.method:
             raise AttributeError("HTTP method is not set")
 
@@ -63,6 +64,12 @@ def get_response(self) -> HttpResponseBase:
         handler = getattr(
             self, self.request.method.lower(), self._http_method_not_allowed
         )
+
+        return handler
+
+    def get_response(self) -> HttpResponseBase:
+        handler = self.get_request_handler()
+
         try:
             result = handler()
         except HttpResponseException as e: