Conversation
Contributor
|
🔄 AI PR Review sedang antri di server...
|
Contributor
Contributor
Author
Contributor
|
mas @habibie11 disaya masih error seperti ini : |
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.



issue # #1652
🎯 Deskripsi
Pull request ini memperbaiki bug pada antarmuka pengguna (UI) di halaman publik Prosedur (
/unduhan/proseduratau/informasi/prosedur), di mana halaman tersebut justru menampilkan data Potensi (seperti Nama Potensi, Deskripsi, Lokasi, dan Gambar) alih-alih menampilkan daftar dokumen Prosedur.Penyebab utamanya adalah view blade untuk halaman publik prosedur (
prosedur.blade.php) menggunakan kode copy-paste dari halaman potensi yang belum pernah disesuaikan, serta ketidaktersediaan API endpoint/prosedurpada sisi frontend. Selain itu, fitur unduh prosedur padaweb.phpsebelumnya menggunakan parameter pencarian string (judul file) yang rentan mengalami isu URL encoding (seperti karakter spasi atau dash), dan elemen modal menggunakan<object>yang dapat menyebabkan PDF gagal dirender di beberapa browser.Dengan perbaikan ini, alur data prosedur telah dirapihkan mulai dari sisi Backend API (Repository, Transformer, Controller) hingga View Blade. Format penampil dokumen PDF pada popup modal juga telah diseragamkan dan ditingkatkan kebesaran kompabilitasnya dengan beralih ke
<iframe>.🛠️ Perubahan yang Dilakukan
1. Penambahan Layer API Frontend Prosedur
Pembuatan komponen backend standar untuk melayani data
DataTablessecara publik:app/Repositories/ProsedurApiRepository.php: Repository untuk query, sorting, dan filter dataProsedur.app/Transformers/ProsedurTransformer.php: Transformer untuk melampirkan URL path file dan route pengunduhan berdasar ID.app/Http/Controllers/Api/Frontend/ProsedurController.php: Controller frontend untuk menangani request AJAX.routes/api-frontend.php: Mendaftarkan route baruGET /api/frontend/v1/prosedur.2.
routes/web.php&app/Http/Controllers/FrontEnd/DownloadController.phpFix — Memperbaiki logika pencarian pada fitur unduh:
{file}/downloaddi mana file adalah ID prosedur).downloadProsedur()dengan beralih dari fungsiwhere('judul_prosedur', ...)ke fungsi bawaan LaravelfindOrFail()demi keandalan data.3.
themes/opendk/default/resources/views/pages/unduhan/prosedur.blade.phpFix — Merekonstruksi View Publik:
#prosedur-table, headerData Prosedur, memanggil AJAX endpoint/prosedur).objectmenjadiiframeuntuk meningkatkan support browser.4.
themes/opendk/default/resources/views/pages/unduhan/regulasi.blade.phpImprovement — Migrasi PDF viewer di Modal Regulasi:
prosedur.blade.php, mengganti tag<object>menjadi<iframe>dan memperbaiki kondisi deteksi formatapplication/pdf.✅ Test Cases yang Diimplementasikan
/informasi/prosedurmenampilkan tabel dengan header "Data Prosedur".Prosedurmaupun ModalRegulasiberhasil merender (menampilkan file PDF) menggunakan elemen<iframe>dengan sempurna tanpa error.📸 Cara Menjalankan Uji Coba Manual
/informasi/prosedur..pdf, pastikan tampil viewer PDF inline dalam frame secara normal.📸 Screenshot atau Video