Skip to content

Fix parameter query tidak diteruskan pada getAllPoint - #1285

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

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

Conversation

@pandigresik

Copy link
Copy Markdown
Contributor

Description

Method getAllPoint di PemetaanService tidak meneruskan parameter $query ke method apiRequest, sehingga filter, pagination, dan parameter lainnya diabaikan saat mengambil data point. Perbaikan menambahkan parameter $query pada pemanggilan apiRequest.

Changes made:

  1. Bug Fix: Menambahkan parameter $query pada pemanggilan $this->apiRequest('/api/v1/point', $query) di method getAllPoint pada app/Services/PemetaanService.php

Reason for change:

  • Parameter Diabaikan: Method getAllPoint menerima parameter $query tetapi tidak diteruskan ke apiRequest, menyebabkan filter dan parameter lainnya tidak sampai ke API
  • Data Tidak Sesuai: Tanpa parameter query, data point yang dikembalikan tidak sesuai dengan filter yang diminta oleh caller
  • Konsistensi: Method lain di service yang sama sudah meneruskan parameter dengan benar

Impact of change:

Query Parameter: Parameter filter, pagination, dan search sekarang diteruskan ke API dengan benar
Data Point: Data point yang dikembalikan sesuai dengan filter yang diminta
Fitur Pemetaan: Fitur yang menggunakan getAllPoint dengan filter berfungsi normal

Related Issue

Closes #1279

#1279

Steps to Reproduce

Before fix (problem):

  1. Panggil method getAllPoint dengan parameter query (filter, pagination)
  2. API menerima request tanpa parameter query
  3. ❌ Data yang dikembalikan tidak sesuai filter, semua data dikembalikan tanpa filter

After fix (solution):

  1. Panggil method getAllPoint dengan parameter query (filter, pagination)
  2. API menerima request dengan parameter query yang benar
  3. ✅ Data yang dikembalikan sesuai dengan filter yang diminta

Testing on related features:

  • Ambil semua data point ✅ Berfungsi normal
  • Filter data point ✅ Parameter diteruskan dengan benar
  • Pagination data point ✅ Berfungsi normal

Checklist

Technical Details

Technical Explanation

Perubahan pada method getAllPoint di app/Services/PemetaanService.php:

 public function getAllPoint(array $query = [])
 {
-    return $this->apiRequest('/api/v1/point');
+    return $this->apiRequest('/api/v1/point', $query);
 }

Konteks:

  • Method getAllPoint mendefinisikan parameter $query tetapi tidak menggunakannya
  • BaseApiService::apiRequest() menerima parameter kedua sebagai query parameters
  • Tanpa meneruskan $query, semua filter/pagination diabaikan

Configuration changes

None

Dependencies added

No new dependencies

Testing

Manual Testing

  • Panggil getAllPoint() tanpa parameter - harus mengembalikan semua data
  • Panggil getAllPoint(['filter' => 'value']) - harus mengembalikan data yang ter-filter
  • Panggil getAllPoint(['page' => 1]) - harus mengembalikan data pagination yang benar
  • Regression testing - pastikan fitur lain tidak terpengaruh

Screenshots / Video

simplescreenrecorder-2026-09-08_13.38.46.mp4

Breaking Changes

None

Migration Guide

Not required

References


Additional notes: Perubahan minimalis (1 baris) - memastikan parameter query diteruskan ke API saat mengambil data point.

@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 #1285

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