# ملاحظات مراجعة النظام القديم

- ملف `files/funcations.php` يعمل كواجهة تنفيذ واحدة عبر `fn` في Query String، وهذا يجعل تتبع الصلاحيات والأخطاء صعبًا.
- كثير من الدوال تبني SQL بدمج القيم مباشرة داخل النص، مما يرفع خطر SQL Injection ويمنع الاستفادة من Bind Parameters.
- تقارير المقارنة السنوية كانت تنفذ الاستعلام مرتين وتستخدم `str_replace` لتغيير السنة، وهذا خطر وبطيء.
- صفحة Revenue تستخدم `jQuery.post` ثم `eval` لتحويل البيانات إلى JavaScript array. في النسخة الجديدة تم استبدال ذلك بـ `json_encode` + `JSON.parse` فقط.
- بيانات الاتصال القديمة كانت داخل ملفات PHP. النسخة الجديدة تستخدم `.env` خارج مجلد `public`.
- الواجهة الجديدة Server Rendered بالكامل ولا تعتمد على API endpoint مثل `funcations.php?fn=...`.
