📊 تقرير تحليل مشروع COREX ERP
بقلم: Senior Laravel Developer (10 سنين خبرة)

🔍 أولاً: ما هو المشروع؟
COREX هو نظام ERP محاسبي متعدد المستأجرين (Multi-Tenant) مبني على Laravel 12، يستهدف السوق العربي. يشمل حالياً:

الموديول	الحالة
📒 المحاسبة (دليل حسابات، قيود يومية، ميزان مراجعة)	✅ موجود
🛒 المبيعات (فواتير، مدفوعات، مرتجعات)	✅ موجود
📦 المشتريات (فواتير، مرتجعات، مدفوعات)	✅ موجود
🏪 المخزون (أصناف، تصنيفات، وحدات، مخازن)	✅ جزئي
👥 العملاء والموردين	✅ موجود
💰 الخزينة والمناوبات	✅ موجود
📈 التقارير (ميزانية، قائمة دخل)	✅ خدمات موجودة - بلا routes!
👤 الصلاحيات والأدوار	❌ غائب تماماً
📋 الموارد البشرية	❌ غائب
🏭 الإنتاج/التصنيع	❌ غائب
🔔 الإشعارات	❌ غائب
🚨 المشاكل الحالية (مرتبة بالأولوية)
🔴 مشاكل حرجة (Critical)
1. لا يوجد نظام صلاحيات (Roles & Permissions)
// الحالة الآن: أي مستخدم يقدر يعمل أي حاجة!
Route::middleware('auth')->group(function () {
    Route::resource('items', ItemController::class); // ❌ بدون permission check
});
الحل: إضافة spatie/laravel-permission مع Gates & Policies.

2. Multi-Tenancy بدوية ومتكررة
// في كل controller بيتكرر نفس الكود!
private function ownerId(): int
{
    $u = auth()->user();
    return (int) ($u->owner_user_id ?? $u->id);
}
هذا الكود مكرر في كل controller (20+ مرة). لازم يكون في Middleware أو Base Controller مرة واحدة.

3. Fat Controllers (Controllers ضخمة جداً)
SalesInvoiceController.php = 798 سطر 🔴
PurchaseInvoiceController.php = 605 سطر 🔴
Business Logic موجودة في Controllers بدل Services. يجب استخدام:

Service Classes (بعضها موجود لكن ناقص)
Action Classes (single-responsibility)
4. لا يوجد تتبع حركات المخزون (Stock Movements)
// لما بتبيع صنف، الكمية مش بتتخصم من المخزن!
// مفيش جدول stock_movements أو inventory_transactions
نظام ERP بدون inventory tracking = نظام ناقص.

5. خدمات التقارير بلا Routes!
// app/Services/Reporting/BalanceSheetGenerator.php ✅ موجود
// app/Services/Reporting/IncomeStatementGenerator.php ✅ موجود
// لكن في web.php مفيش أي route للتقارير ❌
🟠 مشاكل متوسطة (Major)
6. تسمية Models غير متسقة
// ❌ Plural instead of Singular
Items::class      // المفروض Item
Stores::class     // المفروض Store
Treasuries::class // المفروض Treasury
هذا مخالف لـ Laravel conventions.

7. لا يوجد Form Requests لأهم العمليات
StoreInvoiceRequest ❌ غير موجود
StorePurchaseInvoiceRequest ❌ غير موجود
Validation مضمن في Controllers مباشرة (anti-pattern)
8. لا يوجد API Layer
مفيش API routes
مفيش API versioning
مفيش Eloquent API Resources
صعب يتكامل مع mobile app أو third-party
9. لا يوجد نظام Audit Trail حقيقي
// الحالة الحالية: string فقط!
'updated_by' => 'string'
// المفروض: جدول audit_logs كامل بـ old_values, new_values, IP, user_agent
10. عمود مكرر في sales_invoices
// في نفس الجدول:
$table->date('invoice_date'); // ✅
$table->date('date')->nullable(); // ❌ مكرر ونفس الغرض!
11. SalesReturn بلا Edit Route
// في web.php:
Route::get('sales_returns/{id}', ...)    // show ✅
Route::post('sales_returns/{id}/cancel') // cancel ✅
// مفيش edit/update! ❌
12. DB::transaction في Controllers
// في SalesInvoiceController:
return DB::transaction(function () use ($data, $ownerId) {
    // Business logic هنا ❌
});
// المفروض في Service class
🟡 مشاكل ثانوية (Minor)
13. الـ README لسه Default Laravel
مفيش توثيق للمشروع
مفيش شرح للـ modules
مفيش setup instructions حقيقية
14. ملفات Debug كتير في root
COMPLETE_TEST_ANALYSIS.txt
FIXES_APPLIED.txt
REPAIR_PROGRESS.php
quick-check.php
... (20+ ملف debug/report)
هذه ملفات مؤقتة يجب حذفها أو نقلها.

15. لا يوجد Queue Jobs
Posting to ledger يحصل synchronously
PDF generation يحصل synchronously
في production هيسبب timeout
16. لا يوجد Livewire رغم وجوده في الـ stack
الـ stack يشمل Livewire v4 لكن مش مستخدم
Interactive UI محدود
17. Settings Model محدود جداً
// الحالة الحالية:
protected $fillable = ['name', 'phone', 'address', 'logo', ...];

// المفروض يشمل:
// - vat_number, vat_rate
// - fiscal_year_start
// - base_currency
// - invoice_prefix
// - decimal_places
// - enable_inventory_tracking
18. لا يوجد Caching
قائمة الحسابات بتُجلب كل request
الإعدادات بتُجلب كل request
مفيش Redis/Cache layer
💡 المقترحات لتطوير ERP احترافي
المرحلة الأولى: إصلاح الأساس 🏗️
#	المهمة	الأولوية
1	تثبيت spatie/laravel-permission وإضافة Roles & Policies	🔴 عاجل
2	إنشاء TenantMiddleware بدل تكرار ownerId()	🔴 عاجل
3	إنشاء جدول stock_movements وربطه بالمبيعات والمشتريات	🔴 عاجل
4	إضافة Routes للتقارير وإنشاء ReportsController	🔴 عاجل
5	تقسيم SalesInvoiceController لـ Service + Action classes	🟠 مهم
6	إنشاء Form Requests للفواتير	🟠 مهم
7	توحيد أسماء Models (Item, Store, Treasury)	🟠 مهم
المرحلة الثانية: الموديولات الناقصة 📦
HR Module:
├── Employees (موظفين)
├── Departments (أقسام)
├── Attendance (حضور وانصراف)
└── Payroll (رواتب)

Inventory Module (تحسين):
├── Stock Movements (حركة المخزون)
├── Stock Taking (جرد)
├── Reorder Alerts (تنبيهات النفاد)
└── Barcode Scanning

POS Module:
├── Point of Sale Screen
├── Cash Drawer Integration
└── Receipt Printing

Fixed Assets Module:
├── Asset Register
├── Depreciation
└── Asset Disposal
المرحلة الثالثة: الجودة والأداء ⚡
✅ Queue Jobs لـ PDF generation & ledger posting
✅ Redis Caching للبيانات الثابتة (settings, accounts)
✅ API Layer مع versioning (v1)
✅ Audit Trail كامل (spatie/laravel-activitylog)
✅ Notifications (تنبيهات عند اقتراب نفاد المخزون، الفواتير المتأخرة)
✅ Unit & Feature Tests شاملة (Pest)
✅ CI/CD Pipeline
المرحلة الرابعة: Multi-Tenancy حقيقي 🏢
حالياً النظام يستخدم user_id كـ tenant ID وهو محدود. للمستقبل:

Options:
1. Single DB + Row-level isolation (stancl/tenancy) - الحالي محسّن
2. Multiple Databases per tenant - أقوى لكن أعقد
📊 ملخص التقييم
المعيار	التقييم	ملاحظة
Architecture	⭐⭐⭐☆☆	Controllers ضخمة، Services جيدة
Database Design	⭐⭐⭐⭐☆	تصميم جيد لكن ناقص
Code Quality	⭐⭐⭐☆☆	تكرار، inconsistency
Security	⭐⭐☆☆☆	لا permissions، لا audit
Coverage	⭐⭐☆☆☆	موديولات كتير ناقصة
Testing	⭐⭐⭐☆☆	اختبارات محاسبة فقط
Performance	⭐⭐☆☆☆	لا caching، لا queues
المجموع	⭐⭐⭐☆☆	أساس جيد يحتاج تطوير
الخلاصة: المشروع فيه أساس محاسبي قوي وبنية منطقية جيدة، لكنه يحتاج إلى:

نظام صلاحيات عاجل
تتبع المخزون
تقسيم Controllers
إكمال الموديولات الناقصة