8000
Skip to content

feat: Convert 337 stored procedures to PHP classes with detailed documentation - #1

Open
caothu159 wants to merge 3 commits into
mainfrom
feat/stored-procedures-migration
Open

feat: Convert 337 stored procedures to PHP classes with detailed documentation#1
caothu159 wants to merge 3 commits into
mainfrom
feat/stored-procedures-migration

Conversation

@caothu159
Copy link
Copy Markdown
Contributor

Summary

Converted 337 stored procedures from SimbaSql to PHP classes (18.4% of total 1831 as* procedures).

Changes

  • Created PHP classes for 337 stored procedures in src/StoredProcedures/
  • Added comprehensive documentation for each group (docs/procedures-*.md)
  • Updated master PROCEDURES.md with progress tracking
  • All classes include detailed comments for direct usage without reading SQL source
  • Each class uses ProcedureCaller::call() with proper default values
  • PHP syntax validated with php -l

Groups Processed

  • AP (18), AR (11), GL (20), SI (23), IN (22), PO (14), HR (10), FA (6), CA (7), SA (22), CO (19)
  • Get (17), Del (30), Post (10), Rpt (2), Upd (16), Ins (24), Cal (1), Chuyen (2), Filt (9), ReCal (17), Crt (5), Copy (5), Check (2), Chk (5)

Quality Assurance

  • Commented in Vietnamese with full parameter descriptions
  • Default values provided where applicable
  • Consistent structure across all classes
  • No modifications to original SQL source

Next Steps

  • Continue converting remaining 1494 procedures
  • Integrate into Laravel Simba package
  • Add unit tests for critical procedures

…mentation

- Converted 337 stored procedures from SimbaSql to PHP classes
- Created comprehensive documentation for each group (procedures-*.md)
- Updated master PROCEDURES.md with progress tracking
- All classes include detailed comments for direct usage without SQL source
- Groups processed: AP, AR, GL, SI, IN, PO, HR, FA, CA, SA, CO, Get, Del, Post, Rpt, Upd, Ins, Cal, Chuyen, Filt, ReCal, Crt, Copy, Check, Chk
- AsChangePassword, AsDashDemoChart, AsDashExample, AsDashGLKetQuaKinhDoanh, AsDashGLSoSanhKH_TT, AsDashGetParameter
- Updated PROCEDURES.md (total 343)
- Added docs/procedures-remaining-1.md
## 📚 BÀI HỌC MỚI: KHÔNG BAO GIỜ CẬP NHẬT VÀO PR CŨ

### VẤN ĐỀ:
Đã cập nhật vào PR cũ (#31) thay vì tạo PR mới.

### QUY TẮC MỚI (BẮT BUỘC):
**KHÔNG BAO GIỜ CẬP NHẬT VÀO PR CŨ. LUÔN LUÔN TẠO PR MỚI CHO MỖI SET OF CHANGES.**

### CẬP NHẬT TRONG SOUL.md:

#### 1. Thêm Bài học 5:
- **Bài học 5**: Đã cập nhật vào PR cũ (#31) thay vì tạo PR mới (22/02/2026). **Từ nay: KHÔNG BAO GIỜ CẬP NHẬT VÀO PR CŨ. LUÔN LUÔN TẠO PR MỚI CHO MỖI SET OF CHANGES. MỖI TASK MỚI = BRANCH MỚI = PR MỚI.**

#### 2. Cập nhật PR Workflow Rule:
- **KHÔNG BAO GIỜ cập nhật vào PR cũ**.
- **LUÔN LUÔN TẠO PR MỚI** cho mỗi set of changes.

#### 3. Cập nhật Workflow Checklist:
- **Khi có task mới (sau khi PR cũ đã merge)**:
  1. Checkout main
  2. Pull latest
  3. Tạo branch mới
  4. Thực hiện changes
  5. Tạo commit
  6. Báo cáo sếp
  7. Chờ sếp cho phép tạo PR

- **QUAN TRỌNG: KHÔNG BAO GIỜ**:
  - Cập nhật vào PR cũ (đã merge hoặc đang mở)
  - Push vào branch cũ (đã có PR)
  - Thêm commits vào PR đã mở
  - Sửa PR description/body sau khi tạo
  - Tự ý merge/revert/close PR

### TỪ NAY:
1. **Mỗi task mới** = **Branch mới** = **PR mới**
2. **Không bao giờ** cập nhật PR cũ
3. **Luôn luôn** tạo PR mới
4. **Tuân thủ** git workflow rules

### ÁP DỤNG:
- Đã tạo PR #32 mới cho URL restructuring
- Không cập nhật vào PR #31 cũ
- Mỗi set of changes có PR riêng

### GHI NHỚ:
**KHÔNG BAO GIỜ CẬP NHẬT VÀO PR CŨ. LUÔN LUÔN TẠO PR MỚI.**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0