تقرير فحص الأخطاء البرمجية

// File: app/Jobs/Tap/ProcessTapWebhookJob.php
// استبدل handleCancelled() (سطر ١٥٠-١٥٨)

-    private function handleCancelled(Order $order, TapWebhookDTO $dto): void
-    {
-        Order::where('id', $order->id)
-            ->where('status', OrderStatusEnum::Pending)
-            ->update([
-                'status'            => OrderStatusEnum::Cancelled,
-                'tap_charge_status' => 'CANCELLED',
-            ]);
-    }

+    private function handleCancelled(Order $order, TapWebhookDTO $dto): void
+    {
+        // استخدم Eloquent model ->update() عشان الـ Observer يشتغل
+        // والـ ReleaseOrderReservationAction تحرر المخزون.
+        $order->refresh();
+
+        if ($order->status !== OrderStatusEnum::Pending) {
+            return;
+        }
+
+        $order->update([
+            'status'            => OrderStatusEnum::Cancelled,
+            'tap_charge_status' => 'CANCELLED',
+        ]);
+    }

// File: app/Jobs/Tap/ProcessTapWebhookJob.php
// استبدل handleFailed() (سطر ١٤١-١٤٨)

-    private function handleFailed(Order $order, TapWebhookDTO $dto): void
-    {
-        Order::where('id', $order->id)
-            ->where('status', OrderStatusEnum::Pending)
-            ->update(['tap_charge_status' => 'FAILED']);
-
-        Log::warning('ProcessTapWebhookJob: payment failed', ['order_id' => $order->id]);
-    }

+    private function handleFailed(Order $order, TapWebhookDTO $dto): void
+    {
+        $order->refresh();
+
+        if ($order->status !== OrderStatusEnum::Pending) {
+            return;
+        }
+
+        // ألغي الطلب فوراً عشان الـ stock يتحرر عن طريق الـ Observer
+        $order->update([
+            'status'            => OrderStatusEnum::Cancelled,
+            'tap_charge_status' => 'FAILED',
+        ]);
+
+        Log::warning('ProcessTapWebhookJob: payment failed — order cancelled', [
+            'order_id' => $order->id,
+        ]);
+    }

// File: app/Http/Controllers/Payment/TapCallbackController.php
// عرف المتغيرات قبل الـ try block (سطر ٤١-٥٦)

+        $payment_method = null;
+        $amount         = null;
+        $currency       = null;
+
         try {
             $charge         = $tap->getCharge($tapId);
             $success        = ($charge['status'] ?? '') === 'CAPTURED';
             $payment_method = data_get($charge, 'card.brand', null);
             $amount         = data_get($charge, 'amount', null);
             $currency       = data_get($charge, 'currency', null);
         } catch (TapException $e) {
             // دلوقتي المتغيرات معرفة — الـ redirect هيشتغل صح
             return $this->toFailure($order, $payment_method, $amount, $currency);
         }

// الحل — جزء الـ Frontend:
// File: abajora-website/app/services/checkout.service.ts

// ١. ضيف country_id في الـ CheckoutAddress interface:
+   country_id: number    // ← ضيف الـ field ده

// ٢. في mapAddress() ابعت الـ country_id:
+     country_id: formAddr.countryId,

// ─────────────────────────────────────────────────
// الحل — جزء الـ Backend:
// File: app/Http/Requests/Checkout/PlaceOrderRequest.php

// ضيف validation rule:
+ 'shipping_address.country_id' => ['required', 'integer', 'exists:countries,id'],

// File: app/Services/Checkout/CheckoutService.php سطر ٩٣
- countryId: (int) ($data['shipping_address']['country_id'] ?? 0),
+ countryId: (int) $data['shipping_address']['country_id'],

// الحل: احذف resolveCartWeight() واستخدم بيانات buildItems()
// في placeOrder()، حرك الشحن بعد buildItems():

+ $orderItems = $this->buildItems($data['items']);
+ $subtotal = $orderItems->sum('line_total');
+
+ $shipping = $this->shippingCalculator->calculate(
+     countryId:     (int) $data['shipping_address']['country_id'],
+     subtotal:      $subtotal,
+     totalWeightKg: $orderItems->sum('_weight'),
+ );

// في buildItems()، ضيف الوزن:
+ '_weight' => (float) ($product->weight ?? 0) * $qty,

// وامسح resolveCartWeight() خالص (سطر ٣٤٣-٣٥٦)

// migration جديد:
+ Schema::table('orders', function (Blueprint $table): void {
+     $table->unique('order_reference');
+ });

// File: app/Services/Payments/PaymentService.php
// ضيف الحماية بعد سطر ١١٦:

+        // امنع إعادة المحاولة لو في charge لسه شغال
+        if ($order->tap_charge_status === 'INITIATED') {
+            throw new \RuntimeException(
+                "مينفعش تعيد المحاولة — في عملية دفع لسه شغالة للطلب {$order->order_reference}."
+            );
+        }

// File: app/Jobs/Orders/CancelUnpaidOrderJob.php
// ضيف call صريح بعد الـ update (حوالي سطر ٧٠):

         $order->update(['status' => OrderStatusEnum::Cancelled]);

+        // حرر المخزون صراحةً — متعتمدش على الـ Observer لوحده
+        app(ReleaseOrderReservationAction::class)->execute($order);

// File: app/Services/Shipping/ShippingCalculatorService.php

- Cache::tags(['shipping_zones'])->remember(...);

+ // استخدم Cache::remember() بدون tags — بيشتغل على أي driver
+ Cache::remember("shipping_zone_{$countryId}", 3600, function () use (...) {
+     return ...;
+ });

// نفس حل BUG-004 — ضيف validation في الباك اند:
+ 'shipping_address.country_id' => ['required', 'integer', 'exists:countries,id'],

// وعدل الداشبورد يبعت country_id

// وحد response الـ coupon validation API:
+ return response()->json([
+     'valid'                => true,
+     'discount_amount'      => $discount,
+     'total_after_discount' => $grandTotal,
+ ]);

// File: app/Http/Requests/Checkout/PlaceOrderRequest.php
+ 'shipping_address.country_id' => ['required', 'integer', 'exists:countries,id'],
+ 'billing_address.country_id'  => ['required_if:same_as_shipping,false', 'integer', 'exists:countries,id'],

// File: CheckoutService.php سطر ٩٣ — شيل الـ fallback:
- countryId: (int) ($data['shipping_address']['country_id'] ?? 0),
+ countryId: (int) $data['shipping_address']['country_id'],

// ١. migration جديد — ضيف عمود tracking_number:
+ $table->string('tracking_number', 200)->nullable()->after('carrier_name');

// ٢. File: SalasaWebhookService.php سطر ١٨٤:
- $fields['carrier_name'] = $dto->trackingNumber;
+ $fields['tracking_number'] = $dto->trackingNumber;

// ١. ضيف API endpoint في الباك اند:
//    GET /api/user/v1/checkout/shipping-estimate
//    Params: country_id, subtotal
//    Returns: { shipping_cost: number, method: string }

// ٢. في checkout.vue، ضيف:
+ const shippingCost = ref(0)
+ const shippingLoading = ref(false)
+
+ async function fetchShippingEstimate() {
+   shippingLoading.value = true
+   try {
+     const res = await $fetch('/api/user/v1/checkout/shipping-estimate', {
+       params: { country_id: form.shipping.countryId, subtotal: totalPrice.value }
+     })
+     shippingCost.value = res.data?.shipping_cost ?? 0
+   } finally { shippingLoading.value = false }
+ }

// ٣. استدعي fetchShippingEstimate() لما المستخدم يتقدم للخطوة ٣

// خزن الويب هوكس الفاشلة عشان تقدر تعيد تشغيلها:

+ } catch (\Throwable $e) {
+     Log::critical('Salasa webhook processing failed', [
+         'error' => $e->getMessage(),
+         'trace' => $e->getTraceAsString(),
+     ]);
+
+     DB::table('failed_salasa_webhooks')->insert([
+         'payload'    => json_encode($request->all()),
+         'error'      => $e->getMessage(),
+         'created_at' => now(),
+     ]);
+
+     return response()->json(['status' => 'error'], 200);
+ }

// File: app/Services/Salasa/SyncProductsService.php سطر ٢٢٠
// امسح الـ quantity_reserved من السينك:

- 'quantity_reserved' => $reserved,

+ // متعملش سينك للـ quantity_reserved من سلاسة.
+ // الـ quantity_reserved المحلي بيتدار من:
+ //   CheckoutService (حجز) و ReleaseOrderReservationAction (تحرير)
+ // سلاسة الـ "reserved" بتاعها = حجوزات مخزن مختلفة تماماً.

// File: abajora-website/app/pages/checkout.vue
// قلل صلاحية الكوكي وضيف validation للسلة:

- maxAge: 60 * 60 * 24 * 7, // 7 days
+ maxAge: 60 * 60 * 2, // ساعتين بس

// ضيف cart hash عشان تتأكد السلة ما اتغيرتش:
+ const cartHash = computed(() =>
+   items.value.map(i => `${i.product.id}:${i.quantity}`).join(',')
+ )
+ if (checkoutCookie.value?.cartHash &&
+     checkoutCookie.value.cartHash !== cartHash.value) {
+   checkoutCookie.value = null // السلة اتغيرت — امسح البيانات القديمة
+ }