From fddaaf984415623d0d6e90b731a9e19b85a73638 Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Tue, 21 Jan 2025 09:55:44 +1100
Subject: [PATCH] Code cleanup (#8931)

- Remove TemplatePrintBase class
- Not referenced in code
---
 src/backend/InvenTree/report/api.py | 56 ++---------------------------
 1 file changed, 2 insertions(+), 54 deletions(-)

diff --git a/src/backend/InvenTree/report/api.py b/src/backend/InvenTree/report/api.py
index f32e7f7d8b..ac2ce9cfdc 100644
--- a/src/backend/InvenTree/report/api.py
+++ b/src/backend/InvenTree/report/api.py
@@ -6,13 +6,12 @@ from django.template.exceptions import TemplateDoesNotExist
 from django.urls import include, path
 from django.utils.decorators import method_decorator
 from django.utils.translation import gettext_lazy as _
-from django.views.decorators.cache import cache_page, never_cache
+from django.views.decorators.cache import never_cache
 
 from django_filters import rest_framework as rest_filters
 from django_filters.rest_framework import DjangoFilterBackend
 from rest_framework import permissions
 from rest_framework.generics import GenericAPIView
-from rest_framework.request import clone_request
 from rest_framework.response import Response
 
 import common.models
@@ -25,12 +24,7 @@ import report.serializers
 from InvenTree.api import BulkDeleteMixin, MetadataView
 from InvenTree.exceptions import log_error
 from InvenTree.filters import InvenTreeSearchFilter
-from InvenTree.mixins import (
-    ListAPI,
-    ListCreateAPI,
-    RetrieveAPI,
-    RetrieveUpdateDestroyAPI,
-)
+from InvenTree.mixins import ListAPI, ListCreateAPI, RetrieveUpdateDestroyAPI
 from plugin.builtin.labels.inventree_label import InvenTreeLabelPlugin
 from plugin.registry import registry
 
@@ -45,52 +39,6 @@ class TemplatePermissionMixin:
     ]
 
 
-@method_decorator(cache_page(5), name='dispatch')
-class TemplatePrintBase(RetrieveAPI):
-    """Base class for printing against templates."""
-
-    @method_decorator(never_cache)
-    def dispatch(self, *args, **kwargs):
-        """Prevent caching when printing report templates."""
-        return super().dispatch(*args, **kwargs)
-
-    def check_permissions(self, request):
-        """Override request method to GET so that also non superusers can print using a post request."""
-        if request.method == 'POST':
-            request = clone_request(request, 'GET')
-        return super().check_permissions(request)
-
-    def post(self, request, *args, **kwargs):
-        """Respond as if a POST request was provided."""
-        return self.get(request, *args, **kwargs)
-
-    def get(self, request, *args, **kwargs):
-        """GET action for a template printing endpoint.
-
-        - Items are expected to be passed as a list of valid IDs
-        """
-        # Extract a list of items to print from the queryset
-        item_ids = []
-
-        for value in request.query_params.get('items', '').split(','):
-            try:
-                item_ids.append(int(value))
-            except Exception:
-                pass
-
-        template = self.get_object()
-
-        items = template.get_model().objects.filter(pk__in=item_ids)
-
-        if len(items) == 0:
-            # At least one item must be provided
-            return Response(
-                {'error': _('No valid objects provided to template')}, status=400
-            )
-
-        return self.print(request, items)
-
-
 class ReportFilterBase(rest_filters.FilterSet):
     """Base filter class for label and report templates."""