• DevPilot's avatar
    fix(sports-activity): 4 real bugs found while documenting core screens · 28ea8431
    DevPilot authored
    - تقرير الماليات (SaFinanceReportService): الـ switch بتاع تكلفة المدربين
      كان بيقارن بقيم زي 'monthly'/'hourly' مش موجودة في enum الحقيقي
      (monthly_fixed/per_session/per_player/hybrid/salary) — يعني أي مدرب
      مش per_session كان بيتحسب تكلفته صفر، وصافي الربح المعروض في التقرير
      كان مبالغ فيه بشكل منهجي. اتصلحت لتغطي الخمس نماذج كلهم، بنفس صيغة
      الحساب المستخدمة فعلًا في Coaches\Services\CoachPaymentService
      (مدرب الأكاديمية 'salary' متعمّد يفضل صفر — أجره بيتحسب من خلال تسوية
      عقد الأكاديمية مش كبند تكلفة مباشر).
    - FacilityUnit::$fillable ما كانش فيه expected_capacity، فالحقل كان
      بيتقبل في الفورم ويتفلتر بصمت قبل ما يوصل لقاعدة البيانات.
    - EnrollmentService::forceEnroll() كان بيثبّت is_full=1 دايمًا بدل ما
      يحسبه — تجاوز واحد في مجموعة سعتها 20 كان بيقفل التسجيل العادي على
      الـ 19 مكان الفاضيين اللي بعده.
    - زرار سحب اللاعب من المجموعة كان بيتفعّل بصلاحية sa.group.enroll بينما
      الـ route نفسه محتاج sa.group.manage — تضارب بين الاتنين حسب الدور.
    28ea8431
Name
Last commit
Last update
..
Core Loading commit data...
Middleware Loading commit data...
Modules Loading commit data...
Shared Loading commit data...