Skip to content

Fix whitespace in statistics detail link parameters - #1286

Open
pandigresik wants to merge 1 commit into
rilis-devfrom
fix/detail_statistik_penduduk
Open

pandigresik wants to merge 1 commit into
rilis-devfrom
fix/detail_statistik_penduduk

Conversation

@pandigresik

Copy link
Copy Markdown
Contributor

Description

Fix masalah whitespace pada teks kategori statistik penduduk yang digunakan sebagai parameter URL di halaman detail statistik. Perubahan ini menambahkan .trim() pada extracted text dari DOM element .pilih-kategori > a.active untuk memastikan tidak ada karakter whitespace yang tidak diinginkan ikut terkirim ke API.

Catatan penting: Domain OpenKab harus sudah didaftarkan pada API Database Gabungan agar tidak terkena masalah CORS.

Changes made:

  1. Bug Fix: Menambahkan .trim() pada $('.pilih-kategori > a.active').text() di 4 lokasi pada resources/views/statistik/index.blade.php untuk membersihkan whitespace dari teks kategori aktif
  2. Bug Fix: Pembersihan whitespace pada parameter judul (judul detail statistik)
  3. Bug Fix: Pembersihan whitespace pada parameter tipe (tipe kategori statistik)

Reason for change:

  • Whitespace characters: Teks yang diambil dari DOM element sering kali mengandung whitespace/newline karakter tersembunyi yang dapat mempengaruhi parameter URL
  • Konsistensi: Memastikan value yang dikirim ke API bersih dari karakter whitespace yang tidak diinginkan
  • CORS dan API calls: URL parameters yang bersih membantu mencegah masalah pada request ke API Database Gabungan

Impact of change:

  • URL Parameters: Parameter judul dan tipe akan memiliki value yang bersih tanpa whitespace
  • Link Detail: Link ke halaman detail statistik akan memiliki URL yang valid dan bersih
  • User Experience: Judul yang ditampilkan di halaman detail akan terlihat lebih rapi

Related Issue

Steps to Reproduce

Before fix (problem):

  1. Buka halaman Statistik Penduduk
  2. Pilih kategori statistik
  3. Klik link detail pada tabel statistik
  4. ❌ Parameter URL mungkin mengandung whitespace karakter tersembunyi
  5. ❌ Judul di halaman detail mungkin memiliki spasi yang tidak diinginkan

After fix (solution):

  1. Buka halaman Statistik Penduduk
  2. Pilih kategori statistik
  3. Klik link detail pada tabel statistik
  4. ✅ Parameter URL bersih tanpa whitespace karakter
  5. ✅ Judul di halaman detail ditampilkan dengan rapi

Checklist

  • Code follows writing rules
  • Manual testing done in development environment
  • No console errors or warnings
  • No breaking changes
  • Unit test created (tidak diperlukan untuk perubahan frontend minimal)

Technical Details

Technical Explanation

Perubahan dilakukan pada file resources/views/statistik/index.blade.php di 4 lokasi yang menangani pembuatan link detail statistik penduduk. Setiap lokasi menggunakan $('.pilih-kategori > a.active').text() untuk mendapatkan teks kategori aktif, dan perubahan menambahkan .trim() untuk memastikan teks tersebut bersih dari whitespace.

- let judul = $('.pilih-kategori > a.active').text() + ' : ' + data
+ let judul = $('.pilih-kategori > a.active').text().trim() + ' : ' + data

Configuration changes

  • Domain OpenKab harus didaftarkan pada API Database Gabungan untuk menghindari masalah CORS

Dependencies added

  • Tidak ada dependencies baru

Testing

Manual Testing

  • Buka halaman Statistik Penduduk
  • Pilih berbagai kategori statistik
  • Klik link detail pada tabel statistik
  • Verifikasi parameter URL bersih tanpa whitespace
  • Verifikasi judul ditampilkan dengan rapi
  • Test pada multiple browser (Chrome, Firefox)

Screenshots / Video

image

Before:

Tidak ada screenshot - perubahan minimal pada parameter URL

After:

Tidak ada screenshot - perubahan minimal pada parameter URL

Breaking Changes

None

Migration Guide

Not required

References


Additional notes: Pastikan domain OpenKab sudah terdaftar pada API Database Gabungan untuk menghindari masalah CORS. Perubahan ini hanya membersihkan whitespace pada parameter URL dan tidak mempengaruhi fungsi utama halaman statistik.

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

🔄 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 #1286

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