From 801844bc006e61583a6f7b5e5dc5642582fb938f Mon Sep 17 00:00:00 2001 From: Oliver Walters Date: Thu, 18 Jun 2026 09:31:25 +0000 Subject: [PATCH] Revert code, remove create method against StockTrackingList --- src/backend/InvenTree/stock/api.py | 25 +------------------------ src/backend/InvenTree/stock/test_api.py | 19 ------------------- 2 files changed, 1 insertion(+), 43 deletions(-) diff --git a/src/backend/InvenTree/stock/api.py b/src/backend/InvenTree/stock/api.py index 536398cb85..23fdeacfe1 100644 --- a/src/backend/InvenTree/stock/api.py +++ b/src/backend/InvenTree/stock/api.py @@ -1580,7 +1580,7 @@ class StockTrackingFilter(FilterSet): class StockTrackingList( - SerializerContextMixin, DataExportViewMixin, OutputOptionsMixin, ListCreateAPI + SerializerContextMixin, DataExportViewMixin, OutputOptionsMixin ): """API endpoint for list view of StockItemTracking objects. @@ -1661,29 +1661,6 @@ class StockTrackingList( return Response(data) - def create(self, request, *args, **kwargs): - """Create a new StockItemTracking object. - - Here we override the default 'create' implementation, - to save the user information associated with the request object. - """ - # Clean up input data - data = self.clean_data(request.data) - - serializer = self.get_serializer(data=data) - serializer.is_valid(raise_exception=True) - - stock_item = serializer.validated_data['item'] - deltas = dict(serializer.validated_data.get('deltas') or {}) - deltas['quantity'] = float(stock_item.quantity) - - serializer.save(user=request.user, deltas=deltas) - - headers = self.get_success_headers(serializer.data) - return Response( - serializer.data, status=status.HTTP_201_CREATED, headers=headers - ) - filter_backends = SEARCH_ORDER_FILTER ordering = '-date' diff --git a/src/backend/InvenTree/stock/test_api.py b/src/backend/InvenTree/stock/test_api.py index c887d1bfe9..4299cfa83c 100644 --- a/src/backend/InvenTree/stock/test_api.py +++ b/src/backend/InvenTree/stock/test_api.py @@ -2891,25 +2891,6 @@ class StockTrackingTest(StockAPITestCase): assert_fnc=lambda x: x.data['results'][0], ) - def test_create_manual_entry(self): - """Test that a manual stock tracking entry can be created via the API.""" - url = self.get_url() - stock_item = StockItem.objects.first() - - response = self.post( - url, - {'item': stock_item.pk, 'notes': 'Manual tracking entry'}, - expected_code=201, - ) - - entry = StockItemTracking.objects.get(pk=response.data['pk']) - - # user = request.user - self.assertEqual(entry.user, self.user) - - # quantity = StockItem.quantity - self.assertEqual(float(entry.deltas['quantity']), float(stock_item.quantity)) - class StockAssignTest(StockAPITestCase): """Unit tests for the stock assignment API endpoint, where stock items are manually assigned to a customer."""