• DevPilot's avatar
    fix(payments, cashier, treasury): void-type gaps, unpaid fines via queue,... · 366460fd
    DevPilot authored
    fix(payments, cashier, treasury): void-type gaps, unpaid fines via queue, invisible guest payments, missing permission checks
    
    Payments/Installments:
    - PaymentLifecycleService::onPaymentVoided() ما كانتش بترجع حالة العضو
      إلا لـ membership_fee/down_payment بس — رغم إن foreign_membership_fee
      وsports_membership_fee وseasonal_fee كلهم بينشّطوا العضو، إلغاء أي واحدة
      منهم كان بيسيب العضو "فعال" للأبد من غير أي غطاء دفع حقيقي.
      MembershipPaymentGuard::deactivateMember() نفسها كانت أصلًا عارفة تتعامل
      مع الخمس أنواع صح — المشكلة كانت في القايمة اللي بتقرر تنادي عليها.
    - غرامة بتترسل لطابور الدفع (fine) كانت بتتحصّل وتتطبع إيصال، لكن صف
      الغرامة في جدول fines فضل زي ما هو "غير مسدد" للأبد — مفيش حد كان بيسمع
      الحدث ده أصلًا. أي فحص رصيد أو أهلية كارنيه كان هيفضل يقول إن الغرامة
      لسه مستحقة حتى بعد سدادها فعليًا. اتضاف مستمع بيقفل الغرامة المحددة.
    - GET /api/v1/payments كان مكسور تمامًا — بيفلتر بعمود is_archived مش
      موجود أصلًا في جدول payments (الصح is_voided).
    - شاشة كل المدفوعات والتقرير اليومي كانوا بيستخدموا INNER JOIN مع
      members، فأي دفعة لعميل زائر (member_id فاضي — زي تسجيل رياضي لغير
      عضو) كانت تختفي تمامًا من الشاشتين دول رغم إنها محصّلة فعليًا
      (23 دفعة بحوالي 41,800 جنيه في البيانات الحية). اتحول لـ LEFT JOIN.
    - زرار إلغاء الدفعة كان بيتحكم بصلاحية payment.void_receipt بينما الـ
      route بتاعه محتاج payment.void — محدش في الأدوار الحية عنده الصلاحية
      التانية، فكان عمليًا محدش غير super_admin يقدر يلغي دفعة من الشاشة دي.
    - عضوية موسمية بتتفعّل من كاشير مباشرة من غير ما تعدّي على
      activateIncludedDependents — يعني زوجة/أبناء العضو الموسمي المرفقين في
      نفس الرسم ما كانوش بيتفعّلوا معاه.
    - حذف DefaultChecker.php الميتة تمامًا (صفر استدعاء) — نفس منطق فحص
      التعثر بالظبط موجود وشغّال فعليًا في cron/jobs/InstallmentDefaultJob.php.
    
    Treasury/Cashier:
    - شاشات موديول Treasury بالكامل كانت من غير أي فحص can() على الإطلاق —
      كل الأزرار بتظهر لأي حد شايف الشاشة بغض النظر عن صلاحياته الحقيقية.
      اتأكد إن ده مش نظري: أدوار حية زي auditor وtreasury_manager وmain_cashier
      عندهم جزء من صلاحيات الخزنة بس مش كلها، فكانوا بياخدوا 403 على أزرار
      شايفينها. كل زرار دلوقتي بيتحقق من نفس صلاحية الـ route بتاعه.
      نفس الحاجة اتصلحت في زرار فتح/قفل وردية كاشير، وزرار طباعة الإيصال اللي
      كان بيتحكم بصلاحية receipt.print بدل payment.view الحقيقية.
    - عداد "طلبات معلقة" في لوحة الخزنة الفرعية كان بيستخدم قايمة أنواع دفع
      أقدم من القايمة الحقيقية المستخدمة في الطابور نفسه — ناقصة رسوم عضوية
      أجانب/رياضية وبعض أنواع الأنشطة الرياضية.
    
    Plus extreme-detail reference docs for Payments/Installments, Cashier, and Treasury.
    366460fd
Name
Last commit
Last update
..
Academies Loading commit data...
AcademyContracts Loading commit data...
AccessMatrix Loading commit data...
Accounting Loading commit data...
Achievements Loading commit data...
ActivitySubscriptions Loading commit data...
Alerts Loading commit data...
Archive Loading commit data...
Auctions Loading commit data...
Audit Loading commit data...
Auth Loading commit data...
Branches Loading commit data...
Carnets Loading commit data...
Cashier Loading commit data...
Children Loading commit data...
Coaches Loading commit data...
Dashboard Loading commit data...
Death Loading commit data...
Disciplines Loading commit data...
Divorce Loading commit data...
Documents Loading commit data...
Facilities Loading commit data...
FacilityDashboards Loading commit data...
FacilityGrids Loading commit data...
Fines Loading commit data...
Foreign Loading commit data...
Forms Loading commit data...
HR Loading commit data...
Honorary Loading commit data...
Installments Loading commit data...
Interviews Loading commit data...
Inventory Loading commit data...
MatchCenter Loading commit data...
MedicalBoard Loading commit data...
Members Loading commit data...
News Loading commit data...
Notifications Loading commit data...
ParentPortal Loading commit data...
Payments Loading commit data...
PlayerAffairs Loading commit data...
PlayerApi Loading commit data...
PlayerAuth Loading commit data...
PlaygroundAdmin Loading commit data...
PoolManagement Loading commit data...
Pricing Loading commit data...
Procurement Loading commit data...
Receipts Loading commit data...
Rentals Loading commit data...
Reports Loading commit data...
Reservations Loading commit data...
Roles Loading commit data...
Rules Loading commit data...
Sales Loading commit data...
Scheduling Loading commit data...
Seasonal Loading commit data...
ServiceCatalog Loading commit data...
Sessions Loading commit data...
Settings Loading commit data...
Sports Loading commit data...
SportsActivity Loading commit data...
SportsDashboard Loading commit data...
Spouses Loading commit data...
Subscriptions Loading commit data...
Support Loading commit data...
Temporary Loading commit data...
Tournaments Loading commit data...
TrainerPortal Loading commit data...
Training Loading commit data...
TrainingGroups Loading commit data...
Transfers Loading commit data...
Treasury Loading commit data...
Tutorials Loading commit data...
Users Loading commit data...
Waiver Loading commit data...
Workflow Loading commit data...