Fix: perbaikan fields ganda pada menu pejabat daerah

[pandigresik]

Pull Request: Fix field form pejabat daerah tampil ganda

Deskripsi

Field form pada halaman tambah/edit Pejabat Daerah tampil ganda (duplicate). Setiap field (NIP, Email, Deskripsi, Telepon, Jabatan, Departemen) muncul dua kali dalam satu halaman. Perbaikan ini menghapus field-field duplikat sehingga form hanya menampilkan satu set field yang benar.

Perubahan yang dilakukan:

1. Bug Fix: Menghapus field form duplikat (NIP, Email, Deskripsi, Telepon, Jabatan, Departemen) yang muncul dua kali di resources/views/employees/fields.blade.php
2. Bug Fix: Menghapus class col-sm-6 pada field Nama agar konsisten dengan field lainnya
3. Refactor: Memindahkan blok @push('page_scripts') ke bagian atas file (sebelum field HTML) agar script focus() dijalankan setelah DOM ready
4. Bug Fix: Menambahkan nilai old() pada select field position_id (sebelumnya tidak ada fallback value saat validasi error)

Alasan perubahan:

• Root Cause: Field duplikat terjadi akibat kesalahan during migrasi Laravel 9 ke 10 (commit 1ab58208 - "Pasca rilis 2510.0.1"). Saat migrasi Form:: ke Html::, field-field form ditulis ulang namun field lama tidak terhapus, sehingga menjadi ganda.
• Dampak: Form pejabat daerah menjadi tidak usable karena field tampil dua kali, membingungkan pengguna.
• Solusi: Menghapus field duplikat (set pertama dengan class col-sm-6) dan mempertahankan set kedua yang sudah sesuai format Html helper baru.

Dampak perubahan:

• ✅ Form tambah Pejabat Daerah hanya menampilkan satu set field yang benar
• ✅ Form edit Pejabat Daerah hanya menampilkan satu set field yang benar
• ✅ Konsistensi class CSS pada semua form field (tanpa col-sm-6)
• ✅ Script $('#name').focus() berjalan di posisi yang tepat
• ✅ Select field Jabatan (position_id) mempertahankan nilai old() saat validasi error

Masalah Terkait (Related Issue)

• Solusi untuk perbaikan terkait issue #1066

Langkah untuk mereproduksi (Steps to Reproduce)

Sebelum perbaikan (masalah):

1. Login ke halaman admin OpenKab
2. Navigasi ke menu Organisasi > Pejabat Daerah
3. Klik tombol Tambah untuk menambah pejabat daerah baru
4. ❌ Perhatikan bahwa field NIP, Email, Deskripsi, Telepon, Jabatan, dan Departemen muncul dua kali dalam form

Setelah perbaikan (fix):

1. Login ke halaman admin OpenKab
2. Navigasi ke menu Organisasi > Pejabat Daerah
3. Klik tombol Tambah untuk menambah pejabat daerah baru
4. ✅ Form hanya menampilkan satu set field: Nama, NIP, Email, Deskripsi, Telepon, Jabatan, Departemen
5. Klik tombol Edit pada data pejabat daerah yang sudah ada
6. ✅ Form edit juga hanya menampilkan satu set field

Testing pada fitur lain yang terkait:

• Form tambah Pejabat Daerah ✅ Berfungsi normal
• Form edit Pejabat Daerah ✅ Berfungsi normal
• Validasi form (submit tanpa isi) ✅ Error message tampil dengan benar
• Simpan data pejabat daerah ✅ Data tersimpan dengan benar

Daftar Periksa (Checklist)

• Saya telah mematuhi aturan penulisan script.
• Saya telah mengikuti proses review pull request.
• Testing manual telah dilakukan di environment development
• Tidak ada console error atau warning
• Tidak ada perubahan pada database migration
• Tidak ada perubahan pada API endpoint

Teknis Detail

Penjelasan Teknis

File resources/views/employees/fields.blade.php merupakan partial view yang di-include ke dalam:

• resources/views/employees/create.blade.php (form tambah)
• resources/views/employees/edit.blade.php (form edit)

Rute terkait:

Route::middleware('easyauthorize:organisasi-employee')
    ->resource('employees', EmployeeController::class)
    ->except(['show']);

Root cause analysis:

Versi rilis-dev memiliki 92 baris, sedangkan versi fix memiliki 52 baris. Perbedaan 40 baris adalah field-field duplikat yang dihapus.

Field	rilis-dev (duplikat)	fix/hapus
Nama	1x (col-sm-6)	1x (tanpa col-sm-6)
NIP	2x	1x
Email	2x	1x
Deskripsi	2x	1x
Telepon	2x	1x
Jabatan	2x	1x
Departemen	2x	1x

Konfigurasi yang berubah

Tidak ada perubahan konfigurasi.

Dependencies yang ditambahkan

Tidak ada dependencies baru.

Testing

Manual Testing

• Buka halaman Tambah Pejabat Daerah - form tampil normal tanpa duplikat
• Buka halaman Edit Pejabat Daerah - form tampil normal tanpa duplikat
• Submit form kosong - validasi error berfungsi
• Isi form lengkap dan simpan - data tersimpan dengan benar
• Edit data yang sudah ada - data lama ter-load dengan benar
• Field dropdown Jabatan dan Departemen berfungsi dengan benar
• Focus cursor otomatis pada field Nama saat form dibuka

Screenshots / Video

simplescreenrecorder-2026-06-17_15.58.34.mp4

Sebelum:

Form pejabat daerah menampilkan field NIP, Email, Deskripsi, Telepon, Jabatan, dan Departemen sebanyak dua kali.

Sesudah:

Form pejabat daerah menampilkan setiap field hanya sekali, sesuai yang diharapkan.

Breaking Changes

Tidak ada breaking changes.

Migration Guide

Tidak diperlukan.

[github-actions]

🔄 AI PR Review sedang antri di server...

Proses review akan segera dimulai di background — hasil akan muncul sebagai komentar setelah selesai.
Powered by CrewAI · PR #1067