fix: fix order items status update

Author: rahulchaudhary2002

app/Http/Controllers/Orders/OrderItemController.php

@@ -102,11 +102,22 @@ public function bulkUpdateStatus(Request $request): JsonResponse
         return response()->json(['success' => true, 'updated' => $updated]);
     }
 
-    public function updateStatus(Request $request, OrderItem $orderItem, ?Order $order = null): JsonResponse
+    public function updateStatus(Request $request, Order $order, OrderItem $orderItem): JsonResponse
+    {
+        abort_if($orderItem->order_id !== $order->id, 404, 'Item does not belong to this order.');
+        abort_if($orderItem->status === 'cancelled', 422, 'Cannot update a cancelled item.');
+
+        $validated = $request->validate([
+            'status' => ['required', 'string', 'in:sent_to_kitchen,preparing,ready,served'],
+        ]);
+
+        $this->orderService->changeItemStatus($orderItem, $validated['status']);
+
+        return response()->json(['success' => true, 'status' => $orderItem->status]);
+    }
+
+    public function updateItemStatus(Request $request, OrderItem $orderItem): JsonResponse
     {
-        if ($order !== null) {
-            abort_if($orderItem->order_id !== $order->id, 404, 'Item does not belong to this order.');
-        }
         abort_if($orderItem->status === 'cancelled', 422, 'Cannot update a cancelled item.');
 
         $validated = $request->validate([

routes/orders.php

@@ -95,7 +95,7 @@
         ->middleware('permission:order-items-update')
         ->name('orders.items.status');
 
-    Route::patch('order-items/{orderItem}/status', [OrderItemController::class, 'updateStatus'])
+    Route::patch('order-items/{orderItem}/status', [OrderItemController::class, 'updateItemStatus'])
         ->middleware('permission:order-items-update')
         ->name('order-items.status');