fajri/BE_CPONE
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

FHM290626CPONE01 + FHM290626SAS01 - fix T_TestIsPrice, debug AIS, standarisasi PR #4

Merged
fajri merged 1 commits from staging into master 2026-06-29 10:46:53 +07:00
Conversation 0 Commits 1 Files Changed 2 +113
fajri commented 2026-06-29 10:44:33 +07:00
Owner
Copy Link

Latar Belakang Masalah

  1. Fix T_TestIsPrice: Fungsi post_layanan_all di Masterdata.php hanya mendaftarkan layanan dengan T_TestIsPrice = 'Y' ke master AIS, namun fungsi post_transaction_* tidak memfilter hal yang sama sehingga pemeriksaan non-price ikut terkirim dan menyebabkan transaksi ditolak AIS.
  2. Debug AIS: Belum ada cara untuk melihat payload yang akan dikirim ke AIS tanpa benar-benar mengirimnya (dry-run).
  3. Standarisasi PR: Belum ada standar baku penulisan Pull Request sehingga QA kesulitan mengetahui apa yang harus diuji.

Perubahan

FHM290626CPONE01

  • Menambahkan filter AND T_TestIsPrice = 'Y' pada semua JOIN ke tabel t_test di fungsi post_transaction_daily_by_date, post_transaction_daily, dan post_transaction_by_labnumber — Transactionv2.php (21 titik).
  • Menambahkan helper build_curl_command(), should_debug_ais_payload(), debug_ais_payload_response(), fetch_doctor_from_ais() di Masterdata.php untuk fitur dry-run debug payload AIS di semua fungsi post_*.

FHM290626SAS01

  • Menambahkan file PR_PROMPT.md berisi prompt standar dan contoh pembuatan PR.
  • Mengupdate CLAUDE.md dan AGENTS.md dengan section Pull Request Format dan aturan wajib cek semua commit sebelum buat PR.

File yang Diubah

File Keterangan
application/controllers/ais/Transactionv2.php Fix filter T_TestIsPrice = 'Y' — 21 titik JOIN
application/controllers/ais/Masterdata.php Tambah 4 helper debug AIS payload di semua fungsi post_*
AGENTS.md Tambah section PR Format + aturan wajib cek commit sebelum buat PR
PR_PROMPT.md File baru — prompt standar dan contoh pembuatan PR
.gitignore Tambah .env

Test yang Harus Dilakukan QA

Fix T_TestIsPrice

  • Normal: Kirim transaksi dengan pemeriksaan T_TestIsPrice = 'Y' — harus berhasil masuk ke AIS.
  • Edge case: Order dengan campuran IsPrice = 'Y' dan 'N' — hanya yang 'Y' yang terkirim ke AIS.
  • Negatif: Order yang seluruh pemeriksaannya T_TestIsPrice = 'N' — tidak boleh ada layanan terkirim ke AIS.
  • Regression: Pastikan total tagihan, RegpasNominal, dan TrxBayar tetap terhitung benar.

Debug AIS Payload

  • Normal: Aktifkan mode debug, pastikan payload tampil tanpa request dikirim ke AIS.
  • Regression: Pastikan mode non-debug tetap berjalan normal mengirim request ke AIS.
## Latar Belakang Masalah 1. **Fix T_TestIsPrice:** Fungsi `post_layanan_all` di `Masterdata.php` hanya mendaftarkan layanan dengan `T_TestIsPrice = 'Y'` ke master AIS, namun fungsi `post_transaction_*` tidak memfilter hal yang sama sehingga pemeriksaan non-price ikut terkirim dan menyebabkan transaksi ditolak AIS. 2. **Debug AIS:** Belum ada cara untuk melihat payload yang akan dikirim ke AIS tanpa benar-benar mengirimnya (dry-run). 3. **Standarisasi PR:** Belum ada standar baku penulisan Pull Request sehingga QA kesulitan mengetahui apa yang harus diuji. ## Perubahan ### FHM290626CPONE01 - Menambahkan filter `AND T_TestIsPrice = 'Y'` pada semua JOIN ke tabel `t_test` di fungsi `post_transaction_daily_by_date`, `post_transaction_daily`, dan `post_transaction_by_labnumber` — **Transactionv2.php (21 titik)**. - Menambahkan helper `build_curl_command()`, `should_debug_ais_payload()`, `debug_ais_payload_response()`, `fetch_doctor_from_ais()` di `Masterdata.php` untuk fitur dry-run debug payload AIS di semua fungsi `post_*`. ### FHM290626SAS01 - Menambahkan file `PR_PROMPT.md` berisi prompt standar dan contoh pembuatan PR. - Mengupdate `CLAUDE.md` dan `AGENTS.md` dengan section Pull Request Format dan aturan wajib cek semua commit sebelum buat PR. ## File yang Diubah | File | Keterangan | |---|---| | `application/controllers/ais/Transactionv2.php` | Fix filter `T_TestIsPrice = 'Y'` — 21 titik JOIN | | `application/controllers/ais/Masterdata.php` | Tambah 4 helper debug AIS payload di semua fungsi `post_*` | | `AGENTS.md` | Tambah section PR Format + aturan wajib cek commit sebelum buat PR | | `PR_PROMPT.md` | File baru — prompt standar dan contoh pembuatan PR | | `.gitignore` | Tambah `.env` | ## Test yang Harus Dilakukan QA ### Fix T_TestIsPrice - [ ] **Normal:** Kirim transaksi dengan pemeriksaan `T_TestIsPrice = 'Y'` — harus berhasil masuk ke AIS. - [ ] **Edge case:** Order dengan campuran `IsPrice = 'Y'` dan `'N'` — hanya yang `'Y'` yang terkirim ke AIS. - [ ] **Negatif:** Order yang seluruh pemeriksaannya `T_TestIsPrice = 'N'` — tidak boleh ada layanan terkirim ke AIS. - [ ] **Regression:** Pastikan total tagihan, RegpasNominal, dan TrxBayar tetap terhitung benar. ### Debug AIS Payload - [ ] **Normal:** Aktifkan mode debug, pastikan payload tampil tanpa request dikirim ke AIS. - [ ] **Regression:** Pastikan mode non-debug tetap berjalan normal mengirim request ke AIS.
fajri added 1 commit 2026-06-29 10:44:34 +07:00
FHM290626SAS01 - prompt pembuatan PR 589f3f050b
fajri merged commit 2fdac0a347 into master 2026-06-29 10:46:53 +07:00
fajri changed title from FHM290626SAS01 - prompt pembuatan PR to FHM290626CPONE01 + FHM290626SAS01 - fix T_TestIsPrice, debug AIS, standarisasi PR 2026-06-29 10:56:00 +07:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adib fitri hanan mario syfa fajri
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: fajri/BE_CPONE#4
Delete Branch "staging"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?