أمر الشراء يفضل مفتوح للاستلام

اللي طلبته

«أمر الشراء يفضل مفتوح بدل ما يتعملش له إذن، حتى لو اتعمل له فاتورة شراء واتدفعت يبقى لسه ما اتسلمش — ممكن يتعمل له إذن استلام في أي وقت، لأن ساعات بندفع وبعدين بنستلم.»

وده امتداد طبيعي لخيار «الفوترة قبل الاستلام» اللي فتحناه قبل كده: فتحنا الفوترة، بس الأمر كان بيتقفل بعدها فما يبقاش ينفع تستلم عليه.

السبب الجذري — حقل واحد شايل بُعدين

أمر الشراء عنده تلات حقول بيوصفوا حالته:

if ($allBilled) {
    $this->update(['status' => PurchaseOrderStatus::FullyBilled]);
}

@if (o.status === 'confirmed' || o.status === 'partially_received')

النتيجة المجمّعة: بمجرد ما تفوتر أمر ما وصلش، بيدخل في طريق مسدود — الزرار بيختفي، والحالة مابتتحركش، والبضاعة لما توصل مالهاش مكان تتسجّل فيه على الأمر.

اللي غيّرته — تلات تعديلات

١. الفوترة مابقتش تدهس الاستلام خلفية

Modules/Purchases/app/Models/PurchaseOrder.php — recalculateBillStatus()

$stillAwaitingGoods = $items->contains(
    fn ($item) => (float) $item->received_quantity < (float) $item->quantity
);

if ($stillAwaitingGoods) {
    return;   // الأمر لسه عليه بضاعة لم تصل — حالة الاستلام تفضل زي ما هي
}

يعني الأمر اللي لسه مستني بضاعة بيحتفظ بحالة الاستلام بتاعته، والفوترة بتتسجّل في مكانها الصح: bill_status.

٢. الاستلام بقى يقدر يتحرّك وهو في حالة فوترة خلفية

نفس الملف — recalculateReceiveStatus(): ضفت حالتين للقائمة اللي بتسمح بالانتقال لـ«مستلم جزئيًا»:

PurchaseOrderStatus::PartiallyBilled,
PurchaseOrderStatus::FullyBilled,

وكمان لما الاتنين يخلصوا (وصلت كلها + متفوترة كلها) الحالة بتروح fully_billed بدل ما تدهس الفوترة بـfully_received — عشان الحقيقة الأحدث هي اللي تبان.

٣. الزرار بقى بيبان طول ما فيه بضاعة لم تصل واجهة

orders.component.html — الشرط بقى مبني على الحقل اللي بيقول الحقيقة فعلًا:

@if (
  o.receive_status !== 'complete' &&
  (o.status === 'confirmed' ||
   o.status === 'partially_received' ||
   o.status === 'partially_billed' ||
   o.status === 'fully_billed')
)

حالات الفوترة مضافة عشان الأوامر العالقة عندك دلوقتي يرجع لها الزرار، وreceive_status !== 'complete' عشان الزرار يختفي لما البضاعة توصل بالكامل.

إزاي اختبرته

كتبت ملف اختبار انحدار جديد: Modules/Purchases/tests/Feature/OrderStaysReceivableAfterBillingTest.php — خمس حالات:

مع الإصلاح:     5 passed
بعد تعطيل الحارس: 2 failed, 3 passed

وبعدها شغّلت سويت المشتريات كاملة عشان أتأكد إن مفيش حاجة تانية اتكسرت، وبنيت الواجهة (ng build) للتأكد من التعديل.

اللي ما اتغيّرش — عن قصد

• الطريق العادي (استلام ثم فوترة) شغّال بالظبط زي الأول. الحارس الجديد بيشتغل بس لما يكون فيه بضاعة لم تصل — ولو استلمت الأول، الشرط ما بيتحققش أصلًا، فالسلوك حرف بحرف زي ما كان. والاختبار رقم ٤ موجود عشان ده تحديدًا.
• سقف الفوترة ما اتغيّرش. لسه مستحيل تفوتر أكتر من كمية أمر الشراء.
• ما فتحتش الاستلام لحالات جديدة. الأمر لسه لازم يكون confirmed على الأقل — ما ضفتش draft ولا approved ولا الملغي.
• ما لمستش قاعدة البيانات. مفيش أي ترحيل بيانات هنا — التعديل في المنطق بس.

الأوامر العالقة عندك حاليًا

الأوامر اللي اتفوترت قبل الإصلاح واقفة دلوقتي على fully_billed / partially_billed وهي ما وصلتش. تعديل الواجهة عامل حسابهم: الزرار هيرجع يبان لهم فورًا، وأول ما تعمل إذن استلام الحالة هتتصحح لوحدها (بفضل التعديل رقم ٢).