docs: Add comprehensive SimbaSql database documentation and analysis tools

docs/README.md

@@ -0,0 +1,100 @@
+# SimbaSql Database Documentation
+
+## Overview
+This directory contains comprehensive documentation and tools for the SimbaSql database used in the Portal application.
+
+## Files
+
+### 1. Analysis Reports
+- **`SimbaSql_analysis_report.md`** - Detailed technical analysis of the SimbaSql database structure, stored procedures, and performance/security issues
+- **`simba_sql_summary.md`** - Executive summary of findings and recommendations
+
+### 2. Tools & Scripts
+- **`convert_sp_to_php.php`** - Script to automatically convert SQL stored procedures to PHP classes
+- **`example_conversion.php`** - Example conversion of `asGetRight` stored procedure to PHP class
+- **`optimize_procedures.sql`** - SQL scripts for performance optimization and security fixes
+- **`debug_procedures.sql`** - Framework for debugging stored procedures
+
+## Database Statistics
+- **Total SQL Files**: 2,569
+- **Tables**: 440
+- **Stored Procedures**: 2,031
+- **Functions**: 89
+- **Views**: 4
+
+## Key Findings
+
+### Architecture
+- **Multi-tenant design**: All tables include `ma_cty` (company code)
+- **Module-based structure**: GL (Accounting), AR (Receivables), AP (Payables), SI (System), IN (Inventory), etc.
+- **Consistent naming conventions**: Clear prefixes/suffixes for tables and procedures
+
+### Issues Identified
+1. **Performance Issues**:
+   - 142 procedures use `SELECT *`
+   - Multiple procedures use CURSOR instead of set-based operations
+   - 8 procedures use dynamic SQL (`EXECUTE`)
+
+2. **Security Concerns**:
+   - Dynamic SQL in some procedures poses SQL injection risks
+   - Need for parameterized queries and input validation
+
+3. **Code Quality**:
+   - Inconsistent error handling
+   - Missing comments and documentation
+   - Hard-coded values and magic numbers
+
+## Recommendations
+
+### Phase 1: Security & Critical Performance (1-2 weeks)
+1. Fix dynamic SQL injection risks (8 procedures)
+2. Optimize frequently called procedures
+3. Add missing indexes on critical tables
+
+### Phase 2: Performance Optimization (2-3 weeks)
+1. Replace `SELECT *` with explicit column lists (142 procedures)
+2. Convert CURSOR operations to set-based operations
+3. Add proper indexing strategy
+
+### Phase 3: Code Quality & Documentation (1-2 weeks)
+1. Add standard error handling
+2. Improve comments and documentation
+3. Create standardized templates for new procedures
+
+### Phase 4: PHP Integration (2-4 weeks)
+1. Convert stored procedures to PHP classes
+2. Implement repository pattern
+3. Add unit tests and integration tests
+
+## Usage
+
+### Converting Stored Procedures to PHP
+```bash
+php convert_sp_to_php.php [stored_procedure_name]
+```
+
+### Running Optimization Scripts
+```sql
+-- Execute in SQL Server Management Studio
+:r optimize_procedures.sql
+```
+
+### Debugging Procedures
+```sql
+-- Use the debug framework
+:r debug_procedures.sql
+```
+
+## Related Documentation
+- See `../dbo/StoredProcedures/` for all SQL stored procedures
+- Refer to portal documentation for PHP integration examples
+- Check `SimbaSql_analysis_report.md` for detailed technical analysis
+
+## Maintenance
+- Update documentation when database schema changes
+- Run optimization scripts periodically
+- Review security fixes after major updates
+
+## Contributors
+- AI Agent Analysis: 2026-02-23
+- Last Updated: 2026-02-23

docs/SimbaSql_analysis_report.md

@@ -0,0 +1,115 @@
+# BÁO CÁO PHÂN TÍCH SIMBASQL DATABASE
+
+## Tổng quan
+- **Tổng số file SQL**: 2569
+- **Số bảng**: 440
+- **Số stored procedures**: 2031
+- **Số functions**: 89
+- **Số views**: 4
+
+## Phân loại Stored Procedures
+- **SP bắt đầu bằng 'as'**: 1831 (90.2%) - Procedure nghiệp vụ chính
+- **SP bắt đầu bằng 'sp'**: 1 (0.05%) - Procedure hệ thống
+- **SP bắt đầu bằng 'z'**: 188 (9.3%) - Procedure tạm/sao lưu
+- **SP bắt đầu bằng 'af'**: 11 (0.5%) - Aggregate functions
+
+## Cấu trúc Database
+### Các module chính
+1. **Kế toán tổng hợp (GL)**: GlCt, GlCt1, GlDmTk, GlCdTk
+2. **Phải thu (AR)**: ArCt1, ArPh1, ArDmKh, ArTt
+3. **Phải trả (AP)**: ApCt1, ApPh1, ApTt
+4. **Hệ thống (SI)**: SiSetup, SiDmBp, SiDmHd
+5. **Kho (IN)**: InDmLo, InDmVt
+6. **Sản xuất (CO)**: CoDmSpct, CoDmBomCt
+7. **Tài sản cố định (CA)**: CaCt1, CaDmKu
+8. **Nhân sự (HR)**: HrDmKQDanhGia, HrQTHopDongLD
+9. **Ngân hàng (BN)**: BnCt, BnCt1
+
+### Quy ước đặt tên
+- **Tiền tố bảng**: Ap, Ar, Gl, Si, Po, Sa, In, Co, Ca, Bn, Hr
+- **Hậu tố bảng**: Ct (chi tiết), Ph (tổng hợp), Tt (thanh toán), Dm (danh mục), Cd (số dư)
+- **Khóa chính**: Luôn bao gồm `ma_cty` (mã công ty) cho multi-tenant
+- **Audit columns**: cdate, cuser, ldate, luser
+
+## Vấn đề tiềm ẩn
+
+### 1. Performance Issues
+- **142 stored procedures** sử dụng `SELECT *` - có thể gây performance issue khi schema thay đổi
+- **8 stored procedures** sử dụng dynamic SQL (`EXECUTE`) - có nguy cơ SQL injection
+- **Nhiều procedures** sử dụng CURSOR - có thể gây performance issue với dữ liệu lớn
+
+### 2. Security Concerns
+- Dynamic SQL trong một số procedure có thể dẫn đến SQL injection
+- Một số procedure như `GetFolderUpdate` sử dụng dynamic SQL để thêm cột vào bảng
+
+### 3. Code Quality Issues
+- Không có consistent error handling trong nhiều procedure
+- Một số procedure thiếu comments và documentation
+- Sử dụng magic numbers và hard-coded values
+
+### 4. Design Issues
+- Sử dụng nhiều bảng tạm (#temp tables) có thể gây contention trong tempdb
+- Một số procedure quá phức tạp, khó maintain
+
+## Đề xuất tối ưu hóa
+
+### 1. Performance Optimization
+1. **Thay thế SELECT * bằng column list cụ thể**
+   - Ưu tiên các procedure được gọi thường xuyên
+   - Bắt đầu với các procedure reporting
+
+2. **Thay thế CURSOR bằng set-based operations**
+   - Sử dụng CTE, window functions
+   - Xem xét sử dụng WHILE loop với batch processing
+
+3. **Thêm index missing**
+   - Phân tích execution plans
+   - Thêm index trên các cột thường xuyên được filter, join
+
+4. **Tối ưu hóa queries**
+   - Tránh hàm trên cột trong WHERE clause
+   - Sử dụng EXISTS thay vì IN cho subqueries
+   - Tránh implicit conversion
+
+### 2. Security Improvements
+1. **Sử dụng parameterized queries** thay vì dynamic SQL
+2. **Implement proper input validation**
+3. **Sử dụng QUOTENAME()** cho object names trong dynamic SQL
+4. **Review và fix SQL injection vulnerabilities**
+
+### 3. Code Quality Improvements
+1. **Standardize error handling** với TRY-CATCH blocks
+2. **Thêm comments và documentation**
+3. **Remove unused procedures** (bắt đầu bằng 'z')
+4. **Standardize naming conventions**
+
+### 4. Monitoring và Maintenance
+1. **Tạo monitoring scripts** để track performance
+2. **Implement logging** cho các thao tác quan trọng
+3. **Regular index maintenance**
+4. **Update statistics** thường xuyên
+
+## Ưu tiên tối ưu hóa
+
+### High Priority
+1. **Procedure có dynamic SQL** - Security risk
+2. **Procedure reporting** - Performance impact lớn
+3. **Procedure được gọi thường xuyên** - High usage
+
+### Medium Priority
+1. **Procedure với SELECT *** - Performance và maintainability
+2. **Procedure với CURSOR** - Performance với large datasets
+
+### Low Priority
+1. **Cleanup unused procedures**
+2. **Code standardization**
+
+## Tools và Scripts đề xuất
+
+1. **SQL Server Profiler** để identify slow queries
+2. **Execution Plan analysis** để find missing indexes
+3. **DMVs** để monitor performance
+4. **Custom scripts** để automate optimization tasks
+
+## Kết luận
+SimbaSql là một database ERP đầy đủ với cấu trúc rõ ràng. Tuy nhiên, có nhiều cơ hội để cải thiện performance, security và maintainability. Ưu tiên hàng đầu là fix các security issues và optimize các procedure quan trọng.