Skip to content

Tambahkan filter tahun pada statistik seni budaya & pendidikan data presisi#949

Merged
vickyrolanda merged 3 commits intorilis-devfrom
dev-948
Mar 6, 2026
Merged

Tambahkan filter tahun pada statistik seni budaya & pendidikan data presisi#949
vickyrolanda merged 3 commits intorilis-devfrom
dev-948

Conversation

@pandigresik
Copy link
Contributor

@pandigresik pandigresik commented Mar 4, 2026

Perbaikan issue #948
Bergantung pada PR https://github.com/OpenSID/API-Database-Gabungan/pull/330

Summary Perubahan

Detail Perubahan per File

1. resources/views/presisi/statistik/pendidikan.blade.php

  • Status: Dimodifikasi (Modified)
  • Perubahan: +12 baris, -9 baris

Perubahan Utama:

  1. Penggantian Filter Tahun:

    • Menghapus tombol filter collapse yang lama (baris 36-42)
    • Menambahkan komponen <x-filter-tahun /> untuk filter tahun yang lebih konsisten
  2. Pembaruan Referensi ID Filter:

    • Mengubah $("#tahun").val() menjadi $("#filter-tahun").val() pada fungsi exportToExcel (baris 190)
    • Menambahkan parameter "tahun": $('#filter-tahun').val() pada data request DataTable (baris 342)
    • Menambahkan parameter tahun pada URL detail (baris 394)
  3. Penambahan Event Listener:

    • Menambahkan event listener untuk perubahan filter tahun yang akan me-reload data tabel (baris 443-446)
  4. Perbaikan Logika Grafik:

    • Menambahkan kondisi if (data_grafik.length > 2) sebelum memanggil grafikPie() untuk mencegah error saat data kurang dari 3 (baris 358-360)

2. resources/views/presisi/statistik/senibudaya.blade.php

  • Status: Dimodifikasi (Modified)
  • Perubahan: +12 baris, -9 baris

Perubahan Utama:

  1. Penggantian Filter Tahun:

    • Menghapus tombol filter collapse yang lama (baris 36-42)
    • Menambahkan komponen <x-filter-tahun /> untuk filter tahun yang lebih konsisten
  2. Pembaruan Referensi ID Filter:

    • Mengubah $("#tahun").val() menjadi $("#filter-tahun").val() pada fungsi exportToExcel (baris 190)
    • Menambahkan parameter "tahun": $('#filter-tahun').val() pada data request DataTable (baris 342)
    • Menambahkan parameter tahun pada URL detail (baris 393)
  3. Penambahan Event Listener:

    • Menambahkan event listener untuk perubahan filter tahun yang akan me-reload data tabel (baris 440-443)
  4. Perbaikan Logika Grafik:

    • Menambahkan kondisi if (data_grafik.length > 2) sebelum memanggil grafikPie() untuk mencegah error saat data kurang dari 3 (baris 358-360)

Analisis Perubahan

Tujuan Utama Perubahan:

  1. Standardisasi Filter Tahun: Mengganti implementasi filter tahun yang berbeda-beda dengan komponen yang konsisten (<x-filter-tahun />)
  2. Perbaikan Fungsionalitas: Memastikan filter tahun berfungsi dengan baik pada kedua halaman statistik
  3. Penanganan Error: Menambahkan validasi untuk mencegah error pada grafik pie saat data tidak mencukupi
  4. Konsistensi ID: Menggunakan ID yang konsisten (filter-tahun) untuk elemen filter tahun

Dampak Perubahan:

  1. UI/UX: Pengalaman pengguna yang lebih konsisten pada halaman statistik
  2. Maintainability: Penggunaan komponen yang dapat digunakan kembali (reusable) memudahkan pemeliharaan
  3. Stabilitas: Mengurangi kemungkinan error pada tampilan grafik
simplescreenrecorder-2026-03-04_13.11.54.mp4

@pandigresik pandigresik requested a review from vickyrolanda March 4, 2026 06:38
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.

2 participants