Fix: perbaikan menu artikel hilang#1091
Open
pandigresik wants to merge 3 commits into
Open
Conversation
|
🔄 AI PR Review sedang antri di server...
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request: Fix Submenu Artikel OpenSID yang Hilang
Deskripsi
Memperbaiki submenu "Artikel" pada menu "Pengaturan OpenSID" yang tidak tampil lagi, padahal submenu tersebut pernah ditampilkan pada PR #877. Submenu ini mengarah ke halaman manajemen artikel OpenSID (
/master/artikel) yang route-nya sudah tersedia. Fix ini menambahkan kembali definisi submenu padaModulenum dan menjalankan migration yang memanggil commandadmin:menu-updateuntuk memperbarui menu administrator beserta permission-nya.Perubahan yang dilakukan:
app/Enums/Modul.phpdi bawah parent menu "Pengaturan OpenSID", ditempatkan setelah "Kategori Artikel" dengan URLmaster/artikeldan permissionmaster-data-artikel.database/migrations/2026_07_08_063209_update_menu_admin.phpyang memanggil Artisan commandadmin:menu-updateuntuk menyinkronkan menu administrator dengan definisi terbaru dariModulenum.admin:menu-update, seluruh permission CRUD (master-data-artikel-read/write/edit/delete) otomatis dibuatkan (jika belum ada) dan disinkronisasi ke role "administrator".Alasan perubahan:
/master/artikeldan controllerArtikelKabupatenControllerbeserta permissionmaster-data-artikelsebenarnya sudah ada, yang hilang hanya definisi menu-nya padaModulenum dan data menu team administrator yang tersimpan.Artisan::call('admin:menu-update')untuk memperbarui menu administrator setelah ada perubahan diModulenum.Dampak perubahan:
✅ Fungsionalitas: Submenu "Artikel" kembali tampil pada menu "Pengaturan OpenSID" untuk administrator.
✅ Aksesibilitas: Administrator dapat mengakses halaman manajemen artikel OpenSID melalui navigasi menu.
✅ Permission: Permission
master-data-artikelCRUD otomatis tersinkronisasi ke role administrator.✅ Konsistensi: Mengikuti pola migration yang sudah established di codebase.
✅ No Regression: Tidak mengubah route, controller, atau logic aplikasi yang sudah ada — hanya menambahkan entri menu.
Masalah Terkait (Related Issue)
Langkah untuk mereproduksi (Steps to Reproduce)
Sebelum fix (masalah):
Setelah fix (solusi):
/master/artikel) terbuka dengan normal.Testing pada fitur lain yang terkait:
master-data-artikel✅ Tersinkronisasi ke role administratorDaftar Periksa (Checklist)
Teknis Detail
Penjelasan Teknis
Menu administrator didefinisikan di
app/Enums/Modul.phpsebagai struktur default (Modul::Menu), lalu disimpan pada kolommenudi tabelteamsuntuk team "administrator". Fix dilakukan dengan:Modulenum — menambahkan submenu "Artikel" agar terbawa pada struktur menu default.admin:menu-updateyang sudah ada (app/Console/Commands/updateAdminMenu.php). Command ini melakukan:menu-nya denganModul::Menuterbarumenu_orderke null (jika kolom ada)Permission::findOrCreate(iterasiModul::permision:read,write,edit,delete)Pola ini sudah digunakan oleh 17 migration sebelumnya (mis.
2025_11_25_160801_update_menu_admin.php,2024_11_24_220314_add_menu_suplemen.php), sehingga merupakan cara idiomatic di codebase ini untuk memperbarui menu administrator.Konfigurasi yang berubah
Tidak ada perubahan file konfigurasi.
Dependencies yang ditambahkan
Tidak ada dependency baru.
File yang diubah
app/Enums/Modul.phpdatabase/migrations/2026_07_08_063209_update_menu_admin.phpadmin:menu-updateuntuk sync menu & permissionTesting
Manual Testing
php artisan migratedan pastikan tidak ada error/master/artikelterbukamaster-data-artikel-read/write/edit/deletetersimpan ke role administratorBreaking Changes
Tidak ada. Perubahan hanya menambahkan entri menu dan menjalankan sinkronisasi permission melalui command yang sudah ada sebelumnya.
Migration Guide
Jalankan migration standar — tidak ada langkah khusus:
References
admin:menu-update:app/Console/Commands/updateAdminMenu.phpCatatan tambahan: Route
/master/artikel(ArtikelKabupatenController) dan permissionmaster-data-artikelsudah ada sebelumnya, sehingga fix ini bersifat memulihkan tampilan menu tanpa mengubah logic aplikasi. Migration mengikuti konvensiadmin:menu-updateyang sudah digunakan di 17 migration sebelumnya.Screenshot